avoid_logic_in_view_classes

[IvanDembicki]

This rule will help separate view classes from controllers, data models, etc.
Classes that extend base view classes should not contain logic, except for factory constructors. All logical operators should be prohibited in this classes.

class NoLogicView extends StatelessWidget {
  const NoLogicView(this.model);
  
  factory NoLogicView.create(TestModel model) {
    if (model.enabled) return NoLogicViewEnabledChild(); // No error Avoid logic in view classes 
    return NoLogicViewDisabledChild();
  }
  
  final TestModel model;

  @override
  Widget build(BuildContext context) {
    return Container(
      color: model.enabled ? Colors.red : Colors.grey, // Error Avoid logic in view classes 
      child: Row(
        children: [
          if (model.enabled) const SizedBox(height: 10), // Error Avoid logic in view classes
          Container(
            color: containerColor, 
          ),
        ],
      ),
    );
  }
  
  Color get containerColor => model.enabled ? Colors.red : Colors.grey; // Error Avoid logic in view classes 
  
}

class TestModel {
  bool enabled = false;
}

NoLogicView noLogicView = NoLogicView(TestModel());

[srawlins]

@IvanDembicki thanks for the request. Can you explain what specific rules the lint would enforce? Can a class which extends StatelessWidget have any instance members other than build? instance fields are allows? Static members? Are there other "view classes"?

[github-actions]

Without additional information we're not able to resolve this issue, so it will be closed at this time. You're still free to add more info and respond to any questions above, though. We'll reopen the case if you do. Thanks for your contribution!