Beginner Error

I've just started the C programming tutorial and I'm trying to modify the code to practise what I've learnt. This is what I have:

Code:

---

#include <stdio.h>

int main()
{
  int num1, num2, ans1;
  printf( "Insert a number pl0x: " );
  scanf("%d", &num1);
  printf("Insert another number pl0x: ", &num2);
  scanf("%d", &num2);
  ans1 = num1 + num2;
  printf("The two numers entered added together are: %d", &ans1);
  getchar();
  return 0;
}

---

No matter what number I enter, I get the same (wrong response):

Code:

---

Insert a number pl0x: 3
Insert another number pl0x: 3
The two numers entered added together are: 2293580

---

I think it has something to do with the last printf statement. Can anyone point me into the right direction?

Quote:

---

Originally Posted by SvenRL

I think it has something to do with the last printf statement. Can anyone point me into the right direction?

---

For what it's worth, here's my 2c.

Code:

---

#include <stdio.h>

int main()
{
  int num1, num2, ans1;
  printf( "Insert a number pl0x: " );
  scanf("%d", &num1);
  printf("Insert another number pl0x: ", &num2);                  /* why do you need &num2 here?? */
  scanf("%d", &num2);
  ans1 = num1 + num2;
  printf("The two numers entered added together are: %d", &ans1);  /* print the contents of ans1 not its location */
  getchar();                                                      /* why do you need to read more stuff?? */
  return 0;
}

---

Quote:

---

Originally Posted by itCbitC

For what it's worth, here's my 2c.

Code:

---

#include <stdio.h>

int main()
{
  int num1, num2, ans1;
  printf( "Insert a number pl0x: " );
  scanf("%d", &num1);
  printf("Insert another number pl0x: ", &num2);                  /* why do you need &num2 here?? */
  scanf("%d", &num2);
  ans1 = num1 + num2;
  printf("The two numers entered added together are: %d", &ans1);  /* print the contents of ans1 not its location */
  getchar();                                                      /* why do you need to read more stuff?? */
  return 0;
}

---

---

Thank you for the quick response and constructive criticism. I'm guessing my mistake is putting the num2 into the printf statement. I don't need it there.

so....

What I seem to be doing wrong is the mistake mentioned above and the fact that I'm printing the location of ans1 and not the contents of.

The getchar() at the end of my program is simply to prevent the DOS window from closing. Is that not the write way to do it? It's the only way I know of. Suggestions would be great.

The red highlight was showing you what was wrong. You only use & if you need the address of a variable, rather than the value the variable contains. Typically with printf you don't need an address, but a value.


Quzah.

Quote:

---

Originally Posted by SvenRL

The getchar() at the end of my program is simply to prevent the DOS window from closing. Is that not the write way to do it? It's the only way I know of. Suggestions would be great.

---

Here are a list of methods to stop the console from closing.

You're getting the wrong value because you're using &(address of operator) which is not needed in the case of using printf()

the getchar() isn't needed either, I assume you're on Windows and running the program by double clicking.. you could remove the getchar() and launch the program from cmd (Start>Run>cmd) then cd to your working dir and type your programs name :)