whats wrong here?

[soulx`]

#include <stdio.h>

int main()
{

  int i; /*Declaring variable*/
  char text[100]; /*Declaring variable*/
  
  printf("Enter something:"); /*Printing Enter something: to screen*/
  gets(text); /*Getting text typed in and setting the value to text*/
  if(text==0)
  {
  printf("You must enter text");
  }
  else
  {
  for(i=1;i<=5;i++) /* setting the value of i to 1 then setting i to be lesser than or equal to 5 , then adding 1 to to i until it reaches 5*/
  printf("You typed %s and it's been spammed %d times\n" ,text ,i); /*Printing The text followed by the value of text and i %s displays string of text value, and %d displays int value*/
  return(0);
  }
}

how come when i dont enter something is still prints the end message?

[soulx`]

i have tried text==NULL doesnt do anything

[Salem]

> gets(text);
http://faq.cprogramming.com/cgi-bin/...&id=1043284351

> if(text==0)
text is an array, and arrays always exist, so an array can never be equal to NULL
If you're trying to test the first character, then it would be

if ( text[0] == '\0' )

But read the FAQ first. fgets() is not quite the same as gets(), but it is far safer to use.

[soulx`]

great works fine now.. btw what do you mean fget is safer than gets .. i dont understand why its not safe?

[soulx`]

ah never mind read the page, thanks again

[sand_man]

also a better practice would be to have the return(0); outside of the else{}

  else
  {
    for(i=1;i<=5;i++) /* setting the value of i to 1 then setting i to be lesser than or equal to 5 , then adding 1 to to i until it reaches 5*/
        printf("You typed %s and it's been spammed %d times\n" ,text ,i); /*Printing The text followed by the value of text and i %s displays string of text value, and %d displays int value*/

  }
    return(0);
}