What is it I'm forgetting!?

I've been sitting here wondering what it is I'm forgetting and why it is NOT compiling for more than an hour, I've done it via pico and codepad

Can someone please help!

Code:

---

#include <stdio.h>
#include <stdlib.h>

main()

//HW2

{
int n=0;
int m=0;
int a=0;

        printf("Enter the amount of Adults:\n");

        scanf("%d", &n);


    if (n>2 || n<2) {
        printf("Error! you are required to input 2 adults\n");
    exit(1);
    }

        printf("Enter the amount of Children:\n");

        scanf("%d", &m);


    if (m>1 || m<1) {
        printf("Error! you are required to input 1 child\n");
    exit(1);
    }

        printf("How old is your child?:\n");

        scanf("%d", &a);

    if (a=<6)
    printf("Your child may enter the park for free.\n");
        else if (a => 7 || a =< 12)
    printf("Your child will get 10% off\n");
        else if (a > 12)
    printf("You do not have a child and they must pay the adult free.\n");

---

The errors I am getting via codepad: http://codepad.org/hs8IbtAI

The errors via pico

Code:

---

hw2.c: In function 'main':
hw2.c:37: error: expected expression before '<' token
hw2.c:39: error: expected expression before '>' token
hw2.c:42: error: expected declaration or statement at end of input

---

Guess "=<" is not right; try "<=".

Tim S.

It's because it's less-than-or-equal to i.e.

Code:

---

if (a <= 6)

---

Awesome got rid of 2/4 errors!

Now I'm getting

(??what is this) hw2.c: In function 'main':
(last line of the program) hw2.c:42: error: expected declaration or statement at end of input

Post your updated code -- all of it. Do you have a closing bracket for main? You should declare main to explicitly return an int, and put a return 0; at the end of main as well.

Quote:

---

Originally Posted by anduril462

Post your updated code -- all of it. Do you have a closing bracket for main? You should declare main to explicitly return an int, and put a return 0; at the end of main as well.

---

here is my updated script with still the 2 errors:

Code:

---


#include <stdio.h>
#include <stdlib.h>

main()
{
//HW2

{
int n=0;
int m=0;
int a=0;

        printf("Enter the amount of Adults:\n");

        scanf("%d", &n);


        if (n>2 || n<2) {
        printf("Error! you are required to input 2 adults\n");
        exit(1);
        }

        printf("Enter the amount of Children:\n");

        scanf("%d", &m);


        if (m>1 || m<1) {
        printf("Error! you are required to input 1 child\n");
        exit(1);
        }

        printf("How old is your child?:\n");

        scanf("%d", &a);

        if (a <=6 )
        printf("Your child may enter the park for free.\n");
                else if (a >= 7 || a <= 12)
    printf("Your child will get 10% off\n");
                else if (a > 12)
        printf("You do not have a child and they must pay the adult fee.\n");

}

---

here is the errors I am getting

Code:

---


hw2.c: In function 'main':
hw2.c:44: error: expected declaration or statement at end of input

---

Try this for main: int main(int argc, char **argv)

Learn to indent your code properly, and you would see that your brackets don't match up. Read this: SourceForge.net: Indentation - cpwiki

You should indent your code properly, e.g.,

Code:

---

#include <stdio.h>
#include <stdlib.h>

main()
{
    //HW2

    {
        int n = 0;
        int m = 0;
        int a = 0;

        printf("Enter the amount of Adults:\n");

        scanf("%d", &n);

        if (n > 2 || n < 2) {
            printf("Error! you are required to input 2 adults\n");
            exit(1);
        }

        printf("Enter the amount of Children:\n");

        scanf("%d", &m);

        if (m > 1 || m < 1) {
            printf("Error! you are required to input 1 child\n");
            exit(1);
        }

        printf("How old is your child?:\n");

        scanf("%d", &a);

        if (a <= 6)
            printf("Your child may enter the park for free.\n");
        else if (a >= 7 || a <= 12)
            printf("Your child will get 10% off\n");
        else if (a > 12)
            printf("You do not have a child and they must pay the adult fee.\n");

    }

---

Now, it is clear that you have an extra opening brace that should be removed.

Also, you should declare main as explicitly having a return type of int.

Code:

---

if (n > 2 || n < 2)

---

Seriously? Try:

Code:

---

if (n != 2)

---

Also, no need to have 3 ints when they're only being used once linearly.

Code:

---

#include <stdio.h>
#include <stdlib.h>
 
main()
{
    //HW2

    int n = 0;
    int m = 0;
    int a = 0;

    printf("Enter the amount of Adults:\n");
    scanf("%d", &n);

    if (n != 2) {
        printf("Error! you are required to input 2 adults\n");
        exit(1);
    }

    printf("Enter the amount of Children:\n");
    scanf("%d", &m);

    if (m != 1) {
        printf("Error! you are required to input 1 child\n");
        exit(1);
    }

    printf("How old is your child?:\n");
    scanf("%d", &a);

    if (a <= 6)
        printf("Your child may enter the park for free.\n");
    else if (a >= 7 || a <= 12)
        printf("Your child will get 10% off\n");
    else if (a > 12)
        printf("You do not have a child and they must pay the adult fee.\n");
   
    return 0;
   
}

---

Your variable names are awful, a could be for adults or age. n and m make no sense. Don't try to save a few keystrokes at the cost of clarity. Call them num_adults, num_children and child_age. Actually, why do you even bother asking for input for num_adults and num_children if there is only one allowed value for each? Typically, if you have a variable that can only ever have exactly one value, and it never changes, you make a constant. Besides, you aren't using their values for anything, so use them for something or get rid of them all together.

Also, when do you expect this to be false?

Code:

---

else if (a >= 7 || a <= 12)

---

Thanks for all the help guys! I cleaned up the code and will post an updated version in just a bit

Quote:

---

Originally Posted by nonoob

Try this for main: int main(int argc, char **argv)

---

My prof never taught us to

a) declare main
b) add the stuff in the bracket

Is it beneficial to do it that way or just keep it at

Code:

---

main(){
}

---

for now?

Quote:

---

Originally Posted by iamtazb

My prof never taught us to

a) declare main
b) add the stuff in the bracket

Is it beneficial to do it that way or just keep it at

Code:

---

main(){
}

---

for now?

---

FAQ > main() / void main() / int main() / int main(void) / int main(int argc, char *argv[]) - Cprogramming.com