views:

61

answers:

2

I'm trying to do some C++ exercises, but I'm running into a error on build, which doesn't just jump out at me. What am I missing? I'm just getting back to C++ from C# et al after having done it years ago.

[ERROR] syntax error : 'return' [/ERROR]

#include <iostream>
using namespace std;

/* Pre-compiler directives / macros */
#define isValidDrinkChoice(Choice,MaxNumDrinks) ((Choice < MaxNumDrinks) && (Choice > 0))

/* Primary Entry Point for Executable */
int main(const int & argc, char * argv[]){

    const int MaxNumDrinks = 4;
    char ** Drinks;
    Drinks = new char* [MaxNumDrinks];
    Drinks[0] = "Soda";
    Drinks[1] = "Water";
    Drinks[2] = "Coffee";
    Drinks[3] = "Tea";
    Drinks[4] = "Perrier Sparkling Water";

    int Choice = -1;
    do while(!isValidDrinkChoice(Choice, MaxNumDrinks)) {
        cout << "Please select your favorite drink\r\n\r\n" << endl;
        for (int x = 0; x < MaxNumDrinks; x++) cout << "\t" << Drinks[x] << endl;
        cin >> Choice;
        if (isValidDrinkChoice(Choice, MaxNumDrinks)) cout << "\r\n\r\n" << "You chose " << *Drinks[Choice] << endl;
    }
    return 0;
}
+6  A: 

I don't think there's a do while like that in C++. It's do { ... } while (expression);. Or while (expression) { ... }.

Plynx
+3  A: 

The Corrected code sample its the while loop that needs replacement

#include <iostream>

namespace {
  bool isValidDrinkChoice(int Choice, int MaxNumDrinks) {
    return ((Choice < MaxNumDrinks) && (Choice >= 0));
  }
}

/* Primary Entry Point for Executable */
int main() {
    using namespace std;   

    const char *Drinks[] = {
      "Soda", "Water", "Coffee", "Tea", "Perrier Sparkling Water" };
    const int MaxNumDrinks = sizeof(Drinks) / sizeof(*Drinks);

    int Choice = -1;
    do  {
        cout << "Please select your favorite drink\r\n\r\n" << endl;
        for (int i = 0; i < MaxNumDrinks; i++) 
          cout << Drinks[i] << endl;

        cin >> Choice;

        if (isValidDrinkChoice(Choice, MaxNumDrinks)) 
          cout << "\r\n\r\n" << "You chose " << Drinks[Choice] << endl;
    } while(!isValidDrinkChoice(Choice, MaxNumDrinks) && cin) ;

    return cin.good() ? 0 : 1;
}
anijhaw
Sean Ochoa
J.F. Sebastian
@Sean Ochoa: on the namespace wrapper http://stackoverflow.com/questions/154469/unnamed-anonymous-namespaces-vs-static-functions
J.F. Sebastian