Zaciatocnik - kontrola kodu – Java – Fórum – Programujte.com

Ahojte mam za ulohu :

Přečtěte dvě celá čísla a zeptejte se, jakou operaci s nimi máte provést (+, -, *, přesné dělení, celočíselné dělení a zbytek po celočíselném dělení). Vytiskněte výsledek. Pozor na dělení nulou u operací dělení a zbytek po dělení! Pokud dojde k zadání chybného čísla operace, vypište upozornění na obrazovku. Použijte příkaz switch !!! Nezařazujte cyklus.

no a toto je moj vytvor : 

public class uloha {
  
 
 
 	 /* Vrati chybne zadane cislo
	 */
	public static int ctiI() {
		int i = 0;
		byte[] pole = new byte[10];
		boolean nacteno = false;
		do {
			try {
				System.in.read(pole);
				String vstup = new String(pole).trim();
				i = Integer.valueOf(vstup).intValue();
				nacteno = true;
			} catch (Exception e) {
				System.out.print("Chybne, zadajte cele cislo: ");
			}
		} while (!nacteno);
		return i;
	}
	
	public static int ctiJ() {
		int cislo = 0;
		byte[] pole = new byte[10];
		boolean nacteno = false;
		do {
			try {
				System.in.read(pole);
				String vstup = new String(pole).trim();
				cislo = Integer.valueOf(vstup).intValue();
				nacteno = true;
			} catch (Exception e) {
				System.out.print("Chybne, zadajte cele cislo: ");
			}
		} while (!nacteno);
		return cislo;
	}
	
	
	
        
        
	
 public static void main(String[] args){
     
     int cisloOperace;
   
 
        System.out.print("Zadaj 1. cislo: ");       //Zada prve cislo
        int i =ctiI();                            
        System.out.print("Zadaj 2. cislo: ");       //Zada druhe cislo
        int j =ctiJ();
        
        
        System.out.println("1. sčítanie čísel\n2. odčítanie čísel\n3. násobenie čísel\n4. presné delenie čísel\n"
                + "5. celočíselné delenie\n6. Zostatok po celočíselnom delení");
        System.out.print("Zdajte číslo požadovanej operácií: ");
        cisloOperace = ctiI();
        
        switch(cisloOperace) {
            case 1:
                System.out.println("Výsledok po sšítaní je: " +(i+j));
                break;
            case 2:
                System.out.println("Výsledok po odčítaní je: " +(i-j));
                break;
            case 3:
                System.out.println("Výsledok po znásobení je: " +(i*j));
                break;
            case 4:
                if (j == 0) {
         System.out.println("Čsilom nula sa nedá deliť!");
     }
                else {
         System.out.println("Výsledok po presnom delení je: " +((float)i/j));
     }
                break;
            case 5: 
                if (j==0) { 
         System.out.println("Číslom nula sa nedá deliť!");
     }
         
                else {
         System.out.println("Výsledok po celočíselnom delení je: " +(i/j));
     }
                break;
            case 6:
                if (j==0) { 
         System.out.println("Číslom nula sa nedá deliť!");
     }
         
                else {
         System.out.println("Zostatok po celočíselnom delení je: " +(i%j));
     }
                break;
            
            default:
                System.out.println("Chybne ste zadali číslo požadovanej operácií!");
                break;
     
                }
        }
        
                }

chcem sa opytat, ci sa vam to pozdava, ale v prostredi NetBeans mam vystup v poriadku ale v JCreator mi to spravi toto:

Zadaj 1. cislo: 5
Zadaj 2. cislo: 6
1. sčítanie čísel
2. odčítanie čísel
3. násobenie čísel
4. presné delenie čísel
5. celočíselné delenie
6. Zostatok po celočíselnom delení
Zdajte číslo požadovanej operácií: Chybne, zadajte cele cislo:

to Chybne, zadajte cele cislo, by sa tam nemalo ukazovat.

Dakujem za rady, a pripadne vylepsovaky :)

no první vylepšení je,že bych přepracoval to čtení .. obecně platí, že jakmile se kod opakuje, měl by se předělat. A koukni na tvoje metody <i>ctiI , ctiJ</i> ty jsou v podstatě totožné. Navíc přece metoda System.in.read() je přetížená, takže není nutné načítat pole bytů a to potom převádět.

To, že ti to každý komplilátor přelozí jinak není žádná novinka, zkusil bych to načítání předělat jenom metodou system.in.read() a uvidíš jak to půjde

+ system.in.read je blokovací operace, takže dle mě nemusíš to mít v cyklu

Jak dlouho jsi na tom dělal, než jsi to sem vrazil? To načítání čísel máš blbě.

#2 host
Tady se člověk dozví věci...

a zle tam je to , ze tie triedy ctiI a ctiJ su dve? alebo co je tam presne zle? dik