RISOLTO [C++] Dati tre numeri, determinare il maggiore

[Squalo03]

Io non so programmare (non sto scherzando) ma mi sembra che:
linea 11 col 11 ci sono i due punti ( : ) al posto di ;
linea 11 col 10: errore conseguente al primo;
tutti gli altri si commettono perchè max non è stato dichiarato.
Mi astengo dal giudicare il codice perchè non so da quando ha iniziato, spero quest'anno.
--- i due messaggi sono stati uniti ---

Forse do qualcosa per scontato.
Mi ritiro in meditazione...

da settembre

Inviato da SM-J530F tramite App ufficiale di Tom\'s Hardware Italia Forum

 

[clessidra]

da settembre

Inviato da SM-J530F tramite App ufficiale di Tom\'s Hardware Italia Forum

Ok, passerò del tempo a meditare :deserto:.

 

[Squalo03]

cosa posso fare

Inviato da SM-J530F tramite App ufficiale di Tom\'s Hardware Italia Forum

 

[Mursey]

cosa posso fare

Quello che ti ha detto @clessidra è corretto.

Inoltre max non è dichiarato e al momento non usato, puoi stamparlo alla fine
meglio se negli if metti un pò di parentesi graffe

 

[Squalo03]

//librerie
#include <iostream>
using namespace std;

//Consegna esercizio pag. 121 n. 9 punto 1
int main(){
float A,B,C,max;
cout<<"Inserisci il valore di A"<<endl;
cin>>A:
cout<<"Inserisci il valore di B"<<endl;
cin>>B:
cout<<"Inserisci il valore di C"<<endl;
cin>>C:
if(A>B)
{
if(A>C)
{
max=A;
}
}
else
{
max=C;
}
if(B>C)
{
max=B;
}
else
{
max=C;
}
max=B;
}
else
{
max=C;
}
system("PAUSE")
return 0;
}

mi da gli errori

 

[Mursey]

Leggi bene https://forum.tomshw.it/threads/regole-di-sezione-da-leggere-prima-di-creare-una-discussione.597452/
usa il tag CODE

Hai messo : invece che ;

 

[Eduadie]

//librerie
include <iostream>
using namespace std;

//Consegna esercizio pag. 121 n. 9 punto 1
int main(){
    float A,B,C;
    cout<<"Inserisci il valore di A"<<endl;
    cin>>A;
    cout<<"Inserisci il valore di B"<<endl;
    cin>>B:
    cout<<"Inserisci il valore di C"<<endl;
    cin>>C;
    if(A>B)
      if(A>C)
      max=A;
    else
      max=C;
    if(B>C)
      max=B;
    else
      max=C;
    system("PAUSE");
    return 0;
}

va bene la codifica?

come posso buttarmi

Ci sono prima di tutto errori di codice e infine l'errore di logica. Include vuole il cancelletto, al cout di B c'è : e non ;, e la variabile max non è dichiarata.
L'errore di logica è che per come hai scritto tu l'esercizio il max ti verrà trovato solo tra B e C. La logica da seguire è la seguente:
Se A è maggiore di B e se A è maggiore di C allora A è il massimo altrimenti è C, altrimenti se B è maggiore di C il massimo è B altrimenti è C.

 

[Squalo03]

#include <iostream>
using namespace std;

int main(){
float a,b,c,max;
cout<<"Inserisci il valore di a"<<endl;
cin>>a;
cout<<"Inserisci il valore di b"<<endl;
cin>>b;
cout<<"Inserisci il valore di c"<<endl;
cin>>c;
if(a>b)
{
if(a>c)
{
max=a;
}
}
else
{
max=c;
}
if(b>c)
{
max=b;
}
else
{
max=c;
}
max=b;
}
else
{
max=c;
}
getchar();

è corretto ora?

 

[Mursey]

@Squalo03 per cortesia segui quello che ti dico.

Usa il tag CODE !
Leggi bene https://forum.tomshw.it/threads/regole-di-sezione-da-leggere-prima-di-creare-una-discussione.597452/

Vedi che https://forum.tomshw.it/threads/linguaggio-c.753012/page-2#post-7089280 è diverso da quello sopra ?
E' perchè correggo con il tag CODE.

 

[Andretti60]

...
è corretto ora?

Capisco che sei agli inizi, ma non puoi chiedere a noi "e' corretto ora".
Ci sono due tipi di errori che devi imparare a riconoscere:

1. errori di compilazione. Questi te li dice il compilatore stesso, assieme alla linea di codice dove appare l'errore. Il piu delle volte la descrizione dell'errore e' sufficiente a capire che errore sia e come aggiustarlo. Altrimenti guarda la linea e quella precedente, in quanto l'errore puo' essere una conseguenza di quello che e' successo prima (classico esempio, si e' dimenticato un punto e virgola)
2. errore di logica. Una volta che la compilazione e' corretta, lancia il programma e provalo. Entra una serie di dati in ingresso e controlla che i dati di uscita siano corretti. Questo almeno significa che il programma funziona con quei dati. Riguardare il codice.

 

[Squalo03]

quello che non capisco perche dice in funzione int main?

 

[rodhellas]

quello che non capisco perche dice in funzione int main?

Perchè l'errore è all'interno di quella funzione. Controlla le parentesi { } e i ;

 

[Squalo03]

//librerie
#include <iostream>
using namespace std;

//3 numeri
int main(){
    float a,b,c,max;
    cout<<"inserire il valore di a"<<endl;
    cin>>a;
    cout<<"inserire il valore di a"<<endl;
    cin>>b;
    cout<<"inserire il valore di a"<<endl;
    cin>>c;
    if(a>b)
    {
    if(a>c)
    {
    max=a;
    }    
    else
    {
    max=c;
    }
    else
    {
    if(b>c)
    {
    max=b;
    {
    }  
    max=c;
    }
    }
    getchar();
}

io ho messo tutti i ;

 

[Mursey]

Ultimo avvertimento @Squalo03
o segui quello che ti dico, oppure chiudo la discussione e ti sanziono.

Correggo per l'ultima volta.

 

[Squalo03]

Ultimo avvertimento @Squalo03
o segui quello che ti dico, oppure chiudo la discussione e ti sanziono.

Correggo per l'ultima volta.

ho messo code ? che cosa ho fatto di male?

Inviato da SM-J530F tramite App ufficiale di Tom\'s Hardware Italia Forum