syntax error at end of input

Can someone please tell we where the error in this code is i keep getting an error on the last line saying "syntax error at end of input" need help quick please.

Code:

int main()
{
	// Welcome message
	printf("Welcome to RoboSimoClient\n\n");

	// Ask user for server IP address, then connect to it
	int counter;
	int a, b, c, d, port;
	printf("Please enter server IP address: ");
	scanf("%d.%d.%d.%d", &a, &b, &c, &d);
	printf("Please enter port number: ");
	scanf("%d", &port);	
	connect_to_server(a, b, c, d, port);
	// circle the arena
	while(1)
	{
		set_motor_speeds(100,100);
		counter = 1;
		while(read_analog_channel(0) < 120 && read_analog_channel(1) < 120);
		{
			set_motor_directions(1, 1);
		}
		set_motor_directions(0, 0);
		
		if(read_analog_channel(0) > 120 && read_analog_channel(1) < 120);
		{
			while(counter > 0);
			{
				if(read_analog_channel(0) > 120 && read_analog_channel(1) < 120)
				{
					set_motor_directions(1, 0);
				}
				else
				{
					set_motor_directions(1, 1);	
				}
				counter = counter + 1;
		}
		
		if(read_analog_channel(0) < 120 && read_analog_channel(1) > 120);
		{
			while(counter > 0);
			{
				if(read_analog_channel(0) < 120 && read_analog_channel(1) > 120)
				{
					set_motor_directions(0, 1);
				}
				else
				{
					set_motor_directions(0, 0);
				}
				counter = counter + 1;
		}
	}
	return 1;
}

Aside from missing header stdio.h and no function defs for read_analog_channel(), etc, the biggest problem I see is the while() statement and the first if() are missing their closing braces. As such, the compiler goes off the end of the file looking for the missing braces.

HTH

Jeff

Originally Posted by jeffcobb

Aside from missing header stdio.h and no function defs for read_analog_channel(), etc, the biggest problem I see is the while() statement and the first if() are missing their closing braces. As such, the compiler goes off the end of the file looking for the missing braces.

HTH

Jeff

your a hero thank you so much

Some problems were mentioned above, which may/may noe be duplicated here.

Code:

int main() // put "main(void)" instead
{
    // Welcome message
    printf("Welcome to RoboSimoClient\n\n");

    // Ask user for server IP address, then connect to it
    int counter;
    int a, b, c, d, port;
    printf("Please enter server IP address: ");
    scanf("%d.%d.%d.%d", &a, &b, &c, &d);
    printf("Please enter port number: ");
    scanf("%d", &port);    
    connect_to_server(a, b, c, d, port); //function doesnt exist (if it does, you should have included it in your post!)
    // circle the arena
    while(1)
    {
        set_motor_speeds(100,100); // function doesnt exist
        counter = 1;
        while(read_analog_channel(0) < 120 && read_analog_channel(1) < 120); // you probably dont want this trailing semicolon; also the function your calling doesnt exist
        {
            set_motor_directions(1, 1);  // function doesnt exist
         }
        set_motor_directions(0, 0);
        
        if(read_analog_channel(0) > 120 && read_analog_channel(1) < 120); // trailing semi colon probably not wanted
        {
            while(counter > 0); // semicolon
            {
                if(read_analog_channel(0) > 120 && read_analog_channel(1) < 120)
                {
                    set_motor_directions(1, 0);
                }
                else
                {
                    set_motor_directions(1, 1);    
                }
                counter = counter + 1;
           // missing close bracket
        }
        
        if(read_analog_channel(0) < 120 && read_analog_channel(1) > 120); //semi colon
        {
            while(counter > 0); // what the hell
            {
                if(read_analog_channel(0) < 120 && read_analog_channel(1) > 120)
                {
                    set_motor_directions(0, 1);
                }
                else
                {
                    set_motor_directions(0, 0);
                }
                counter = counter + 1;
            // missing close bracket for while loop 
        }
    }
    return 1;
}

Originally Posted by sauoon

your a hero thank you so much

The easiest way to pick up on these kinds of brace-related errors is for me twofold:
1. Don't write any function or code-block larger than a screen in length. If you need to make (say) a while() loop shorter, export the guts of it to an external function and then simply calling them from the while. For example is this not easier to catch?

Code:

while(some_condition)
{
       doSomeFunction();
}

2. If your editor has a built-in code-formatter, use that on your entire file. That is what I used to find your issues. In Emacs it was Ctrl-Alt-|. When I did that it was obvious.

Happy Sunday...

Jeff

This is not C++...
I would advise to post in the proper section next time.
Furthermore, quick help is not a good title. Don't use it. People don't hurry to your post regardless. Instead type a proper title and there is a bigger chance people will read it.