loop for not waiting the end of process

sierdzio

If the process works, and you see the output file being created, then I'd say there is no problem at all... it works. If you mean that next loop iteration overwrites your process or something, then do as I suggested and create new QProcess instance inside the loop.

PLease note that waitForFinished will wait at most the specified amount of time. If process ends sooner, it will stop waiting. So if your process finishes extremely fast, you may not even notice any delay.

BTW. You probably should call waitForStarted() before calling waitForFinished().

filipdns

@sierdzio it almost work, the problem is only one output file is created, but it suppose to have many one (images.count() return 5)

using .bat with

for %%f in ("\\DGS1109N007\Echanges_SAA\Affichage\Messages\*.*") do ( \\DGS1109N007\Echanges_SAA\Affichage\Display\imagmagick\magick convert -density 300 -trim "\\DGS1109N007\Echanges_SAA\Affichage\Messages\*.*" -quality 100 "\\DGS1109N007\Echanges_SAA\Affichage\Display\Messages\output.jpg" )

in the ouput folder I got all 5 files named output.jpg, output-0.jpg, output-1.jpg etc...

SGaist

Hi,

You should try to properly build the argument list as a QStringList as show in QProcess' doc.

filipdns

@filipdns I will try to change the process to . in place of file name, I will see if it work but I doubt because in the .bat, I have to have for %%f in to make it convert all files, that why I pass trough loop for or foreach but without success

filipdns

@SGaist Hi SGaist, building with argument was my first attempt, but I have exactly the same result

filipdns

@filipdns

here how was my first code

program=QDir::toNativeSeparators("//DGS1109N007/Echanges_SAA/Affichage/Display/imagmagick/magick.exe ")
QStringList arguments;
arguments<< "convert -density 300 -trim "<<QDir::toNativeSeparators("\"//DGS1109N007/Echanges_SAA/Affichage/Messages/")+filename<<" -quality 100 "<<QDir::toNativeSeparators("\"//DGS1109N007/Echanges_SAA/Affichage/Display/Messages/output.jpg");

myProcess.start(program,arguments);

jsulm

@filipdns said in loop for not waiting the end of process:

arguments<< "convert -density 300 -trim "<<QDir::toNativeSeparators(""//DGS1109N007/Echanges_SAA/Affichage/Messages/")+filename<<" -quality 100 "<<QDir::toNativeSeparators(""//DGS1109N007/Echanges_SAA/Affichage/Display/Messages/output.jpg");

This is wrong. EVERY parameter needs to be an element in parameter list. Like:

arguments<< "convert" << "-density" << "300" << "-trim "<<QDir::toNativeSeparators("\"//DGS1109N007/Echanges_SAA/Affichage/Messages/")+filename << "-quality" << "100 "<<QDir::toNativeSeparators("\"//DGS1109N007/Echanges_SAA/Affichage/Display/Messages/output.jpg");

filipdns

@jsulm Hello, I try that but it's not working.

I solve my issue with:

        program=QDir::toNativeSeparators("//DGS1109N007/Echanges_SAA/Affichage/Display/imagmagick/magick ")+ "convert -density 300 -trim "+ QDir::toNativeSeparators("\"//DGS1109N007/Echanges_SAA/Affichage/Messages/"+filename)+"\" -quality 100 "+QDir::toNativeSeparators("\"//DGS1109N007/Echanges_SAA/Affichage/Display/Messages/"+outfilename+".jpg");

        if(myProcess.state()!= QProcess::Running)
        {
            myProcess.start(program);
            myProcess.waitForFinished(5000);
            i=i+1;
        }

thank you very much for you help!!

sierdzio

@filipdns said in loop for not waiting the end of process:

i=i+1;

That means you are skipping one image for every loop iteration.

if(myProcess.state()!= QProcess::Running)

And that means you may skip many more if process is still running. You could have avoided that if you created local QProcess inside the loop...

filipdns

@sierdzio hi, no it's not skipping any image but what do you mean with process inside the loop?

sierdzio

@filipdns said in loop for not waiting the end of process:

@sierdzio hi, no it's not skipping any image

OK then you probably have removed i++ from first line of your loop? Then that has another side effect - until your process exits you are forcing a tight, endless loop. That will eat your CPU, don't do it.

Also, if QProcess gets stuck for any reason, that loop will never exit.

but what do you mean with process inside the loop?

for (whatever) {
  QProcess process;
  process.start(program);
  process.waitForStarted(5000);
  if(process.waitForFinished(5000) == false)
    qInfo() << "Process had some trouble" << process.errorString() << program;
}

filipdns

@sierdzio Yes I removed the I++ from first line. I didn't know that will have effect on my cup, I will change that and see if your advise can do the job, thanks a lot

sierdzio

@filipdns said in loop for not waiting the end of process:

@sierdzio Yes I removed the I++ from first line. I didn't know that will have effect on my cup, I will change that and see if your advise can do the job, thanks a lot

You can check the endless loop easily. Print a message at the beginning of the loop:

for (...) {
  qDebug() << "Loop block begins";
}

If you see lots and lots of these lines (more than the number of images you have) then it's looping tightly as I mentioned in my last reply.

filipdns

@sierdzio I try to move the i=i+1 to the i++ in the line for(int i=0;i<images.count();i++) and it's working fine only if I keep the waitForFinished(5000) otherwise, I have infinit loop

    for(int i=0;i<images.count();i++) {
        //do whatever you need to do
        QString filename=images[i];
        QString outfilename=filename.split(".")[0];
        program2=QDir::toNativeSeparators("//DGS1109N007/Echanges_SAA/Affichage/Display/imagmagick/magick ")+ "convert -density 300 -trim "+ QDir::toNativeSeparators("\"//DGS1109N007/Echanges_SAA/Affichage/Messages/"+filename)+"\" -quality 100 "+QDir::toNativeSeparators("\"//DGS1109N007/Echanges_SAA/Affichage/Display/Messages/"+outfilename+".jpg");

        if(myProcess2.state()!= QProcess::Running)
        {
            myProcess2.start(program2);
            myProcess2.waitForFinished(5000);
            //i=i+1;
        }

    }

jsulm

@filipdns You still don't use QProcess inside the loop as suggested by @sierdzio

filipdns

@jsulm Hello, I seen this advise but I didn't understand how I have to do else than put the process in the loop

jsulm

@filipdns @sierdzio provided the code...

filipdns

@filipdns oh he is meaning the process declaration may be?

In this case ok, I try

filipdns

@filipdns I try the provided code and it didn't work...

sierdzio

@filipdns said in loop for not waiting the end of process:

@jsulm Hello, I seen this advise but I didn't understand how I have to do else than put the process in the loop

Well there's nothing else. Just put it inside. Then you will be certain 100% that next time the loop iterates, it will delete the old process and create a new, fresh one.