Thread: decision program ??

Code:

#include <iostream>
#include <string>


using namespace std;

int main(void)
{
    string name;
   string math;
   ;int a;
   ;int d;
   ;int matha;
   
   ;cout << "Enter name: \n"
    ;cin >> name;
    
   ;cout << "Enter math grade " << name << " got: \n"
   ;cin >> math;
   if (matha = a){cout << name << " passed \n"; }
   else 
   if (matha = d){cout << name << " failed \n";}
   ;system("PAUSE");
    return 0;
}

Moved to C++ programming forum.

One possible problem is because you are assigning (matha = a) instead of comparing for equality (matha == a).

Oh, and kindly indent your code more consistently and get rid of those superfluous empty statements.

Originally Posted by laserlight

Moved to C++ programming forum.

One possible problem is because you are assigning (matha = a) instead of comparing for equality (matha == a).

Oh, and kindly indent your code more consistently and get rid of those superfluous empty statements.

okay but its for C not C++ and please explain what you mean by indentting my code??
*edit* Ive placed two == but now it doesnt state whether its a pass or fail it completely ignores that ???

please explain what you mean by indentting my code??

Well, you are nearly there, but a little more consistency is good. For example:

Code:

#include <iostream>
#include <string>
#include <cstdlib>

using namespace std;

int main()
{
    string name;
    string math;
    int a;
    int d;
    int matha;

    cout << "Enter name: \n";
    cin >> name;

    cout << "Enter math grade " << name << " got: \n";
    cin >> math;
    if (matha = a)
    {
        cout << name << " passed \n";
    }
    else if (matha = d)
    {
        cout << name << " failed \n";
    }

    system("PAUSE");
    return 0;
}

I took the liberty of removing the superfluous empty statements as including <cstdlib> which is needed for system() (which you do not need to use in the first place), but I did not make the fixes I suggested.