78 Zeilen, wer bietet weniger?

DFPWare

Hallo Leute,
ich wollte hier einfach mal mein erstes C++ Skript vorstellen, es ist, wie könnte es anders sein ein Taschenrechner
Außerdem wollte ich mal wissen wie man den Code kürzer und effizienter gestalten kann, weil ich nicht unbedingt anfangen will sofort nur langsamen Code zu programmieren, also postet doch einfach mal eine gekürzte Version davon

Spoiler anzeigen

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <string>
using namespace std ;

char auswahl;
char nochmal;
bool berechnet = false;
bool exitloop = false;
double zahl1;
double zahl2;
double ergebnis;

void rechner()
{
cout << "Wollen sie addieren(a) subtrahieren(s) multiplizieren(m) oder dividieren(d)" << endl;
cin >> auswahl;
cout << "Geben sie Zahl 1 an:" << endl;
cin >> zahl1;
cout << "Geben sie Zahl 2 an:" << endl;
cin >> zahl2;
switch (auswahl)
{
case 'a' :
ergebnis = zahl1 + zahl2;
berechnet = true;
break;
case 's' :
ergebnis = zahl1 - zahl2;
berechnet = true;
break;
case 'm' :
ergebnis = zahl1 * zahl2;
berechnet = true;
break;
case 'd' :
if (zahl2 != 0)
{
ergebnis = zahl1 / zahl2;
berechnet = true;
}
else
{
berechnet = false;
}
break;
}
if (berechnet == true)
{
cout << "Das Ergebnis ist: " << ergebnis << endl;
}
else
{
cout << "FAIL!" << endl;
}
}

int _tmain()
{
rechner();
while (exitloop == false)
{
cout << "Nochmal(y) oder nicht (n)" << endl;
cin >> nochmal;
switch (nochmal)
{
case 'y' :
rechner();
break;
default :
exitloop = true;
break;
}
}
_getch();
return 0;
}

Danke
DFPWare

**Greenhorn**

Hi,

sieht doch ganz gut aus.
Allerdings enthält es noch einen schweren Fehler. Bei einer Division musst Du unbedingt darauf achten, dass der Divisor niemals null ist, andernfalls, wenn durch null geteilt wird, wird eine Ausnahme ausgelöst (Interrupt 0) und das Programm wird mit einer Fehlermeldung beendet.

Gruß
Greenhorn

DFPWare

Hey Danke, ich habs oben ergänzt
Jetzt muss ich wohl den Titel des Threads ändern
DFPWare

L3viathan

Zumindest die Zeilenanzahl kann ich kürzen

C

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <string>
char auswahl, nochmal; bool berechnet = false, exitloop = false; double zahl1, zahl2, ergebnis; void rechner() {std::cout << "Wollen sie addieren(a) subtrahieren(s) multiplizieren(m) oder dividieren(d)" << endl; std::cin >> auswahl; std::cout << "Geben sie Zahl 1 an:" << endl; std::cin >> zahl1; std::cout << "Geben sie Zahl 2 an:" << endl; std::cin >> zahl2; switch (auswahl) { case 'a' : ergebnis = zahl1 + zahl2; berechnet = true; break; case 's' : ergebnis = zahl1 - zahl2; berechnet = true; break; case 'm' : ergebnis = zahl1 * zahl2; berechnet = true; break; case 'd' : if (zahl2 != 0) { ergebnis = zahl1 / zahl2; berechnet = true; } else { berechnet = false; } break; } if (berechnet == true) { std::cout << "Das Ergebnis ist: " << ergebnis << endl; } else { std::cout << "FAIL!" << endl; } }  int _tmain() { rechner(); while (exitloop == false) { std::cout << "Nochmal(y) oder nicht (n)" << endl; std::cin >> nochmal; switch (nochmal) { case 'y' : rechner(); break; default : exitloop = true; break; } } _getch(); return 0; }

Tada, 5 Zeilen

**Greenhorn**

Ich biete 43 ...
Mit ein wenig quetschen wären noch ein paar Zeilen drin (aber wer will das dann noch lesen)...

Spoiler anzeigen

C

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <string>
using namespace std ;




double rechner (double operand_1, double operand_2, char* _operator) {


	if      (*_operator == '+') return (operand_1 + operand_2);
	else if (*_operator == '-') return (operand_1 - operand_2);
	else if (*_operator == '*') return (operand_1 * operand_2);
	else if (*_operator == '/') {
		if (operand_2 != 0)		// Divisor ungleich null ?
			return (operand_1 / operand_2);
		else
			cout << endl << "Ungültiger Divisor!" << endl << endl;
	} else
		cout << endl << "Ungültiger Operator!" << endl << endl;

	return 0;
}


int _tmain(int argc, _TCHAR* argv[]) {


	double zahl1, zahl2;
	char auswahl, nochmal = 'y';


	while (nochmal == 'y') {
		cout << endl << "Wollen Sie addieren(+), subtrahieren(-), multiplizieren(*) oder dividieren(/) ?" << endl;
		cin >> auswahl;
		cout << "Geben Sie Zahl 1 an: ";
		cin >> zahl1;
		cout << "Geben Sie Zahl 2 an: ";
		cin >> zahl2;


		cout << zahl1 << ' ' << auswahl << ' ' << zahl2 << " = " << rechner(zahl1, zahl2, &auswahl) << endl << endl;


		cout << "Nochmal(y) oder nicht(n) ?" << endl;
		cin >> nochmal;
	}
	return 0;
}

Alles anzeigen

Gruß
Greenhorn

DevFly

Ich weiß ist fies aber bin auf 57 Zeilen

Spoiler anzeigen

C

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <string>
using namespace std ;
char auswahl;
char nochmal;
bool berechnet = false;
bool exitloop = false;
double zahl1;
double zahl2;
double ergebnis;
void rechner() {
cout << "Wollen sie addieren(a) subtrahieren(s) multiplizieren(m) oder dividieren(d)" << endl;
cin >> auswahl;
cout << "Geben sie Zahl 1 an:" << endl;
cin >> zahl1;
cout << "Geben sie Zahl 2 an:" << endl;
cin >> zahl2;
switch (auswahl) {
case 'a' :
ergebnis = zahl1 + zahl2;
berechnet = true;
break;
case 's' :
ergebnis = zahl1 - zahl2;
berechnet = true;
break;
case 'm' :
ergebnis = zahl1 * zahl2;
berechnet = true;
break;
case 'd' :
if (zahl2 != 0) {
ergebnis = zahl1 / zahl2;
berechnet = true;
}else{
berechnet = false;}
break;}
if (berechnet == true){
cout << "Das Ergebnis ist: " << ergebnis << endl;
}else{
cout << "FAIL!" << endl;}}
int _tmain()
{ rechner();
while (exitloop == false) {
cout << "Nochmal(y) oder nicht (n)" << endl;
cin >> nochmal;
switch (nochmal){
case 'y' :
rechner();
break;
default :
exitloop = true;
break;}}
_getch();
return 0;}

Alles anzeigen

Edit: Mist da waren einige schneller

DFPWare

So hab jetzt selber noch ein wenig rumgespielt und ein paar Klammern weggelassen
Bin jetzt bei 59 Zeilen:)

Spoiler anzeigen

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <string>
using namespace std ;

char auswahl, nochmal = 'y';
bool berechnet = false;
double zahl1, zahl2, ergebnis;

void rechner()
{
cout << "Wollen sie addieren(a) subtrahieren(s) multiplizieren(m) oder dividieren(d)" << endl;
cin >> auswahl;
cout << "Geben sie Zahl 1 an:" << endl;
cin >> zahl1;
cout << "Geben sie Zahl 2 an:" << endl;
cin >> zahl2;
berechnet = false;
switch (auswahl)
{
case 'a' :
ergebnis = zahl1 + zahl2;
berechnet = true;
break;
case 's' :
ergebnis = zahl1 - zahl2;
berechnet = true;
break;
case 'm' :
ergebnis = zahl1 * zahl2;
berechnet = true;
break;
case 'd' :
if (zahl2 != 0)
{
ergebnis = zahl1 / zahl2;
berechnet = true;
}
else
berechnet = false;
break;
}
if (berechnet == true)
cout << "Das Ergebnis ist: " << ergebnis << endl;
else
cout << "FAIL!" << endl;
}

int _tmain()
{
while (nochmal == 'y')
{
rechner();
cout << "Nochmal(y) oder nicht (n)" << endl;
cin >> nochmal;
}
return 0;
}

@L3V: Mist ich hab vergessen, dass C++ nich zeilenorientiert ist -.-
Greenhorn: Danke, dank dir weiß ich jetzt, dass man nicht überall Klammern braucht
DevFly: Mist immer noch 2 Zeilen weniger

DFPWare

PS: Greenhorn wie hast du die Formatierung hinbekommen, ich hab FF und der unterstützt doch eigentlich alles oder?

progandy

Zitat von DFPWare

PS: Greenhorn wie hast du die Formatierung hinbekommen, ich hab FF und der unterstützt doch eigentlich alles oder?

Code wird in der Quellcode-Ansicht des Editors eingefügt und abgesendet

DFPWare

Spoiler anzeigen

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <string>
using namespace std ;

char auswahl, nochmal = 'y';
bool berechnet = false;
double zahl1, zahl2, ergebnis;

void rechner()
{
cout << "Wollen sie addieren(a) subtrahieren(s) multiplizieren(m) oder dividieren(d)" << endl;
cin >> auswahl;
cout << "Geben sie Zahl 1 an:" << endl;
cin >> zahl1;
cout << "Geben sie Zahl 2 an:" << endl;
cin >> zahl2;
berechnet = false;
switch (auswahl)
{
case 'a' :
ergebnis = zahl1 + zahl2;
berechnet = true;
break;
case 's' :
ergebnis = zahl1 - zahl2;
berechnet = true;
break;
case 'm' :
ergebnis = zahl1 * zahl2;
berechnet = true;
break;
case 'd' :
if (zahl2 != 0)
{
ergebnis = zahl1 / zahl2;
berechnet = true;
}
else
berechnet = false;
break;
}
if (berechnet == true)
cout << "Das Ergebnis ist: " << ergebnis << endl;
else
cout << "FAIL!" << endl;
}

int _tmain()
{
while (nochmal == 'y')
{
rechner();
cout << "Nochmal(y) oder nicht (n)" << endl;
cin >> nochmal;
}
return 0;
}

klappt nich
egal wie ich es drehe und wende
DFPWare

**Greenhorn**

Packe es in [code] Tags ...

Ich habe noch ein paar grobe Schnitzer abgefedert, aber ganz fehlerfrei ist es immer noch nicht ...

Spoiler anzeigen

C

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <string>
using namespace std ;




double rechner (double operand_1, double operand_2, char _operator) {


	if      (_operator == '+') return (operand_1 + operand_2);
	else if (_operator == '-') return (operand_1 - operand_2);
	else if (_operator == '*') return (operand_1 * operand_2);
	else if (_operator == '/') {
		if (operand_2 != 0)	  return (operand_1 / operand_2);
		else
			cout << endl << "Ungültiger Divisor!" << endl << endl;
	} else
		cout << endl << "Ungültiger Operator!" << endl << endl;

	return 0;
}


int _tmain(int argc, _TCHAR* argv[]) {


	double zahl1, zahl2;
	char auswahl [260], nochmal [260] = {'y','\0'};


	while (nochmal[0] == 'y') {
		cout << endl << "Wollen Sie addieren(+), subtrahieren(-), multiplizieren(*) oder dividieren(/) ?" << endl;
		cin >> auswahl;
		cout << "Geben Sie Zahl 1 an: "; cin >> zahl1;
		cout << "Geben Sie Zahl 2 an: "; cin >> zahl2;


		cout << zahl1 << ' ' << auswahl << ' ' << zahl2 << " = " << rechner(zahl1, zahl2, auswahl[0]) << endl << endl;


		cout << "Nochmal(y) oder nicht(n) ?" << endl;
		cin >> *nochmal;
	}
	return 0;
}

Alles anzeigen

Ah ja, habe noch um eine Zeile kürzen können, ohne das es unlesbar wird ...

EDIT:
Noch mal zwei Zeilen und immer noch lesbar ... (Aber nun ist Schluss)

Gruß
Greenhorn