Calcolatrice in C++

[DjCrash]

Ciao ragazzi,sto programmando una calcolatrice,ma con dev c++ mi da un errore.
Questo è il codice [C++] Calcolatrice c++ - Pastebin.com

Grazie in anticipo,

-DjCrash

 

[Tinwor]

NON e' C++. E' C con librerie C++. Finche' nn scrivete giusto nn rispondero'

 

[pabloski]

In effetti è un brutto mix di C e C++. Ma esattamente l'errore qual'è? Senza sapere che errore ti dà, è difficile dire cosa può essere.

 

[signore del tempo]

Utilizzi cout/cin/endl senza <iostream>.
Oltretutto l'algoritmo va rifatto, ci sono troppe ripetizioni.
Un'idea di come dovrebbe essere:

Spoiler

#include <stdio.h>


main()
{
   printf("Inserisci il primo numero, il segno, ed il secondo numero:\n");
   int numero1, numero2;
   char segno;
   scanf("%i %c", &numero1, &segno);
   while(getchar() != '\n');
   scanf("%i", &numero2);
   printf("Risultato: ");


   switch(segno)
   {
       case '+': printf("%i", numero1 + numero2); break;
       case '-': printf("%i", numero1 - numero2); break;
       
       case '*': 
       case 'x': printf("%i", numero1 * numero2); break;
       case '/': printf("%i", numero2 != 0 ? numero1 / numero2 : 0); break;
       default: puts("Segno non riconosciuto!");
   
   
   }
   
   puts("");
   return 0;
}[/spoiler]

 

[Alessandro27]

Questo è quello che feci a scuola qualche mese fa. Semplice (per me :asd:) e funzionale :asd:

Spoiler

#include<iostream>
main() {
using namespace std;
long double altro, oper, ris, ris1, ris2, a, b , pow ;
cout <<"Created by Cerrito Alessandro"<<endl;
cout << "Che operazione si vuole eseguire?" << endl;
cout << "1 = Addizione" << endl;
cout << "2 = Sottrazione" << endl;
cout << "3 = Moltiplicazione" << endl;
cout << "4 = Divisione" << endl;
cout << "5 = exit" << endl;
cin >> oper;
if (oper == 1) {
cout << "Inserisci il primo numero:" << endl;
cin >> a;
cout << "Inserisci il secondo numero:" << endl;
cin >> b;
ris = a + b;
cout << "Il risultato e' " << ris << endl << endl;
}
if (oper == 2) {
cout << "Inserisci il primo numero:" << endl;
cin >> a;
cout << "Inserisci il secondo numero:" << endl;
cin >> b;
ris = a - b;
cout << "Il risultato e' " << ris << endl << endl;
}
if (oper == 3) {
cout << "Inserisci il primo numero:" << endl;
cin >> a;
cout << "Inserisci il secondo numero:" << endl;
cin >> b;
ris = a * b;
cout << "Il risultato e' " << ris << endl << endl;
}
if (oper == 4) {
cout << "Inserisci il primo numero:" << endl;
cin >> a;
cout << "Inserisci il secondo numero:" << endl;
cin >> b;
ris = a / b;
cout << "Il risultato e' " << ris << endl << endl;
}
if (oper >= 12) {
exit(0);
}
cout << "Vuoi eseguire altri calcoli con la calcolatrice?" << endl;
cout << "1 = SI" << endl;
cout << "2 = NO" << endl;
cin >> altro;
if (altro == 1) {
system("cls");
main();
}
return 0;
}

Dio quanto amo programmare in C++ :blush::blush:

 

[signore del tempo]

Questo è quello che feci a scuola qualche mese fa. Semplice (per me :asd:) e funzionale :asd:

#include<iostream>
main() {
     ......
    if (altro == 1) {
        system("cls");
      [SIZE=4][B]main();[/B][/SIZE]
    }
    return 0;
}

Dio quanto amo programmare in C++ :blush::blush:

Dovresti prima imparare ad usarlo. :rolleyes:

 

[Alessandro27]

Dovresti prima imparare ad usarlo. :rolleyes:

Non so perchè ma senza il main () alla fine mi dava errore :skept:
Invece così funge alla perfezione :D
(La cosa importante è che funzioni :asd:)


Però forse, adesso che ci penso... Qualche errore me lo dava, ma non ricordo dove... Però risolsi subito... E comunque Djcrash deve prenderlo come spunto non deve fare copia-incolla ;)

 

[signore del tempo]

Non so perchè ma senza il main () alla fine mi dava errore :skept:
Invece così funge alla perfezione :D
(La cosa importante è che funzioni :asd:)

Chiamare main è un gravissimo errore, oltre che violazione dello standard.
Usi la exit() senza includere cstdlib.

 

[Alessandro27]

In quanto al main posso sorvolare, ma adesso mi hai rinfrescato la memoria.. Era proprio exit che non andava, proprio perchè ci mancava "cstdlib"... :thanks: :luxhello:

 

[signore del tempo]

La soluzione corretta è usare un ciclo tipo

do
 //operazioni
...
cout << "Continuare? s/n" << endl;
char r;
cin >> r;
while (r == 's');

Leggi questo

 

[Ivan Sala]

(La cosa importante è che funzioni :asd:)

Scusa, ma se io da un grattacielo di 20 piani devo arrivare a terra, prendo l'ascensore seguendo tutte le prassi o uso il metodo veloce e mi butto dal tetto? La seconda opzione è certamente la più veloce, ma non per forza corretta e funzionale... (oltre che riporta danni collaterali quali, al esempio la morte :D )

Comunque volevo solo sottolineare il fatto che non bisognerebbe limitarsi a dire "funziona" ma bisognerebbe trovare il miglior metodo seguendo tutte le convenzioni e gli standard dettati, se no anche se funziona non vale un ciuffolo :D

 

[Alessandro27]

Scusa, ma se io da un grattacielo di 20 piani devo arrivare a terra, prendo l'ascensore seguendo tutte le prassi o uso il metodo veloce e mi butto dal tetto? La seconda opzione è certamente la più veloce, ma non per forza corretta e funzionale... (oltre che riporta danni collaterali quali, al esempio la morte :D )

Comunque volevo solo sottolineare il fatto che non bisognerebbe limitarsi a dire "funziona" ma bisognerebbe trovare il miglior metodo seguendo tutte le convenzioni e gli standard dettati, se no anche se funziona non vale un ciuffolo :D

Dai ragazzi, mi state massacrando per 2 errori :asd:
Degli errori ci possono stare, ma se non comportano nessuna anomalia passano in secondo piano ;)

 

[Ivan Sala]

Tranquillo, gli errori ci stanno. L'importante però, per me, non è metterli in secondo piano, bensì capirli e risolverli e migliorarsi :D

 

[incolto]

Dai ragazzi, mi state massacrando per 2 errori :asd:
Degli errori ci possono stare, ma se non comportano nessuna anomalia passano in secondo piano ;)

Ma anche no!
Ecco spiegato perchè ogni tanto le cose funzionano alla coda di mosca.
Ad ogni modo... le anomalie non devono passare in secondo piano :sisi:

 

[Alessandro27]

Ma anche no!
Ecco spiegato perchè ogni tanto le cose funzionano alla coda di mosca.
Ad ogni modo... le anomalie non devono passare in secondo piano :sisi:

Ma tu preferisci una calcolatrice che funziona bene seppur con 1-2 errori, oppure una calcolatrice che non funziona proprio? :asd: