Thread: First Project-Calculator

Code:

:mMenu;

This should be

Code:

mMenu:

Code:

cout<<div1"/"div2"="divO"\n";

You can't string stuff together like that, try this:

Code:

cout<<div1<<"/"<<div2<<"="<<divO<<"\n";

Code:

    if(mMenuc==3)
    {
                 break;
    }

Change this to

Code:

    if(mMenuc==3)
    {
                return 0;
    }

EDIT
People may also advise against the use labels and system calls.

EDIT 2
I don't know if it was intentional or not but you spelt your name wrong.
Nebuchadnezzar

Code:

int main(int argc, char *argv[])

should be

Code:

int main(int argc, char argv[])

Code:

:mMenu;

i don't even know why this is here

fixed, you'd better read some c books before write the code.
and, don't use goto unless you have to.

Code:

//Mini calculator used for multiplication and division
#include <cstdlib>
#include <iostream>
#include <cstdio>

using namespace std;

int main(int argc, char *argv[])
{
    //ask for conversion of Multiplacation and Division
    :mMenu;
    system("cls");
    cout<<"Calculator Mini v1\n\n";
    cout<<"1. Multiplacation.\n";
    cout<<"2. Division.\n";
    cout<<"3. Exit.\n";
    int mMenuc;
    cout<<"\nYour decision (1,2,3):";
    cin>>mMenuc;
    
    //Multiplacation
    if (mMenuc==1)
    {
                  //value 1
                  float multi1;
                  system("cls");
                  cout<<"Mini Calculator v1\n\n";
                  cout<<"Please enter value 1:";
                  cin>>multi1;
                  
                  //value 2
                  float multi2;
                  system("cls");
                  cout<<"Mini Calculator v1\n\n";
                  cout<<"Please enter value 2:";
                  cin>>multi2;
                  
                  //Output
                  float multiO;
                  multiO=multi1*multi2;
                  system("cls");
                  cout<<"Mini Calculator v1\n\n";
                  cout<<multi1"x"multi2"="multiO"\n";                  
                  system("PAUSE");
                  goto mMenu;
    }
    
    //division
    if (mMenuc==2)
    {
                  //value 1
                  float div1;
                  system("cls");
                  cout<<"Mini Calculator v1\n\n";
                  cout<<"Please enter value 1:";
                  cin>>div1;
                  
                  //value 2
                  float div2;
                  system("cls");
                  cout<<"Mini Calculator v1\n\n";
                  cout<<"Please enter value 2:";
                  cin>>div2;
                  
                  //Output
                  float divO;
                  divO=div1/div2;
                  system("cls");
                  cout<<"Mini Calculator v1\n\n";
                  cout<<div1"/"div2"="divO"\n";                  
                  system("PAUSE");
                  goto mMenu;
    }
    
    //Exit
    if(mMenuc==3)
    {
                 break;
    }
    
    //failsafe 1
    if(mMenuc>=4)
    {
                 system("cls");
                 cout<<"ERROR: Incorrect input, restarting";
                 system("pause");
                 goto mMenu;
    }
    
    //failsafe 2
    if(mMenuc<=0)
    {
                 system("cls");
                 cout<<"ERROR: Incorrect input, restarting.";
                 system("pause");
                 goto mMenu;
    }
    return EXIT_SUCCESS;
}

I know, I know I suck at coding but I have to start somewhere.

OK, thx for all of your replies and help, also I Was reading a C++ book, however it was rather old, and as such alot of the code isnt compatible with my compiler (so the authors website said.)
addressed to The Brain: unless you can tell me another way of allowing the user to return to the main menu, I think Ill have to stick with goto etc.

And BTW, Nebbuchadnezzar IS spelt with 1 b, although Ive always thought it was 2, Nebuchadnezzar was Babylonian general and Nebuchadnezzar II had the hanging gardens of Babylon, one of the wonders of the ancient world, built for his wife. The ruins of Babylon can be found south of Baghdad.

Again thank you for your help.

Instead of using the goto function, you could place the main menu in a function, so that it can be called later in your program. For example...

Code:

int main(...)
{
menu();
return 0;
}

void menu()
{
// main menu goes here
}