Utilisation de system()

Version imprimable

Bonjour,

je suis en train de mettre au point une petite interface graphique pour des besoins spécifiques. Le but est de faciliter l'utilisation de FFmpeg: ne pas avoir à taper les commandes à la main.

J'ai donc fait mon application en C++ et Qt. Sous Linux tout fonctionne bien les commandes sont générées et exécutées correctement.

Mais sous Windows, pour une raison que je ne connais pas, aucune commande ne passe avec un system("...") ... sauf system("pause") qui elle marche. J'ai essayé de prendre la chaine de caractère générée qui correspond à la commande et de la coller directement dans l'invité de commande: ça marche très bien. :aie:

Voici le petit bout de code, qui fait les exécutions de commandes pour la HD (High Definition):
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 14 if (needHd) { for(int i=0;i<nbrCommandsHd;i++) { currentCommand ++; tempCommand = commandsEncode.commandsHd[i].commandline; generateCommand(&tempCommand, pathApp, pathInfile, basename, timePreview, idHd); //std::cout << tempCommand << std::endl; commandResultCurrent = system(StringToConstChar(tempCommand)); system("pause"); commandResultHdOk = commandResultHdOk && (commandResultCurrent==0); progressbarEncode->setValue(getPercentage(i + 1, nbrCommands)); } }
Si vous avez une explication ou une piste de réflexion, n'hésitez pas !

Thx.

10/02/2011, 00h09
Mat007

Salut,

Que vaut le résultat de StringToConstChar(tempCommand) (voire quelle est l'implémentation de la fonction, même) ?

MAT.
10/02/2011, 09h14
Raphyy
Voici un exemple de ligne de commande générée pour Linux:

Citation:

/home/raphy/Workspace/Netbeans/TYPVideoEncoder/ffmpeg/ffmpeg -y -threads 8 -i /home/raphy/Bureau/short.mp4 -s 1280x720 -aspect 16:9 -r 24 -b 1536k -bt 1792k -vcodec libx264 -pass 2 -vpre /home/raphy/Workspace/Netbeans/TYPVideoEncoder/ffmpeg/hq.ffpreset -acodec libfaac -ac 2 -ar 44100 -ab 192k /home/raphy/Bureau/shortHD.flv

Voici un exemple de ligne de commande générée pour Windows, qui elle marche quand je teste à la main, mais pas avec system() :

Citation:

"C:\Documents and Settings\Administrateur\Mes documents\NetBeansProjects\TYPVideoEncoder\ffmpeg\ffmpeg.exe" -y -threads 8 -i "C:\short.mp4" -s 1280x720 -aspect 16:9 -r 24 -b 1536k -bt 1792k -vcodec libx264 -pass 2 -vpre "C:\Documents and Settings\Administrateur\Mes documents\NetBeansProjects\TYPVideoEncoder\ffmpeg\hq.ffpreset" -acodec libfaac -ac 2 -ar 44100 -ab 192k "C:\shortHD.flv"

Et enfin le code de StringToConstChar :
Code:

1 2 3 4 5 6 7 const char * StringToConstChar(std::string text) { size_t size = text.size() + 1; char * buffer = new char[size]; strncpy(buffer,text.c_str(),size); return buffer; }
10/02/2011, 10h41
Mat007

La fonction StringToConstChar ne sert à rien (et elle fuit de la mémoire), utilise directement text.c_str(), c'est fait pour ça.

Sinon il faut que les \ soient doublés dans les chaînes parce que ça sert à échapper les caractères spéciaux.

MAT.
10/02/2011, 11h10
JolyLoic

Je n'ai pas regardé ton problème spécifique, mais globalement, system est rarement l'idéal pour ce genre de fonction, du moins sous windows. En effet, il existe d'autres API qui renvoient un handle sur le process lancé, ce qui permet entre autre de le tuer aisément, par exemple sur un bouton cancel pressé par l'utilisateur.
10/02/2011, 11h42
Raphyy

@Mat007 : j'ai suivi ton conseil en utilisant .c_str(). Pour les caractères "\", je les ai effectivement doublés pour l'échappement, de même j'en place aussi sous Linux pour gérer les espaces. Peux-tu m'en dire plus sur la fuite de mémoire ?

@JolyLoic : je suis en train de faire des recherches là-dessus, si tu as des choses précises en tête, je suis preneur !
10/02/2011, 11h55
Mat007

Citation:

Envoyé par Raphyy

Peux-tu m'en dire plus sur la fuite de mémoire ?

StringToConstChar fait un new et y'a jamais de delete.

MAT.
16/02/2011, 10h28
Raphyy
Après quelques tests supplémentaires, j'ai trouvé d'où vient le problème.
Pour exécuter cette commande :

Citation:

"C:\Documents and Settings\Administrateur\Mes documents\NetBeansProjects\TYPVideoEncoder\ffmpeg\ffmpeg.exe" -y -threads 8 -i "C:\short.mp4" -s 1280x720 -aspect 16:9 -r 24 -b 1536k -bt 1792k -vcodec libx264 -pass 2 -vpre "C:\Documents and Settings\Administrateur\Mes documents\NetBeansProjects\TYPVideoEncoder\ffmpeg\hq.ffpreset" -acodec libfaac -ac 2 -ar 44100 -ab 192k "C:\shortHD.flv"

Il faut rajouter des guillemets pour que l'appel avec system() fonctionne :

Citation:

" "C:\Documents and Settings\Administrateur\Mes documents\NetBeansProjects\TYPVideoEncoder\ffmpeg\ffmpeg.exe" -y -threads 8 -i "C:\short.mp4" -s 1280x720 -aspect 16:9 -r 24 -b 1536k -bt 1792k -vcodec libx264 -pass 2 -vpre "C:\Documents and Settings\Administrateur\Mes documents\NetBeansProjects\TYPVideoEncoder\ffmpeg\hq.ffpreset" -acodec libfaac -ac 2 -ar 44100 -ab 192k "C:\shortHD.flv" "

Maintenant que j'ai validé le principe de mon projet: ça fonctionne. Je vais pour faire quelque chose de plus propre: ne pas utiliser system().

Quelques ressources intéressantes trouvées lors de mes recherches, ça peut rendre service :
- Pourquoi il ne faut pas utiliser system() : http://www.cplusplus.com/forum/articles/11153/
- Solution de remplacement pour Unix : http://pubs.opengroup.org/onlinepubs...ions/exec.html
- Solution de remplacement pour Windows : http://msdn.microsoft.com/en-us/libr...8VS.85%29.aspx
Att.
16/02/2011, 17h54
darkrojo

L'utilisation de system() me fait bien rire alors que tu utilises la framework Qt qui contient une sous-partie nommée QProcess, spécialisée dans la gestion de processus séparés ;)
16/02/2011, 21h09
Raphyy

Citation:

tu utilises la framework Qt qui contient une sous-partie nommée QProcess

Dieu soit en location !

Trêve de blagues, je suis totalement débutant avec Qt, donc je ne connaissais pas cette possibilité. Tu fais bien de me le dire, c'est le genre de trucs qui m'aide ! ;) Je vais jeter un œil là-dessus très prochainement.

D'ailleurs tant que j'y suis, je bosse avec Netbeans et un dépôt Subversion ... jusque là ça va, tout marche parfaitement. J'ai donc mis à jour mon projet Netbeans (sous Windows) avec le code du dépôt ... et là, message d'insulte à la compilation:

Citation:

"/usr/bin/make" -f nbproject/Makefile-Debug.mk QMAKE=/C/Qt/2010.05/qt/bin/qmake.exe SUBPROJECTS= .build-conf
make[1]: Entering directory `/c/Documents and Settings/Administrateur/Mes documents/NetBeansProjects/TakeYourPillsVideoEncoder'
/C/Qt/2010.05/qt/bin/qmake.exe VPATH=. -o qttmp-Debug.mk nbproject/qt-Debug.pro
mv -f qttmp-Debug.mk nbproject/qt-Debug.mk
"/usr/bin/make" -f nbproject/qt-Debug.mk dist/Debug/MinGW_1-Windows/TYPVideoEncoder.exe
make[2]: Entering directory `/c/Documents and Settings/Administrateur/Mes documents/NetBeansProjects/TakeYourPillsVideoEncoder'
C:/Qt/2010.05/qt/binmoc.exe -DUNICODE -DQT_LARGEFILE_SUPPORT -DQT_DLL -DQT_NO_DEBUG -DQT_GUI_LIB -DQT_CORE_LIB -DQT_THREAD_SUPPORT -DQT_NEEDS_QMAIN -I"c:/Qt/2010.05/qt/include/QtCore" -I"c:/Qt/2010.05/qt/include/QtGui" -I"c:/Qt/2010.05/qt/include" -I"c:/Qt/2010.05/qt/include/ActiveQt" -I"." -I"nbproject" -I"." -I"c:/Qt/2010.05/qt/mkspecs/default" -D__GNUC__ -DWIN32 form.h -o moc_form.cpp
make[2]: Leaving directory `/c/Documents and Settings/Administrateur/Mes documents/NetBeansProjects/TakeYourPillsVideoEncoder'
make[1]: Leaving directory `/c/Documents and Settings/Administrateur/Mes documents/NetBeansProjects/TakeYourPillsVideoEncoder'
/bin/sh: C:/Qt/2010.05/qt/binmoc.exe: No such file or directory
make[2]: *** [moc_form.cpp] Error 127
make[1]: *** [.build-conf] Error 2
make: *** [.build-impl] Error 2

BUILD FAILED (exit value 2, total time: 1s)

Ce qui me tue dans les messages précédents, c'est le :

Citation:

/bin/sh: C:/Qt/2010.05/qt/binmoc.exe: No such file or directory

J'ai pas la moindre idée de ce qu'il faut faire pour corriger le problème ! 8O
17/02/2011, 11h49
Raphyy

La solution est ici : http://forums.netbeans.org/post-88590.html

Ça s'invente pas ...