Why my if doesn't work?

Code:

scanf("%s",&data);
   while ((fgets(test_line,512,eoftest))!=NULL)
   {

      if(Contactos[z].data==data)
      {  
         
         printf("test");
      }
      z++;
   }

Hey guys I am having a problem this isn't working what's up?
I have tested the data with a printf before the if and I have tested the z with a printf as well and I have tested the Contactos[z].data with a printf

The results of my testes are Contactos[1].data=27.05.88
data=27.05.88
z=0,1,2,3,4
why doesn't it write what I ask it to in the printf
This is: test. Contactos[1].data is the same as data shouldn't it be printing test??

Because data and Contactos[1].data are pointers. Even though the things they point to are equal, they do not "live" in the same place, so their addresses are different. To compare strings, use strcmp.

More code is needed to tell.

if data is a pointer, scanf("%s",&data); is wrong.

Originally Posted by robwhit

More code is needed to tell.

if data is a pointer, scanf("%s",&data); is wrong.

Ooh, good point. Although, when it comes down to it, what declaration would make that statement right? (If &data is a char *, then data is a char, but how do I know that I have a pile of memory right behind it that I can write into? A union, I guess; I don't see anything else.)

It worked fine with strcmp ;-)

Originally Posted by tabstop

Ooh, good point. Although, when it comes down to it, what declaration would make that statement right? (If &data is a char *, then data is a char, but how do I know that I have a pile of memory right behind it that I can write into? A union, I guess; I don't see anything else.)

I guess anything with sizeof(data) >= 2. But not with type correctness. not sure if you can overwrite a union like that.

Code:

scanf("%s",&data);

Since OP had confirmed that strcmp had worked , it should definitely be the char * pointer. Which he should seriously has to change the way he is calling the scanf function.

ssharish

Of course, using scanf with %s is unsafe as it stands, anyway. At least use %20s (or whatever your size is), or go with fgets.

Perhaps, i can see that he is using scanf and fgets to read string, which makes me wonder what is he trying to do. He should be knowing the consequences of using scanf function on reading string.

OP: What is it are you trying to achieve?

ssharish

I get the feeling Elysia would have a field day with this one.

Read this
Need help with altering info on a txt file
regarding your scanf, please.