Questo sito utilizza cookies solo per scopi di autenticazione sul sito e nient'altro. Nessuna informazione personale viene tracciata. Leggi l'informativa sui cookies.
//prototipi dei nostri sottoprogrammi (funzioni) dico cosa faccio int menu(); int Calcolo_Coppie_Uguale(); int Calcola_Coppie_Divisibili();
int main() { //inizio la seconda funzione del menu(aggiunto da me) int sceltaMain; sceltaMain=menu(); //chiamo la funzione menu che restituisce la scelta if(sceltaMain==1){ int risultato=Calcolo_Coppie_Uguale(); printf("Il numero di coppie uguali è %d\n",risultato); } else if(sceltaMain==2){ int risultato=Calcola_Coppie_Divisibili(); printf("Il numero di coppie Divisibili è %d\n",risultato);
} else{ //stampare un messaggio di errore }
return 0; } //definisco come fanno le cose i prototipi
//funzione che restituisce la scelta fatta dall'utente int menu(){ int scelta; //locale al menu printf("Quale operazione vuoi eseguire?\n"); printf("Inserisci 1 per coppie uguali consecutive\n"); printf("Inserisci 2 per coppie divisibili consecutive\n"); scanf("%d",&scelta); return scelta; }
//acquisiti i numeri restituisce il numero di coppie consecutive uguali int Calcolo_Coppie_Uguale(){ int contatore; //variabile locale a questo sottoprogramma int numCorrente; int numPrecedente; contatore=0; printf("Comincia ad inserire i numeri (0 termina inserimento)!!\n"); scanf("%d",&numCorrente); numPrecedente=0; while(numCorrente>0){ if(numCorrente==numPrecedente){ contatore++; } numPrecedente=numCorrente; printf("Inserisci un nuovo numero (zero per terminare)\n"); scanf("%d",&numCorrente);
} return contatore; }
//acquisiti i numeri restituisce il numero di cppie consecutive divisibili int Calcola_Coppie_Divisibili(){ int contatore; //variabile locale a questo sottoprogramma int numCorrente; int numPrecedente; contatore=0; printf("comincia ad inserire i numeri (0 termina inserimento)!!\n"); scanf("%d",&numCorrente); numPrecedente=0; while(numCorrente>0);{ if(numCorrente%numPrecedente==0){ contatore++; } numPrecedente=numCorrente; printf("Inserisci un nuovo numero (zero per terminare)\n"); scanf("%d",&numCorrente); } return contatore; }
// non capisco perchè la seconda funzione non funziona
Ti è scappato un punto e virgola di troppo tra la parentesi chiusa e la graffa aperta. Quel punto e virgola dà luogo a un ciclo potenzialmente infinito (se il primo numero inserito è maggiore di 0).
Codice sorgente - presumibilmente Plain Text
while(numCorrente>0);{ // l'errore è qui...
if(numCorrente%numPrecedente==0){
contatore++;
}
numPrecedente=numCorrente;
printf("Inserisci un nuovo numero (zero per terminare)\n");
scanf("%d",&numCorrente);
}
Come potrebbe testimoniare qualsiasi parroco un minimo esperto, è certo che una funzione che non funziona è poco funzionale.
Ultima modifica effettuata da AldoBaldo il 03/11/2020 alle 21:05
ATTENZIONE! Sono un hobbista e l'affidabilità delle mie conoscenze informatiche è molto limitata. Non prendere come esempio il codice che scrivo, perché non ho alcuna formazione accademica e rischieresti di apprendere pratiche controproducenti.
Oltre al punto e virgola che ti ha fatto notare AldoBaldo, non puoi scrivere: numPrecedente = 0
e poi farci una divisione anche se vuoi solo il resto: if(numCorrente % numPrecedente == 0)
Codice sorgente - presumibilmente C++
int Calcola_Coppie_Divisibili(){
int contatore =0;//variabile locale a questo sottoprogramma
int numCorrente =0;
int numPrecedente =1;
printf("comincia ad inserire i numeri (0 termina inserimento)!!\n");
scanf("%d",&numCorrente);
while(numCorrente>0){
if(numCorrente % numPrecedente == 0){
contatore++;
}
numPrecedente=numCorrente;
printf("Inserisci un nuovo numero (zero per terminare)\n");
scanf("%d",&numCorrente);
}
return contatore;
}
Ultima modifica effettuata da Carlo il 03/11/2020 alle 21:36
Vero! Non avevo fatto caso alla letale divisione per zero.
Basta aggiungere una controllo di condizione che assicuri che il numero precedente sia diverso da zero e in caso contrario eviti la divisione incriminata.
ATTENZIONE! Sono un hobbista e l'affidabilità delle mie conoscenze informatiche è molto limitata. Non prendere come esempio il codice che scrivo, perché non ho alcuna formazione accademica e rischieresti di apprendere pratiche controproducenti.
printf("Inserisci un nuovo numero (zero per terminare)\n");
scanf("%d",&numCorrente);
}
return contatore;
}
&& salta il confronto successivo e ti mette al riparo dall'errore, ma la logica di conteggio è sbagliata, il risultato non è sempre correto, Milton ci deve lavorare un po'...
Ultima modifica effettuata da Carlo il 03/11/2020 alle 22:53
Direi che con il controllo numPrecedente!=0 il conteggio è corretto, perché...
riga 6. viene inserito numCorrente
riga 8. se numCorrente è <1 il ciclo while viene saltato integralmente
riga 9. arrivati qui, numPrecedente e' SICURAMENTE >=0, perché al primo ciclo è 0 per inizializzazione, mentre dopo è >0 essendo che la condizione viene controllata all'ingresso del while
riga 10. si entra in questo blocco solo a partire dal secondo ciclo (nel primo ciclo non c'è comunque nulla da confrontare, dato che si ha un solo numero e non si può fare alcuna divisione con %) e solo se il resto della divisione è 0
riga 12. numPrecedente riceve un valore che è SICURAMENTE >0
Vedi qualche falla? A me sembra tutto a posto.
Ultima modifica effettuata da AldoBaldo il 04/11/2020 alle 16:59
ATTENZIONE! Sono un hobbista e l'affidabilità delle mie conoscenze informatiche è molto limitata. Non prendere come esempio il codice che scrivo, perché non ho alcuna formazione accademica e rischieresti di apprendere pratiche controproducenti.
@ AldoBaldo,
l'errore è nella logica, ti è sfuggito perché pensavi che chi ha scritto lo ha fatto logicamente, invece ha solo copiato e incollato il codice delle coppie, e cercato di faro funzionare con i divisibili.
inserisco il primo numero = 10
contatore = 0;
numCorrente = 10;
numPrecedente = 0;
entro nel while e la condizione (numPrecedente != 0) è falsa si salta a
numPrecedente=numCorrente;
contatore = 0;
numCorrente = 10;
numPrecedente = 10;
inserisco il secondo numero = 5
contatore = 0;
numCorrente = 5;
numPrecedente = 10;
ritorno all'if (numPrecedente != 0) è vera, (numCorrente % numPrecedente == 0) è falsa perché 5 % 10 non da zero
contatore non si incrementa.
i numeri immessi sono 10 e 5, sono divisibili e contatore si doveva incrementare.
numCorrente % numPrecedente, sono invertiti, nel calcolo delle coppie consecutive è ininfluente, in una divisione no...
E' una diatriba tra noi perché Milton ha posto il quesito, ma non se ne interessa.
corretto:
Codice sorgente - presumibilmente C++
#include <stdio.h>
//prototipi dei nostri sottoprogrammi (funzioni) dico cosa faccio
int menu();
int Calcolo_Coppie_Uguale();
int Calcola_Coppie_Divisibili();
int main()
{//inizio la seconda funzione del menu(aggiunto da me)
int sceltaMain;
sceltaMain=menu();//chiamo la funzione menu che restituisce la scelta
if(sceltaMain==1){
int risultato=Calcolo_Coppie_Uguale();
printf("Il numero di coppie uguali sono %d\n",risultato);
}
elseif(sceltaMain==2){
int risultato=Calcola_Coppie_Divisibili();
printf("Il numero di coppie Divisibili sono %d\n",risultato);
}
else{
//stampare un messaggio di errore
}
return0;
}
//definisco come fanno le cose i prototipi
//funzione che restituisce la scelta fatta dall'utente
int menu(){
int scelta;//locale al menu
printf("Quale operazione vuoi eseguire?\n");
printf("Inserisci 1 per coppie uguali consecutive\n");
printf("Inserisci 2 per coppie divisibili consecutive\n");
scanf("%d",&scelta);
return scelta;
}
//acquisiti i numeri restituisce il numero di coppie consecutive uguali
int Calcolo_Coppie_Uguale(){
int contatore =0;//variabile locale a questo sottoprogramma
int numCorrente =0;
int numPrecedente =0;
contatore=0;
printf("Comincia ad inserire i numeri (0 termina inserimento)!!\n");
scanf("%d",&numCorrente);
numPrecedente=0;
while(numCorrente>0){
if(numCorrente==numPrecedente){
contatore++;
}
numPrecedente=numCorrente;
printf("Inserisci un nuovo numero (zero per terminare)\n");
scanf("%d",&numCorrente);
}
return contatore;
}
//acquisiti i numeri restituisce il numero di cppie consecutive divisibili
int Calcola_Coppie_Divisibili(){
int contatore =0;//variabile locale a questo sottoprogramma
int numCorrente =0;
int numPrecedente =0;
printf("comincia ad inserire i numeri (0 termina inserimento)!!\n");
Macché diatriba! E' che a me piace questo tipo di confronto, ci sguazzo come una carpa in una risaia.
Ho capito il punto, che avevo notato, ma non lo considero un errore di logica, ovvero considero che quando il numero precedente è 10 e quello corrente è 5, in effetti il contatore non deve essere aumentato, proprio perché 5 non è divisibile per 10. Cioè, considero parte delle "specifiche" (capirai...) richieste il fatto che sia l'ultimo numero inserito ad essere divisibile per quello precedente, e che il contrario non faccia parte delle richieste.
Invece, a quanto ho capito, secondo te dovrebbe essere verificata sia la divisibilità del numero precedente per quello corrente, sia la divisibilità del numero corrente per quello precedente. Si può fare senza neppure alzare un sopracciglio, ma... è poi quello che bisognerebbe fare secondo le "specifiche" dell'esercizio?
Codice sorgente - presumibilmente C++
int Calcola_Coppie_Divisibili(){
int qDiv =0;// qDiv: quantita' dei divisori
int nCorr =0;// nCorr: numero corrente
int nPrec =0;// nPrec: numero precedente
printf("Inserisci il primo numero (zero per terminare): ");
printf("Inserisci un nuovo numero (zero per terminare): ");
scanf("%d", &nCorr );
}
return qDiv;
}
ATTENZIONE! Sono un hobbista e l'affidabilità delle mie conoscenze informatiche è molto limitata. Non prendere come esempio il codice che scrivo, perché non ho alcuna formazione accademica e rischieresti di apprendere pratiche controproducenti.
Postato originariamente da AldoBaldo: Invece, a quanto ho capito, secondo te dovrebbe essere verificata sia la divisibilità del numero precedente per quello corrente, sia la divisibilità del numero corrente per quello precedente. Si può fare senza neppure alzare un sopracciglio, ma... è poi quello che bisognerebbe fare secondo le "specifiche" dell'esercizio?
Il menu di scelta dice: "Inserisci 2 per coppie divisibili consecutive".
se inserisco 10 e poi 5 (esco con 0), la risposta è: "il numero di coppie divisibili è zero", intanto questo risultato è sbagliato.
Poi se vuoi interpretare che la divisibilità va controllata anche al contrario, io non ci avevo pensato, ma il menù di scelta non lo vieta e potresti avere ragione, ma comunque non confuta il fatto che il programma originale da risultati sbagliati.
Carino il sitema per incrementare qDiv
Ultima modifica effettuata da Carlo il 04/11/2020 alle 22:37
| 
