使用Bloc实现CRUD操作的更优方案咨询

你的初始实现已经符合Bloc的不可变状态核心原则，这里提供几个更简洁、健壮的优化方向：

1. 提取公共状态更新逻辑 #

重复的列表复制与状态生成代码可以抽成公共方法，减少冗余：

class TodoBloc extends Bloc<TodoEvent, TodoState> {
  TodoBloc() : super(TodoState(todos: const [])) {
    on<TodoEventAdd>(_handleAdd);
    on<TodoEventDelete>(_handleDelete);
    on<TodoEventIsCompleted>(_handleToggleCompleted);
  }

  // 封装状态更新逻辑，专注于列表修改规则
  TodoState _updateTodos(List<TodoModel> Function(List<TodoModel>) updateFn) {
    final newTodos = updateFn(List.from(state.todos));
    return state.copyWith(todos: newTodos);
  }

  void _handleAdd(TodoEventAdd event, Emitter<TodoState> emit) {
    emit(_updateTodos((todos) => [...todos, event.todo]));
  }

  void _handleDelete(TodoEventDelete event, Emitter<TodoState> emit) {
    emit(_updateTodos((todos) => todos.where((t) => t.id != event.todo.id).toList()));
  }

  void _handleToggleCompleted(TodoEventIsCompleted event, Emitter<TodoState> emit) {
    emit(_updateTodos((todos) => todos.map((t) => 
      t.id == event.todo.id ? t.copyWith(isCompleted: event.isCompleted) : t
    ).toList()));
  }
}

2. 避免依赖索引，用ID定位Todo #

原实现中通过索引更新完成状态的逻辑存在隐患——当列表排序、过滤后，索引会失效。改用ID匹配更可靠：

// 调整事件类，传递目标Todo而非索引
class TodoEventIsCompleted extends TodoEvent {
  final TodoModel todo;
  final bool isCompleted;

  const TodoEventIsCompleted({required this.todo, required this.isCompleted});
}

3. 强制状态不可变性 #

在TodoState中使用不可变集合，防止外部代码意外修改状态：

class TodoState {
  final UnmodifiableListView<TodoModel> todos;

  const TodoState({required List<TodoModel> todos}) 
    : todos = UnmodifiableListView(todos);

  TodoState copyWith({List<TodoModel>? todos}) {
    return TodoState(
      todos: todos != null ? UnmodifiableListView(todos) : this.todos,
    );
  }
}

4. 简化集合操作逻辑 #

利用Dart集合的where、map方法替代手动修改列表，代码更紧凑且可读性强：

• 删除操作直接过滤掉目标Todo
• 更新操作遍历替换目标Todo的状态

这些优化既保持了Bloc的核心规范，又提升了代码的可维护性与鲁棒性，你可以根据项目复杂度选择适配。

内容的提问来源于stack exchange，提问作者CCP