Thread: Need Help!

Question2 take a char pointer instread of a single char.

Code:

int question2 ( char *user_input,int age);

Code:

#include <iostream.h>
#include <string.h>

int question1( int age);
int question2 ( char *user_input,int age);

int main()
{
    char string[256];          //define variables
    char user_input[10];
    int age;

    cout<<"This is a question and answer game!\n";   //Question 1:
    cout<<"Question 1: How old are you? ";           //How old are you
    cin>>age;                                        //input
    question1(age);                                  //goto function q1
   cin.ignore(1);
    cout<<"Question 2: Do you get pocket money?\n";
    cin.getline(user_input, 10, '\n');
    question2(user_input, age);                        //unknown error

    
}

int question1 (int age)
{
    if (age > 13)                          //if age more then 13
    {
      cout<<"You are older then me!\n";   //output you are older then me on the screen
    }
    else
    {
     cout<<"You are younger then me!\n";  //otherwise say you are younger then me
    }
    return 0;                              //return 0
}
int question2 ( char *user_input, int age)
{
   if (!strcmpi("no", user_input))                     //no error now
   {
    if (age < 15)
    {
      cout<<"LOL you dont get pocket money\n";
    }

   }
   return 0;
}

this works now..

changes made:
>int question2 ( char *user_input,int age);
>cin.ignore(1);

and dont define functions inside functions.

Thx. Why musnt i use system() functions?

:sigh: another error it says "Parse error at end of input line 68" there is only 67 lines?

Code:

int question2 ( char *user_input, int age)
{
  if (!strcmpi("no", user_input))
   {
    if (age > 15)
    {
      cout<<"LOL you dont get pocket money and you are"<<age<<"\n";
    }
    else
    {
      cout<<"If I were you I'd beg your parents for money\n";
    }
   }
    else
   {
    if (age > 12)
   {
    cout<<"You are lucky you have money in your pocket so DONT waste it!\n";
   }
   else
   {
    cout<<"When I was your age I didnt get pocket money\n";
   }

   return 0;
}

>> #include <iostream.h>

That is an outdated and non-standard header file. You should be using the new, standard headers. In this case, that would be <iostream>. The new headers put names into something called a namespace, so you'll have to handle just that small change. I prefer to add std:: in front of cout, cin, endl, and anything else I use from the standard library, but the simplest solution is to just put the line "using namespace std;" without the quotes under the #include lines.

Also, since you are just learning C++, you should be learning C++ instead of C style C++. That means learning the C++ string class instead of character arrays. Consider finding newer tutorials that teach modern C++.

Do you know where I can find these "Modern" tutorials.

Honestly, no, but I've never really looked. Basically, look for tutorials that have <iostream> instead of <iostream.h>, and that teach the C++ string class. I personally would use a book like Accelerated C++. You should at least find a tutorial that teaches <iostream>. Tutorials that teach the string class are unfortunately not all that common, since the C++ standard hasn't been around for that long. The tutorials on this site might do it. You can also search the forum to see other people asking for good tutorials.

The string class is actually pretty easy. It acts like a built-in type like int or double. You can find information about it at a reference site like cppreference.com.

Tutorials, right under your nose. Oh, man!

I am learning C++ from those tutorials. My compiler cant use <iostream> only <iostream.h> i have dev C++

I think I am using V4.0. I dont like using beta versions. I'll try it anyway