Minimizzare loop in riferimento alle operazioni di lettura / scrittura

Question

l'ultima cosa che voglio minimizzare oggi è il codice che contiene po 'e do-while loop. Ecco l'originale:

class Vereinfache3_edit { 

        public static void main(String [] args) {

           int c1 = Integer.parseInt(args[0]) ;
           int c2 = Integer.parseInt(args[1]) ;
           int c3 = Integer.parseInt(args[2]) ;

/*  1 */           c1 += 7 ; 
/*  2 */           System.out.println( c1 ) ; 

/*  3 */       while (c1 % 8 != 0)
/*  4 */              if ( c1 % 16 == 0 ) ; 
/*  5 */              else
/*  6 */         do 
/*  7 */                 {
/*  8 */                    c1 += 7 ; 
/*  9 */                    System.out.println( c1 ) ; 
/* 10 */                    if ( c2 < c3 )
/* 11 */                       { c1 = c1+c1 ; 
/* 12 */                         c3 ++ ; 
/* 13 */                         c1 /= 2 ; 
/* 14 */                         c3 -= 1 ; 
/* 15 */                       }
/* 16 */                 }
/* 17 */                 while ( c1 % 8 != 0 ) ;

/* 18 */           c1 += 7 ; 
/* 19 */           System.out.println( c1 ) ; 
        }          

}  // end of class Vereinfache3

Ed ecco la mia versione ridotta:

class Vereinfache3 { 

        public static void main(String [] args) {

               int c1 = Integer.parseInt(args[0]);
               int c2 = Integer.parseInt(args[1]) ;
               int c3 = Integer.parseInt(args[2]) ;

               do{
                   c1 += 7 ; 
                   System.out.println( c1 ) ;               
               }while (c1 % 8 != 0);

/* 18 */       c1 += 7 ; 
/* 19 */       System.out.println( c1 ) ; 
        }          

}  // end of class Vereinfache3

Esso genera la stessa uscita per me. Vedete eventuali errori o cose che possono essere migliorate?

Soprattutto questo codice sembra essere ingannevole:

/*  3 */       while (c1 % 8 != 0)
/*  4 */              if ( c1 % 16 == 0 ) ; 
/*  5 */              else
/*  6 */          do{}

Come faccio a trattare con il tempo? Cosa contiene il ciclo while?

Answer 1

La versione minimizzata produce lo stesso risultato come la versione originale, uscite diverse per la soluzione originale e la versione minimizzata sono possibili solo nel caso in cui c1 % 8 != 0 and c1 % 16 == 0, questo caso è impossibile dal momento che tutti i multipli di 16 sono multipli 8 troppo ..

Rimozione del inutilizzato declarations..is l'unico miglioramento potrei suggerire. Io ho una soluzione con meno code..Not che rende le cose meglio ... non mi piace troppi sysouts nel codice ..:)

    int finalBound = ((c1 + 7) % 8 == 0) ? c1 + 7 : 8 * c1;
    for (; c1 <= finalBound; c1 += 7) {
        System.out.println(c1 + 7);
    }

La speranza che aiuta.