Está errado porque você não pode comparar mais que dois operandos ao mesmo tempo. Isto não existe:
if ( n1 > n2 > n3) {
O correto seria:
if ( n1 > n2 && n2 > n3) {
Coloquei no GitHub para referência futura.
Desta forma você tem três expressões com dois operandos cada: n1 > n2
, n2 > n3
e a operação relacional &&
que tem como operandos os resultados da expressões anteriores.
O mesmo vale para os demais. Acho que já pegou o jeito.
Tem maneiras melhores de fazer isto. Usar Math.max()
e Math.min()
seria uma delas. Não vou colocar porque me parece que você está fazendo isto para fins de estudo e não quer a melhor solução.