>max=(n1>n2) ? n1:n2;
これって()要らないんじゃない?
max=n1>n2 ? n1:n2;
でいいんじゃない?