Thread: Just started programming in C, and so far, it has been great...

However, I am running into a syntax error.

Code:

#include <stdio.h>

int main (void)


{
    int num1;
    int num2;
    int sum;
    int product;
    
    sum = num1 + num2;
    product = (num1 * num2);
    
    
    printf("Enter your numbers!\n");
    
    scanf("%d", &num1);
    scanf("%d", &num2);
    
    
    printf("Your sum is: %d", sum);
    
    if (num1 > num2)
    {
             printf("\nThe first number is greater than the second one!\n");
             }
             
             if (num1 < num2) 
             {
                      printf("\nThe first number is less than the second number!");
                      }
                      
                      char choice;
                      
                      printf("Would you like to multiply the two numbers?");
                      scanf("Yes");
                      scanf("No");
                      
                      if (choice == "Yes") {
                      printf("The product of the two numbers is... \n%d", product);
                      }
                      
                      
                      
                      
                      
}

It says that there is a "comparison between point and integer" on the line with the bold text.

Can you correct my issue? I am making a basic calculator.

You have defined choice to be a single character. That means you can't compare this single character to the string constant "Yes".
Also this snippet is incorrect:

Code:

scanf("Yes");
                      scanf("No");

I really don't know what exactly you are trying to do with these two lines, but I suggest you look up the documentation for scanf().

Jim

Another thing i would state is that comparison between string is done with strcmp and not with the operator ==

Another thing i would state is that comparison between string is done with strcmp and not with the operator ==

True but strcmp() requires two strings, not a string and a character. The OP needs to decide if he wants the user to enter a single character or a string then use the proper method.

Jim

Originally Posted by jimblumberg

True but strcmp() requires two strings, not a string and a character. The OP needs to decide if he wants the user to enter a single character or a string then use the proper method.

Jim

I would say that by the "YES" and "NO" we see he/she wanted to compare to strings,but ok we could wait

Am also a learner like you pal, hope i can offer a lil assistance.
I checked it and i think the pointer issue is present cos of this:

Code:

if(choice=="Yes")

On more thing, as earlier said, you calculated sum and product too earlier, you should atleast get the inputs of the person before calculating. Theres a possibility that the results of the calculation wont be correct!
In your code, this is in the wrong position.

Code:

char choice;

it should be done at the beginning.

No we do not kidding you .We stated what you said about the string comparison above