can't find why this code ain't working!

Hi,
This is a program finds out the date for Eastern. I get these two errors when compiling:
-invalid operands to binary % (line 14 and 16)
-int format,double arg<arg 2> (line 19)
I don't know how to fix it. Does anyone have a tips on this?

Code:

---

#include<stdio.h>
#include<stdbool.h>
#include<math.h>
int main()
{
  int year;
  float A,B,C,S,D,T,E,date;
  printf("Choose a year between 1900 to 2099");
  year=getchar();
  A= year % 19;
  B= year %4;
  C= year % 7;
  S=19*A+ 24;
  D=S % 30;
  T=2*B+4*C+6*D+5;
  E=T % 7;
  date=22 + D + E;
  if(date==32){
    printf("The eastern's date is:%d",date);
  }
  else if(date!=32){
    date=D+E-9;
    if(date==26){
      printf("The eastern is April 19th.");
    }
    if(date==25&&A==16&&D==28){
      printf("The eastern is April 18th.");
    }


  }



}

---

-invalid operands to binary &#37; (line 14 and 16)
You can't perform modulo arithmetic on a float (why are they floats anyway)

-int format,double arg<arg 2> (line 19)
Read the manual page to find out what formats correspond to what types.

Your attempt to read a year needs work.

Can you think of better variable names for some of those single letter identifiers?

Tnx for the help.
I should have used int instead of float.(right?) cause now I don't get that error longer.
And I used scanf to read "year" instead of getchar. But now I get this error :
"format argument is not a pointer."
what does it mean?

Basically, you have to pass the variable's memory address to scanf() using the & operator. So the call will be:

Code:

---

scanf("%d", &year);

---

Let me clarify that a bit more. Each variable is stored in memory and referenced using a memory address. When you call a function, you actually pass it's value, not the variable itself. So the function can't modify that variable.

A pointer points to a variable's memory address. It's operator is '&'. So when you want to pass the memory address of a variable to a function, you'll have to precede it with '&'.

Another way to do it is using a pointer variable. This is a variable which stores the memory address of another variable. A pointer variable is declared using

Code:

---

type *name

---

where type is a type of variable and name is the name of the pointer variable.
You then must assign the pointer variable the address of the original variable. It will look like this:

Code:

---

ptr_variable = &variable

---

Now you can call scanf and pass it the pointer variable without the & operator:

Code:

---

scanf("%d", ptr_variable);

---

Hope I did a good job explaining it. If anyone can fix this up anymore than I can, please feel free to comment.

ooops! I was being careless forgetting "&" in scanf.
but tnx to your explanations I know now why I should use it =)
but there's another problem too. When the program asks abt the year, It just reads it and jumps out of the program.

http://faq.cprogramming.com/cgi-bin/...&id=1043284385

Quote:

---

Originally Posted by Salem

http://faq.cprogramming.com/cgi-bin/...&id=1043284385

---

tnx for d link, very helpful cause I get that problem alot ;) but this time it is not the command window which disappears, it's the program that ends it self ;like if I have a break or sth in the program which I don't have. It doesn't go to any of loops at all =(

The above link is pretty much what you want. It describes it very well. The program runs fine, but closes directly because your program ends.
Have another look in there and look in the FAQ for ways to wait for a keypress before closing the app.

Code:

---

    }
  }
  getchar();
  return 0;
}

---

include this at the end of code. One to read a char from the stdin. So that it will wait until a key is press and the main should return a value.

ssharish