QListWidget вызывает ошибку сегментации при нажатии и перетаскивании по нескольким элементам - PullRequest
Новая
       0

QListWidget вызывает ошибку сегментации при нажатии и перетаскивании по нескольким элементам

0 голосов
/ 08 декабря 2018

У меня есть QListWidget с несколькими элементами, каждый из которых действует как кнопка, реагирующая на нажатие.Я столкнулся с проблемой, при которой, если вы щелкнете по одному элементу и перетащите мышку в любое место на экране, кроме элемента, по которому вы щелкнули, тогда программа вызовет ошибку сегментации и сбой.Кто-нибудь знает, как я могу это исправить?

Я включил весь код, который я написал ниже, хотя этот код также основан на проприетарном коде, который я не думаю, что я могу опубликовать здесь

window.cc 

#include "globals.h"

#include <QLabel>
#include <QBoxLayout>
#include <QScrollArea>
#include <QListWidget>
#include <QListWidgetItem>
#include <QPushButton>
#include <QMessageBox>

#include "windowheader.h"

namespace{

class statisticsTab : public QWidget {
public:
    statisticsTab();
private:
    QGridLayout * layout;
    QLabel * title;
    QLabel * userListTitle;
    QLabel * branchListTitle;
    UserListWidget * userList;
    BranchListWidget * branchList;
    QListWidget * statsPage;
};

inline statisticsTab::statisticsTab() : QWidget() {
    layout = new QGridLayout();
    cur_repo = new GITPP::REPO();

    title = new QLabel("Repository Statistics");
    title->setStyleSheet("QLabel {font-weight: bold;}");
    layout->addWidget(title, 0, 0, 1, 2);

    userListTitle = new QLabel("Contributors");
    layout->addWidget(userListTitle, 1, 0, 1, 1);

    branchListTitle = new QLabel("Branches");
    layout->addWidget(branchListTitle, 1, 1, 1, 1);

    statsPage = new QListWidget();
    layout->addWidget(statsPage, 3, 0, 1, 2, Qt::AlignTop);
    QListWidgetItem * statsPageDefault = new QListWidgetItem(QString("Click on a contributor or branch to get started!"), 0, 0);
    statsPage->addItem(statsPageDefault);

    userList = new UserListWidget(statsPage);
    layout->addWidget(userList, 2, 0, 1, 1);

    branchList = new BranchListWidget(statsPage);
    layout->addWidget(branchList, 2, 1, 1, 1);

    if(cur_repo != nullptr) {
        GITPP::COMMITS commits = cur_repo->commits();
        std::vector <std::string> contributors;

        for(auto commit : commits) {
            contributors.push_back(commit.author());
        }

        std::sort(contributors.begin(), contributors.end());
        contributors.erase(unique(contributors.begin(), contributors.end()), contributors.end());

        for(auto contributor : contributors) {
            QString contributorName = QString::fromStdString(contributor);
            QListWidgetItem * contributorNameItem = new QListWidgetItem(contributorName);
            userList->addItem(contributorNameItem);
        }

        GITPP::BRANCHES branches = cur_repo->branches();

        for(auto branch : branches) {
            QListWidgetItem * branchName = new QListWidgetItem(QString::fromStdString(branch.name()), 0, 0);
            branchList->addItem(branchName);
        }
    } else {
        QListWidgetItem * branchListDefault = new QListWidgetItem(QString("No branches found"), 0, 0);
        branchList->addItem(branchListDefault);

        QListWidgetItem * userListDefault = new QListWidgetItem(QString("No users found"), 0, 0);
        userList->addItem(userListDefault);
    }

    setLayout(layout);
}

INSTALL_TAB(statisticsTab, "Statistics");

}

windowheader.h 

#ifndef WINDOWHEADER_H
#define WINDWOHEADER_H

#include <QListWidget>
#include <string>
#include <limits.h>
#include <unistd.h>
#include <QPushButton>
#include <QFileDialog>
#include <QMessageBox>

#include "globals.h"

class UserListWidget : public QListWidget {
    Q_OBJECT
public:
    UserListWidget(QListWidget * statsPage);

private slots:
    void updateStatsPage();
private:
    QListWidget * statsPage;
};

inline UserListWidget::UserListWidget(QListWidget * statsPageArg) : QListWidget() {
    connect(this, SIGNAL(itemSelectionChanged()), this, SLOT(updateStatsPage()));
    statsPage = statsPageArg;
}

inline void UserListWidget::updateStatsPage() {
    GITPP::CONFIG config = cur_repo->config();
    GITPP::COMMITS commits = cur_repo->commits();


    QString statsTitle = QString("Here are some stats about the user ") + this->currentItem()->text();
    QListWidgetItem * statsTitleItem = new QListWidgetItem(statsTitle);
    statsPage->clear();
    statsPage->addItem(statsTitleItem);

    for(auto thing : config) {
        QString statsInfo = QString::fromStdString(thing.name());
        QListWidgetItem * statsInfoItem = new QListWidgetItem(statsInfo);
        statsPage->addItem(statsInfoItem);
    }

    selectionModel()->clear();
}

class BranchListWidget : public QListWidget {
    Q_OBJECT
public:
    BranchListWidget(QListWidget * statsPage);

private slots:
    void updateStatsPage();
private:
    QListWidget * statsPage;
};

inline BranchListWidget::BranchListWidget(QListWidget * statsPageArg) : QListWidget() {
    connect(this, SIGNAL(itemSelectionChanged()), this, SLOT(updateStatsPage()));
    statsPage = statsPageArg;
}

inline void BranchListWidget::updateStatsPage() {
    GITPP::CONFIG config = cur_repo->config();
    GITPP::COMMITS commits = cur_repo->commits();

    QString statsTitle = QString("Here are some stats about the branch ") + this->currentItem()->text();
    QListWidgetItem * statsTitleItem = new QListWidgetItem(statsTitle);
    statsPage->clear();
    statsPage->addItem(statsTitleItem);

    for(auto thing : config) {
        QString statsInfo = QString::fromStdString(thing.name());
        QListWidgetItem * statsInfoItem = new QListWidgetItem(statsInfo);
        statsPage->addItem(statsInfoItem);
    }

    selectionModel()->clear();
}

#endif

Выход Valgrind: 

==9475== Invalid read of size 8
==9475==    at 0x1158B5: UserListWidget::updateStatsPage() (in /home/alexis/Desktop/programming/uni_work/comp_2811/cw2/ui_cw3/2811_gui)
==9475==    by 0x5F5D5E8: QMetaObject::activate(QObject*, int, int, void**) (in /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.7.1)
==9475==    by 0x5F5D5E8: QMetaObject::activate(QObject*, int, int, void**) (in /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.7.1)
==9475==    by 0x5EE0BE6: QItemSelectionModel::selectionChanged(QItemSelection const&, QItemSelection const&) (in /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.7.1)
==9475==    by 0x5EE544A: QItemSelectionModel::emitSelectionChanged(QItemSelection const&, QItemSelection const&) (in /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.7.1)
==9475==    by 0x5EE8F91: QItemSelectionModel::select(QItemSelection const&, QFlags<QItemSelectionModel::SelectionFlag>) (in /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.7.1)
==9475==    by 0x54DF9B4: QListView::setSelection(QRect const&, QFlags<QItemSelectionModel::SelectionFlag>) (in /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5.7.1)
==9475==    by 0x54C6B3E: QAbstractItemView::mouseMoveEvent(QMouseEvent*) (in /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5.7.1)
==9475==    by 0x54E6386: QListView::mouseMoveEvent(QMouseEvent*) (in /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5.7.1)
==9475==    by 0x52B8277: QWidget::event(QEvent*) (in /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5.7.1)
==9475==    by 0x53A0A0D: QFrame::event(QEvent*) (in /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5.7.1)
==9475==    by 0x54C7502: QAbstractItemView::viewportEvent(QEvent*) (in /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5.7.1)
==9475==  Address 0x0 is not stack'd, malloc'd or (recently) free'd
==9475== 
==9475== 
==9475== Process terminating with default action of signal 11 (SIGSEGV)
==9475==  Access not within mapped region at address 0x0
==9475==    at 0x1158B5: UserListWidget::updateStatsPage() (in /home/alexis/Desktop/programming/uni_work/comp_2811/cw2/ui_cw3/2811_gui)
==9475==    by 0x5F5D5E8: QMetaObject::activate(QObject*, int, int, void**) (in /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.7.1)
==9475==    by 0x5F5D5E8: QMetaObject::activate(QObject*, int, int, void**) (in /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.7.1)
==9475==    by 0x5EE0BE6: QItemSelectionModel::selectionChanged(QItemSelection const&, QItemSelection const&) (in /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.7.1)
==9475==    by 0x5EE544A: QItemSelectionModel::emitSelectionChanged(QItemSelection const&, QItemSelection const&) (in /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.7.1)
==9475==    by 0x5EE8F91: QItemSelectionModel::select(QItemSelection const&, QFlags<QItemSelectionModel::SelectionFlag>) (in /usr/lib/x86_64-linux-gnu/libQt5Core.so.5.7.1)
==9475==    by 0x54DF9B4: QListView::setSelection(QRect const&, QFlags<QItemSelectionModel::SelectionFlag>) (in /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5.7.1)
==9475==    by 0x54C6B3E: QAbstractItemView::mouseMoveEvent(QMouseEvent*) (in /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5.7.1)
==9475==    by 0x54E6386: QListView::mouseMoveEvent(QMouseEvent*) (in /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5.7.1)
==9475==    by 0x52B8277: QWidget::event(QEvent*) (in /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5.7.1)
==9475==    by 0x53A0A0D: QFrame::event(QEvent*) (in /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5.7.1)
==9475==    by 0x54C7502: QAbstractItemView::viewportEvent(QEvent*) (in /usr/lib/x86_64-linux-gnu/libQt5Widgets.so.5.7.1)

1 Ответ

0 голосов
/ 08 декабря 2018

Похоже, что ошибка сегментации происходит в методе UserListWidget::updateStatsPage().

Учитывая ограниченную доступную информацию и то, что вы сказали, что эта проблема возникает, когда вы перетаскиваете мышь от элемента, который был первоначально нажат, Я подозреваю , что this->currentItem() возвращает нулевой указатель в строке кода, где метод вызывает this->currentItem()->text().Когда вы впервые нажимаете на элемент списка виджетов, я предполагаю, что метод UserListWidget::updateStatsPage() вызывается с ненулевым указателем, возвращаемым из this->currentItem().Но затем вы перетаскиваете мышь и, если вы перетаскиваете ее из текущего элемента, генерируется другой сигнал itemSelectionChanged().Если вы полностью утащили мышь из QListWidget, я думаю, что сигнал будет вызван в тот момент, когда this->currentItem() вернет нулевой указатель, указывающий, что ничего не выбрано.

Попробуйте проверить, является ли this->currentItem() является нулем и только разыменовывает его, если не ноль.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.

Нет похожих вопросов

...