Thread: How to parse the data and retrieve the data.

Hi,

I am developing a parser for reading a file and retrieving the data of he file. I want to read individual part of data and store it in the local variable.I am able to read & store the first string, but not able to read & retrieve the remaining string in the file.Can any one help me in solving this problem.I am attaching the code and a sample file.

Code:

[headers]
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
[/headers]

[main]
int main(int argc, char ** argv)
{
	FILE *fp;
	char s[80];
	char * tempbuf = new char[100];
	char * lastTok = new char[100];
	char seps[] = ", // { }";
	unsigned short x[50];
	unsigned short y[50];

	fp = fopen("file1.txt","r");
	if (fp == NULL)
	{
		puts ("Cannot open file");
	     	return 0;
	}
	while(fgets(s,79,fp)!= NULL)
	{
		int buffer=1;
		strcpy(tempbuf, s);
		if((buffer==1) && (strncmp(tempbuf,"const u",7)==0))
		{
			buffer = 0;
		}
		if (strlen(tempbuf) > 32)
		{
			buffer = 1;
			if(strchr(tempbuf, '}')!=NULL)
			{
			exit(0);
			}

			else	if(buffer == 1)
				{
					lastTok = strtok(tempbuf," ");
					while(lastTok != NULL)
					{
						printf("%s\n",lastTok);
						for(int i=0; i<5; i++)
						{
							x[i] = atoi(lastTok);
							printf("%d\n",x[i]);
						}
						lastTok = strtok (NULL, seps);
					
					}
				}
		}

		if((strcmp(tempbuf,"map")>0)|| (strlen(tempbuf) > 32)) 
		{
			buffer = 1;
			{
				if(strrchr(tempbuf, '}')!=NULL)
				{
					exit(0);
				}
	
				if(buffer == 1)
				{
					lastTok = strtok(tempbuf," ");
					while(lastTok != NULL)
					{
						for(int i=0; i<1; i++)
						{
							y[i] = atoi(lastTok);
						}
						lastTok = strtok (NULL, seps);
					}
				}
			}
		}
	}
return 0;
}
[/main]

[File]

[string1]
const u8 pal[5 * 5] =
{
0x7c1f,0x8bef,0x8acf,0x0000,0xfec1,
0x7c1f,0x8bef,0x8acf,0x0000,0xfec1,
0x7c1f,0x8bef,0x8acf,0x0000,0xfec1,
0x7c1f,0x8bef,0x8acf,0x0000,0xfec1,
0x7c1f,0x8bef,0x8acf,0x0000,0xfec1
}
[/string1]

[string2]
const u16 map[8 * 8]=
{
0x5aed,0x32ac,0x9bdc,0x231c,0x8cde,0xfbce,0x4cde,0 x756b,
0x5aed,0x32ac,0x9bdc,0x231c,0x8cde,0xfbce,0x4cde,0 x756b,
0x5aed,0x32ac,0x9bdc,0x231c,0x8cde,0xfbce,0x4cde,0 x756b,
0x5aed,0x32ac,0x9bdc,0x231c,0x8cde,0xfbce,0x4cde,0 x756b,
0x5aed,0x32ac,0x9bdc,0x231c,0x8cde,0xfbce,0x4cde,0 x756b,
0x5aed,0x32ac,0x9bdc,0x231c,0x8cde,0xfbce,0x4cde,0 x756b,
0x5aed,0x32ac,0x9bdc,0x231c,0x8cde,0xfbce,0x4cde,0 x756b,
0x5aed,0x32ac,0x9bdc,0x231c,0x8cde,0xfbce,0x4cde,0 x756b
}
[/string2]

[/File]

Thanks in advance,
Kane.

So is this C or C++?
The 'new' say C++, but most of the rest says C

> strcmp(tempbuf,"map")>0
What are you trying to achieve with > 0

> lastTok = strtok(tempbuf," ");
You have a memory leak here - you don't need to allocate lastTok at all.
It simply points to somewhere in tembuf

In fact, all you really need is
char * tempbuf[100];
char * lastTok;

Sorry for the errors in the program.Infact I am using both C/C++.Modified code is as below.

Code:

[headers]
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
[/headers]

[main]
int main (int argc, char **argv)
{
	FILE *fp;
	char *read;
	char s[255];
	char * tempbuf = new char[100];
	char * lastTok = new char[100];
	char seps[] = "{ # , // } ;";
	int color[255];

	fp = fopen("file1.txt", "r"); //Open the file

	while (fp == NULL)
	{
		printf("Cannot open file");
		return 0;
	}

	while(fgets(s,200,fp)!= NULL)  //Read the file until EOF 
	{  
		strcpy(tempbuf, s);
	        int enter = 1;
		
		if((strchr(tempbuf , '{')!=NULL) && (enter == 1)) 
			printf("Do Not Read\n");

		if(strchr(tempbuf, '}')!=NULL)
		{
			exit(0);
		}
		else if (strlen(tempbuf) > 32) 
		{
			lastTok = strtok(tempbuf, seps); //Start deleting the symbols
			while(lastTok != NULL)
			{
				printf("%s\n",lastTok);
				lastTok = strtok(NULL, seps);
			}
		}
	}
	return 0;

}
[/main]

Like they said, don't mix languages, it just makes problems down the line, and that's if you get the source code working.
To take the string, just take the whole line of input, instead of 255, which might not be long enough, set it to 1024 and then use a sizeof operation to only take what you need.

Well, I tried - but no body listened....

Thanks for all. Infact, there are so many errors in that. I'll take care not to make mistakes like that.

Kane.