Un mio semplice programma: come miglioralo?

[zanardi84]

Sto scrivendo un semplice programma di cui vi inserirò il codice.
Nella versione non completa attuale e funzionante ho cercato di inserire ciò che ho sino ad ora imparato, cioè poco: funzioni e vettori. Il mio scopo è stato quello di cercare di scrivere più funzioni possibile, versatili per scopi disparati, adattabili e semplici. Il main doveva essere poco più di una sequenza di chiamate a funzione.

C'è sicuramente un bug comune che andrebbe risolto con una funzione ad hoc, ma che non sono in grado di trovare e di scrivere: il menu del mio programma è strutturato in modo che la scelta sia sempre numerica. Nel momento in cui inserisco un solo carattere letterale va in loop, e ciò non è ovviamente ammissibile.

Due cose che non mi piacciono:
1) l'interfaccia delle chiamate, forse un po' complicate da gestire.. qualche consiglio in proposito?
2) Il primo elemento del vettore ha indice 0. Vorrei che avesse indice 1. Ho qualche dubbio su come fare.

Vorrei un vostro parere sull'operato, sino a questo punto.
Quando imparerò liste e strutture lo potenzierò.

N.B. Intanto è solo un "prototipo" con il vettore già riempito, per simulare la situazione.

[!fazz]

per risolvere il loop

int inserimento (string messaggio1, string messaggio2, int estremo_inferiore, int estremo_superiore)
{
      int dato;
      char cdato
      cout << messaggio1<<endl;
      cout <<endl;
      cout << messaggio2<<endl;
      cin >>cdato;
      dato=atoi(cdato)

     if (dato==0)&&(cdato!='0')
      cout<<"errore";
else{

      while (dato < estremo_inferiore || dato > estremo_superiore)
      {
            cout << "Errore! Il valore non deve essere inferiore di " <<estremo_inferiore<< " e maggiore di " <<estremo_superiore<< "! Ridigitare"<<endl;
            cin >> dato;
      }
}
      return dato;
}

[zanardi84]

Originariamente inviato da !fazz

per risolvere il loop

int inserimento (string messaggio1, string messaggio2, int estremo_inferiore, int estremo_superiore)
{
      int dato;
      char cdato
      cout << messaggio1<<endl;
      cout <<endl;
      cout << messaggio2<<endl;
      cin >>cdato;
      dato=atoi(cdato)

     if (dato==0)&&(cdato!='0')
      cout<<"errore";
else{

      while (dato < estremo_inferiore || dato > estremo_superiore)
      {
            cout << "Errore! Il valore non deve essere inferiore di " <<estremo_inferiore<< " e maggiore di " <<estremo_superiore<< "! Ridigitare"<<endl;
            cin >> dato;
      }
}
      return dato;
}

Qualcosa non quadra, ho errori di compilazione.
71 invalid conversion from `char' to `const char*'

71 initializing argument 1 of `int atoi(const char*)'

73 expected identifier before '(' token

73 expected `;' before '(' token

[!fazz]

Originariamente inviato da zanardi84

Qualcosa non quadra, ho errori di compilazione.
71 invalid conversion from `char' to `const char*'

71 initializing argument 1 of `int atoi(const char*)'

73 expected identifier before '(' token

73 expected `;' before '(' token

vabbè ho scritto di fretta nel forum senza un ide sotto, lo scopo era farti vedere come si risolve non risolverti il problema fatto e finito

(cmq per l'esattezza manca il ; nella dichiarazione di cdato)

i loop li hai quando inserisci un carattere al posto di un intero la soluzione è acquisire un carattere e dopo trasformarlo in un intero mediante la funzione atoi

il problema è che atoi restituisce 0 se il dato inserito non è un numero quindi se vuoi usare anche lo 0 nel menu devi controllare che il valore inserito sia effettivamente 0 l'if te l'ho messo li a caso devi integrare il controllo nel tuo codice

[zanardi84]

Originariamente inviato da !fazz

vabbè ho scritto di fretta nel forum senza un ide sotto, lo scopo era farti vedere come si risolve non risolverti il problema fatto e finito

(cmq per l'esattezza manca il ; nella dichiarazione di cdato)

i loop li hai quando inserisci un carattere al posto di un intero la soluzione è acquisire un carattere e dopo trasformarlo in un intero mediante la funzione atoi

il problema è che atoi restituisce 0 se il dato inserito non è un numero quindi se vuoi usare anche lo 0 nel menu devi controllare che il valore inserito sia effettivamente 0 l'if te l'ho messo li a caso devi integrare il controllo nel tuo codice

Da quel che ho capito implementando un piccolo programma per testare il funzionamento, atoi funge così, per esempio:

Inserisco 11 -> restituisce 11.
Inserisco A -> restituisce 0.
Inserisco 123a -> restituisce 123 ignorando la a.
Inserisco "nel mezzo del cammin di nostra vista..." -> restituisce 0.
Inserisco A4 -> restituisce 0.

e così via.
Buon funzionamento, ma voglio prevedere l'uso dello 0 per qualche ragione. Come posso fare?

[!fazz]

Originariamente inviato da zanardi84

Da quel che ho capito implementando un piccolo programma per testare il funzionamento, atoi funge così, per esempio:

Inserisco 11 -> restituisce 11.
Inserisco A -> restituisce 0.
Inserisco 123a -> restituisce 123 ignorando la a.
Inserisco "nel mezzo del cammin di nostra vista..." -> restituisce 0.
Inserisco A4 -> restituisce 0.

e così via.
Buon funzionamento, ma voglio prevedere l'uso dello 0 per qualche ragione. Come posso fare?

guarda l'if che ho inserito

per controllare lo 0 devi controllare quando atoi restituisce 0 se il carattere inserito è effettivamente 0

[zanardi84]

Originariamente inviato da !fazz

guarda l'if che ho inserito

per controllare lo 0 devi controllare quando atoi restituisce 0 se il carattere inserito è effettivamente 0

Non posso fare il controllo diretto perchè dato è un intero e cdato è una stringa. Devo trovare un modo alternativo per fare il confronto, ma al momento non mi viene in mente nulla.

[!fazz]

Originariamente inviato da zanardi84

Non posso fare il controllo diretto perchè dato è un intero e cdato è una stringa. Devo trovare un modo alternativo per fare il confronto, ma al momento non mi viene in mente nulla.

a parte che dopo questo ti consiglio di riguardarti le basi di programmazione

cdato è un carattere non una stringa

se ti ho detto di guardare l'if che ti ho scritto appositamente io ci sarà un motivo no?

if (dato==0)&&(cdato!='0')
cout<<"errore";

[zanardi84]

Originariamente inviato da !fazz

a parte che dopo questo ti consiglio di riguardarti le basi di programmazione

cdato è un carattere non una stringa

se ti ho detto di guardare l'if che ti ho scritto appositamente io ci sarà un motivo no?

if (dato==0)&&(cdato!='0')
cout<<"errore";

Ho sbagliato a chiamare un carattere come stringa, ma è su queste (col tipo string) che sto lavorando attualmente nei programmi (e l'implementazione che ho provato con successo, ma senza gesitone dello 0, prevede vettore di char che è una stringa
Se io quell'if lo inserisco nel compilatore, l'unico errore che restituisce è quello di compilazione perchè non può confrontare un char come cdato con un int come dato.

[!fazz]

Originariamente inviato da zanardi84

Ho sbagliato a chiamare un carattere come stringa, ma è su queste (col tipo string) che sto lavorando attualmente nei programmi (e l'implementazione che ho provato con successo, ma senza gesitone dello 0, prevede vettore di char che è una stringa
Se io quell'if lo inserisco nel compilatore, l'unico errore che restituisce è quello di compilazione perchè non può confrontare un char come cdato con un int come dato.

non so a casa tua ma a casa mia quell'if confronta un intero con una costante intera e un carattere con una costante carattere

[zanardi84]

Originariamente inviato da !fazz

non so a casa tua ma a casa mia quell'if confronta un intero con una costante intera e un carattere con una costante carattere

Se io inserisco il tuo codice, mettendo il ; dopo la dichiarazione della variabile e la chiamata di funzione e sistemando la sintassi dell'if, ottengo quanto puoi vedere nello screen.



da cui non so uscire al momento perchè sto cercando di imparare.

Qualche idea?

[!fazz]

Originariamente inviato da zanardi84

Se io inserisco il tuo codice, mettendo il ; dopo la dichiarazione della variabile e la chiamata di funzione e sistemando la sintassi dell'if, ottengo quanto puoi vedere nello screen.



da cui non so uscire al momento perchè sto cercando di imparare.

Qualche idea?

comincia con il sistemare le parentesi


if ((dato==0)&&(cdato!='0'))

[zanardi84]

Originariamente inviato da !fazz

comincia con il sistemare le parentesi


if ((dato==0)&&(cdato!='0'))

Non sono servite

L'errore è sempre alla riga dell'atoi.

[!fazz]

Originariamente inviato da zanardi84

Non sono servite

L'errore è sempre alla riga dell'atoi.

allora il problema non è l'if

comunque rileggendo l'errore pare che la tua atoi legga solo le stringhe e non i caratteri

sostituisci cdato con una stringa il cin con un cin.getline ecc ecc

ma comunque sappi che quel codice non ha nessuna valenza pratica è fondamentalmente sbagliato

a che scopo dare estremo inferiore e superiore per comunicare solo due messaggi? dovresti sistemarlo almeno con una matrice di caratteri in grado di creare un menu ndimensionale

[zanardi84]

Originariamente inviato da !fazz

allora il problema non è l'if

comunque rileggendo l'errore pare che la tua atoi legga solo le stringhe e non i caratteri

sostituisci cdato con una stringa il cin con un cin.getline ecc ecc

ma comunque sappi che quel codice non ha nessuna valenza pratica è fondamentalmente sbagliato

a che scopo dare estremo inferiore e superiore per comunicare solo due messaggi? dovresti sistemarlo almeno con una matrice di caratteri in grado di creare un menu ndimensionale

Guarda bene l'interfaccia con cui chiamo "inserimento" asso due messaggi come string e due interi che sono gli estremi:
Ho trovato una funzione generica già pronta (l'avevo preparata per un altro programma semplice) e l'ho riutilizzata col minimo delle modiiche (cioè ho aggiunto un parametro string in più), ma potrei potenziarla aggiungendole qualche if (o un costrutto switch) per farle fare altre operazioni come chiamare altre funzioni alla pressione di un tasto, ma l'inferfaccia sarebbe sempre quella, con gli estremi numerici fuori dai quali avrei la non accettabilità. Ovviamente conosco il limite del controllo, cioè se metto un letterale al posto di un numero mi crasha tutto.

Torniamo pertanto alla questione principale! Voglio che mi prenda, per il mio scopo, solo numeri e non caratteri letterali o "grafici".

[-MiStO-]

Originariamente inviato da zanardi84

Non sono servite

L'errore è sempre alla riga dell'atoi.

ad atoi devi passargli l'indirizzo del char

dato=atoi(&cdato);

comunque il while è errato, non ripete il controllo numerico

[zanardi84]

Originariamente inviato da -MiStO-

ad atoi devi passargli l'indirizzo del char

dato=atoi(&cdato);

comunque il while è errato, non ripete il controllo numerico

Dunque, ho implementato un semplice programmino che analizza la situazione (il while poi si aggiusta, ma mi interessava vedere il comportamento effettivo dell'atoi) e devo dire che fa quasi tutto quello che voglio: gestisce anche lo 0 con l'if creato, ma manca ancora una cosa.

Se io digito in input per esempio 23er45, vorrei che mi dicesse che è un dato sbagliato. Ho provato a implementare usando la stringa di caratteri da convertire in intero, così

int i;
  char input[256];
  cout << "Inserire un numero: ";
  cin >> input;
  
  i = atoi(input);
  
  if((i==0) && (*input!='0'))  //se il dato inserito è 
  cout << "Dato non accettato"<<endl;
  
  else if ((i==0) && (*input='0')) 
  cout << "inserito lo 0: "<<i<<endl;
  
  else 
  
  cout << "dato inserito: "<<i;

ma pur funzionando, invece di darmi l'avviso di dato non corretto, mi tronca il dato a partire dal letterale.
Cioè 23er45 per lui è 23, mentre vorrei che mi dicesse che è errore.
Lavora a dovere se scrivo i primi versi della Divina Commedia (segnala errore) o se gli digito un numero, per esempio 23 è proprio 23 (e se gli faccio calcolare il doppio viene 46). Ok anche se gli metto 000000000002, mi tronca gli zeri.

Cosa mi suggerite per risolvere?

[!fazz]

Originariamente inviato da zanardi84

Dunque, ho implementato un semplice programmino che analizza la situazione (il while poi si aggiusta, ma mi interessava vedere il comportamento effettivo dell'atoi) e devo dire che fa quasi tutto quello che voglio: gestisce anche lo 0 con l'if creato, ma manca ancora una cosa.

Se io digito in input per esempio 23er45, vorrei che mi dicesse che è un dato sbagliato. Ho provato a implementare usando la stringa di caratteri da convertire in intero, così

int i;
  char input[256];
  cout << "Inserire un numero: ";
  cin >> input;
  
  i = atoi(input);
  
  if((i==0) && (*input!='0'))  //se il dato inserito è 
  cout << "Dato non accettato"<<endl;
  
  else if ((i==0) && (*input='0')) 
  cout << "inserito lo 0: "<<i<<endl;
  
  else 
  
  cout << "dato inserito: "<<i;

ma pur funzionando, invece di darmi l'avviso di dato non corretto, mi tronca il dato a partire dal letterale.
Cioè 23er45 per lui è 23, mentre vorrei che mi dicesse che è errore.
Lavora a dovere se scrivo i primi versi della Divina Commedia (segnala errore) o se gli digito un numero, per esempio 23 è proprio 23 (e se gli faccio calcolare il doppio viene 46). Ok anche se gli metto 000000000002, mi tronca gli zeri.

Cosa mi suggerite per risolvere?

controlla la lunghezza della stringa che leggi e confrontala con il dato

se è un numero da 0-9 la stringa deve essere 1 carattere
se è tra 10-99 due caratteri

ecc ecc

[zanardi84]

Originariamente inviato da !fazz

controlla la lunghezza della stringa che leggi e confrontala con il dato

se è un numero da 0-9 la stringa deve essere 1 carattere
se è tra 10-99 due caratteri

ecc ecc

Questa è un'ottima idea!!!!

[zanardi84]

Originariamente inviato da !fazz

controlla la lunghezza della stringa che leggi e confrontala con il dato

se è un numero da 0-9 la stringa deve essere 1 carattere
se è tra 10-99 due caratteri

ecc ecc

Implementata, funziona perfettamente. In un lampo l'ho adattata alla mia interfaccia! L'unica cosa è che adesso modificherò, se posso, le chiamate di funzione, sostituendo i numeri con delle costanti che metto a inizio codice in modo tale da fare modifiche al volo se devo ampliare i vettori.
Poi continuerò a sviluppare il programma aggiungendo sempre più cose come per esempio una bella gestione delle prenotazioni (adesso è molto grezza e minimale), file, sotituisco il vettore con una bella lista (e all'interfaccia potrebbe cambiare solo il parametro del vettore sostituendolo con uno che identifichi la lista e sostituendo la ricerca vettore con quella della lista) e chi più ne ha più ne metta!