Qt中为QAbstractListModel创建CurrentIndexChanged信号遇C2662错误的解决

咱们先搞懂为啥会报这个C2662错误：

• Qt的信号本质是类的成员函数，默认是非const的。而你的data()函数被const修饰，这意味着在这个函数内部，this指针是const BookListModel*类型——编译器绝对不允许用一个const指针去调用非const的成员函数（也就是你的currentIndexChanged信号），所以才会出现“无法将this指针从const BookListModel转换为BookListModel&”的报错。
• 另外你发射信号的时候还漏传了参数i，就算解决了const的问题，这也会触发另一个编译错误。

这里有个关键的设计误区得先提：data()函数根本不应该用来发射currentIndexChanged信号。data()的核心职责是当视图请求数据时返回对应值，比如列表项要显示文本、图标时都会调用它，如果在这里发射信号，会导致视图每次刷新数据都触发信号，完全不符合“当前索引变化”的语义。

正确的做法是把信号发射时机放在真正触发索引变化的场景里：

• 如果是视图选中项变化导致的，可以在视图的QAbstractItemView::currentChanged信号里响应；或者在模型里写一个专门的setCurrentIndex(int index)函数，当主动设置当前索引时，更新内部状态并发射信号：

  void BookListModel::setCurrentIndex(int i) {
      // 可以先加个判断，比如索引是否合法、是否和当前值一致
      emit currentIndexChanged(i);
  }
  

• 如果是模型内部数据变化导致当前索引需要更新，也应该在处理数据变化的函数里发射信号，而不是data()中。

如果你确实有特殊需求，必须在const修饰的data()函数里发射信号，可以借助Qt 5.10及以上版本支持的const信号特性：

1. 修改头文件中的信号声明，加上const修饰：

   signals:
       void currentIndexChanged(int i) const;
   

2. 在data()函数中正确传递参数发射信号：

   QVariant BookListModel::data(const QModelIndex &index, int role) const{
       int i = index.row();
       emit currentIndexChanged(i); // 别忘了传参数i
       // 这里返回对应role的数据
       return QVariant();
   }
   

再次提醒：这种做法违背了Qt模型视图的设计原则，只有万不得已时才考虑使用。

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