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) ;

---

Where is the while loop? You cannot use strcpy in this way, and it does not fit your description of what you are trying to do.

'check' is an integer, so you shouldn't be pretending it's a string.
'arra' is ... no one knows, because you didn't show us in your example. More than likely you are accessing a single character, and not a 'word'/'string'/'row' of it.


Quzah.

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"?

Well, it could be as simple as:

Code:

---

check = arra[0][0];

---

Quote:

---

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"

Post the smallest and simplest program that demonstrates the error. It should be a program of not more than say, 10 lines, that you expect to compile perfectly well but which results in that warning.

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

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 ','.

1. 'row' should start at 0, not at 1.
2. the 'for' loop only executes the first 'while', not all of the ones after it.
3. You 'character' plows off the end of all of your arrays.
4. Why do you have so many huge arrays?

Quzah.

Quote:

---

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.