1

次のコードで少し問題が発生しています。InternetReadFileを使用してファイルに保存する例をたくさん見てきました。しかし、私はそれを見つけることができないか、char[]のためにそれを機能させることができません。holdBuffを取得するためにszBufferを追加してから、コンテンツをholdBuffと等しく設定したいと思います。

#include <stdio.h>
#include <tchar.h>
#include <string.h>
#include <windows.h>
#include <WinInet.h>

HINTERNET hSession;
void urlToChar(char* url, char** content);

int main()
{
    hSession = InternetOpen("Mozilla/4.0 (compatible) Poison", INTERNET_OPEN_TYPE_PRECONFIG, NULL, NULL, 0);
    char* content;
    urlToChar("http://google.com/", &content);
    printf("%s",content);
    return 0;
}

void urlToChar(char* url, char** content)
{
    HINTERNET hConnect = InternetConnect(hSession, _T(""),INTERNET_DEFAULT_HTTP_PORT, NULL, NULL, INTERNET_SERVICE_HTTP, 0, 1);
    HINTERNET hRequest = InternetOpenUrl(hSession, url, NULL, 0, 0, 0);
    if (hRequest)
    {
        char holdBuff[] = "";
        char szBuff[1025];
        memset(szBuff, 0x00, sizeof(szBuff));
        DWORD bytesRead;
        while (InternetReadFile(hRequest, szBuff, 1024, &bytesRead) == TRUE && bytesRead > 0)
        {
            // Cat szBuff to holdBuff
            memset(szBuff, 0x00, sizeof(szBuff));
        }
        *content = holdBuff;
        // memset(holdBuff, 0x00, sizeof(holdBuff));  <-- Need this?
    }
    InternetCloseHandle(hRequest);
    InternetCloseHandle(hConnect);
}
4

2 に答える 2

2

変数宣言

char xyz[] = "Hello World!";

文字列の内容をスタックに置くようにコンパイラに指示します。もちろん、関数が戻るとスタックはなくなります。

だからあなたの場合：

char holdBuff[] = "";
...
*content = holdBuff;

これは、ローカル変数として長さ1の文字列（NULLターミネータ）を作成するようにコンパイラに指示します。の値をcontentに設定したからといって、指し示しholdBuffていたものがもう存在するわけではありません。holdBuff

2つのことを修正する必要があります。まず、strcpy()または同様の機能を使用する必要があります。次に、に十分なスペースを割り当てる必要がありますholdBuff。

例：

char holdBuff[4096];    // or some other sufficiently large size
...
*content = malloc (strlen(holdBuff) + 1);
strcpy (*content, holdBuff);

使い終わったらfree(content)、参加する必要があります。main()

さて、実際に連結を行う方法について：使用することszBuffをまったく忘れて、に直接書き込むと、パフォーマンスが大幅に向上しholdBuffます。

char* temp = holdBuff;
while (InternetReadFile(hRequest, temp, 1024, &bytesRead) == TRUE && bytesRead > 0)
{
    temp += bytesRead;
}
*temp = '\0';    // manually append NULL terminator

holdBuffこれで、中間の連結を必要とせずに、必要なデータが得られます。

于 2010-11-10T02:12:12.133 に答える
0 
void urlToChar(char* url, char** content)
{
    HINTERNET hConnect = InternetConnect(hSession, _T(""),INTERNET_DEFAULT_HTTP_PORT, NULL, NULL, INTERNET_SERVICE_HTTP, 0, 1);
    HINTERNET hRequest = InternetOpenUrl(hSession, url, NULL, 0, 0, 0);
    if (hRequest)
    {
        char holdBuff[] = "";   //do not use fixed size char array and allocate buffer in stack,you can allocate large enough buffer in heap,but recommend you can use string or CString
        char szBuff[1025];
        memset(szBuff, 0x00, sizeof(szBuff));
        DWORD bytesRead;
        while (InternetReadFile(hRequest, szBuff, 1024, &bytesRead) == TRUE && bytesRead > 0)
        {
            // Cat szBuff to holdBuff
            //strcat when using char array
            // operator+ when using stl string
            // [stl string][1]
            memset(szBuff, 0x00, sizeof(szBuff));
        }
        *content = holdBuff;
        // memset(holdBuff, 0x00, sizeof(holdBuff));  <-- Need this?
    }
    InternetCloseHandle(hRequest);
    InternetCloseHandle(hConnect);
}
于 2010-11-10T02:18:01.010 に答える