Il modo più semplice per capovolgere un valore booleano?
-
03-07-2019 - |
Domanda
Voglio solo girare un booleano in base a quello che è già. Se è vero, fallo 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;
}
Soluzione
Puoi capovolgere un valore in questo modo:
myVal = !myVal;
quindi il tuo codice si accorcerebbe in:
switch(wParam) {
case VK_F11:
flipVal = !flipVal;
break;
case VK_F12:
otherVal = !otherVal;
break;
default:
break;
}
Altri suggerimenti
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, puoi fare flipval ^ = 1
.
Soluzione più semplice che ho trovato:
x ^= true;
Solo per informazione - se invece di un numero intero il 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 ... Heh. 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);
Suppongo che i suoi vantaggi siano:
- Molto conciso
- Non richiede la diramazione
E uno svantaggio altrettanto evidente è
- Molto conciso
Questo è vicino alla soluzione di @ korona usando?: ma ha fatto un (piccolo) passo in avanti.
Solo perché il mio modo preferito di ballare dispari per attivare 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 affermazione 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 i tipi mascherati da booleani. Ciò consente:
template<typename T> bool Flip(const T& t);
Puoi quindi specializzarlo per diversi tipi che potrebbero fingere di essere booleani. Ad 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 sono serio.