ShellExecute i iconv.exe problem z przekazaniem parametrów

[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 sprawdza czy jest coś w rurze jednak wiesza program wiec nie jest prawidłowy...

Tutaj znalazłem coś o zawieszaniu programu przez PeekNamedPipe
http://www.duckware.com/tech/peeknamedpipe.html
ale i bez tej funkcji program się zawiesza przy odczytywaniu ...

[polymorphism]

Tak czy inaczej podany przez Ciebie kod sprawdza czy jest coś w rurze jednak wiesza program wiec nie jest prawidłowy...

Przecież pisałem, że ten kod wzbudza pewne moje obawy, jak się później okazało trafne, dlatego też podałem algorytm, który powinien rozwiązać problem. Sprawdzałeś go?

a skąd te Twoje 10000

Z głowy.

---- DODANE ----

ale i bez tej funkcji program się zawiesza przy odczytywaniu ...

No bo ta funkcja na tym etapie nie jest problemem, tylko algorytm. Zresztą mam Win XP z SP2 i wszystko gra i buczy...

[Cyfrowy Baron]

Włączę się na chwilę.

Skoro funkcja ReadFile czyta plik o rozmiarze 41 000 bajtów a rozmiar bufora wynosi 10*1024 bajtów to pojemność zmiennej jest prawie cztery razy za mała.

BOOL ReadFile( HANDLE hFile, LPVOID lpBuffer, DWORD nNumberOfBytesToRead, LPDWORD lpNumberOfBytesRead, LPOVERLAPPED lpOverlapped );

lpBuffer - A pointer to the buffer that receives the data read from a file.
nNumberOfBytesToRead - The maximum number of bytes to read.

---

Wydaje mi się, że skoro funkcja ReadFile działa w pętli to może się zdarzyć, że gdy funkcja PeekNamedPipe wykonuje kolejny obieg to funkcja ReadFile blokuje dojście do pliku uniemożliwiając jego uaktualnienie, ale mogę się mylić.

[Cyfrowy Baron]

A co powiesz na temat moich uwag, gdyż szczerze mówiąc nie sprawdzałem tego.

[polymorphism]

(...) to pojemność zmiennej jest prawie cztery razy za mała.

Dlatego to wszystko jest w pętli.

Wydaje mi się, że skoro funkcja ReadFile działa w pętli to może się zdarzyć, że gdy funkcja PeekNamedPipe wykonuje kolejny obieg to funkcja ReadFile blokuje dojście do pliku uniemożliwiając jego uaktualnienie, ale mogę się mylić.

W kodzie, który podałem PeekNamedPipe sprawdza tylko, ile mogę w danej chwili przeczytać bez blokowania (coś jak socketowy select), więc wywołanie ReadFile nie będzie blokować przy pustej rurze, ponieważ pętla zostanie wcześniej przerwana.

[Darek_C++]

Zauważyłem coś takiego, że po uruchomieniu kodu program się zawiesza i pojawia się proces php-cgi.exe którego zabicie w Menadżerze zadań Windows powoduje odblokowanie programu jednak dane wyczytane z rury mają ok 4,00 KB (bajtów: 4 099) przy zapisaniu jako plik a nie 40KB wiec nie są całkowicie wczytane. Przy zapisaniu z wyjścia CreateProces bezpośrednio do pliku mają ok 40KB.

polymorphism ja się z Tobą nie wykłócam tylko męczę się z pewnym problemem i liczyłem na pomoc ... podajesz kod który nie działa więc problem pozostaje nadal, a szkoda, bo już całkiem dużo zostało w tej kwestii osiągnięte..

Tu też jest o tym problemie:
http://www.codeguru.com/forum/archive/i ... 63149.html

[Darek_C++]

Twoja koncepcja:

Kod: Zaznacz cały

char buffer[1024];
do
{
   PeekNamedPipe(hRead,NULL,0,NULL,&bytes_avail,NULL);
   ShowMessage("Są dane "+ (String) bytes_avail);
   if(bytes_avail == 0) return;
   ReadFile(hRead,buffer,1023,&bytes_read,NULL);
   out.append(buffer,buffer + bytes_read);

}while(1);

Tylko że od razu zawiesza program przy większej ilości danych z PHP nawet ani razu pętla się nie wykona. Wiec dlaczego tak się dzieje ?
-----------
Nawet bez pobierania danych w pętli nie idzie sprawdzić czy coś jest w rurze, bo też program się zawiesza.

Kod: Zaznacz cały

   if(PeekNamedPipe(hRead,NULL,0,NULL,&bytes_avail,NULL))
   {
      ShowMessage("Są dane "+ (String) bytes_avail);
   }

Coś mi się wydaje, że błąd jest gdzie indziej, ale już nie mam dziś do tego głowy. Oto caly kod który wiesza program:

Kod: Zaznacz cały

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

   HANDLE hRead;
   HANDLE hWrite;

   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  = (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);

   }
   DWORD bytes_read;
   DWORD bytes_avail;
   DWORD  dw,read;
   std::string out; //<--- to jest przykladowo

   if(PeekNamedPipe(hRead,NULL,0,NULL,&bytes_avail,NULL))
   {
      ShowMessage("Są dane "+ (String) bytes_avail);
   }


}

Twoja kolej skoro wiesz jak to rozwiązać to słucham...
-----------

Widzę, że "Tu" się blokuje
WaitForSingleObject(ProcessInfo.hProcess,INFINITE);
po zamianie tego na np
WaitForSingleObject(ProcessInfo.hProcess,1000);
Program się nie wiesza wiec ciekawe ze Twój program działał....
Masz teraz okazję to udowodnić stosownym kodem - działającym kodem.

[Cyfrowy Baron]

Na tej stronie: http://forum.ks-ekspert.pl/topic/122563-delphi-uruchomienie-aplikacji-dosowej/ jest interesujący kod w C++.

Poza tym nie mam dobrych wieści. Jak wynika z opisów na wielu forach funkcja PeekNamedPipe i ReadFile w systemie WindowsXP nie będą działały prawidłowo.

Na jednym z nich znalazłem taką informację:

You can add some sleep between peeks to avoid too much CPU utilization: WaitForSingleObject(hProcess, 100); Perhaps it would be better to do overlapped I/O in a thread with WaitForMultipleObjects, and wait until either the process finishes, or the overlapped I/O triggers. Then you don't have to do polling.

Spróbuj tego może pomoże.

[Darek_C++]

Ale jak już pisałem po zamianie w WaitForSingleObject(pi.hProcess, 0); program - kod działa zgodnie z oczekiwaniem
Nie rozumiem tylko kwestii, że gdy dane "szły" do pliku, a nie rury kod działał i z parametrem INFINITE, a przy wujściu na rurę się zawieszał.

[Cyfrowy Baron]

Poza tym zasygnalizowałem konkretny problem z zwiszaniem programu w miejscu

Kod: Zaznacz cały

WaitForSingleObject(ProcessInfo.hProcess,INFINITE); //<--- czekamy...

INFINITE oznacza, że funkcja ma czekać w nieskończoność, co może powodować zawieszenie programu, jeżeli proces nie zostanie zakończony. Zamiast INFINITE możesz podać wartość liczbową określającą po ilu milisekundach proces ma zostać zakończony, wtedy program odblokuje się po zadanym czasie. 1000 = 1 sekunda.

[Cyfrowy Baron]

Ale jak już pisałem po zamianie w WaitForSingleObject(pi.hProcess, 0); program - kod działa zgodnie z oczekiwaniem

To wydaje się chyba naturalne. Zmniejszyłeś czas oczekiwania do 0, więc równie dobrze możesz usunąć tą funkcję. Oczekiwanie w nieskończoność o którym pisałem wcześniej, wiąże się z blokowaniem procesu do czasu zakończenia wykonywanej operacji, przez co kod znajdujący się poniżej nie mógł być wywołany, gdyż program cały czas czekał. Sugerowałbym funkcji WaitForSingleObject zadać jednak jakiś czas oczekiwania, by nie zdarzyła się sytuacja, gdyż rura będzie próbować czytać z pliku, którego nie ma.

[Darek_C++]

Dodam jeszcze, że przy parametrze 0
WaitForSingleObject(pi.hProcess, 0);
proces też czeka na zakończenie uruchomionego PHP co można zaobserwował dając w kodzie PHP funkcje sleep(5); // 5 sekund jednym słowem całość działa zgodnie z oczekiwaniem

[Cyfrowy Baron]

To że czeka to normalne, ale dlaczego 0 sprawia że czeka na zakończenie operacji na pliku?

---

...po chwili

Plik pomocy wyjaśnia to zjawisko:

If dwMilliseconds is zero, the function tests the object's state and returns immediately. If dwMilliseconds is INFINITE, the function's time-out interval never elapses.

Tak więc, gdy wartość argumentu dwMilliseconds wynosi 0, funkcja sprawdza stan obiektu i powraca natychmiast, jeżeli wartość jest INFINITE działa w nieskończoność. Nie ma więc w tym niczego odkrywczego.

[Darek_C++]

To nie jest operacja na pliku, bo ja już nie testuje to z programem iconv.exe do zmiany kodowania plików, ale programem php.exe który wykonuje kod PHP z pliku zewnętrznego. Z kolei PHP uruchomione z kodem w którym jest funkcja sleep(5) musi czekać te 5 sekund zwłoki zanim może się jako program zakończyć.

-----
Ależ ja nie czuje się odkrywcą tylko piszę w jaki sposób rozwiązałem swój problem. Poza tym myślałem, że to właśnie INFINITE sprawia, że czeka kod czeka do zakończenia uruchomionego procesu, a podanie jakiegoś nie zerowego parametru spowoduje, że po tym czasie program ruszy dalej nie czekając na zakończenie uruchomionego procesu co w moim wypadku było by niewłaściwe.

[Cyfrowy Baron]

No, ale o takich zmianach to powinieneś pisać od razu, gdyż my tutaj zastanawiamy się co jest nie tak w testowanym kodzie, a Ty zwyczajnie zmieniłeś warunki eksperymentu, przez co my zastanawiamy się nad jednym, a Ty robisz zupełnie coś innego.