PDA

View Full Version : C++ ...perchè questo codice non funziona?


Kratos
16-09-2002, 19:14
ho provato a risolvere una semplice traccia dell'esame di Fondamenti II...la gestione di un array attraverso gli oggetti.
Credevo di aver fatto tutto più o meno bene...ma il programma non funziona. Nell'array i valori non vengono inseriti, in effetti non compare mai il messaggio di controllo "valore inserito" che ho messo per verificare questo fatto, ma non riesco a capire perchè.
Qualcuno mi sà dire cosa ho sbagliato?
Oltretutto la lista di opzioni mi appare solo una volta nonostante sia inserita in un ciclo infinito...

Ecco il codice...è un po' incasinato perchè ho tentato di cambiare qualcosa qui e là per cercare di risolvere...

#include <iostream.h>
#include <stdlib.h>
#include <stdio.h>
int i;

class Array {
public:

int CurNum;
int val[30];


Array() { CurNum = 0; }
void Inserisci(int num);
void Modifica (int num, int pos);
void Visualizza();
void Cancella();
};

void Array::Inserisci(int num)
{
int MaxNum = 30;
if (CurNum < MaxNum)
{

for (i = CurNum; i >= 0; --i)
{
val[i+1] = val[i];
}
val[0] = num;
CurNum++;
cout << "Valore inserito\n";
}
else {cout << "Spazio nell'array esaurito\n";}
}

void Array::Modifica(int num, int pos)
{
if (pos <= CurNum)
{
val[pos-1] = num;
}
else cout << "Indice non valido\n";
}

void Array::Visualizza()
{

cout << "Valori presenti nell'array:\n";
for (i = 0; i < CurNum; i++)
{
cout << val[i] << "\n";
}
cout << CurNum << " valori visualizzati\n";
}

void Array::Cancella()
{
CurNum = 0;
val[0] = 0;
}


int main()
{
Array array1;
int numero, posizione;
char c;
for (;;)
{
cout << "Scegliere un'opzione:\n";
cout << "1. Inserisci un valore in cima all'array\n";
cout << "2. Modifica un valore dell'array\n";
cout << "3. Visualizza l'array\n";
cout << "4. Cancella l'array\n";
cout << "5. Esci dal programma\n";

c = getchar();
switch(c)
{
case '1':
cout << "\nImmetti il nuovo valore:";
cin >> numero;
array1.Inserisci(numero);

case '2':
cout << "\nImmetti l'indice del valore che vuoi modificare:";
cin >> posizione;
cout << "\nImmetti il nuovo valore:";
cin >> numero;
array1.Modifica(numero, posizione);

case '3':
array1.Visualizza();

case '4':
array1.Cancella();

case '5':
system("PAUSE");
return 0;
default:
cout << "\nScelta errata.";
}
}
}

Grazie in anticipo :)

/\/\@®¢Ø
16-09-2002, 20:49
L'errore piu' grave e' che dimentichi i break nello switch ! Se non lo fai vengono eseguiti oltre allo switch che scegli, anche tutti i successivi !
Mi sono permesso anche un altro paio di correzioni (non indispensabili comunque)


[b]/* in C++ sarebbe piu' adatto
usare questo header:
*/[/b]
#include <iostream>
#include <cstdlib>
#include <cstdio>


[b]//int i;[/b]

class Array {
public:

int CurNum;
int val[30];


Array() { CurNum = 0; }
void Inserisci(int num);
void Modifica (int num, int pos);
void Visualizza();
void Cancella();
};

void Array::Inserisci(int num)
{
int MaxNum = 30;
if (CurNum < MaxNum)
{
[b]// attento... se CurNum == MaxNum-1,
// con val[i+1] al primo passo sfori l'array !
// meglio usare variabili locali inoltre...
[/b]
for ([b]int[/b]i = CurNum[b]-1[/b]; i >= 0; --i)
{
val[i+1] = val[i];
}
val[0] = num;
CurNum++;
cout << "Valore inserito\n";
}
else {cout << "Spazio nell'array esaurito\n";}
}

void Array::Modifica(int num, int pos)
{
if (pos <= CurNum)
{
val[pos-1] = num;
}
else cout << "Indice non valido\n";
}

void Array::Visualizza()
{

cout << "Valori presenti nell'array:\n";
for (i = 0; i < CurNum; i++)
{
cout << val[i] << "\n";
}
cout << CurNum << " valori visualizzati\n";
}

void Array::Cancella()
{
CurNum = 0;
val[0] = 0;
}


int main()
{
Array array1;
int numero, posizione;
char c;
for (;;)
{
cout << "Scegliere un'opzione:\n";
cout << "1. Inserisci un valore in cima all'array\n";
cout << "2. Modifica un valore dell'array\n";
cout << "3. Visualizza l'array\n";
cout << "4. Cancella l'array\n";
cout << "5. Esci dal programma\n";


c = getchar();
switch(c)
{
case '1':
cout << "\nImmetti il nuovo valore:";
cin >> numero;
array1.Inserisci(numero);
[b]break[/b];
case '2':
cout << "\nImmetti l'indice del valore che vuoi modificare:";
cin >> posizione;
cout << "\nImmetti il nuovo valore:";
cin >> numero;
array1.Modifica(numero, posizione);
[b]break[/b];

case '3':
array1.Visualizza();
[b]break[/b];

case '4':
array1.Cancella();
[b]break[/b];

case '5':
system("PAUSE");
return 0;
[b]break[/b];
default:
cout << "\nScelta errata.";
}
}
}