QT中QStringList::const_iterator实现图片上一张功能失败求助

最近我在开发一款Android平台的QT图片浏览应用，需要实现点击按钮切换上一张、下一张图片的功能。一开始我用QStringList::const_iterator来控制图片切换逻辑，结果发现下一张功能正常，但上一张完全失效，甚至在按钮函数里加addPixmap(*m_imageIt)会直接导致应用崩溃。后来改用整数索引的方式修改代码，功能终于恢复正常。这里就来分析一下迭代器版本的问题所在，以及如何正确用迭代器实现需求。

问题回顾：初始迭代器版本代码 #

showpic.h #

#ifndef SHOWPIC_H
#define SHOWPIC_H
#include <QWidget>
#include <QTimer>
namespace Ui {
class ShowPic;
}
class ShowPic : public QWidget {
    Q_OBJECT
public:
    explicit ShowPic(QWidget *parent = 0);
    ~ShowPic();
public:
    void addPixmap(const QPixmap &pixmap);
    void startPlay(QStringList infilenames);
private slots:
    void on_pushButton_3_clicked();
    void tick();
    void on_pushButton_clicked();
    void on_pushButton_2_clicked();
    void on_pushButton_4_clicked();
private:
    Ui::ShowPic *ui;
    QStringList::const_iterator m_imageIt;
    QTimer m_timer;
    QStringList filenames;
    bool flag = false;
};
#endif // SHOWPIC_H

showpic.cpp（关键问题函数） #

void ShowPic::on_pushButton_clicked() //move to the prev image
{
    if(!filenames.isEmpty()){
        m_timer.stop();
        m_imageIt--;
        if(m_imageIt != filenames.begin()){
            addPixmap(*m_imageIt);
        }else{
            m_imageIt = filenames.end();
            //addPixmap(*m_imageIt); // 解开注释直接崩溃
        }
    }
}

问题根源分析 #

迭代器版本的上一张功能失效甚至崩溃，主要有三个核心问题：

1. 非法迭代器的产生：当m_imageIt已经指向filenames.begin()（也就是第一张图片）时，直接执行m_imageIt--会让迭代器指向容器起始位置之前的非法内存区域，这时候解引用迭代器必然触发未定义行为（崩溃）。
2. 尾后迭代器的错误使用：代码里试图把迭代器设为filenames.end()来循环到最后一张，但end()返回的是尾后迭代器，它不指向任何有效元素，解引用它肯定会导致应用崩溃。
3. 隐藏赋值错误：startPlay函数里的flag == false是比较操作，不是赋值，应该写成flag = false，这个在索引版本里也存在，虽然没影响功能，但属于逻辑错误。

迭代器版本的正确实现 #

只要修正边界处理逻辑，迭代器版本也能稳定工作，以下是修改后的关键代码：

修改后的showpic.cpp核心函数 #

// 上一张按钮的正确逻辑
void ShowPic::on_pushButton_clicked() //move to the prev image
{
    if(filenames.isEmpty()) return;
    m_timer.stop();
    
    // 先判断是否在起始位置，再移动迭代器
    if(m_imageIt == filenames.begin()){
        // 循环到最后一张：先移动到尾后迭代器，再往前移一位
        m_imageIt = filenames.end();
    }
    --m_imageIt;
    addPixmap(QPixmap(*m_imageIt));
}

// 下一张按钮逻辑优化（更严谨）
void ShowPic::on_pushButton_2_clicked() //move to the next image
{
    if(filenames.isEmpty()) return;
    m_timer.stop();
    
    ++m_imageIt;
    if(m_imageIt == filenames.end()){
        m_imageIt = filenames.begin();
    }
    addPixmap(QPixmap(*m_imageIt));
}

// 修复startPlay里的赋值错误
void ShowPic::startPlay(QStringList infilenames){
    filenames = infilenames;
    m_imageIt = filenames.begin();
    m_timer.start();
    flag = false; // 这里把==改成=
    QString d("Stop");
    ui->pushButton_3->setText(d);
}

为什么整数索引版本能正常工作？ #

整数索引的边界处理逻辑更直观：当index < 0时直接设置为filenames.size()-1（最后一张的索引），当index >= filenames.size()时设置为0（第一张的索引），不会出现迭代器那种指向非法内存的问题，因为索引是基于整数的，边界判断清晰，不容易触发未定义行为。

不过迭代器版本和索引版本在功能上没有本质区别，只要正确处理边界，两者都能稳定运行。对于图片文件名这种小型列表来说，两者的性能差异可以忽略不计，选择哪种方式完全看个人习惯。

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