Set QVector data to QTreeWidgetItem

gangerM

The problem in your link is that 'programIcon' is only set when there is a new icon
This means that if an application has no icon it will use the icon that was there previously

I'll advice you to search some things about scope, pointers, references (and with that memory management) for c++
Maybe make a function that copies from 1 vector to another and after that use the QTreeWidgetItem. (So you can try some things)
I hope you can do something with this...
I can't really grasp what you are trying to do with the vector and I think somewhere around there is the problem.

Cobra91151

@gangerM

I think that I don't need vectors at all in this case, it just complicates the issue, all data works fine but the problem is with icons. When icon isNull() I set default icon but then it sets on all apps to default one. So I will change code back when all works and try to fix the issue with icons.

Cobra91151

The main post has been updated to illustrate the current issue.

Cobra91151

void Test::setAppData(QVector<AppsData> data)
{
    qDebug() << data.data()->name;
}

This code return only 1 name value: "HHD Software Free Hex Editor Neo 6.24".

jsulm

@Cobra91151 What do you expect it to return?
Why do you use data()?
Just do

void Test::setAppData(QVector<AppsData> data)
{
    for (AppsData element : data)
        qDebug() << element.name;
}

This will print out names of all elements in the vector. Alternative:


void Test::setAppData(QVector<AppsData> data)
{
    for (int i = 0; i < data.size(); ++i)
        qDebug() << data[i].name;
}

jsulm

@Cobra91151

void Test::setAppData(QVector<AppsData> data)
{
    for (int i = 0; i < data.count(); i++) {
        allApplicationsItem = new QTreeWidgetItem(allApplications);
        allApplicationsItem->setText(0, data[i].name); // I assume you want to set name, is this correct? You cannot just use the whole struct as you need to pass a string
    }
}

Cobra91151

@jsulm

Thanks. But I have figured it out how to retrieve data from QVector structure. The only problem is, it sets only one row. I have checked loop and it sets only one.

Code:

    AppsData appsData;
    QVector<AppsData> dataVector;

    for (QString key : allCurrentUserKeys) {
         if (!key.isEmpty()) {
            if (key.contains("DisplayIcon") && registryKeyCurrentUser.value(key).toString() != "") {
                displayIcon = registryKeyCurrentUser.value(key).toString();

                if (displayIcon.contains("\"")) {
                    displayIconSplitted = displayIcon.split("\"");
                    iconSplittedData = displayIconSplitted.value(displayIconSplitted.length() - 2);
                    iconFile = iconSplittedData;
                } else if (displayIcon.contains(",")) {
                    displayIconSplitted = displayIcon.split(",");
                    iconSplittedData = displayIconSplitted.value(displayIconSplitted.length() - 2);
                    iconFile = iconSplittedData;
                } else {
                    iconFile = displayIcon;
                }

                if (iconFile.contains(".ico")) {
                    programIcon = QIcon(iconFile);
                } else {
                    QFileInfo fileInfo(iconFile);
                    programIcon = fileIconProvider.icon(fileInfo);
                }

                appsData.icon = programIcon;
                //programIcon = setDefaultIcon();
            }

            if (key.contains("DisplayName") && registryKeyCurrentUser.value(key).toString() != "") {
                displayName = registryKeyCurrentUser.value(key).toString();
                appsData.name = displayName;
                //emit appData(programIcon, displayName);
            }
        }
    }

    dataVector.push_back(appsData);
    emit appData(dataVector);

For example there are 5 entries in registry but only latest is set to a QVector structure. How to get all registry entries?

jsulm

@Cobra91151 said in Set QVector data to QTreeWidgetItem:

dataVector.push_back(appsData);

Shouldn't

dataVector.push_back(appsData);

be inside the loop?


    QVector<AppsData> dataVector;

    for (QString key : allCurrentUserKeys) {
         AppsData appsData;
         if (!key.isEmpty()) {
            if (key.contains("DisplayIcon") && registryKeyCurrentUser.value(key).toString() != "") {
                displayIcon = registryKeyCurrentUser.value(key).toString();

                if (displayIcon.contains("\"")) {
                    displayIconSplitted = displayIcon.split("\"");
                    iconSplittedData = displayIconSplitted.value(displayIconSplitted.length() - 2);
                    iconFile = iconSplittedData;
                } else if (displayIcon.contains(",")) {
                    displayIconSplitted = displayIcon.split(",");
                    iconSplittedData = displayIconSplitted.value(displayIconSplitted.length() - 2);
                    iconFile = iconSplittedData;
                } else {
                    iconFile = displayIcon;
                }

                if (iconFile.contains(".ico")) {
                    programIcon = QIcon(iconFile);
                } else {
                    QFileInfo fileInfo(iconFile);
                    programIcon = fileIconProvider.icon(fileInfo);
                }

                appsData.icon = programIcon;
                //programIcon = setDefaultIcon();
            }

            if (key.contains("DisplayName") && registryKeyCurrentUser.value(key).toString() != "") {
                displayName = registryKeyCurrentUser.value(key).toString();
                appsData.name = displayName;
                //emit appData(programIcon, displayName);
            }
        }
        dataVector.push_back(appsData);
    }

    
    emit appData(dataVector);

Cobra91151

@jsulm

It should be, I have tried it also, but it returns multiple the same values.

Screenshot:
alt text

jsulm

@Cobra91151 You should move

dataVector.push_back(appsData);

into

if (!key.isEmpty()) {
            if (key.contains("DisplayIcon") && registryKeyCurrentUser.value(key).toString() != "") {

So

QVector<AppsData> dataVector;

    for (QString key : allCurrentUserKeys) {
         AppsData appsData;
         if (!key.isEmpty()) {
            if (key.contains("DisplayIcon") && registryKeyCurrentUser.value(key).toString() != "") {
                displayIcon = registryKeyCurrentUser.value(key).toString();

                if (displayIcon.contains("\"")) {
                    displayIconSplitted = displayIcon.split("\"");
                    iconSplittedData = displayIconSplitted.value(displayIconSplitted.length() - 2);
                    iconFile = iconSplittedData;
                } else if (displayIcon.contains(",")) {
                    displayIconSplitted = displayIcon.split(",");
                    iconSplittedData = displayIconSplitted.value(displayIconSplitted.length() - 2);
                    iconFile = iconSplittedData;
                } else {
                    iconFile = displayIcon;
                }

                if (iconFile.contains(".ico")) {
                    programIcon = QIcon(iconFile);
                } else {
                    QFileInfo fileInfo(iconFile);
                    programIcon = fileIconProvider.icon(fileInfo);
                }

                appsData.icon = programIcon;
                //programIcon = setDefaultIcon();
               if (key.contains("DisplayName") && registryKeyCurrentUser.value(key).toString() != "") {
                displayName = registryKeyCurrentUser.value(key).toString();
                appsData.name = displayName;
                //emit appData(programIcon, displayName);
              }
              dataVector.push_back(appsData);
            }
       }
    }

    emit appData(dataVector);

Cobra91151

@jsulm

Yes, but the apps data will not be in appropriate columns.

Screenshot:
alt text

Code:

for (int i = 0; i < vectorData.count(); i++) {
      allApplicationsItem = new QTreeWidgetItem(allApplications);
      allApplicationsItem->setIcon(0, vectorData[i].icon);
      allApplicationsItem->setText(0, vectorData[i].name);
      allApplicationsItem->setText(1, vectorData[i].version);
      allApplicationsItem->setText(2, vectorData[i].publisher);
      allApplicationsItem->setText(3, vectorData[i].installLocation);
      allApplicationsItem->setText(4, vectorData[i].uninstallLocation);
}

jsulm

@Cobra91151 It looks like you don't get icons for all apps - you need to fix your code, I currently don't have time for this.

Cobra91151

@jsulm

In this case I have all apps icons but we dataVector.push_back(appsData); setting in if (key.contains("DisplayName") or icons key. That's why this issue, one first row is empty.

I have changed to dataVector.push_back(appsData); on the last check:

if (key.contains("UninstallString") && registryKeyCurrentUser.value(key).toString() != "" && !key.contains("QuietUninstallString")) {
    uninstallLocation = registryKeyCurrentUser.value(key).toString();
    appsData.uninstallLocation = uninstallLocation;
    dataVector.push_back(appsData);
}

And now it setting all data properly, but I will check for other registry keys.

Screenshot (HKEY_CURRENT_USER):
alt text

Cobra91151

The same issue with icons still exists. It's time to change code to Win API and try to fix this issue.