[C++] Diversi errori segnalati in compilazione di un programma

[Pacio88]

Da pochi giorni ho iniziato a studiare il linguaggio c++ attraverso un manuale. Nello studio di una dichiarazione di funzione mi sono imbattuto in un errore. Riportando il codice scritto sul libro nell'editor Dev-C++ e compilandolo non mi riportava nessun errore tuttavia una volta che lo facevo eseguire non ottenevo il risultato prefisso.

#include <iostream.h>
#include <stdlib.h>
#include <iostream.h>


void max()
{
    int n1, n2; // variabili imput

    /* Imput */

    cout << "Inserisci il primo numero: ";
    cin >> n1;
    cout << "Inserisci il secondo numero: ";
    cin >> n2;

    /* Elaborazione e Output */

    if (n1 >= n2)
     {
      cout << "Il numero maggiore digitato è: "
           << n1
           << " .\n";
     }
    else
     {
      cout << "Il numero maggiore digitato è: "
           << n2
           << " .\n";
     }
}

void main()
{
  void max();
  system("pause");
  return 0;
}

Grazie

[Stoicenko]

posta l'errore..

cmq il se il main è dichiarato come "void" non serve ritornare 0 (che è un int) sennò devi usare int main()

in più devi inserire

using namespace std;

altrimenti cout e cin non può usarli

[Pacio88]

Scusa ma essendo alle prime armi mi sono espresso male, cerco di spiegarmi meglio. Al momento che eseguo il programma mi viene visualizzata una finestra con scritto: "Premere un tasto per continuare...", quindi premendo poi un tasto mi esce e quindi non esegue effettivamente il programma, spero di essermi spiegato.

Per quanto rigurda il pezzo di codice:

system("pause");
return 0;

Sinceramente non mi è stato ancora spiegato il significato, se mi potessi dare delle delucidazioni anche in merito te ne sarei grato.

[oregon]

#include <iostream>
#include <stdlib.h>

using namespace std;

Nel main non devi scrivere

void max();

ma solamente

max();

int main()
{
  max();
  system("pause");
  return 0;
}

[pallinopinco]

1) La funzione main deve restituire int e non void

2) Input si scrive con la N di Nabucodonosor

3) Qualsiasi compilatore/libro (d|r)ecente ti segnala con un warning che non bisogna usare gli header standard con il .h

4) Non gestisci bene il caso di numeri uguali

5) Nel main togli il void davanti a max()

6) La funzione max dovrebbe soltanto effettuare una verifica, non gestire l'I/O quindi è preferibile dichiararla come

int max(int a, int b)

e richiamarla con

esito = max(n1, n2)

(o in un blocco switch)

[Pacio88]

Vi ringrazio, scusate l'ignoranza ma è due giorni che studio il C++.

P.S. Pallinopinco potresti spiegarmi il punto 6 eventualmente con un esempio.

GRAZIE MILLE

[pallinopinco]

Te lo scrivo al volo, dovrebbe compilare:

#include <iostream>
#include <cstdlib>

using namespace std;

int max(int a, int b)
{
 if (a > b) return 1; // Primo maggiore del secondo
     else if (a < b) return 2; // Secondo maggiore del primo
          else return 0; // I 2 numeri sono uguali
}

int main(void)
{
   int n1, n2;

    cout << "Inserisci il primo numero: ";
    cin >> n1;
    cout << "Inserisci il secondo numero: ";
    cin >> n2;

  // Analizza i casi
  switch(max(n1, n2))
  {
   case 0: cout << "UGUALI!" << endl; break;
   case 1: cout << n1 << " > " << n2 << endl; break;
   case 2: cout << n2 << " > " << n1 << endl;
  }
  
  system("pause");
  return 0;
}