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.

Quote:

---

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 ???

Quote:

---

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.

Quote:

---

Originally Posted by laserlight

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.

---

thanks