increaments that wont work!!

[Optimus Prime]

ok ive wrote an example program to help me with a project but its not working.
basically,the user has to enter the number "1".after this is done my program SHOULD display the amount of times it has been entered correctly,and incorrectly.(the answer to each of these answers will either be 1 or zero,no loop)

but the problem is,even when i enter it incorrectly,it says i have entered it correctly once,which i havent!

#include <stdio.h>
main()
{
     int crt, incrt, num;
     
     
     
     incrt=0 ;
     crt=0 ;
     num = 1 ;
     
    
     
     printf("please enter the num 1\n") ;
     scanf("%d",&num) ;
     
     if  (num=1)
     {
                  printf("Correct!!") ;
                  crt++ ;
     }//end if crt
     
     else if (num != 1)
     {
                printf("Incorrect!!") ;
                incrt++ ;
     }//end if incrt
     
     
     printf("Incorrectly entered = %d",incrt) ;
     printf("Correctly entered = %d",crt) ;
    
     
     
     
     
      }//end main

any help at all will be much appreciated!

[divineleft]

     if  (num==1)
     {
                  printf("Correct!!") ;
                  crt++ ;
     }//end if crt

heh. watch out for that.

     printf("Incorrectly entered = %d",incrt) ;
     printf("Correctly entered = %d",crt) ;

what are these doing there?

[manutd]

if  (num=1)...

This sets num to 1. It should be:

if (num == 1)...

EDIT: Whoops. divineleft beat me to it

[Optimus Prime]

oh so if i change it it will work?


the 2 printfs at the end should show how many times its been entered righ or wrong..

[divineleft]

Yes, it will work.

I still don't know why you need them, if it's correct, you know you entered it correctly once (same applies to != 1). Unless you wanted it to be a loop.

[manutd]

Yes. = will assign the second argument to the first, thus it sets num equal to one, making that statement always true. == compares the values and returns true if they are equal.
EDIT: Aaaaaah! Stop beating me to the post!

[Prelude]

>main()
Omitting the type doesn't relieve you of your responsibility to return an int. In fact, "good style" C and the latest standard dictate that you must always specify the return type.

>oh so if i change it it will work?
Did you try it?

[Optimus Prime]

yes thank you,i have to aply this to myproject thats why i had the printfs there.thanks to all.and yes i had a main()..

[divineleft]

yes thank you,i have to aply this to myproject thats why i had the printfs there.thanks to all.and yes i had a main()..

Prelude meant to use a type with main()

so

int main()
{
     //code...
     return (0); // returns 0 because main is an int
}

instead of just "main()"