C / C++ / MFC

kitty5 wrote:

data[a] = "q";

for (int a = 0; a < 200; a++)
{
    data[a] = new char[5]; // room for 0x## plus a terminating \0 character
    
    data[a][0] = 'q';
    data[a][1] = '\0';
    // or
    strcpy(data[a], "q");
}

kitty5 wrote:

while( (feof(fpread) == 0 || ferror(fpread) == 0) )

"The largest fire starts but with the smallest spark." - David Crow

"Judge not by the eye but by the heart." - Native American Proverb

<br />
<blockquote class="FQ"><div class="FQA">DavidCrow wrote:</div>You must allocate memory for data[a] before you can assign anything to it. Consider:<br />
<br />
<br />
for (int a = 0; a < 200; a++)<br />
{    <br />
data[a] = new char[5]; <br />
// room for 0x## plus a terminating \0 character        <br />
data[a][0] = 'q';    <br />
data[a][1] = '\0';    <br />
// or    <br />
strcpy(data[a], "q");<br />
}</blockquote><br />
<br />

kitty5 wrote:

I've also added the above code. However, I'm still getting the same error.

"The largest fire starts but with the smallest spark." - David Crow

"Judge not by the eye but by the heart." - Native American Proverb

void main( void )
{
    char *data[200];
    FILE *pFile;
    int i = 0;
 
    pFile = fopen("c:\\data.txt", "r");
    if (pFile != NULL )
    {
        for (int a = 0; a < 200; a++ )
            data[a] = new char[5];
 
        while ((feof(pFile) == 0 && ferror(pFile) == 0))
        {
            fscanf(pFile, "%s", data[i]);
            printf("data = %s \n", data[i]);
            i++;
        }
 
        fclose(pFile);
    }
}

"The largest fire starts but with the smallest spark." - David Crow

"Judge not by the eye but by the heart." - Native American Proverb

kitty5 wrote:

//initialize data
for ( int a = 0; a < 200; a++ )
{
data[a] = "q";
}

kitty5 wrote:

fscanf(fpread, "%s", &data[i]); <- problem is here... i think

int _tmain(int argc, _TCHAR* argv[])
{
	char *data[200] = {0};	// initialize it here
	char* buffer = NULL;		// buffer for file
	FILE *fpread;
	int i = 0;

	if ((fpread = fopen("data.txt", "r")) == NULL)
	{
		printf("Cannot open file for reading. \n");
		return -1;
	}

	if (0 != fseek(fpread, 0, SEEK_END))
	{
		printf("Cannot seek end of file.\n");
		fclose(fpread);
		return -1;
	}

	size_t fileSize = ftell(fpread);
	if (0 != fseek(fpread, 0, SEEK_SET))
	{
		printf("Failed to seek beginning of file.\n");
		fclose(fpread);
		return -1;
	}

	buffer = new char[fileSize + 1];
	memset(buffer, 0, fileSize + 1);
	if (fileSize != fread(buffer, sizeof(char), fileSize, fpread))
	{
		printf("Failed to read file.\n");
		fclose(fpread);
		delete [] buffer;
		buffer = NULL;
		return -1;
	}

	// parse buffer here

	delete [] buffer;
	buffer = NULL;
	return 0;
}

Zac Howland wrote:

if (fileSize != fread(buffer, sizeof(char), fileSize, fpread))
{
printf("Failed to read file.\n");
fclose(fpread);
delete [] buffer;
buffer = NULL;
return -1;
}

size_t size;<br />
	size = fread(buffer, sizeof(char), fileSize, fpread);<br />
	printf("fileSize = %d\n", fileSize);<br />
	printf("fread() = %d\n", size);

Jay03 wrote:

Does that make sense to you guys????

void inc() {
  static iCpt = 0;
  iCpt++;
  printf("%d\n", iCpt);
}
 
void main(void) {
  for (int i = 0; i < 10; i++) {
    inc();
  }
}

1
2
3
4
...

Jay03 wrote:

Static sounds like a CONSTANT.

YourFunction()
{
  static int st = 0;   // only when the function is first called
  st++;                // every time when the function is called
}

Jay03 wrote:

Static sounds like a CONSTANT.

"The largest fire starts but with the smallest spark." - David Crow

"Judge not by the eye but by the heart." - Native American Proverb

"The largest fire starts but with the smallest spark." - David Crow

"Judge not by the eye but by the heart." - Native American Proverb