Thread: unclear about typecasting and creating pointers

I am trying make a variable (check) that will change in a while loop that pulls a single character from an array of strings (arra). It was giving me lots of errors about creating pointers without casts, I looked up typecasting but I'm not sure if I'm understanding it very well. I ended up with this code, but I am getting the error: "warning: cast to pointer from integer of different size" on the strcpy line. Any other combination of casts gives lines of errors. Any idea what I'm still doing wrong?

Code:

int character = 0 ;
int check ;
row = 1 ;

strcpy ((char *) check, (char *) arra[row][character]) ;
printf ("first character: %s", &check) ;

I tried creating the pointer straight out and it gave me a whole bunch of errors so I tried using strcpy. The while loops are after this part of code to go through the entire array called "arra" which i specified in my explanation. I am trying to assign "check" a single character from the array "arra". I changed "check" to an integer because the compile was giving me an error that I should be casting it as an character later. I'm very confused at this point.

My question is this: How do I reference a single character of an array of strings and assign it to the variable "check"?

Originally Posted by laserlight

Well, it could be as simple as:

Code:

check = arra[0][0];

This was my original try

Code:

check = arra[row][character];

it gives this error:
"warning: assignment makes pointer from integer without a cast"

I tried taking the check variable entirely out of it. This is the part of the code I am having trouble with. It is giving me a segmentation fault.

Code:

while (fgets (line, sizeof line, fp ) != NULL) /*read a line*/
{
	strcpy (arra[row], line) ;
	row++;
}
				
/*split up rows into separate arrays*/
char codes [2000][4] ;
char names [2000][15] ;
char alts [2000][10];
char lats [2000][10] ;
char longis [2000][10] ;
char states [2000][2];
int character = 0 ;
char temp ;
			
row = 1 ;

for ( row = 1; row <2000 ; row ++) 
	while(arra[row][character++] != ',');
	{
		temp = arra [row][character] ;
		codes [row][character ] = temp ;
	}
	while(arra[row][character++] != ',');
	{
		temp = arra [row][character] ;
		names[row][character] = temp ;
	}
	while(arra[row][character++] != ',');
	{
		temp = arra [row][character] ;
		states[row][character] = temp ;
	}
	while(arra[row][character++] != ',');
	{
		temp = arra [row][character] ;
		lats[row][character] = temp ;
	}
	while(arra[row][character++] != ',');
	{
		temp = arra [row][character] ;
		longis[row][character] = temp ;
	}
	while(arra[row][character++] != ',');
	{
		temp = arra [row][character] ;
		alts[row][character] = temp ;
	}
	fclose(fp) ;

the fclose (fp) should be one tab back

Originally Posted by laserlight

The thing is, you showed even more code, yet the declaration of arra is still missing.

EDIT:
That said, one thing to note is that your character variable is incremented, but never reset. It looks like you do not even need temp. You are probably looking to use strtok() to tokenise the string based on ','.

Sorry, it's kind of a long code. here's the declaration of arra:

Code:

char arra [2000] [100] ;

strtok sounds like what I want to use, I'll try that.