ShellExecute i iconv.exe problem z przekazaniem parametrów

[Darek_C++]

Kolejnym sposobem na zmianę kodowania znaków z użyciem iconv programu jest program iconv.exe i wywoływany z lini poleceń np:
cd E:\iconv
E:\iconv>iconv.exe -f ISO-8859-2 -t UTF-8 <test.txt> wyjscie.txt
i tak działa. Działa również przy wywołaniu w programie funkcji:

Kod: Zaznacz cały

system("E:\\iconv\\iconv.exe -f ISO-8859-2 -t UTF-8 E:\\iconv\\test.txt > E:\\iconv\\wyjscie2.txt");

ale za nic nie chce zadziałać wywołany z

Kod: Zaznacz cały

ShellExecute(Form1->Handle, NULL , "E:\\iconv\\iconv.exe", "-f ISO-8859-2 -t UTF-8 E:\\iconv\\test.txt > E:\\iconv\\wyjscie3.txt", "", SW_SHOWNORMAL);

tak samo dzieje się gdy podaje argumenty do programu Wyświetlanie zawartości konsoli CMD.exe w Memo - wg. pomysłu kinio.
[Scieżka do procesu] E:\iconv\iconv.exe
[Parametry wywolania] -f ISO-8859-2 -t UTF-8 E:\iconv\test.txt > E:\iconv\wyjscie3.txt
[Zwracane jest]Invalid argument
No such file or directory
-----------
Najbarzdiej zależało by mi na możliwości uruchomienia działania programu funkcją CreateProcess której jak myślę odpowiednia wywołanie spowoduje oczekiwanie programu głównego na zakończenie wywołania danego procesu co ma kluczowy związek z jego dalszym działaniem.

W załączniku zamieszczam program iconv.exe jak by ktoś z Was chciał się zmierzyć z tym problemem
Pozdrawiam

[Cyfrowy Baron]

Spróbuj za pomocą funkcji CreateProcess:

Kod: Zaznacz cały


 String eValue ="E:\\iconv\\iconv.exe";
  
  String val1 = "-f ISO-8859-2 -t UTF-8 E:\\iconv\\test.txt > E:\\iconv\\wyjscie3.txt";

 if(!FileExists(eValue)) return; // jeżeli brak pliku to przerwij
 eValue = "\"" + eValue + "\" \"" + val1 + "\" \""; // program i parametr zostają ujęte w cudzysłowa.

 STARTUPINFO StartupInfo;
 ;ZeroMemory( &StartupInfo, sizeof(STARTUPINFO));
 StartupInfo.cb = sizeof(STARTUPINFO);

 PROCESS_INFORMATION ProcessInfo;

 if(CreateProcess(NULL, eValue.w_str(), NULL, NULL, true, NORMAL_PRIORITY_CLASS,
                  NULL, NULL, &StartupInfo, &ProcessInfo))
 {
  CloseHandle(ProcessInfo.hProcess);
  CloseHandle(ProcessInfo.hThread);
 }
 

[Darek_C++]

Niestety, ale też nie działa
---
Jak dotąd zadziałało tylko przy wywołaniu funkcją system

[Cyfrowy Baron]

Spróbuj podanego przeze mnie sposobu, ale ujmij każdy element linii komend w oddzielny cudzysłów, czyli:

Kod: Zaznacz cały


  String val1 = "\"-f\" \"ISO-8859-2\" \"-t UTF-8\" \"E:\\iconv\\test.txt >\" \"E:\\iconv\\wyjscie3.txt\"";

  if(!FileExists(eValue)) return;
  eValue = "\"" + eValue + "\" " + val1;
 

[Darek_C++]

Dałem:

Kod: Zaznacz cały

String eValue ="E:\\iconv\\iconv.exe";
   String val1 = "\"-f\" \"ISO-8859-2\" \"-t UTF-8\" \"E:\\iconv\\test.txt >\" \"E:\\iconv\\wyjscie3.txt\"";
   if(!FileExists(eValue)) return; // jeżeli brak pliku to przerwij
   eValue = "\"" + eValue + "\" \"" + val1 + "\" \""; // program i parametr zostają ujęte w cudzysłowa.

I też nie działa

[Cyfrowy Baron]

Ale ten element:

Kod: Zaznacz cały


   if(!FileExists(eValue)) return; // jeżeli brak pliku to przerwij
   eValue = "\"" + eValue + "\" \"" + val1 + "\" \""; // program i parametr zostają ujęte w cudzysłowa.
 

powinieneś teraz zapisać tak:

Kod: Zaznacz cały


   if(!FileExists(eValue)) return; // jeżeli brak pliku to przerwij
   eValue = "\"" + eValue + "\" " + val1


Nie należy ponownie ujmować zmiennej val1 w cudzusłowa, gdyż pojawią się podwójne.

Poza tym ten element:

Kod: Zaznacz cały

String val1 = "\"-f\" \"ISO-8859-2\" \"-t UTF-8\" \"E:\\iconv\\test.txt >\" \"E:\\iconv\\wyjscie3.txt\""; 

też źle rozpisałeś, gdyż chodzi o to, że każdy element oddzielony spacją należy ująć w cudzuysłowa. Powinno być:

Kod: Zaznacz cały

String val1 = "\"-f\" \"ISO-8859-2\" \"-t\" \"UTF-8\" \"E:\\iconv\\test.txt>\" \"E:\\iconv\\wyjscie3.txt\""; 

między -t i UTF-8 zabrakło cudzysłowów. Poprawiłem też ten element: E:\\iconv\\test.txt> na E:\\iconv\\test.txt>

Spróbuj z poprawkami.

[Cyfrowy Baron]

Spróbuj może za pomocą pliku wsadowego *.BAT. Niech program zapisze dane do pliku wsadowego a następnie uruchom ten plik.

---

Symbol > użyty jako parametr przekazywany do programu nie ma zastosowania poza konsolą CMD, spróbuj więc bez tych symboli:

Kod: Zaznacz cały



 String eValue ="E:\\iconv\\iconv.exe";
  
 String val1 = "-f ISO-8859-2 -t UTF-8 E:\\iconv\\test.txt E:\\iconv\\wyjscie3.txt";

if(!FileExists(eValue)) return;
 eValue = "\"" + eValue + "\" \"" + val1 + "\" \"";

 

[polymorphism]

Panowie, problem polega na tym, że macie do czynienia z przekierowaniem standardowego wejścia/wyjścia na pliki, a to - chodzi o zapis - ma zastosowanie pod konsolą.

Tu macie kod przykładowy, nieładny, bo spłodzony na szybko, ale mam nadzieję, że tłumaczący idee:

Kod: Zaznacz cały

    SECURITY_ATTRIBUTES sa;
    sa.nLength              =   sizeof  (   SECURITY_ATTRIBUTES);
    sa.lpSecurityDescriptor =   NULL;
    sa.bInheritHandle       =   TRUE;

    STARTUPINFO si;
    ZeroMemory( &si, sizeof(STARTUPINFO));

    si.cb = sizeof(STARTUPINFO);
    si.dwFlags = STARTF_USESTDHANDLES;

    /* wejście */
    si.hStdInput = CreateFile("D:\\iconv\\test.txt",
                GENERIC_READ,
                FILE_SHARE_READ,
                &sa,
                OPEN_EXISTING,
                FILE_ATTRIBUTE_NORMAL,
                NULL);

    /* wyjście */
    si.hStdOutput = CreateFile("D:\\iconv\\out.txt",
                GENERIC_WRITE,
                FILE_SHARE_READ | FILE_SHARE_WRITE,
                &sa,
                CREATE_ALWAYS,
                FILE_ATTRIBUTE_NORMAL,
                NULL);

    PROCESS_INFORMATION ProcessInfo;

    if(CreateProcess(NULL,
                    "D:\\iconv\\iconv.exe -f ISO-8859-2 -t UTF-8", 
                    NULL, 
                    NULL, 
                    TRUE, 
                    NORMAL_PRIORITY_CLASS, 
                    NULL, 
                    NULL, 
                    &si, 
                    &ProcessInfo))
    {
        WaitForSingleObject(ProcessInfo.hProcess,INFINITE); //<--- czekamy...
        CloseHandle(ProcessInfo.hProcess);
        CloseHandle(ProcessInfo.hThread);
    }

    CloseHandle(si.hStdInput);
    CloseHandle(si.hStdOutput); 

[Cyfrowy Baron]

Rzeczywiście, teraz gdy to napisałeś, jest to oczywiste. Ja osobiście traktowałem to jako parametr przekazywany do programu i w zasadzie nie zastanawiałem się nad tym co ten program ma zrobić.

[Darek_C++]

Brawo Panie polymorphism - w końcu zadziałało.
Jeszcze chciał bym się jeżeli można dowiedzieć jak uruchomić program by wyeliminować mignięcie okna konsoli jakie się pojawia przy wykonywaniu tego kodu ?

[polymorphism]

Poczytaj w dokumentacji o STARTUPINFO, tam chyba jest coś o tym.

[Cyfrowy Baron]

Struktura STARTUPINFO posiada element wShowWindow, jeżeli przypisze mu wartość SW_HIDE to okno konsoli uruchamiane za pomocą funkcji CreateProcess zostanie uruchomione jako ukryte, czyli niewidoczne. Element wShowWindow jest brany pod uwagę tylko wtedy, gdy inny element tej struktury - dwFlags ma wartość STARTF_USESHOWWINDOW, w przeciwnym razie element wShowWindow jest ignorowany.

Kod: Zaznacz cały


 STARTUPINFO StartupInfo;
 ;ZeroMemory( &StartupInfo, sizeof(STARTUPINFO));
 StartupInfo.cb = sizeof(STARTUPINFO);
 StartupInfo.dwFlags = STARTF_USESHOWWINDOW;
 StartupInfo.wShowWindow = SW_HIDE;
 

Czyli w przypadku kodu podanego przez polymorphism, powinno może to wyglądać tak:

Kod: Zaznacz cały


SECURITY_ATTRIBUTES sa;
sa.nLength              =   sizeof  (   SECURITY_ATTRIBUTES);
sa.lpSecurityDescriptor =   NULL;
sa.bInheritHandle       =   TRUE;

STARTUPINFO si;
ZeroMemory( &si, sizeof(STARTUPINFO));
si.cb = sizeof(STARTUPINFO);

si.dwFlags     = STARTF_USESTDHANDLES;
si.wShowWindow = SW_HIDE;                  // <-- patrz tutaj

/* wejście */
si.hStdInput = CreateFile("D:\\iconv\\test.txt",
                    GENERIC_READ,
                    FILE_SHARE_READ,
                    &sa,
                    OPEN_EXISTING,
                    FILE_ATTRIBUTE_NORMAL,
                    NULL);

/* wyjście */
si.hStdOutput = CreateFile("D:\\iconv\\out.txt",
                    GENERIC_WRITE,
                    FILE_SHARE_READ | FILE_SHARE_WRITE,
                    &sa,
                    CREATE_ALWAYS,
                    FILE_ATTRIBUTE_NORMAL,
                    NULL);

PROCESS_INFORMATION ProcessInfo;

if(CreateProcess(NULL, "D:\\iconv\\iconv.exe -f ISO-8859-2 -t UTF-8", 
                        NULL, 
                        NULL, 
                        TRUE, 
                        NORMAL_PRIORITY_CLASS, 
                        NULL, 
                        NULL, 
                        &si, 
                        &ProcessInfo))
{
 WaitForSingleObject(ProcessInfo.hProcess, INFINITE); //<--- czekamy...
 CloseHandle(ProcessInfo.hProcess);
 CloseHandle(ProcessInfo.hThread);
}

CloseHandle(si.hStdInput);
CloseHandle(si.hStdOutput); 
 

[Darek_C++]

Jest coś takiego jak si.wShowWindow = SW_HIDE; ale dodanie tego parametru nic nie zmienia i nadal pojawia się na chwilę okno konsoli
-------
Znalazłem musi być tak:
si.dwFlags = STARTF_USESHOWWINDOW | STARTF_USESTDHANDLES;
si.wShowWindow = SW_HIDE;
-------

Jeszcze jedna kwestia mnie nurtuje z którą raczej sam sobie nie poradzę czy można zmienić standardowe wyjście hStdOutput w tym wypadku z zapisania do pliku na zmianną , lub TMemo ?

[Cyfrowy Baron]

To raczej niezwykłe, gdyż u mnie to działa tak jak podałem. Poza tym w pomocy jest podane:

STARTF_USESHOWWINDOW - The wShowWindow member is valid.

STARTF_USESTDHANDLES - The hStdInput, hStdOutput, and hStdError members are valid.

Wynika więc z tego, że wartość STARTF_USESTDHANDLES nie ma nic wspólnego z elementem wShowWindow, gdyż go nie dotyczy. Poza tym mozemy również przeczytać:

wShowWindow If dwFlags specifies STARTF_USESHOWWINDOW, this member can be any of the SW_ constants defined in Winuser.h. Otherwise, this member is ignored. For GUI processes, wShowWindow specifies the default value the first time ShowWindow is called. The nCmdShow parameter of ShowWindow is ignored. In subsequent calls to ShowWindow, the wShowWindow member is used if the nCmdShow parameter of ShowWindow is set to SW_SHOWDEFAULT.

---

po chwili...

Już wiem gdzie tkwi błąd. Powinno być tak jak podałem ja, a nie Ty, czyli wartość STARTF_USESHOWWINDOW, tylko, że w kodzie polymorphism mamy podaną wartość STARTF_USESTDHANDLES, stąd ten błąd, poprawiony kod powinien wyglądać tak:

Kod: Zaznacz cały


SECURITY_ATTRIBUTES sa;
sa.nLength              =   sizeof  (   SECURITY_ATTRIBUTES);
sa.lpSecurityDescriptor =   NULL;
sa.bInheritHandle       =   TRUE;

STARTUPINFO si;
ZeroMemory( &si, sizeof(STARTUPINFO));
si.cb = sizeof(STARTUPINFO);

si.dwFlags     = STARTF_USESHOWWINDOW | STARTF_USESTDHANDLES; // <-- tutaj zmieniłem
si.wShowWindow = SW_HIDE;                  // <-- patrz tutaj

/* wejście */
si.hStdInput = CreateFile("D:\\iconv\\test.txt",
                    GENERIC_READ,
                    FILE_SHARE_READ,
                    &sa,
                    OPEN_EXISTING,
                    FILE_ATTRIBUTE_NORMAL,
                    NULL);

/* wyjście */
si.hStdOutput = CreateFile("D:\\iconv\\out.txt",
                    GENERIC_WRITE,
                    FILE_SHARE_READ | FILE_SHARE_WRITE,
                    &sa,
                    CREATE_ALWAYS,
                    FILE_ATTRIBUTE_NORMAL,
                    NULL);

PROCESS_INFORMATION ProcessInfo;

if(CreateProcess(NULL, "D:\\iconv\\iconv.exe -f ISO-8859-2 -t UTF-8", 
                        NULL, 
                        NULL, 
                        TRUE, 
                        NORMAL_PRIORITY_CLASS, 
                        NULL, 
                        NULL, 
                        &si, 
                        &ProcessInfo))
{
 WaitForSingleObject(ProcessInfo.hProcess, INFINITE); //<--- czekamy...
 CloseHandle(ProcessInfo.hProcess);
 CloseHandle(ProcessInfo.hThread);
}

CloseHandle(si.hStdInput);
CloseHandle(si.hStdOutput); 
  

Ta druga wartość podana przez ciebie czyli STARTF_USESTDHANDLES jest potrzebna dla elementów hStdInput i hStdOutput, gdyż wypełniasz zarówno element wShowWindow jak też elementy hStdInput i hStdOutput.

Drobne przeoczenie, ale teraz wszystko się zgadza i nie ma w tym niczego niezwykłego.

[Darek_C++]

A jak by wyglądało z tym:
Jeszcze jedna kwestia mnie nurtuje z którą raczej sam sobie nie poradzę czy można zmienić standardowe wyjście hStdOutput w tym wypadku z zapisania do pliku na zmianną , lub TMemo ?