Ad:

C++ Discussion Thread
Marked Solved
Views: 919

Page 1
2
Next

You are currently viewing page 1 of this multi-page discussion thread

Jul 21st, 2009

C++ - Need help solving a problem with my Calculator Script.

Expand Post »

Hello, this is Metalcrunch. I've been learning C++ over the past days and I thought I'd make a calculator. So I looked over the existing code on the internet on how to make a calculator, understood it, then made my own from the very scratch.

However, I'm having a few problems with it - Hope you guys can help me out!

main_calculator.cpp

c++ Syntax (Toggle Plain Text)
//Calculator Script by Sass.
//main_calculator.cpp
 
//Default include's.
#include <iostream>
#include <windows.h>
using namespace std;
 
//Custom header file.
#include "header_calculator.h"
 
//Main()
int main()
{
	//One-time welcome greeting:
	 SetConsoleTitle("Calculator v1.1");
	 cout << "Kalkulaator v1.1 by MetalCrunch." << endl;
	 cout << "---------------------------------------" << endl << endl;
 
    //Main program:
	 for (;;) {
	  do {
	   cout << "1 - Addition." << endl;
	   cout << "2 - Subtraction." << endl;
	   cout << "3 - Multiplication." << endl;
	   cout << "4 - Division." << endl;
	   cout << "5 - Abimees." << endl;
	   cin >> choice; }
 
    switch (choice) {
     //Addition.
	 case 1:
	   cout << "Enter the first number: ";
        cin >> num1;
	   cout << "\nEnter the second number: ";
		cin >> num2;
	    result = num1 + num2;
	   cout << "Answer: ";
	   cout << result;
	   break;
 
     //Subtraction.
	 case 2:
	   cout << "Enter the first number: ";
        cin >> num1;
	   cout << "\nEnter the second number: ";
		cin >> num2;
		result = num1 - num2;
	   cout << "\nAnswer: ";
	   cout << result;
	   break;
 
     //Multiplication.
	 case 3:
		 cout << "Enter the first number: ";
		  cin >> num1;
		 cout << "\nEnter the second number: ";
		  cin >> num2;
		  result = num1 * num2;
		 cout << "\nAnswer: ";
		 cout << result;
		  break;
 
     //Division.
	 case 4:
		 cout << "Enter the first number: ";
		  cin >> num1;
	     cout << "\n Enter the second number: ";
		  cin >> num2;
		  result = num1 / num2;
		 cout << "\nAnswer: ";
		 cout << result;
		  break;
 
     //"Information".
	 case 5:
		 cout << "Calculator v1.1 by Metalcrunch" << endl;
		 cout << "Enter the number 1-5, then answer the questions..." << endl;
		 cout << "The program will solve it all." << endl;
		 cout << "But please don't enter a value over 9 numbers, the program will crash!" << endl;
		 break;
 
	 default:
	     cout << "test";
	     break; }
	 }
 return 0;
}
//Calculator Script by Sass.
//main_calculator.cpp

//Default include's.
#include <iostream>
#include <windows.h>
using namespace std;

//Custom header file.
#include "header_calculator.h"

//Main()
int main()
{
	//One-time welcome greeting:
	 SetConsoleTitle("Calculator v1.1");
	 cout << "Kalkulaator v1.1 by MetalCrunch." << endl;
	 cout << "---------------------------------------" << endl << endl;

    //Main program:
	 for (;;) {
	  do {
	   cout << "1 - Addition." << endl;
	   cout << "2 - Subtraction." << endl;
	   cout << "3 - Multiplication." << endl;
	   cout << "4 - Division." << endl;
	   cout << "5 - Abimees." << endl;
	   cin >> choice; }

    switch (choice) {
     //Addition.
	 case 1:
	   cout << "Enter the first number: ";
        cin >> num1;
	   cout << "\nEnter the second number: ";
		cin >> num2;
	    result = num1 + num2;
	   cout << "Answer: ";
	   cout << result;
	   break;

     //Subtraction.
	 case 2:
	   cout << "Enter the first number: ";
        cin >> num1;
	   cout << "\nEnter the second number: ";
		cin >> num2;
		result = num1 - num2;
	   cout << "\nAnswer: ";
	   cout << result;
	   break;

     //Multiplication.
	 case 3:
		 cout << "Enter the first number: ";
		  cin >> num1;
		 cout << "\nEnter the second number: ";
		  cin >> num2;
		  result = num1 * num2;
		 cout << "\nAnswer: ";
		 cout << result;
		  break;

     //Division.
	 case 4:
		 cout << "Enter the first number: ";
		  cin >> num1;
	     cout << "\n Enter the second number: ";
		  cin >> num2;
		  result = num1 / num2;
		 cout << "\nAnswer: ";
		 cout << result;
		  break;

     //"Information".
	 case 5:
		 cout << "Calculator v1.1 by Metalcrunch" << endl;
		 cout << "Enter the number 1-5, then answer the questions..." << endl;
		 cout << "The program will solve it all." << endl;
		 cout << "But please don't enter a value over 9 numbers, the program will crash!" << endl;
		 break;

	 default:
	     cout << "test";
	     break; }
	 }
 return 0;
}

header_calculator.h

c++ Syntax (Toggle Plain Text)
//Calculator Script
//header_calculator.h
 
#ifndef HEADER_CALCULATOR_H
#define HEADER_CALCULATOR_H
 
//Main int's, for calculation.
int num1, num2;
int result;
 
//The "choice" int.
int choice;
 
#endif
//Calculator Script
//header_calculator.h

#ifndef HEADER_CALCULATOR_H
#define HEADER_CALCULATOR_H

//Main int's, for calculation.
int num1, num2;
int result;

//The "choice" int.
int choice;

#endif

The build log.

C++ Syntax (Toggle Plain Text)
Compiling...
main_calculator.cpp
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(31) : error C2059: syntax error : 'switch'
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(31) : error C2143: syntax error : missing ';' before '{'
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(33) : error C2046: illegal case
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(44) : error C2046: illegal case
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(55) : error C2046: illegal case
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(66) : error C2046: illegal case
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(77) : error C2046: illegal case
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(84) : error C2047: illegal default
Compiling...
main_calculator.cpp
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(31) : error C2059: syntax error : 'switch'
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(31) : error C2143: syntax error : missing ';' before '{'
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(33) : error C2046: illegal case
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(44) : error C2046: illegal case
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(55) : error C2046: illegal case
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(66) : error C2046: illegal case
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(77) : error C2046: illegal case
c:\documents and settings\*********\my documents\visual studio 2008\projects\project1\calculator\calculator\main_calculator.cpp(84) : error C2047: illegal default

I can't honestly figure it out ... I'd really, really like some help!
metalcrunch

Similar Threads

how to solve this in IT Professionals' Lounge
Problem to find smallest no divisible by all of the first 20 natural numbers in C++
Delete columns in Perl
Help solving this problem in C++
isFixReg() ??? in Java

metalclunch

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

21 posts
since Jul 2009

Permalink

Jul 21st, 2009

Re: C++ - Need help solving a problem with my Calculator Script.

its because you are missing the while part of a do while loop
do {

} while (some condition);

AceofSpades19

Reputation Points: 61

Solved Threads: 10

Junior Poster in Training

Offline

61 posts
since Jun 2008

Permalink

Jul 21st, 2009

Re: C++ - Need help solving a problem with my Calculator Script.

C++ Syntax (Toggle Plain Text)
 
#include <iostream>
double a = 0;
double b = 0;
double c = 0;
double d = 0;
double add = 0;
double sub = 0;
double mul = 0;
double f = 0;
int main () {
	std::cout << "Enter any one of the following " << std::endl;
	std::cout << "1 for ADDITION " << std::endl;
	std::cout << "2 for SUBTRACTOIN " << std::endl;
	std::cout << "3 for MULTIPLICATOIN " << std::endl;
	std::cout << "4 for DIVISION " << std::endl;
	std::cin >> a;
	std::cout << ' ' << std::endl;
	std::cout << "Enter first number: " <<std::endl;
	std::cin >> b;
	std::cout<< ' ' << std::endl;
	std::cout << "Enter second number: " << std::endl;
	std::cin >> c;
	std::cout<< ' ' << std::endl;
	if ( a == 1) {
		add = b + c;
		std::cout << add <<  std::endl;
	}else if ( a == 2){
		sub = b - c;
		std::cout << sub <<  std::endl;
	}else if (a == 3) {
		mul = b * c;
		std::cout << mul <<  std::endl;
    }else if (a == 4){
		f = b / c;
		std::cout << f <<  std::endl;
 
	}
 
 
 
}
#include <iostream>
double a = 0;
double b = 0;
double c = 0;
double d = 0;
double add = 0;
double sub = 0;
double mul = 0;
double f = 0;
int main () {
	std::cout << "Enter any one of the following " << std::endl;
	std::cout << "1 for ADDITION " << std::endl;
	std::cout << "2 for SUBTRACTOIN " << std::endl;
	std::cout << "3 for MULTIPLICATOIN " << std::endl;
	std::cout << "4 for DIVISION " << std::endl;
	std::cin >> a;
	std::cout << ' ' << std::endl;
	std::cout << "Enter first number: " <<std::endl;
	std::cin >> b;
	std::cout<< ' ' << std::endl;
	std::cout << "Enter second number: " << std::endl;
	std::cin >> c;
	std::cout<< ' ' << std::endl;
	if ( a == 1) {
		add = b + c;
		std::cout << add <<  std::endl;
	}else if ( a == 2){
		sub = b - c;
		std::cout << sub <<  std::endl;
	}else if (a == 3) {
		mul = b * c;
		std::cout << mul <<  std::endl;
    }else if (a == 4){
		f = b / c;
		std::cout << f <<  std::endl;

	}
	


}

well here is simple solution to your very simple project

Last edited by Ancient Dragon; Jul 21st, 2009 at 10:47 am. Reason: correct code tags

ALI INAM

Reputation Points: 2

Solved Threads: 1

Newbie Poster

Offline

3 posts
since Jul 2009

Permalink

Jul 21st, 2009

Re: C++ - Need help solving a problem with my Calculator Script.

ALI INAM, you should urgently stop posting such rubbish solutions, BTW: why making every variable global?
Ever heard of local variables?

Do you really think the OP will learn from such solutions?
By just giving code to the OP, you won't help him, remember that.
It's against Daniweb's homework policy.

Links for (I hope) a permanent reminder:
http://www.daniweb.com/forums/announcement118-2.html
http://www.daniweb.com/forums/announcement118-3.html

tux4life

Reputation Points: 2125

Solved Threads: 243

Postaholic

Offline

2,105 posts
since Feb 2009

Permalink

Jul 21st, 2009

Re: C++ - Need help solving a problem with my Calculator Script.

Hi, me again. Well, first of all, big thanks to AceOfSpade19 for clearing that up to me.
Second of all, thank you ALI INAM for trying to help me, however next time try putting the [.code=c++][./code] tags.

And third of all, I'm still stuck. Here's how the Main Program part looks like currently.

c++ Syntax (Toggle Plain Text)
    //Main program:
	 for (;;) {
	  do {
	   cout << "1 - Addition." << endl;
	   cout << "2 - Subtraction." << endl;
	   cout << "3 - Multiplication." << endl;
	   cout << "4 - Division." << endl;
	   cout << "5 - Abimees." << endl;
	   cin >> choice; }
 
    switch (choice) {
     //Addition.
    //Main program:
	 for (;;) {
	  do {
	   cout << "1 - Addition." << endl;
	   cout << "2 - Subtraction." << endl;
	   cout << "3 - Multiplication." << endl;
	   cout << "4 - Division." << endl;
	   cout << "5 - Abimees." << endl;
	   cin >> choice; }

    switch (choice) {
     //Addition.

Now, that was the default Main Code. If I add the "while" command to it like this,

c++ Syntax (Toggle Plain Text)
    //Main program:
	 for (;;) {
	  do {
	   cout << "1 - Addition." << endl;
	   cout << "2 - Subtraction." << endl;
	   cout << "3 - Multiplication." << endl;
	   cout << "4 - Division." << endl;
	   cout << "5 - Abimees." << endl;
	   cin >> choice; }
	  while (choice < 1 || choice > 5) {
	   break; }
 
    switch (choice) {
     //Addition.
    //Main program:
	 for (;;) {
	  do {
	   cout << "1 - Addition." << endl;
	   cout << "2 - Subtraction." << endl;
	   cout << "3 - Multiplication." << endl;
	   cout << "4 - Division." << endl;
	   cout << "5 - Abimees." << endl;
	   cin >> choice; }
	  while (choice < 1 || choice > 5) {
	   break; }

    switch (choice) {
     //Addition.

Then it will get me this error.

C++ Syntax (Toggle Plain Text)

error C2143: syntax error : missing ';' before '{'

If I follow the error, and put it like this,

c++ Syntax (Toggle Plain Text)
    //Main program:
	 for (;;) {
	  do {
	   cout << "1 - Addition." << endl;
	   cout << "2 - Subtraction." << endl;
	   cout << "3 - Multiplication." << endl;
	   cout << "4 - Division." << endl;
	   cout << "5 - Abimees." << endl;
	   cin >> choice; }
	  while (choice < 1 || choice > 5); {
	   break; }
 
    switch (choice) {
     //Addition.
    //Main program:
	 for (;;) {
	  do {
	   cout << "1 - Addition." << endl;
	   cout << "2 - Subtraction." << endl;
	   cout << "3 - Multiplication." << endl;
	   cout << "4 - Division." << endl;
	   cout << "5 - Abimees." << endl;
	   cin >> choice; }
	  while (choice < 1 || choice > 5); {
	   break; }

    switch (choice) {
     //Addition.

It will compile "succesfully" but the script is broken, if I enter any number/letter and press Enter it will exit immediately... Right..

I also tried to make it like this.

c++ Syntax (Toggle Plain Text)
    //Main program:
	 for (;;) {
	  do {
	   cout << "1 - Addition." << endl;
	   cout << "2 - Subtraction." << endl;
	   cout << "3 - Multiplication." << endl;
	   cout << "4 - Division." << endl;
	   cout << "5 - Abimees." << endl;
	   cin >> choice; }
	  while (choice < 1 || choice > 5)
	   break;
 
    switch (choice) {
     //Addition.
    //Main program:
	 for (;;) {
	  do {
	   cout << "1 - Addition." << endl;
	   cout << "2 - Subtraction." << endl;
	   cout << "3 - Multiplication." << endl;
	   cout << "4 - Division." << endl;
	   cout << "5 - Abimees." << endl;
	   cin >> choice; }
	  while (choice < 1 || choice > 5)
	   break;

    switch (choice) {
     //Addition.

However, that didn't help much, because I get this error.

C++ Syntax (Toggle Plain Text)

syntax error : missing ';' before 'break'

Well, I'm stuck... Again.. Would be nice if anyone would told me what to do... Well, would be even better, if I would be told what to put in the "while" statement...

metalclunch

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

21 posts
since Jul 2009

Permalink

Jul 21st, 2009

Re: C++ - Need help solving a problem with my Calculator Script.

the syntax for a do-while loop is this:

c++ Syntax (Toggle Plain Text)
do
{
     //stuff to loop through
}
while(conditions); //note the colon
do
{
     //stuff to loop through
}
while(conditions); //note the colon

and maybe consider changing your 'switch' statement to a series of if-else since there's more than 1 or 2 lines of code for each option. I think that's more of a style issue.

Edit: one more thing I noticed

you have your 'for(;

' but once you run through your do-while loop, you have a break, exiting the for loop. So you only run through the for loop once, maybe look at why you have a break there

Hope that's helpful

Last edited by jesseb07; Jul 21st, 2009 at 9:03 am.

jesseb07

Reputation Points: 76

Solved Threads: 15

Junior Poster

Offline

111 posts
since Dec 2006

Permalink

Jul 21st, 2009

Re: C++ - Need help solving a problem with my Calculator Script.

Well, thanks, mate... This is what I got, and it works..

c++ Syntax (Toggle Plain Text)
    //Main program:
	 for (;;) {
	  do {
	   cout << "1 - Addition." << endl;
	   cout << "2 - Subtraction." << endl;
	   cout << "3 - Multiplication." << endl;
	   cout << "4 - Divide." << endl;
	   cout << "5 - Help." << endl;
	   cin >> choice; }
	  while (choice < 1 || choice > 5);
 
    switch (choice) {
	 default:
	     cout << "If you see me, report the 'default' bug!";
 
     //Liitmine.
    //Main program:
	 for (;;) {
	  do {
	   cout << "1 - Addition." << endl;
	   cout << "2 - Subtraction." << endl;
	   cout << "3 - Multiplication." << endl;
	   cout << "4 - Divide." << endl;
	   cout << "5 - Help." << endl;
	   cin >> choice; }
	  while (choice < 1 || choice > 5);

    switch (choice) {
	 default:
	     cout << "If you see me, report the 'default' bug!";

     //Liitmine.

Although, I have to admit... I don't fully understand why do I need this.

c++ Syntax (Toggle Plain Text)
	  while (choice < 1 || choice > 5);
	  while (choice < 1 || choice > 5);

If I change it to like, "choice > 1 || choice < 5" then it stops working. But that seems more logical, doesn't it?

I won't mark it as solved, yet, because I'd like to know why do I need the "while" statement to be like that (shown in the code tags, previously). Is it because if you put a number below 1 and over 5 it will "re-do" the Do function? Seems to function like so...

Thank you all, though.
metalclunch

Last edited by metalclunch; Jul 21st, 2009 at 9:52 am.

metalclunch

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

21 posts
since Jul 2009

Permalink

Jul 21st, 2009

Re: C++ - Need help solving a problem with my Calculator Script.

Click to Expand / Collapse Quote originally posted by metalclunch ...

Although, I have to admit... I don't fully understand why do I need this.
c++ Syntax (Toggle Plain Text)
	  while (choice < 1 || choice > 5);
while (choice < 1 || choice > 5);
If I change it to like, "choice > 1 || choice < 5" then it stops working. But that seems more logical, doesn't it?

>Although, I have to admit... I don't fully understand why do I need this.

c++ Syntax (Toggle Plain Text)
	  while (choice < 1 || choice > 5);
	  while (choice < 1 || choice > 5);

Well, this way works because: if the choice entered by the user is a number, lower than one, OR a number higher than five, then the choice is invalid.

>If I change it to like, "choice > 1 || choice < 5" then it stops working. But that seems more logical, doesn't it?
I know what you mean, if you want to have it work in that way, then you have to replace the logical OR by a logical AND, so that it becomes: choice > 0 && choice < 5 (I also changed the one to a zero here).
(Because the number has to be higher than zero AND lower than five, it's not enough that just one of these conditions is true, they must be both true).

Last edited by tux4life; Jul 21st, 2009 at 9:59 am.

tux4life

Reputation Points: 2125

Solved Threads: 243

Postaholic

Offline

2,105 posts
since Feb 2009

Permalink

Jul 21st, 2009

Re: C++ - Need help solving a problem with my Calculator Script.

Thanks, many thanks indeed. I've got my Calculator script fully working now. I also "optimized" the code, by cleaning it up and also making it more readable.

I guess I can Mark this as Solved. +rep to the people who helped me, of course!

metalclunch

Reputation Points: 10

Solved Threads: 0

Newbie Poster

Offline

21 posts
since Jul 2009

Permalink

Jul 21st, 2009

Re: C++ - Need help solving a problem with my Calculator Script.

Quote ...

choice > 0 && choice < 5 (I also changed the one to a zero here).

Hmm... if the range is 1 - 5 (5 included), shouldn't we use choice > 0 && choice <6 or choice > 0 && choice <= 5 ?

23.12.2012

Reputation Points: 32

Solved Threads: 1

Newbie Poster

Offline

18 posts
since Jul 2009

Page 1
2
Next

This thread is solved

Either the thread starter or a moderator has marked this thread as solved. You can most likely trust the responses and answers given. There is most likely no reason for any further responses to be posted here. If you have a related question, please start a new thread in this forum instead.

This thread is more than three months old

No one has posted to this discussion for at least three months. Please let old threads die and do not reply to them unless you feel you have something new and valuable to contribute that absolutely must be added to make the discussion complete. Otherwise, please start a new thread in this forum instead.

Previous Thread in C++ Forum Timeline: AfxBeginThread Memory Leak?

Next Thread in C++ Forum Timeline: I really don't know how to make programs....please Solve this for me

DaniWeb Message

Cancel Changes

User Name
Password