[H.....z]

Jakis Mirek moglby spojrzec czy moj kod jest poprawny? Otoz, czy poprzez memcpy nie bada nadpisywane moje zamiast zapisywania ich w ciagu?

do { 
 char buffer[2000];
 InternetReadFile(hData, (LPVOID)buffer, _countof(buffer), &dwBytesRead);
 data = (char*)realloc(data, strlen(buffer) * sizeof(char));
 memcpy(data, buffer, strlen(buffer));

} while ( (dwBytesRead > 0));

#cpp #programowanie

[z.....1]

Jeśli chcesz, żeby w zmiennej data na koniec były wszystkie bajty, to powyższy kod nie zadziała.

Realloc powinien przyjmować poprzedni rozmiar tablicy data + rozmiar buffer'a. A memcpy powinien kopiować tablice buffer pod adres data + ile danych już zostało do niego zapisane.

[z.....1]

@Horvitz: Mogłoby to wyglądać na przykład tak:
HINTERNET hData;

(...)

const DWORD bufferSize = 2000;
char buffer[bufferSize];
DWORD dataSize = 0;
char *data;
DWORD dwBytesRead;

do {
InternetReadFile(hData, (LPVOID)buffer, bufferSize, &dwBytesRead);
data = (char*)realloc(data, dataSize + dwBytesRead);
memcpy(data + dataSize, buffer, dwBytesRead);
dataSize += dwBytesRead;
} while((dwBytesRead > 0));

(...)

[H.....z]

@zido11: Wielkie dzieki kolego, zaraz sie temu przyjrze.

[H.....z]

@zido11: Faktycznie to dziala tak jak chcialem, jeszcze raz wielkie dzieki,

[z.....1]

@Horvitz: Nie ma za co :)

[H.....z]

InternetReadFile

@zido11: Cale dzialanie tej procedury jest dla mnie okropne ( ͡° ʖ̯ ͡°) .

[z.....1]

Zastanawiam się, czemu nie skorzystasz z C#, tam takie zabawy powinny być to o wiele łatwiejsze niż w C++. Chyba, że masz takie wymaganie, to inna sprawa.

[z.....1]

A samo działanie wydaje się proste (piszę na podstawie dokumentacji, bo nigdy nie używałem tej funkcji).

Bierzesz stream(coś ala czytanie z stdin) typu HINTERNET i wczytujesz z niego MAKSYMALNIE bufferSize bajtów pod adres buffer, dodatkowo metoda modyfikuje wartość zmiennej dwBytesRead wpisując tam ile bajtów udało się przeczytać(zauważ, że przekazujesz wskaźnik na ta zmienną, dzięki czemu ta metoda jest w stanie nadpisać jej wartość).

No i teraz jeśli stream jest pusty to przeczytasz