Nochmal XML-File einlesen

[Pitter *c*]

Hallo Big Brother,

ich sach ja... DAU am Start, aber ich versuch mich da rein zu kämpfen...

Ok, mal abgesehen davon, dass das return 0 in Zeile 30 da weg muss...

Warum kann denn das Script die XML nicht öffnen und in INVALID_HANDLE_VALUE läuft? Die Datei ist definitiv in dem Verzeichnis drin.

HANDLE hFile6 = CreateFile(TEXT("\\My Flash Disk\\navigation\\licenses\feature.xml\0"), GENERIC_READ , FILE_SHARE_READ, NULL, OPEN_EXISTING,  FILE_ATTRIBUTE_NORMAL, NULL);

	if (hFile6 == INVALID_HANDLE_VALUE)

Wie muss ich denn die obige Zeile umschreiben, dass mir beim Entfernen der Zeile

FILE *file = fopen("\\My Flash Disk\\navigation\\licenses\feature.xml", "rb");

das Script nicht in Fehler läuft?

[Big Brother]

Ich sehe momentan gar nicht, wozu du das hFile6 Handle überhaupt brauchst ...?
Das Öffnen geht vermutlich in beiden Fällen schief, weil ein Backslash zwischen licenses und feature.xml fehlt:
"\\My Flash Disk\\navigation\\licenses****feature.xml"
Die \0 im TEXT Macro ist überflüssig.
Du brauchst nur eine Zeile aus der Datei zu bearbeiten und liest dafür die ganze Datei ein?

[Pitter *c*]

Hallo,

Ja *patsch* den Backslash hab ich übersehen...

Trotzdem liest das Script die Datei nicht ein, sondern schreibt mir das obligatorische 'NO' in das Textfile.

Wenn ich den Bereich mit CreateFile raus nehme, hängt sich das Proggie auf dem PNA komplett auf...

Das hfile wird vorher schon des Öfteren benötigt. So komme ich halt auf hfile6.
Das Proggie ermittelt ja nicht nur die hier gezeigten beiden Werte aus der XML, sondern holt unter Anderem die Grösse einer bestimmten DLL, schaut, ob eine bestimmte Datei auf der SD-Karte liegt usw...

Das Grundgerüst des Scriptteiles hab ich in dem defekten Thread von jemandem erhalten und umgeschrieben...

Wie könnte ich das denn besser anstellen, statt die ganze XML einlesen???

Kann sein, dass das für Euch Basics sind... aber ich bin ja noch am Anfang...

[Big Brother]

Lässt sich die Datei mit fopen öffnen?
Pack doch mal diese Zeilen vor Zeile 3 und probier mal:

FILE *file = fopen("\\My Flash Disk\\navigation\\licenses\\feature.xml", "rb"); 
    if (!file)
	{
		puts("Datei kann nicht geoffnet werden!");
		fprintf(stdout, "%s\n", strerror(errno));
	}
	else
		puts("Datei wurde geoffnet!");

[Pitter *c*]

Mit fopen gehts, allerdings mag er
fprintf(stdout, "%s\n", strerror(errno)); nicht.

Wenn ich das Auslesen des Speichers komplett auskommentiere läuft die Exe komplett durch und hinterlässt auch das 'NO' nicht in der TXT.

Ansonsten hängt sich die EXE mit Fehler auf.

Das Einlesen klappt also schon mal folgendermassen:

FILE *hfile6 = fopen("\\My Flash Disk\\navigation\\licenses\\feature.xml", "rb");
		 if (!hfile6)
			{
			 wcscat(variable, L"NO");
			}
		 else
			{
                   	 fseek(hfile6, 0, SEEK_END);                //get file size
                   	 size_t filesize = ftell(hfile6); //files >4GB?
	                    char *buffer = (char*)malloc(filesize+1);                //get memory to contain the whole file
              		 if (!buffer) puts("Buffer ist leer!"); 
                   	 size_t temp;
              		  if ((temp=fread(buffer, 1, filesize+1, hfile6))!=filesize)
				{
 //                   		  free(buffer);
                      	 	 fclose(hfile6);
           			 }
			}

So... nun jetzt geht es ans Auslesen des Buffers... und da hänge ich jetzt...

Edit sacht dazu: Wenn ich hinter flcose(hfile6); noch ein wcscat(variable, L"YES"); einfüge, dann bekomme ich dies auch im Textfile ausgegeben:

Sirf|mfd|-|4920320|NO|storage|484737024|480|272|YES

[Big Brother]

Öffnen klappt, das ist ja schonmal was. Wie war das nun? Brauchst du die Werte aus einer bestimmten Zeile? Ist diese Zeile immer an der gleichen Position (z.B. Zeile Nr. 3) und die Werte in der Zeile auch?
Wenn ja, vermindert sich der Aufwand um einiges, z.B. kannst du dann das Parsen der Variablennamen weglassen. Wenn nicht, musst du wohl einen ähnlichen Weg gehen, wie nwp2 in deinem ersten Thread vorgeschlagen hat.

Für strerror und errno musst du die Header einbinden, errno.h war das glaube ich.

[Pitter *c*]

Jep, das steht alles in Zeile 4

<?xml version="1.0" encoding="UTF-8"?>
<!DOCTYPE FeatureLicense SYSTEM "feature.dtd">

<FeatureLicense vendor="MEDION" appVersion="5.0" appName="GoPal Navigator" editionName="AE" master="yes" version="1.0" >

Aus dieser Zeile benötige ich nur die Werte von appVersion= und editionName=

Alles Andere wird nicht benötigt.

[Big Brother]

Da du die Datei nun auch schon im Puffer hast, kannst du appVersion und editionName mit strstr suchen und dich zu den Werten durchhangeln.
Willst du die Datei nicht komplett einlesen, würde ich so machen wenn es für mich wäre, reicht hier eine einfache Suche nach den Variablenamen:

#define SMAX 255

long sfind( FILE* fp, char* s ) // Sucht nach der Zeichenkette s in der zum Lesen geöffneten Datei ("rb").
{
	int buf[SMAX+1] = {0}, i=0, n = strlen(s);
	assert ( n <= SMAX );
	fseek( fp, 0, SEEK_SET );
	while ( ( buf[i] = fgetc(fp) ) != EOF ) {
		if ( buf[i] != s[i] || !s[i] ) {
			i=0;
			continue;
		}
		i++;
		if ( i==n )
			return ftell(fp); // Position hinter dem ersten s in der Datei, oder negativer Wert wenn ftell fehlschlägt.
	}
	return -1; // Die Zeichenkette s wurde nicht gefunden.
}

Auch hier musst du dich noch zu den Werten durchackern. SMAX kannst du einen ausreichend großen Wert geben, z.B. 255. Das ist die maximale Anzahl der Zeichen, die ein Variablenname annehmen darf.

[Pitter *c*]

Hallo Big Brother,

sorry, aber da krieg ich keinen Kopf und keinen Ar**** dran. Vielleicht sollte ich doch besser mit LEGO spielen...

Liegt nur an mir, weil ich das nicht schnalle...

long sfind( FILE* fp, char* s )

da hab ich folgendes Fehlermeldungen zu:
error #2157: Unrecognized statement.
error #2001: Syntax error: expected ';' but found 'sfind'.
warning #2027: Missing prototype for 'sfind'.
error #2066: Illegal use of type name 'FILE'
error #2048: Undeclared identifier 'fp'.
error #2168: Operands of * have incompatible types 'void' and 'int'.
error #2039: Illegal expression.
error #2048: Undeclared identifier 's'.

Kannst Du mir da noch mal bitte nen Schubs in die richtige Richtung geben???

[;fricky]

Pitter (c) schrieb:

Kannst Du mir da noch mal bitte nen Schubs in die richtige Richtung geben???

nimm 'nen fertigen xml-parser, damit ersparst du dir viel arbeit. vielleicht den: http://www.minixml.org/

[Pitter *c*]

Hallo fricky,

danke für den Tipp, aber das ist nicht das was ich wollte...

[Big Brother]

Pitter (c) schrieb:

long sfind( FILE* fp, char* s )

da hab ich folgendes Fehlermeldungen zu:
....

Was soll denn die obige Zeile sein, eine Vorwärtsdeklaration? Ein Funktionsaufruf? Deinem Code nach zu urteilen scheinst du einen Aufruf mit einer Deklaratiion zu verwechseln.

[Pitter *c*]

Hallo Big Brother,

wenn ich da nur nen Kopf dran kriegen würde...
Hab mir schon die Finger wund gesucht, wie ich das hinbekommen könnte.

Wie deklariere ich denn 's' und 'fp'?

FILE *hfile6 = fopen("\\My Flash Disk\\navigation\\licenses\\feature.xml", "rb");
	if (!hfile6)
          {
             wcscat(variable, L"NO");
          }
         else
	{
 		long sfind( FILE* fp, char* s ); // Sucht nach der Zeichenkette s in der zum Lesen geöffneten Datei ("rb").
		{
  	 	 int temp[SMAX+1] = {0}, i=0, n = strlen(s);
   	 	assert ( n <= SMAX );
   		 fseek( fp, 0, SEEK_SET );
  	 	 while ( ( temp[i] = fgetc(temp) ) != EOF )
			 {
       			 if (temp[i] != s[i] || !s[i] )
				 {
         		 		 i=0;
          		 	continue;
      	 			 }
       			 i++;
       			 if ( i==n )
           			return ftell(fp); // Position hinter dem ersten s in der Datei, oder negativer Wert wenn ftell fehlschlägt.
				wchar_t hStr6[100];
				swprintf(hStr6, L"%d", (fp));
				wcscat(variable, hStr6);
  	 	 	}
            		 wcscat(variable, L"Error");
		}
	}

Ich hoffe, Ihr verliert nicht die Geduld mit mir???

[Pitter *c*]

Hallo,

ich habs nun doch noch hinbekommen.

Vielleicht durch die Brust ins Auge, aber es funktioniert.

//Get GoPal-Version #######################################

          FILE *hfile6 = fopen("\\My Flash Disk\\navigation\\licenses\\feature.xml", "rb");
	if (!hfile6)
          {
             wcscat(variable, L"NO");
          }
         else
	{

		wchar_t hStr6[100];
		char Str6[100];
		char appVersion[20];
		wchar_t hStr7[100];
		char Str7[100];
		char editionName[20];

			// look for appVersion###########################

		char buffer[] = "<FeatureLicense vendor=\"MEDION\" appVersion=\"5.0\" appName=\"GoPal Navigator\" editionName=\"AE\" master=\"yes\" version=\"1.0\" >";	
		char * position = strstr(buffer, "appVersion="); // search in string
		int appVersionLength = 0;
		char * temp = position + sizeof("appVersion=");
		for (appVersionLength = 0; *temp != '"'; temp++)
			appVersionLength++;

		strncpy(appVersion, position + sizeof("appVersion="), appVersionLength);
		sprintf(Str6, "%s", appVersion);			//Variable in ASCII

		MultiByteToWideChar(CP_ACP, 0, Str6, strlen(Str6), hStr6, 200);  	//convert from ASCII to WideChar

		wprintf(L"%ls\n", variable); 
		wcscat(variable, hStr6);
		wcscat(variable, L" ");

			//look for editionName##########################

		char * position1 = strstr(buffer, "editionName=");
		int editionNameLength = 0;
		char * temp1 = position1 + sizeof("editionName=");
		for (editionNameLength = 0; *temp1 != '"'; temp1++)
			editionNameLength++;

		strncpy(editionName, position1 + sizeof("editionName="), editionNameLength);
		sprintf(Str7, "%s", editionName);

		MultiByteToWideChar(CP_ACP, 0, Str7, strlen(Str7), hStr7, 200);  	//  convert from ASCII to WideChar

		wprintf(L"%ls\n", variable); 

		wcscat(variable, hStr7);
   	}

Vielen Dank für Eure Hilfe... mir qualmt der Schädel...

Ich wünsche euch Schöne Weihnachten und einen guten Rutsch nach 2010. :xmas1:

[Pitter *c*]

Zu früh gefreut...

Das war so nix.

Bin jetzt einen Schritt weiter, aber hab da mal eine DAU-Frage zu:

Warum läuft mir das Script immer in wcscat(variable, L"Error")???
Kann mir das vielleicht jemand erklären??? Büdde... :xmas1:

wchar_t fp[200];
char *s = ("appVersion=");
wchar_t hStr6[100];
char Str6[200];

    FILE *hfile6 = fopen("\\My Flash Disk\\navigation\\licenses\\feature.xml", "rb");
    if (!hfile6)
         	 {
            	 wcscat(variable, L"NO");
         	 }
         else
    		{
        	 long sfind( FILE* fp, char* s ); // Sucht nach der Zeichenkette s in der zum Lesen geöffneten Datei ("rb").
       			{
           		 int temp[SMAX+1] = {0}, i=0, n = strlen(s);
           		 assert ( n <= SMAX );
            		fseek( fp, 0, SEEK_SET );
           		 while ( (temp[i] = fgetc(temp) ) != EOF )
            			{
                   			 if (temp[i] != s[i] || ! s[i] )
                			 {
                          		 i=0;
                      			 continue;
                 			}
                   			 i++;
                   			 if ( i==n )
                      		 	return ftell(fp); // Position hinter dem ersten s in der Datei, oder negativer Wert wenn ftell fehlschlägt.
					sprintf(Str6, "%s",fp);
					MultiByteToWideChar(CP_ACP, 0, Str6, strlen(Str6), variable, 200);
                			swprintf(fp, L"%d", (hStr6));
                			wcscat(variable, hStr6);
             			}
             		 wcscat(variable, L"Error");
        		}
    		}

[Big Brother]

Das Thema Funktionen solltest du dir dringend mal in einem C Buch oder Tutorial angucken(Deklaration, Definition, Rückabewerte, Aufruf, etc).

[Pitter *c*]

Hallo,
bin schon seit ein paar Tagen dabei zu lesen und zu testen, aber ich krieg das nicht ans rennen.

[Big Brother]

Wieso baust du dann immer noch Prototypen mitten in den Code? Warum lässt du die Beispielfunktionen von nwp2 und mir nicht Funktionen bleiben, sondern verstreust sie irgendwo im Quellcode?

[Pitter *c*]

Vielleicht sollte ich es besser dran geben mit diesem Script, oder mir jemanden suchen, der mir das Teil baut...

Einfache Sachen kriege ich hin, aber das ist mir gerade alles viel zu hoch....
Möglicherweise denke ich auch viel zu kompliziert....

[Pitter *c*]

Big Brother schrieb:

Wieso baust du dann immer noch Prototypen mitten in den Code? Warum lässt du die Beispielfunktionen von nwp2 und mir nicht Funktionen bleiben, sondern verstreust sie irgendwo im Quellcode?

Hallo Big Brother,

ich geb da nicht auf... und hab den "Bretterzaun" verstanden...
Also hab ich Deine Funktion vor das Hauptprogramm gestellt.

Im Hauptprogramm öffne ich die XML mit

//Get GoPal-Version 

wchar_t variable[20];
wchar_t Str7[200];

FILE* fp;

fp = fopen("\\My Flash Disk\\navigation\\licenses\\feature.xml", "rb");
  if (!fp)
	{        
	wcscat(variable, "--");
	}
else
 	{
 	int s = sfind  (fp, "appVersion=");

...und dann??? Dann steh ich schon wieder doof da.