Il modo più semplice per capovolgere un valore booleano?

Voglio solo capovolgere un booleano in base a ciò che è già. Se è vero, rendilo falso. Se è falso, rendilo vero.

Ecco il mio estratto di codice:

switch(wParam) {

case VK_F11:
  if (flipVal == true) {
     flipVal = false;
  } else {
    flipVal = true;
  }
break;

case VK_F12:
  if (otherVal == true) {
     otherValVal = false;
  } else {
    otherVal = true;
  }
break;

default:
break;
}

Puoi capovolgere un valore in questo modo:

myVal = !myVal;

quindi il tuo codice si ridurrebbe a:

switch(wParam) {
    case VK_F11:
    flipVal = !flipVal;
    break;

    case VK_F12:
    otherVal = !otherVal;
    break;

    default:
    break;
}

Chiaramente hai bisogno di un modello di fabbrica!

KeyFactory keyFactory = new KeyFactory();
KeyObj keyObj = keyFactory.getKeyObj(wParam);
keyObj.doStuff();


class VK_F11 extends KeyObj {
   boolean val;
   public void doStuff() {
      val = !val;
   }
}

class VK_F12 extends KeyObj {
   boolean val;
   public void doStuff() {
      val = !val;
   }
}

class KeyFactory {
   public KeyObj getKeyObj(int param) {
      switch(param) {
         case VK_F11:
            return new VK_F11();
         case VK_F12:
            return new VK_F12();
      }
      throw new KeyNotFoundException("Key " + param + " was not found!");
   }
}

: D

</sarcasm>

Se sai che i valori sono 0 o 1, potresti farlo flipval ^= 1.

La soluzione più semplice che ho trovato:

x ^= true;

Solo per informazione: se invece di un numero intero il tuo campo richiesto è un singolo bit all'interno di un tipo più grande, usa invece l'operatore "xor":

int flags;

int flag_a = 0x01;
int flag_b = 0x02;
int flag_c = 0x04;

/* I want to flip 'flag_b' without touching 'flag_a' or 'flag_c' */
flags ^= flag_b;

/* I want to set 'flag_b' */
flags |= flag_b;

/* I want to clear (or 'reset') 'flag_b' */
flags &= ~flag_b;

/* I want to test 'flag_b' */
bool b_is_set = (flags & flag_b) != 0;

Questo sembra essere un libero per tutti ... Eh. Ecco un'altra variante, che immagino sia più nella categoria "intelligente" di qualcosa che consiglierei per il codice di produzione:

flipVal ^= (wParam == VK_F11);
otherVal ^= (wParam == VK_F12);

Immagino che i suoi vantaggi siano:

• Molto conciso
• Non richiede ramificazione

E uno svantaggio altrettanto ovvio è

• Molto conciso

Questo è vicino alla soluzione di @ korona usando?: Ma ha fatto un (piccolo) passo avanti.

Solo perché il mio modo strano preferito per attivare o disattivare un bool non è elencato ...

bool x = true;
x = x == false;

funziona anche. :)

(sì, x = !x;è più chiaro e più facile da leggere)

La soluzione codegolf'ish sarebbe più simile a:

flipVal = (wParam == VK_F11) ? !flipVal : flipVal;
otherVal = (wParam == VK_F12) ? !otherVal : otherVal;

Preferisco la soluzione di John T, ma se vuoi andare tutto in codice, la tua dichiarazione si riduce logicamente a questo:

//if key is down, toggle the boolean, else leave it alone.
flipVal = ((wParam==VK_F11) && !flipVal) || (!(wParam==VK_F11) && flipVal);
if(wParam==VK_F11) Break;

//if key is down, toggle the boolean, else leave it alone.
otherVal = ((wParam==VK_F12) && !otherVal) || (!(wParam==VK_F12) && otherVal);
if(wParam==VK_F12) Break;

flipVal ^= 1;

lo stesso vale per

otherVal

Chiaramente hai bisogno di una soluzione flessibile in grado di supportare tipi mascherati da booleani. Quanto segue lo consente:

template<typename T>    bool Flip(const T& t);

Puoi quindi specializzarlo per diversi tipi che potrebbero fingere di essere booleani. Per esempio:

template<>  bool Flip<bool>(const bool& b)  { return !b; }
template<>  bool Flip<int>(const int& i)    { return !(i == 0); }

Un esempio di utilizzo di questo costrutto:

if(Flip(false))  { printf("flipped false\n"); }
if(!Flip(true))  { printf("flipped true\n"); }

if(Flip(0))  { printf("flipped 0\n"); }
if(!Flip(1)) { printf("flipped 1\n"); }

No, non dico sul serio.

Per i numeri interi con valori di 0 e 1 puoi provare:

value = abs(value - 1);

MWE in C:

#include <stdio.h>
#include <stdlib.h>
int main()
{
        printf("Hello, World!\n");
        int value = 0;
        int i;
        for (i=0; i<10; i++)
        {
                value = abs(value -1);
                printf("%d\n", value);
        }
        return 0;
}

Solo perché mi piace mettere in discussione il codice. Propongo che tu possa utilizzare anche il ternario facendo qualcosa del genere:

Esempio:

bool flipValue = false;
bool bShouldFlip = true;
flipValue = bShouldFlip ? !flipValue : flipValue;