ShellExecute i iconv.exe problem z przekazaniem parametrów

[polymorphism]

Wynika więc z tego, że wartość STARTF_USESTDHANDLES nie ma nic wspólnego z elementem wShowWindow, gdyż go nie dotyczy.

Tyle że flaga ta musi być ustawiona, żeby uwzględniało uchwyty plików hStdInput i hStdOutput .

(...) czy można zmienić standardowe wyjście hStdOutput w tym wypadku z zapisania do pliku na zmianną , lub TMemo ?

Hmm... na zmienną może nie, ale rurę (pipes) możesz podłączyć.

[Darek_C++]

Przyznam się, że chyba jestem za cienki by sam sobie z rura poradzić

[polymorphism]

CreatePipe + ReadFile + trochę pomyślunku i będzie git. Wszystkie niezbędne szczegóły są w dokumentacji, pod linkiem, który podałem.

[Cyfrowy Baron]

Tyle że flaga ta musi być ustawiona, żeby uwzględniało uchwyty plików hStdInput i hStdOutput .

Przecież tak właśnie napisałem:

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.

[Darek_C++]

Chyba będę musiał posiłkować się pośrednictwem zapisu do pliku i wczytania z niego danych zamiast tej rury

[polymorphism]

Przecież tak właśnie napisałem: (...)

Dopisałeś. Odnosiłem się do wersji pierwotnej.

@Darek_C++, nie przesadzasz? Z tego co widzę w dokumentacji, stworzenie i użycie z rury nie wygląda na coś bardziej skomplikowanego niż otworzenie i czytanie pliku za pomocą funkcji WinAPI. Więc zamiast biadolić, że coś Ci nie wyjdzie, poeksperymentuj, spróbuj chociaż.

[Darek_C++]

Stworzyłem coś takiego:

Kod: Zaznacz cały

void __fastcall TForm1::Button2Click(TObject *Sender)
{
   float Start = GetTickCount();
   SECURITY_ATTRIBUTES sa;
    sa.nLength              =   sizeof  (   SECURITY_ATTRIBUTES);
    sa.lpSecurityDescriptor =   NULL;
   sa.bInheritHandle       =   TRUE;

   HANDLE hRead;
  HANDLE hWrite;
  char buffer[1024];
  DWORD bytesRead;

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

    si.cb = sizeof(STARTUPINFO);
   si.dwFlags    = STARTF_USESHOWWINDOW | STARTF_USESTDHANDLES;
   si.wShowWindow = SW_HIDE;

   //Tworzenie potoku...
   sat.nLength  = sizeof(SECURITY_ATTRIBUTES);
   sat.lpSecurityDescriptor =  NULL;
   sat.bInheritHandle = TRUE;
   CreatePipe((PHANDLE)&hRead, (PHANDLE)&hWrite, &sat, NULL);

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

   /* wyjście */
   /**
   si.hStdOutput = CreateFile("E:\\php\\outtest.txt",
                GENERIC_WRITE,
                FILE_SHARE_READ | FILE_SHARE_WRITE,
                &sa,
                CREATE_ALWAYS,
                FILE_ATTRIBUTE_NORMAL,
            NULL);
   */
   si.hStdOutput  = (PHANDLE)hWrite;
   si.hStdError  = (PHANDLE)hWrite;

    PROCESS_INFORMATION ProcessInfo;


    if(CreateProcess(NULL,
                    "E:\\php\\php-cgi.exe",
                    NULL,
                    NULL,
                    TRUE,
               NORMAL_PRIORITY_CLASS,
                    NULL,
                    NULL,
                    &si,
                    &ProcessInfo))
    {
        WaitForSingleObject(ProcessInfo.hProcess,INFINITE); //<--- czekamy...
        CloseHandle(ProcessInfo.hProcess);
        CloseHandle(ProcessInfo.hThread);
   }

   RtlZeroMemory(buffer, 1024);
    ReadFile(hRead, buffer, 1023, &bytesRead, NULL);
   Memo1->Lines->Add( buffer );

    CloseHandle(si.hStdInput);
   CloseHandle(si.hStdOutput);
   float End = GetTickCount();
   Edit1->Text =  FloatToStr((End - Start) / 1000) + " s.";
   //   Memo1->Lines->LoadFromFile("E:\\php\\outtest.txt");
}

W tym wypadku uruchamiam PHP który wykonuje jakiś kod jeśli efekt wykonania kodu nawet

<?php echo file_get_contents('http://www.google.pl'); ?> czyli pobranie kodu strony google idzie do pliku jest OK, ale jeśli ma iść na Memo1 program się zawiesza. Program działa jeśli dane wyjściowe są mniejsze.

---

@Darek_C++, nie przesadzasz?

Zależy jak Podchodzisz do klimatu forum i pomoc jako takiej. Dla kogoś dobrze znającego dany język pewne kwestie wydają się bardzo łatwe. Jednak dla kogoś kto mniej zna sprawa wygląda inaczej...

[polymorphism]

Dla kogoś dobrze znającego dany język pewne kwestie wydają się bardzo łatwe. Jednak dla kogoś kto mniej zna sprawa wygląda inaczej...

Widzisz, tu nie chodzi o język, tylko o umiejętność samodzielnego tudzież kreatywnego myślenia. To w programowaniu jest bardzo ważne i tego wymagam od ludzi zajmujących się programowaniem. Dla mnie pomoc to podpowiedź lub wskazanie błędu, a nie pisanie za kogoś kodu. Przez moment odniosłem wrażenie, że czekasz na gotowy kod, pisząc "nie umiem, jestem za cienki", pomimo że dostałeś pewne wskazówki. Takie podejście rzadko spotyka się ze zrozumieniem (no i się mści, bo po paru latach programowania taki delikwent nic nie umie ).

--- za moment odniosę się do kodu ---

[Darek_C++]

Panie Kolego nie od razu Kraków zbudowano....
Usiłowałem też odczytać to w ten sposób:

Kod: Zaznacz cały

const int bufSize = 1024;
   char buf[bufSize];
  // DWORD bytesRead;
   while (ReadFile(hRead, buf, bufSize-1, &bytesRead, 0) && (bytesRead > 0))
   {
     buf[bytesRead] = 0;
     // dopisz buf do stringu, editboxa...
   }

ale program się zawiesza

[polymorphism]

Zawiesza się dlatego, ponieważ ReadFile blokuje w momencie, gdy rura jest opróżniona. Funkcja zachowuje się podobnie do recv, która czeka na dane jeśli nie ma ich w sockecie.

Check it out:

Kod: Zaznacz cały

char buffer[10000];
DWORD bytes_read;
DWORD bytes_avail;
string out; //<--- to jest przykladowo

while(PeekNamedPipe(hReadPipe,NULL,0,NULL,&bytes_avail,NULL) &&
    bytes_avail > 0 &&
    ReadFile(hReadPipe,buffer,10000,&bytes_read,NULL))
{
    out.append(buffer,buffer + bytes_read);
} 

Wywołaj to za WaitForSingleObject, bo wtedy jest pewność, że aplikacja coś wtłoczyła do rury, choć nie wiem jak to będzie wyglądać z dużymi ilościami danych, przy przepełnieniu bufora rury. Najlepiej byłoby czytać przed zamknięciem aplikacji, ale to nie przejdzie z tą pętlą, trzeba by jakiś timeout ustawić jeśli bytes_avail == 0. No ale nieważne...

--- DODANE ---

...a może i ważne Tu masz pewniejszy algorytm czytania:

1. funkcją PeekNamedPipe sprawdź, czy w rurze są jakieś dane do przeczytania.
   
   1. jeśli są, skok do 2
   2. jeśli nie ma, sprawdź funkcją WaitForSingleObject, czy aplikacja jest aktywna.
      
      • jeśli jest, daj Sleep(1) i skocz do 1
      • jeśli nie jest, skok do 3
2. czytasz dane funkcją ReadFile i skok do 1
3. koniec pętli

Całość dajesz zaraz po CreateProcess.

Panie Kolego nie od razu Kraków zbudowano....

...ale zbudowano. Nikt za budowniczych tego nie zrobił

[Darek_C++]

Zostawmy Kraków na inną okazję, bo program nadal nie działa do końca poprawnie
Działa jeśli dane wynikowe z PHP są niewielkie, ale jak ich jest więcej np kod wywołania funkcji :

<?php phpinfo(); ?>

jest dosyć duży plik utworzony przy wyjściu nie do rury, ale na pilik ma 40,1 KB (bajtów: 41 105) przy wyjściu do pliku działa [0,094 s.]
Ale przy odczytywaniu przez rurę program się zawiesza.
Poniżej zamieszczam cały kod może będzie łatwiej dojść dlaczego tak się dzieje - co mam źle w tym kodzie;

Kod: Zaznacz cały

void __fastcall TForm1::Button2Click(TObject *Sender)
{
   float Start = GetTickCount();
   SECURITY_ATTRIBUTES sa;
    sa.nLength              =   sizeof  (   SECURITY_ATTRIBUTES);
    sa.lpSecurityDescriptor =   NULL;
   sa.bInheritHandle       =   TRUE;

   HANDLE hRead;
   HANDLE hWrite;

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

    si.cb = sizeof(STARTUPINFO);
   si.dwFlags    = STARTF_USESHOWWINDOW | STARTF_USESTDHANDLES;
   si.wShowWindow = SW_HIDE;

   //Tworzenie potoku...
   sat.nLength  = sizeof(SECURITY_ATTRIBUTES);
   sat.lpSecurityDescriptor =  NULL;
   sat.bInheritHandle = TRUE;
   CreatePipe((PHANDLE)&hRead, (PHANDLE)&hWrite, &sat, NULL);

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

   /* wyjście */
   /**
   si.hStdOutput = CreateFile("E:\\php\\outtest.txt",
                GENERIC_WRITE,
                FILE_SHARE_READ | FILE_SHARE_WRITE,
                &sa,
                CREATE_ALWAYS,
                FILE_ATTRIBUTE_NORMAL,
            NULL);
   */
   si.hStdOutput  = (PHANDLE)hWrite;
   si.hStdError  = (PHANDLE)hWrite;

   PROCESS_INFORMATION ProcessInfo;


   if(CreateProcess(NULL,
               "E:\\php\\php-cgi.exe",
                    NULL,
                    NULL,
                    TRUE,
               NORMAL_PRIORITY_CLASS,
                    NULL,
                    NULL,
                    &si,
                    &ProcessInfo))
    {
        WaitForSingleObject(ProcessInfo.hProcess,INFINITE); //<--- czekamy...
        CloseHandle(ProcessInfo.hProcess);
        CloseHandle(ProcessInfo.hThread);
   }
   // odczytanie tego co proces wypluł na stdout;

    char buffer[10240];
    DWORD bytesRead;
    DWORD bytes_read;
    DWORD bytes_avail;
   std::string out; //<--- to jest przykladowo

   while(PeekNamedPipe(hRead,NULL,0,NULL,&bytes_avail,NULL) &&
        bytes_avail > 0 &&
      ReadFile(hRead,buffer,10239,&bytes_read,NULL))
    {
        out.append(buffer,buffer + bytes_read);
   }
   Memo1->Lines->Add(out.c_str() );

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

   float End = GetTickCount();
   Edit1->Text =  FloatToStr((End - Start) / 1000) + " s.";

}

Więc ponownie zwracam się z prośba o pomoc...

[polymorphism]

Czyli moje obawy się potwierdziły... U góry masz inny algorytm czytania, który dopisałem.

[Darek_C++]

Ale po co mam sprawdzać czy dane są w rurze skoro wiem, że są. Zostaje kwestia ich odczytania.... i po co stosować mam te sekundowe opóźnienie ?
Poza tym jak już pisałem problem występuje przy czytaniu większej ilości danych z rury.

[polymorphism]

Ale po co mam sprawdzać czy dane są w rurze skoro wiem, że są.

Tak? A wiesz dokładnie ile bajtów jest w rurze i co ile bajtów aplikacja flushuje swój bufor, że możesz sobie robić takie założenia na sztywno?

Skąd to 10240?!

Zostaje kwestia ich odczytania....

Ot, drobiazg...

i po co stosować mam te sekundowe opóźnienie ?

Żeby nie zarzynać procesora. I nie sekundowe, tyko milisekundowe. Dokumentacja się kłania...

[Darek_C++]

Skąd to 10240?!.

a skąd te Twoje 10000 char buffer[10000];
moje to 10 * 1024
Dokumentacja się nie musi się od razu kłamać, bo pomyliłem z PHP gdzie jest to właśnie sleep(1) jedna sekunda.
----
Tak czy inaczej podany przez Ciebie kod sparwdza czy jest coś w rurze jednak wiesza program wiec nie jest prawidłowy...