Problem mit der while Schleife

flenders

Vielleich solltest es mal so versuchen:

if(status == FILE_OK) {
  result = TRUE;
}
else {
  result = FALSE;
}

Müsste aber eigentlich das gleiche bewirken wie KPC schon gesagt hat!
(Hast du BOOL result = FALSE; auch in der while-Schleife, also nicht davor!?!)
Falls es immer noch nicht geht, was genau passiert denn bzw. inwiefern funktioniert es nicht?

eonic+d

Hab es versucht was ihr beide mir mir gesagt habt aber es geht leider immer noch nicht.

Also nochmal kurz die Problem Lage:

In der Datei string.txt steht:

forumstart.htm
xyz.htm
forumstart.htm

Das Programm soll nun prüfen welche Datei vorhanden ist, er sollte dann normalerweie folgende Ausgabe machen:

Datei gefunden! Datei nicht gefunden! Datei gefunden

Das Programm gibt aber:

Datei gefunden! Datei gefunden! Datei gfeunden!

aus.

Hat vieleicht einer eine ganz andere LÖsung wie ich das bewerkstelligen kann, oder noch einen Vorschlag zur abänderung?

glaub zwar nich das das was bringt aber wie währs mal mit {} klammern um die if s ?

eonic+d

Leider nicht...

eonic+d

Hatt den keiner noch eine Idee, werde noch total verrückt mit dem Problem.
Bitte um Hilfe!

MagiC Creator

Hmm... stell mal die Reihenfolge der Dateinamen um, also zuerst die xyz.htm, wenn die dann als nicht gefunden erkannt wird, dann stimmt deine Überprüfung, aber deine Schleife nicht :). Wär vielleicht'n Anfang, herauszufinden, wo der Fehler überhaupt liegt.

cya

[ Dieser Beitrag wurde am 26.12.2002 um 10:45 Uhr von MagiC Creator editiert. ]

Also...

Datei:
xyz.htm
forumstart.htm
forumstart.htm

Ausgabe:
Datei nicht gefunden! Datei nicht gefunden! Datei gefunden!

Das "BOOL result = false;" hab ich jetzt nochmal nach oben getan.

// > bcc32 http_request.cpp -l wininet.lib

#define WIN32_LEAN_AND_MEAN
#include <windows.h>
#include <wininet.h>
#include <iostream>
#include <fstream.h>
#include <string>
using namespace std;

const DWORD FILE_OK = 3158066;
HINTERNET hnet, hhost, hdata;
DWORD status = 0;
DWORD size = sizeof(DWORD);
BOOL result = FALSE;
char buffer[80];

int main()
{

    hnet  = InternetOpen("Inet",INTERNET_OPEN_TYPE_DIRECT,NULL,NULL,0);
        hhost = InternetConnect(hnet,"www.c-plusplus.net",INTERNET_DEFAULT_HTTP_PORT,NULL,NULL,INTERNET_SERVICE_HTTP,0,0);

        ifstream file;
    file.open("string.dat", ios::in | ios::binary);

    if (!file.is_open())
            cout << "Error opening file"; 

        while (!file.eof())
        {
        file.getline(buffer,80);

        hdata = HttpOpenRequest(hhost,"HEAD",buffer,NULL,NULL,NULL,0,0);

        if(hdata == NULL)
                result = FALSE;

            HttpSendRequest(hdata,NULL,0,0,0);

            if(HttpQueryInfo(hdata,HTTP_QUERY_STATUS_CODE,&status,&size,0))
                    if(status == FILE_OK)
                        result = TRUE;

            if(result == TRUE)
                    cout << "Datei vorhanden!";
            else
                    cout << "Datei nicht vorhanden!";
        }

    file.close();

    getchar();

    InternetCloseHandle(hdata);
    InternetCloseHandle(hhost);
        InternetCloseHandle(hnet);

    return 0;
}

KPC

Wie gesagt so kann es nicht klappen, weil result dann sobald eine Datei gefunden wurde auf true gesetzt wird. Es wird aber nie mehr zurückgesetzt! Also entweder ein result=false an den Anfang der while Schleife oder

if(status == FILE_OK)
                    result = TRUE;
         else
                    result = FALSE;

Wenn es dann nicht klappt, hab ich keine Ahnung, aber geh doch mal mit Einzelschritten durch das Programm mit deinem Debugger und guck dir result an!

Gibt das Gleiche in grün

Ausgabe:
Datei nicht gefunden! Datei nicht gefunden! Datei gefunden!

MagiC Creator

Schreib mal unter "cout << "Datei nicht gefunden!"" noch "result = FALSE;", vielleicht funktioniert's dann.

cya

Leider nicht

KPC

Original erstellt von <eonic+d>:
**Gibt das Gleiche in grün

Ausgabe:
Datei nicht gefunden! Datei nicht gefunden! Datei gefunden!**

Zeig doch mal bitte den Code wo du das drinhast, ich kann mir das nicht recht vorstellen.

#define WIN32_LEAN_AND_MEAN
#include <windows.h>
#include <wininet.h>
#include <iostream>
#include <fstream.h>
#include <string>
using namespace std;

const DWORD FILE_OK = 3158066;
HINTERNET hnet, hhost, hdata;
DWORD status = 0;
DWORD size = sizeof(DWORD);

char buffer[80];

int main()
{

hnet = InternetOpen("Inet",INTERNET_OPEN_TYPE_DIRECT,NULL,NULL,0);
hhost = InternetConnect(hnet,"www.c-plusplus.net",INTERNET_DEFAULT_HTTP_PORT,NULL,NULL,INTERNET_SERVICE_HTTP,0,0);

ifstream file;
file.open("string.dat", ios::in | ios::binary);

if (!file.is_open())
cout << "Error opening file";

while (!file.eof())
{
BOOL result = FALSE; // Ist doch richtig, oder?
file.getline(buffer,80);

hdata = HttpOpenRequest(hhost,"HEAD",buffer,NULL,NULL,NULL,0,0);

if(hdata == NULL)
result = FALSE;

HttpSendRequest(hdata,NULL,0,0,0);

if(HttpQueryInfo(hdata,HTTP_QUERY_STATUS_CODE,&status,&size,0))
if(status == FILE_OK)
result = TRUE;

if(result == TRUE)
cout << "Datei vorhanden!";
else{
cout << "Datei nicht vorhanden!"; result = FALSE;}

}

file.close();

getchar();

InternetCloseHandle(hdata);
InternetCloseHandle(hhost);
InternetCloseHandle(hnet);

return 0;
}

KPC

Sorry da weiß ich wirklich nicht weiter.

MagiC Creator

Probier mal normale Strings aus, die nicht aus der Datei kommen, vielleicht stimmt was am Einlesen der Datei nicht.

cya

eonic+d

Wie würdet ihr es denn machen, wenn ihr Strings aus einer Text Datei einlesen und per HttpOpenRequest einzeln überprüfen müsstet?

eonic+d

zu spät gesehen...

@Magic

das habe ich schon gemacht und da funzt es, nur weil es sehr viele später einmal werden sollen, wollte ich die in einer Datei anlegen

MagiC Creator

Also wenn du Strings nimmst funzt es? Probier mal nen String-Array in deiner while-Schleife. Wenn das dann auch noch funzt, dann stimmt was mit dem Auslesen nicht.

cya

eonic+d

Weiss jetzt nicht ob ich das jetzt so richtig verstehe, denn die while Schleife hab ich nur deswegen eingeführt wegen dem Datei Handling, denn als ich das mit den Strings gemacht habe, hatte ich eine for Schleife verfendet.

Den Code hatte ich aber auch nur von Kane bekommen:

#define WIN32_LEAN_AND_MEAN

#include <windows.h>
#include <wininet.h> //wininet.lib linken
#include <iostream>
#include <string>

using namespace std;

const DWORD FILE_OK = 3158066;

bool TestFile(string path);
HINTERNET hnet,hhost;

int main()
{
    char host[50];
    string files[2] = {{"index.asp"}, {"home.asp"}};

    cout << "Hostname: ";
    cin >> host;

    hnet = InternetOpen("Inet",INTERNET_OPEN_TYPE_DIRECT,NULL,NULL,0);
    hhost = InternetConnect(hnet,host,INTERNET_DEFAULT_HTTP_PORT,NULL,NULL,INTERNET_SERVICE_HTTP,0,0);

    if((hnet || hhost) == NULL)
        cout << "Error! Keine Verbindung\n";

    for(int i=0; i<2; i++)
    {
        if(TestFile(files[i]))
        {
            cout << "File: " << files[i] << "\t[   Ok   ]\n";
        }
        else
        {
            cout << "File: " << files[i] << "\t\t[ Failed ]\n";
        }
    }

    getchar();

    InternetCloseHandle(hhost);
    InternetCloseHandle(hnet);

    return 0;
}

bool TestFile(string path)
{
    DWORD status = 0;
    DWORD size = sizeof(DWORD);
    bool result = false;

    HINTERNET hdata = HttpOpenRequest(hhost,"HEAD",path.c_str(),NULL,NULL,NULL,0,0);
    if(hdata == NULL)
        return result;
    HttpSendRequest(hdata,NULL,0,0,0);

    if(HttpQueryInfo(hdata,HTTP_QUERY_STATUS_CODE,&status,&size,0))
        if(status == FILE_OK)
            result = true; // Datei ok

    InternetCloseHandle(hdata);
    return result;
}

MagiC Creator

Und das funktioniert? Ok. Dann würde ich gleich die Funktion TestFile weiternehmen, nicht alles in die main schreiben. Also irgendwas an deinem Datei-Zeugs stimmt nicht... bloß was?

cya