getting error message for greedy expeted identifier

Question

jharvard@appliance (~/Dropbox/pset1): make greedy
clang -ggdb3 -O0 -std=c99 -Wall -Werror    greedy.c  -lcs50 -lm -o greedy
greedy.c:13:4: error: expected identifier or '('
   { 
   ^
1 error generated.
make: *** [greedy] Error 1

Here is the code it refers to:

#include <stdio.h>
#include <cs50.h>
#include <math.h>

int main (void);

int money;
int quater;
int dime; 
int nickle;
int penny;

{ 
do 
{
    printf("Please give me the amount of change owed.\n")
    int money = Getfloat ();
}
while 
     money > 0.00;

When I remove the curley braces it tells me the do and the while are incorrect. I have cross referenced it with other project and I have it set up the same way and that program is perfect in check50.

I am getting some failed checks when the vm boots up and I am not sure if this has something to do with this or not. I have tried shutting it down and rebooting several times.

Answer 1

You have a series of issues here, all to do with correct program structure. Some of them are hiding others.

First, int main (void); has a semicolon. This is causing problems and masking others. main should be structured like this:

// includes go here
#include....

// followed by main
int main(void)
{
    //main's code goes here, including variable definitions
}

Next, do/while is structured like this:

do
{
    // code goes here
} 
while ( test condition );

That should get you started. You would benefit from reviewing the material on program syntax, curly braces, and while loops.

If this answers your question, please accept this answer to remove the question from the unanswered question pool. Let's keep up on forum maintenance. ;-)