Significado da expressão: "wvar = id == null;"

Isso é uma péssima prática de programação que significa que você está atribuindo o valor de id a wvar e, na sequência, checando se isto é null.

(Isto deveria ser refatorado, com certeza).

[]´s

asaudate:

você está atribuindo o valor de id a wvar e, na sequência, checando se isto é null.

Na verdade a variável wvar vai receber o resultado da comparação entre id e null, ou seja, se a variável id for null, então wvar vai receber true.

Não é uma má prática de programação. Conforme dito pelo colega acima, é apenas uma atribuição de um valor booleano a uma variável. Outro código que faria a mesma coisa seria:

if ( id == null ) 
  wvar = true;
else wvar = false;

Da forma mencionada por você, o código fica mais limpo e não vejo problemas de interpretação.

Um parentheses cairia bem ali.

asaudate:

Isso é uma péssima prática de programação que significa que você está atribuindo o valor de id a wvar e, na sequência, checando se isto é null.

(Isto deveria ser refatorado, com certeza).

[]´s

acho que o colega se confundiu, pq em C/C++ por ex é possível fazer algo do tipo…

char c;
while (c = get_char() != NULL) {
    // do something
}

informando que wvar é um boolean ficaria mais claro de entender…

boolean wvar = (id == null);

ou

boolean wvar;
// do something
wvar = id == null;