Har 2 spøgsmål til din løsning:
1. Er det nødvendigt at lave et nyt objekt af din scanner for hver iteration af løkken eller kunne du oprette et objekt ude af løkken og bruge dette inde i løkken.
2. Kan du se nogen som helst mulighed for at x>y eller x==y er sande, hvis du lige har verificeret at x<y er sand, hvis ikke så har en if sætning en mulig else blok eller evt. har en løkke en continue statement.
Helt generelt bør man aldrig, synes jeg, teste for noget der aldrig kan lade sig gøre.
Som f.eks.
if(a<0){
bla bla
}
if(a==0){
bla bla
}
if(a>0){
bla bla
}
Hvis bare en af de ovenstående test er sand vil de 2 andre altid være falske ergo spilder du cpu på at teste dem så:
if(a<0){
bla bla
}else{
if(a==0){
bla bla
}else{
bla bla
}
}
Et andet eksempel er at når du tester på to ting i samme if sætning og den ene af disse er den samme i alle if sætninger:
if( a==0 && B){
bla bla bla
}
if(a==0 && C){
bla bla
}
if(a==0 && D){
bla bla
}
Der kan man tage den ens del af testen udtil en overordnet if sætning:
if(a==0){
if(B){
bla bla bla
}
if(C){
bla bla
}
if(D){
bla bla
}
}