Está errado porque você não pode comprar mais que dois operandos ao mesmo tempo. Isto não existe: if ( n1 > n2 > n3) { O correto seria: if ( n1 > n2 && n2 > n3) { O mesmo vale para os demais. Acho que já pegou o jeito.