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.

To me it says that this a warning.We will fix it,but first consider that we use to end main by the line return 0;,which says that the program terminated successfully.
Now the basic problem i think is that you calculate sum and product before even taken the numbers from the user.You should first take the input(check it maybe if you want) and then do the calculations.

scanf("YEs"); and scnaf("NO") has no reason of living.You should create a string(an array of char maybe) that will be used to hold the input of the user.As you wrote scanf about the integers try to write it again for a char.

As for the comparison between pointer and integer,well the char choice is not exactly what you need.You need something to store a word(a string),not something to store just a characters,as char choice does.You need an array of char(an array of char maybe)

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 ==

Quote:

---

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

Quote:

---

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.

Are you guys kidding me? Even if choice was an array instead of a single char you still wouldn't compare its "value" to "Yes" or "No" using ==. The OP's problem is that he has no idea how scanf works as indicated by Jim and that he doesn't understand that string comparison involves using library functions not ==.

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