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

Stato
Discussione chiusa ad ulteriori risposte.

Squalo03

Bannato a Vita
443
92
CPU
amd fx tm 6300 six core processor
GPU
nvidia geforce gtx 650
Monitor
Samsung
Periferiche
mouse
OS
Windows 10 pro
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.
Post unito automaticamente:


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

Inviato da SM-J530F tramite App ufficiale di Tom\'s Hardware Italia Forum
 
  • Mi piace
Reazioni: Ancora79

Squalo03

Bannato a Vita
443
92
CPU
amd fx tm 6300 six core processor
GPU
nvidia geforce gtx 650
Monitor
Samsung
Periferiche
mouse
OS
Windows 10 pro
C:
//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
 
  • Mi piace
Reazioni: Ancora79

Eduadie

Utente Attivo
204
24
C:
//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.
 
  • Mi piace
Reazioni: Ancora79 e Mursey

Squalo03

Bannato a Vita
443
92
CPU
amd fx tm 6300 six core processor
GPU
nvidia geforce gtx 650
Monitor
Samsung
Periferiche
mouse
OS
Windows 10 pro
C:
#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?
 
Ultima modifica da un moderatore:
  • Mi piace
Reazioni: Ancora79

Andretti60

Utente Èlite
6,440
5,091
...
è 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.
 
  • Mi piace
Reazioni: Mursey

Squalo03

Bannato a Vita
443
92
CPU
amd fx tm 6300 six core processor
GPU
nvidia geforce gtx 650
Monitor
Samsung
Periferiche
mouse
OS
Windows 10 pro
quello che non capisco perche dice in funzione int main?
 

rodhellas

Utente Èlite
1,522
427
CPU
Ryzen 5 3600
Dissipatore
GELID Phantom
Scheda Madre
MSI B450 Gaming Plus Max
HDD
500GB m.2 + 2TB HDD
RAM
16GB Corsair LPX 3000mhz
GPU
Gigabyte GTX 960 OC
Audio
Integrata
Monitor
SyncMaster 223BW
PSU
Antec HCG-520M
Case
Meshify C
Net
Gigabit Fastweb
OS
Windows 10 64bit
  • Mi piace
Reazioni: Ancora79

Squalo03

Bannato a Vita
443
92
CPU
amd fx tm 6300 six core processor
GPU
nvidia geforce gtx 650
Monitor
Samsung
Periferiche
mouse
OS
Windows 10 pro
C:
//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 ;
 
Ultima modifica da un moderatore:
  • Mi piace
Reazioni: Ancora79

Mursey

Super Moderatore
Staff Forum
Utente Èlite
8,191
5,631
Ultimo avvertimento @Squalo03
o segui quello che ti dico, oppure chiudo la discussione e ti sanziono.

Correggo per l'ultima volta.
 
Stato
Discussione chiusa ad ulteriori risposte.

Entra

oppure Accedi utilizzando
Discord Ufficiale Entra ora!

Discussioni Simili