[C] Operatore logico OR e condizione sempre vera

[pastoreerrante]

Un saluto al forum,

non riesco a capire come mai questo codice non funzioni come dovrebbe:

#include <stdio.h>

int main ()

{

   int valore;

   printf ("Inserire l'intero 1 o l'intero 2:");
   scanf ("%d", &valore);

   if ( valore != 1 || valore != 2 )
      printf ("Il valore inserito non è corretto. I valori ammessi sono solo 1 e 2\n");

   else
      printf ("Il valore inserito è corretto\n");

   return 0;

}

Il problema è la condizione if ( valore != 1 || valore != 2 ): il senso dell'istruzione dovrebbe essere "Se valore è diverso da 1 O da 2, stampa "Il valore inserito non è corretto..."". Invece anche se immetto 1 0 2 mi restituisce sempre "il valore inserito non è corretto".

Come posso risolvere?

Grazie

[king64]

Modifica:

if ( valore != 1 || valore != 2 )

con

if ( valore != 1 && valore != 2 )

Saluti

[oregon]

Originariamente inviato da pastoreerrante
... "Se valore è diverso da 1 O da 2, ...

No ... il senso dovrebbe essere

"Se valore è diverso da 1 E da 2, ...

[mondobimbi]

per chiarezza del codice e facilitare l'umana comprensione è meglio scriverla nella forma

   if ( valore == 1 || valore == 2 )
      printf ("Il valore inserito è corretto\n");
   else
      printf ("Il valore inserito non è corretto. I valori ammessi sono solo 1 e 2\n");

in questo caso potrebbe anche essere di più immediata comprensione uno switch

ciao

sergio

[oregon]

Originariamente inviato da mondobimbi
per chiarezza del codice e facilitare l'umana comprensione

D'accordo Sergio, ma sarebbe bastata un po' di "umana conoscenza dell'algebra booleana" ...

[pastoreerrante]

Grazie a tutti, ho risolto.

[mondobimbi]

Originariamente inviato da oregon
D'accordo Sergio, ma sarebbe bastata un po' di "umana conoscenza dell'algebra booleana" ...

mi riferivo al fatto che un and di due valori negati non si usa nel linguaggio comune, infatti si dice

"mi compro una mela o una pera"

ciao
sergio