String Split

jondoe

Somebody can help me please I dont know how can I code this split and loop :(

aha_1980

QString files = "test1.dll,test2.dll,test3.dll";
const QStringList list = files.split(',');
for (const QString &name : list {
  qDebug() << name;
}

jondoe

@aha_1980

How can I use this with download function ?

Gojir4

@jondoe said in String Split:

@aha_1980

How can I use this with download function ?

for (const QString &name : list {
  downloadFile(name);
}

aha_1980

@jondoe

Try this:

QString dlls = "test1.dll,test2.dll,test3.dll";
const QStringList list = dlls.split(',');

for (const QString &name : list) {
  const QString url = QString("http://sitename.com/dlls/") + name;
  const QString file = QString("C:\\Users\\user\\Desktop\\testfile\\") + name;
  qDebug() << "Downloading" << url << "to" << file << "...";
  HRESULT hr = URLDownloadToFileW(0, (LPCWSTR)url.utf16(), (LPCWSTR)file.utf16(), 0, 0);
}

jondoe

@Gojir4

I tried like this but I got error.

QString getdlls = "http://sitename/dlls.php";
  QByteArray dllssonuc = QResolveUrl(getdlls);

  //qDebug() << "testedek" << dllssonuc;

  QString files = dllssonuc;
  const QStringList list = files.split(',');
  for (const QString &name : list) {
    //qDebug() << "deneyelim" << name;

      if (name != "")
      {
    HRESULT hr;
        LPCTSTR Url = _T("http://sitename.com/dlls/" + name), File = _T("C:\\Users\\user\\Desktop\\testklasor\\" + name);
        hr = URLDownloadToFile (0, Url, File, 0, 0);
        switch (hr)
            {
                case S_OK:
                    qDebug() << "Successful download\n";
                    break;
                case E_OUTOFMEMORY:
                    qDebug() << "Out of memory error\n";
                    break;
                case INET_E_DOWNLOAD_FAILURE:
                    qDebug() << "Cannot access server data\n";
                    break;
                default:
                    qDebug() << "Unknown error\n";
                    break;
            }
        }


  }

jondoe

@aha_1980

Error like this -> invalid conversion from 'const wchar_t*' to 'char' [-fpermissive] LPCTSTR Url

TobbY

Hi,

you need to do conversion for that. please refer this code if that help.

Required Libs

#include <Windows.h>
#include <urlmon.h>
#include <Wininet.h>
#pragma comment(lib, "urlmon.lib")
#pragma comment(lib,"wininet.lib")

To Query DLLs

    QString data = "test1.dll,test2.dll,test3.dll";
    QStringList list = data.split(",");
    for (const QString &name : list ) {
        if (name != "") downloadFile(name);
    }

To Download DLL

void MainWindow::downloadFile(QString name)
{
    HRESULT hr;
    LPCTSTR Url = (LPTSTR)QString("http://sitename.com/dlls/" + name).utf16(); // Convert QString to LPCTSTR 
    LPCTSTR File = (LPTSTR)QString("C:\\Users\\user\\Desktop\\testklasor\\" + name).utf16();
    hr = URLDownloadToFile (0, Url, File, 0, 0);
    switch (hr)
    {
    case S_OK:
        qDebug() << "Successful download\n";
        break;
    case E_OUTOFMEMORY:
        qDebug() << "Out of memory error\n";
        break;
    case INET_E_DOWNLOAD_FAILURE:
        qDebug() << "Cannot access server data\n";
        break;
    default:
        qDebug() << "Unknown error\n" << hr;
        break;
    }
}

jondoe

@TobbY

Thank you for answer but debug like this.

Unknown error
-2146697211
Unknown error
-2146697211

TobbY

@TobbY said in String Split:

URLDownloadToFile

Hi,
Have you tried it with correct url and dlls ?

jondoe

@TobbY yes but still same error.

TobbY

Hi,
Please check error code. INET_E_RESOURCE_NOT_FOUND (0x800C0005L or -2146697211)

https://msdn.microsoft.com/en-us/library/bb268233(v=vs.85).aspx

aha_1980

@jondoe if you don't copy my code 1:1, you'll have to fix yourself.

Cobra91151

@aha_1980 said in String Split:

@jondoe

Try this:

QString dlls = "test1.dll,test2.dll,test3.dll";
const QStringList list = dlls.split(',');

for (const QString &name : list {
  const QString url = QString("http://sitename.com/dlls/") + name;
  const QString file = QString("C:\\Users\\user\\Desktop\\testfile\\") + name;
  qDebug() << "Downloading" << url << "to" << file << "...";
  HRESULT hr = URLDownloadToFileW(0, (LPCWSTR)url.utf16(), (LPCWSTR)file.utf16(), 0, 0);
}

Hi! You forgot closing ) in the for loop)).

aha_1980

@Cobra91151 said in String Split:

Hi! You forgot closing ) in the for loop)).

Psst. Don't tell anyone. It's part of the homework ;)

VRonin

If you are using Qt I don't see why you have to complicate your life using LPCTSTR and stuff like that.

const QString dlls = "test1.dll,test2.dll,test3.dll";
const QStringList list = dlls.split(',');
QEventLoop waitLoop;
int dllsToDownLoad =  list.size();
auto manager = new QNetworkAccessManager;
for (const QString &name : list){
    auto destinationFile = new QFile("C:/Users/user/Desktop/testfile/" + name);
    if (!destinationFile.open(QFile::WriteOnly)){
        --dllsToDownLoad ;
        continue;
    }
    QNetworkReply* reply = manager->get(QNetworkRequest(QUrl::fromUserInput("http://sitename.com/dlls/" + name)));
    QObject::connect(reply, &QNetworkReply::readyRead(), [destinationFile, reply]() -> void {
        destinationFile->write(reply->readAll());
    });
    QObject::connect(reply, &QNetworkReply::finished(), [&dllsToDownLoad, reply, &waitLoop]() -> void {
        reply->deleteLater();
        destinationFile->deleteLater();
        if (--dllsToDownLoad == 0){
            manager->deleteLater();
            waitLoop.quit();
        }
    });
}
waitLoop.exec();
}