why is the charcter constant?

Here is what I have...

Code:

#include <stdio.h> 
#include <stdlib.h>
#include <time.h>

#define SIZE 500000 

int main(void) 
{ 
  
  int i = 0, b = 0, d = 0; 
  FILE *fp1; 
  fp1 = fopen("c:\\blah.html", "r"); 
  

  char html[SIZE]; 
  char url[150][512]; 
  char c; 
  int count = 0; 
  
  fread(html, sizeof(char), SIZE, fp1); 
 
  for (c = html[i]; c != sizeof(html); i++)
  { 
   if (c == '\"') 
    { 
     count++;
      continue; 
    } 
    if (count == 1) 
    { 
	
		url[b][d] = c;
		printf("%c", url[b][d]);
		d++;
	} 
	if (count == 2)
	{
		url[b][d] = '\0';
		printf("\n");
		b++;
		count = 0;
	}
	
  } 
  
  fclose(fp1); 


  return 0; 
}

For some reason, when I check with the debugger, the variable c, does not change. The html[i], changes, but I don't understand why c does not change if it is always assigned a new character. Any help is appreciated.

Code:

  for (c = html[i]; c != sizeof(html); i++)

Do your chars hold values up to 500000?

oops, yeah it's not supposed to be that large. But I don't think that is the casue of the problem.

Besides the loop initialization, when do you assign a new value to c?

it never changes because its only set once -- when the loop starts. After that, c is never reset after incrementing i.

Code:

for (c = html[i]; i <  sizeof(html); i++)
{
    c = htm[i];
    // rest of your code here
}

That is really a badly formatted loop anyway. you should start by setting i to 0 or something, like this:

Code:

for (i=0; i <  sizeof(html); i++)
{
    c = htm[i];
    // rest of your code here
}

Oh, gotcha, thanks a bunch.

> fread(html, sizeof(char), SIZE, fp1);
> for (c = html[i]; c != sizeof(html); i++)
Who said the fread() filled the array exactly?
What if there is only 1278 bytes of HTML data, what happens then?
What if theres 12345678 bytes of HTML data, and byte 500000 happens to be a " ?

> fp1 = fopen("c:\\blah.html", "r");
This is between two declarations, you slipped into C++ or C99 coding.
Also, where is the error checking?