[C] min e max tra 3 num pt2

[Metallica]

Allora ho questa funzione in un file chiamato func.c:

int min (int a, int b, int c)
{
int min;

/* DETERMINAZIONE DEL MINIMO */
	if (a<b) if (a<c){
	   min = a;} 
	else if (b<a) if (b<c){
	   min = b;}
	else if (c<a) if(c<b){
	   min = c;}
return min;
}

Poi ho il programma chiamante:

#include <stdio.h>
#include "func.c"

main()
{
    int Min;
      
    Min = min(2, 3, 8);
    
    printf("Min: %d\n", Min);
}

In questo caso la funzione và bene e mi restituisce

Min: 2

ma purtroppo funziona solo se il minimo è a.
ovvero se la sequenza è:
2, 3, 8 -> restituisce 2
ma se è:
4, 2, 8 -> restituisce 0
e anche se è:
5, 6, 2 -> restituisce 0.

Perchè? Dove sbaglio??

[ff8mania]

Fai così:

main.cpp

#include <stdio.h>
#include "funz.h"

void main() {

int Nmin = 0;

Nmin = min(21,34,42);
printf("Il numero + piccolo è: %d\n", Nmin);

}

funz.h

#include <stdio.h>

int min(int, int, int);

int min(int a, int b, int c) {
if(a<b && b<c) {
return a;
}
else if(b<a && a<c) {
return b;
}
else if(c<b && b<a) {
return c;
}
return -1;
}

Bye

[LeleFT]

Originariamente inviato da Metallica

int min (int a, int b, int c)
{
int min;

/* DETERMINAZIONE DEL MINIMO */
	if (a<b) if (a<c){
	   min = a;} 
	else if (b<a) if (b<c){
	   min = b;}
	else if (c<a) if(c<b){
	   min = c;}
return min;
}

Perchè? Dove sbaglio??

Dovresti imparare ad indentare un po' meglio il codice, in questo modo ti accorgeresti che stai sbagliando a scrivere il codice (ecco cosa si intende per BASI DELL'INFORMATICA...).

Questo è il tuo codice meglio indentato:

if (a<b) 
   if (a<c){
      min = a;
   }else 
      if (b<a)
         if (b<c){
            min = b;
         }else
            if (c<a)
               if(c<b){
                  min = c;
               }

Ecco, invece, quello che avresti dovuto scrivere tu:

if (a<b) && (a<c) min = a;
else {
   if (b<a) && (b<c) min = b;
   else min = c;
}

Molto più compatto, molto più leggibile, molto più corretto.


Ciao.

[sapore di mare]

Molto più pignolo!!!! :gren: :gren: :tongue:
Ciao ciao :adhone:

[LeleFT]

Originariamente inviato da sapore di mare
Molto più pignolo!!!! :gren: :gren: :tongue:
Ciao ciao :adhone:

Beh... più che pignolo direi che una buona indentazione, in questo caso, fa la differenza fra un codice CORRETTO e uno SBAGLIATO. :gren:


Ciao.

[Metallica]

a me aveva detto un ragazzo che non si poteva usare if (COND && COND2) ma if (cond) if (cond)

non era colpa dell'indentazione

[LeleFT]

Originariamente inviato da LeleFT

if (a<b) && (a<c) min = a;
else {
   if (b<a) && (b<c) min = b;
   else min = c;
}

Ciao.

Ebbene sì! Mancano delle parentesi:

if ((a<b) && (a<c)) min = a;
else {
   if ((b<a) && (b<c)) min = b;
   else min = c;
}

Proviamo a risolverlo senza l'uso dei connettivi logici:

if (a<b) {
   if (a<c) min = a;
   else min = c;
} else {
   if (b<c) min = b;
   else min = c;
}

Ecco il risultato. Nettamente differente da ciò che avevi scritto tu.

Ciao.

[S4NdrOx]

ghgh lele vedo che vai deciso! massì spakkali >>> :metallica

[LeleFT]

No... non vado deciso. Ho solo messo in evidenza che a volte l'indentazione di un programma semplifica notevolmente la vita sia al programmatore che redige il codice, sia a chi, poi, lo deve rivedere. Certi errori, come si vede bene, vengono facilmente risolti dall'indentazione stessa, che mette sullo stesso piano tutte le condizioni che sono allo stesso livello logico di esecuzione.

Siccome penso che in questo forum (come in altri) la regola sia "cercare di aiutare gli altri al meglio, non limitandosi al caso specifico, ma, possibilmente, facendo capire il procedimento con cui vengono risolti i casi particolari, per poterli estendere al caso generale", cerco anch'io di fare la mia parte (visto che ho avuto modo, anch'io come molti, di imparare delle cosucce nuove, leggendo le discussioni poste da altri).


Ciao.

[Andrea Simonassi]

Scusate se mi intrometto, vorrei solo dire che Lele ha perfettamente ragione, una buona identazione aiuta a tenere sott'occhio il flusso logico del programma, cosa molto utile in lunghe catene di if.

Non è una questione di gusti, infatti nell caso specifico, non solo il codice risulta poco leggibile, ma è anche sbagliato.