Chyba v kódu Javy - poradna.net

Zdravim,
nenapadá prosím Vás někoho kde je chyba v tomto kódu ?

public class ukol {
    public void sudecislo (int[] pole){
        int pocet = 0;
        for (int i = 0; i < pole.length; i++)
            if (pole[i]%2 == 0){
                pocet++;           
        }
       
        System.out.println(pocet);
       
    }
        public static void main(String[] args) {
 
       ukol5 h = new ukol5();
       int[] testpole = {7,9,15,19,21, 22, 44, 256};
       h.sudecislo(testpole);
       
        }
}

Zadání úkolu bylo: Vytvořte metodu, která vrátí počet prvků pole, které mají sudou hodnotu.
Získal jsem 3 body z 5; a jediná chyba která mě napadla je malé počáteční písmeno v názvu třídy.

Předem děkuji za pomoc

Předmět	Autor	Datum
Ja neviem jake body si ziskal kde, ani preco si myslis ze tam je nejaka chyba, ale ked ma metoda nie… nový MM.. 13.01.2016 13:32	MM..	13.01.2016 13:32
BTW: ukol5 h = new ukol5(); trieda sa vola predsa ukol, a ne ukol5 nový MM.. 13.01.2016 13:34	MM..	13.01.2016 13:34
to je jen chyba ted pri opisovani z notebooku nový AldyDD 13.01.2016 13:36	AldyDD	13.01.2016 13:36
Jak říká MM. public class ukol { public int sudecislo (int[] pole){ //<---- ZMENA int pocet = 0; fo… nový MašinkaTomáš 13.01.2016 14:24	MašinkaTomáš	13.01.2016 14:24
Hlavně vůbec nemusí dělat třídu, která slouží za účelem získání počtu sudých čísel. Ta metoda může b… nový freez2 13.01.2016 14:29	freez2	13.01.2016 14:29
Ano, to nemusí. poslední MašinkaTomáš 13.01.2016 14:51	MašinkaTomáš	13.01.2016 14:51

Ja neviem jake body si ziskal kde, ani preco si myslis ze tam je nejaka chyba, ale ked ma metoda nieco vratit, tak to ma vratit (nema byt void), a ne vypisat. V zadani nebolo nikde napisane ze mas nieco vypisovat.
Velksot pismen je nepodstana (ak to nehodnotil nejaky magor).

BTW:
ukol5 h = new ukol5();
trieda sa vola predsa ukol, a ne ukol5

to je jen chyba ted pri opisovani z notebooku

Jak říká MM.

public class ukol {

    public int sudecislo (int[] pole){ //<---- ZMENA
        int pocet = 0;
        for (int i = 0; i < pole.length; i++)
            if (pole[i] % 2 == 0){
                pocet++;           
        }       
        return pocet; //<---- ZMENA       
    }

    public static void main(String[] args) { 
       ukol h = new ukol(); //<---- ZMENA
       int[] testpole = {7,9,15,19,21, 22, 44, 256};
       int pocetSudychCisel =  h.sudecislo(testpole); //<---- ZMENA
       System.out.println(pocetSudychCisel); //<---- ZMENA
    }      
}

Jinak, pokud hodnotili i to, jak "to je hezký", tak bych měl pár připomínek:
sudecislo -> z toho je prd poznat, dal bych něco jako ziskejPocetSudychCisel
kdyz uz sudecislo -> tak prvni pismeno druheho slova velke "sudeCislo".
Trida velke pismenko :)
Nemusíš ukládat referenci jen na jedno použití. Tady jde:

int pocetSudychCisel = new ukol().sudecislo(testpole);

Pokud ti učitel dal 3z5 měl by ti být schopen říct PROČ.

Hlavně vůbec nemusí dělat třídu, která slouží za účelem získání počtu sudých čísel. Ta metoda může být klidně statická a volána přímo z mainky.

Ano, to nemusí.

Zpět do poradny Odpovědět na původní otázku Nahoru