progetto c++

CIao a tutti, sono alle prime armi con il c++ e volevo chiedere se avevo fatto bene questo:

Codice:

---

//programma calcolo equazioni 2° grado
#include <cstdlib>
#include <iostream>
#include <math.h>

using namespace std;
double a, b, c, x1, x2, d;
int main ()
{
    cout<<"Inserisci il valore di a: ";
    cin>>a;
    cout<<"Inserisci il valore di b: ";
    cin>>b;
    cout<<"Inserisci il valore di c: ";
    cin>>c;
   
    d = b * b - 4*a*c; // calcola il delta
    if ( d>0 )
    {
        x1 = (-b -sqrt(d))/2*a;
       
        x2 = (-b +sqrt(d))/2*a;
       
        cout<<"x1= "<< x1 << "x2= "<<x2<< endl;
        } else
          {
              cout<<"L'equzione non e' possibile in R perche' il delta e' negativo." ;
             
              }
              system ("pause");
              }

---

Prova a compilarlo e se non lo compila posta gli errori.
EDIT:
sostituisci:

Codice:

---

#include <math.h>

---

con

Codice:

---

#include <cmath>

---

ancora non lo avevo compilato perchè lo avevo scritto solo con il blocco note.
adesso l'ho compilato e non mi da errori con math.h. che differenza c'è tra i due?


EDIT: mmmh, mi sa ke ho sbagliato la parte matematica perchè mi sbaglia a fare un equazione.

C'è un errore logico: in realtà l'equazione è risolvibile anche se d == 0 (nel qual caso, come confermato dalla teoria, le due soluzioni sono uguali). Il tuo programma invece crede che con d == 0 l'equazione sia irriducibile...

Altro errore: metti tra parentesi (2 * a), giusto per garantirti che prima non divida per 2 e moltiplichi poi IL RISULTATO per a... a proposito, e se a fosse zero, cosa succede?

Stammi bene...

ciao,
mi diverto a programmare in c++ da un po'.. mi ha colpito il tuo programmino..
devi inserire comunque:
else if (d==0)
{
x1 = -b/(2*a); //mi sembra sia così
}

mi spiegate cos'é cmat? grazie...ciao!

EDIT: ps, ora che ci penso nn server, farebbe b - delta = 0.. metti semplicemente => e dovrebbe andare..ciao!

Citazione:

---

C'è un errore logico: in realtà l'equazione è risolvibile anche se d == 0 (nel qual caso, come confermato dalla teoria, le due soluzioni sono uguali). Il tuo programma invece crede che con d == 0 l'equazione sia irriducibile...

Altro errore: metti tra parentesi (2 * a), giusto per garantirti che prima non divida per 2 e moltiplichi poi IL RISULTATO per a... a proposito, e se a fosse zero, cosa succede?

Stammi bene...

---

grazie dell'aiuto delle parentesi, per l'uguale a zero lo devo finire.

Citazione:

---

ciao,
mi diverto a programmare in c++ da un po'.. mi ha colpito il tuo programmino..
devi inserire comunque:
else if (d==0)
{
x1 = -b/(2*a); //mi sembra sia così
}

mi spiegate cos'é cmat? grazie...ciao!

EDIT: ps, ora che ci penso nn server, farebbe b - delta = 0.. metti semplicemente => e dovrebbe andare..ciao!

---

grazie

EDIT: ho riscritto tutto il codice ma lo compilo con dev c++ mi da un errore alla riga 19" expected `;' before "endl" e questo " In function `int main()': "

Codice:

---

//programma calcolo equazioni 2° grado
#include <cstdlib>
#include <iostream>
#include <math.h>

using namespace std;
double a, b, c, x1, x2, d;
int main ()
{
    cout<<"Inserisci il valore di a: ";
    cin>>a;
    cout<<"Inserisci il valore di b: ";
    cin>>b;
    cout<<"Inserisci il valore di c: ";
    cin>>c;
    d = (b*b)-(4*a*c);
 
  if (d<0)
  {cout<<"Non esistono soluzioni reali!" endl;
}
else {
    if(d==0)
{ x1= -b/(2*a);
x2= -b/(2*a);
}
else {
    x1=(-b-sqrt(d))/(2*a);
    x2=(-b+sqrt(d))/(2*a);
    }
    cout<<"x1= "<<x1<<"x2= "<<x2<<endl;}

              system ("pause");
              }

---

ciao,
prova così:

Codice:

---

if (d<0)
  {
cout<<"Non esistono soluzioni reali!"<<endl;
}

---

ho dato solo un occhaita, ora provo a metterlo giù bene...fammi sapere se funziona!

ho riscrittto un po' il codice:

Codice:

---

#include <cstdlib>
#include <iostream>
#include <math.h>

using namespace std;
double a, b, c, x1, x2, d;

int main ()
{

    cout << "Inserisci il valore di a: ";
    cin >> a;
    cout << "Inserisci il valore di b: ";
    cin >> b;
    cout << "Inserisci il valore di c: ";
    cin >> c;

    d = (b*b)-(4*a*c);

  if (d >= 0)
    {

    x1 = (-b-sqrt(d))/(2*a);
    x2 = (-b+sqrt(d))/(2*a);

        cout << "x1= " << x1 << endl << "x2= " << x2;

    } else {

        cout << "Non esistono soluzioni reali!";

    }

return 0;
}

---

sembrerebbe funzionare... ed é anche piu pulito..! ciaoooo!

Piccolo consiglio: indenta meglio il codice, così risulta più facile capire dove magari potresti aver sbagliato ;)

cosa vorresti dire con indenta?

EDIT:
bluster==>anche facendo le modifiche che mi hai detto te mi da sempre gli stessi errori.

molto strano, lo compilato pure io il mio codice, e funziona ;)
cambia compilatore dev c++ é fermo... usa code::blocks, io lo utilizzo e mi ci trovo molto bene...ciao!

Ti dò qualche altro consiglio:
- puoi dichiarare le variabili anche all'inizio del metodo main(), invece di fare dichiarazioni globali come hai fatto tu. Nel tuo caso non comporta grosse conseguenze... ma ti potrà essere davvero d'aiuto (o potrebbe essere la tua condanna, a seconda dei punti di vista) quando inizierai a sviluppare funzioni diverse dal main().
- altra cosa: se vuoi perfezionare davvero il tuo programma, dovresti prevedere il caso in cui a == 0. In questo caso il tuo programma molto probabilmente genererà errore in esecuzione (divisione per zero), ma l'equazione in sé non è irrisolvibile... è solo che diventa di primo grado e quindi... (a te il resto).

Stammi bene...

bluster==>code::block e solo in inglese? se c'è in ita mi dai il link?

io ce l'ho in inglese, vabbé sono poche parole che devi consocere per utilizzarlo ;) e poi se programmi un po di inglese lo sai.. cmq so che c'é il modo di cambiare compilatore, prendi quello di code::blocks (che é aggiornato) e lo sostituisci a quello di dev c++, cerca in google, nn so come si fa, ma so che si può fare...ciao!

ho scaricato code::block ma mi da gli stessi errori quando compilo. bo.

ti riposto il codice

Codice:

---

int main ()
{
    cout<<"Inserisci il valore di a: ";
    cin>>a;
    cout<<"Inserisci il valore di b: ";
    cin>>b;
    cout<<"Inserisci il valore di c: ";
    cin>>c;
    d = (b*b)-(4*a*c);
 
  if (d<0)
  {
          cout<<"Non esistono soluzioni reali!" endl;
}
else {
    if(d==0)
{ x1= -b/(2*a);
x2= -b/(2*a);
}
else {
    x1=(-b-sqrt(d))/(2*a);
    x2=(-b+sqrt(d))/(2*a);
    }
    cout<<"x1= "<<x1<<"x2= "<<x2<<endl;}

              system ("pause");
              }

---

EDIT: gli errori sono:
-F:\Ale\equazionia.cpp||In function `int main()':|
F:\Ale\equazionia.cpp|20|error: expected `;' before "endl"|
||=== Build finished: 1 errors, 0 warnings ===|

Codice:

---

#include <cstdlib>
#include <iostream>
#include <math.h>

using namespace std;
double a, b, c, x1, x2, d;

int main ()
{

    cout << "Inserisci il valore di a: ";
    cin >> a;
    cout << "Inserisci il valore di b: ";
    cin >> b;
    cout << "Inserisci il valore di c: ";
    cin >> c;

    d = (b*b)-(4*a*c);

  if (d >= 0)
    {

    x1 = (-b-sqrt(d))/(2*a);
    x2 = (-b+sqrt(d))/(2*a);

        cout << "x1= " << x1 << endl << "x2= " << x2;

    } else {

        cout << "Non esistono soluzioni reali!";

    }

return 0;
}

---

prova a compilare il mio codice, ricorda inoltre che esiste else if (), e poi un altro consiglio che é stato gia detto, identa meglio il codice, é piu facile trovare gli errori ;)

cosa serve return 0 alla fine?

non ritorna alcun valore, ma permette di farti vedere l'applicazione, system("PAUSE") non é molto consigliato...ciao!
PS: funziona ora?

no sempre gli stessi errori.

provato su code blocks e dev c++

Citazione:

---

Originalmente inviato da versionilatino

ho scaricato code::block ma mi da gli stessi errori quando compilo. bo.

ti riposto il codice

Codice:

---

int main ()
{
    cout<<"Inserisci il valore di a: ";
    cin>>a;
    cout<<"Inserisci il valore di b: ";
    cin>>b;
    cout<<"Inserisci il valore di c: ";
    cin>>c;
    d = (b*b)-(4*a*c);
 
  if (d<0)
  {
          cout<<"Non esistono soluzioni reali!" endl;
}
else {
    if(d==0)
{ x1= -b/(2*a);
x2= -b/(2*a);
}
else {
    x1=(-b-sqrt(d))/(2*a);
    x2=(-b+sqrt(d))/(2*a);
    }
    cout<<"x1= "<<x1<<"x2= "<<x2<<endl;}

              system ("pause");
              }

---

EDIT: gli errori sono:
-F:\Ale\equazionia.cpp||In function `int main()':|
F:\Ale\equazionia.cpp|20|error: expected `;' before "endl"|
||=== Build finished: 1 errors, 0 warnings ===|

---

ti da errore perchè il primo "endl" non è concatenato (<<) al resto della stringa

Citazione:

---

Originalmente inviato da bluster

non ritorna alcun valore, ma permette di farti vedere l'applicazione, system("PAUSE") non é molto consigliato...ciao!
PS: funziona ora?

---

Sbagliato... il return del main() non ha nulla a che vedere con la funzione system(), è solo il codice di errore che viene restituito al sistema operativo. Lo zero indica solo che è andato tutto bene...

Stammi bene...

hai ragione adesso va. mancavano quei due minori!!!

si vabbé, te l'avevo scritto qualche post piu su ;) ti avevo psotato anche tutto il codice corretto ;) ciao!