Počítání metody – Java – Fórum – Programujte.com

Zdravím vás všechno...

Potřeboval bych poradit jak mohu přičítat a odečítat podle toho kolikrát se ta metoda použije...

Já jsme to udělal takhle

 
int corner = 0;
void step(){
move();
corner = corner + 1;
}

Bohužel se mi to nefunguje..

A potřeboval bych a by nejdřív přičítal a potom odečítal a jak mile se dostane zpátky do nuly tak celí program skončí...

Děkuji moc za každou radu, jak bych si s tím mohl poradit...

#1 lekejs
Tohle by fungovat mělo. Asi jsi zapomněl uvést tu část, která nefunguje.

na co tam ma ten corner? nikam nepredavas, jen inicializujes pred tim nez se definitivne zahodi tak prictes...

#3 KIIV
Myslel jsem si, že to má uvnitř třídy, kterou nám tady neukázal.

mě zajímá jak potom vypadá algoritmus proto aby se mi to zase odečítalo jak mile sčítáni splní podmínku.. Jak udělat aby se to odečítalo a v okamžiku, kdy skončí podmínka pro sčítaní... A aby se s pomocí toho odečítání vrátil zpět do corner = 0...

já jsem to udělal pomocí while a mám tam podmínku, že jakmile program udělá dostatek kroků, tak se za se změní podmínka, že se musí vrátit na zpět a tam se ukončit...

#3 KIIV
ten corner to je můj název, který jsem tam použil... co myslíš tím zahodí??

#6 lekejs
Že ta proměnná po opuštění bloku přestane existovat. Pokud není v objektu, který tady nemáš uveden.

#7 Kit
i kdyby byla v i v objektu, tak pokud tam bude mit presne tudle posloupnost prikazu za sebou, tak tu objektovou prekreje lokalni verzi

#8 KIIV
vysvetli to pls ... deklarovana premenna je mimo metody a v metode neni deklarovana dalsia cize sa vola ta vonkajsia ... ak nezanikne objekt ktory obsahuje metodu a zaroven premennu tak preco by mala zaniknut premenna?

#8 KIIV
Podívej se pořádně. Deklaraci a definici "corner" má mimo metodu step(). To znamená, že je provedena pouze jednou před spuštěním konstruktoru.

Kód však není kompletní, chyba může být ještě jinde.

ale kdyz budes mit jen:

step() {
  if (corner) ... 
}

metodaX() {
  int corner = 0;
  step();
  ++corner;
}

tak to nemuze prece v zivote fungovat.. step ji nesmi videt z principu (co kdyby to pak zavolal od nekud jinud.. treba z jine metody Y, ktera to nedefinuje?)

ale nemusime se o tom hadat.. dokud tu nebude presnejsi kod, tak je to jako vesteni z logru na dne salku kafe... popis "nefunguje" a kus ocividne blbyho kodu nestaci

Neviem o co sa snazis, cize mozno lepsi popis veci ktoru robis by sa hodil, nechcelo sa mi citat vsetky prispvky, ale zjavne to stale nefunguje. Tu je spon nejaky napad:

private int corner = 0;
private int next = 1;
void step() {
	move();
	if(conditionStisfied())
		next = -1;
	corner += next;
}

pripadne to pripocitovanie a odcitavanie mozes urobit priamo v tom if-e:

if(conditionSatisfied()) {
	corner--;
} else {
	corner++;
}

Omlouvá se vám, že tady chodím okolo horké kaše...

int corner = 0;
    
    void step(){
        move();
        corner = corner + 1;
      }
    void back(){
        move();
        corner = corner - 1;
    }
    
    
    boolean wallOnRight(){
    turnLeft();
        turnLeft();
        turnLeft();
    if (!isInFrontOfWall()){
        turnLeft();
        return true;
    }
    else {
        turnLeft();
        return false;
    }
}
    boolean wallOnLeft(){
        turnLeft();
        if(!isInFrontOfWall()){
        turnLeft();
        turnLeft();
        turnLeft();
        return true;
        
        }else{
            turnLeft();
            turnLeft();
            turnLeft();
            
            return false;
        }
    }
    
    void followWallLeft(){
        if(isInFrontOfWall()){
        turnLeft();
        turnLeft();
        turnLeft();
        }else{
            back();
            turnLeft();
            if(isInFrontOfWall()){
                turnLeft();
                turnLeft();
                turnLeft();
            }else{
                back();
                turnLeft();
                if(isInFrontOfWall()){
                    turnLeft();
                    turnLeft();
                    turnLeft();
                }else{
                    back();
                }
            }
        }
    }
    
    void followWallRight(){
     
    if (isInFrontOfWall()) {
       turnLeft();
        }
    else {
       step();
       turnLeft();
           turnLeft();
           turnLeft();
           if (isInFrontOfWall()) {
        
                turnLeft();
           }
           else {
            step();
        turnLeft();
                turnLeft();
                turnLeft();
                if (isInFrontOfWall()) {
         
          turnLeft();
               }  
        else {
              
                  step();
                }
          }
        }
}
    
    

    
    @Override
    public void execute() {
        int numberOfSteps = 1;
        
        for (int i = 0; i < numberOfSteps; i++) {

        while(!isOnBeeper()){
          wallOnRight();
          followWallRight();

        while(isOnBeeper()){
                wallOnLeft();
                pickBeeper();
              followWallLeft();
        }
             }
           
            }
        turnOff();
        }
    }

já potřebuji aby while(!isOnBeeper) přičítal a while(isOnBeeper) odečítal a jak mile se dostane do nuly tak se program zastaví a vypne...

#13 lekejs
To všechno máš v jedné třídě? Jak se v tom můžeš vyznat?

#13 lekejs
no budes to muset zaprve poradne poodsazovat... pak mozna i uvidis tu chybu ....

Jo tohle vse mam v jedne tride...

Zasláno z mobilního telefonu.

#15 KIIV
No jo, nechal jsem si editorem přerovnat odsazení a hned jsem tu chybu uviděl. Snad na to přijde i lekejs.

Jak bych to mohl udelat a bych na ni prisel??

Zasláno z mobilního telefonu.

#18 lekejs
Nechej si editorem automaticky odsadit zdrojový kód. Dělám to u svých programů poměrně často. Chyby ve struktuře jsou pak vidět pouhým pohledem.

#18 lekejs
no rika se tomu odsazovani .. znamena to ze kazda uroven zanoreni je odsazena o vicero znaku (a stejna uroven stale o stejny pocet) je to jedna z naprosto zakladnich veci v kazdym coding-standardu (prevazne je to o formatovani kodu)

napriklad srovnej svuj kod:

    boolean wallOnRight(){
    turnLeft();
        turnLeft();
        turnLeft();
    if (!isInFrontOfWall()){
        turnLeft();
        return true;
    }
    else {
        turnLeft();
        return false;
    }
}

se spravne odsazenym kodem:

    boolean wallOnRight() {
        turnLeft();
        turnLeft();
        turnLeft();
        if (!isInFrontOfWall()){
            turnLeft();
            return true;
        } else {
            turnLeft();
            return false;
        }
    }

vdzy pak vis co k sobe patri

mimochodem, kdyz sem se ted podival na ten kod, co sem menil odsazeni, tak by to asi bylo spravne i takto: 

    boolean wallOnRight() {
        // otocil se jako korouhvicka o 360stupnu a je zase ve stejnem smeru
        return !isInFrontOfWall();
    }

Kde bych tu chybu mel hledat?? Ja uz do toho ziram nekolikatej den a uz my z toho hrabe...

Zasláno z mobilního telefonu.

#21 KIIV
Nebylo.

#22 lekejs
Když správně odsadíš, tak to uvidíš na první pohled. Nesmíš to však udělat ručně, ale musí to udělat editor automaticky.

#23 Kit
proc ne?  ma tam 4x turnLeft(); to je jako zustan jak si.. (pokud teda turn left nedela jeste nejake akce, o kterych nemame tuseni - ale to uz je zase "bad practice")

#22 lekejs
doporucuju dobre poodsazovat ty while cykly - pak to tam musi byt videt i pro slepyho

#25 KIIV
Bohužel je tam hromada těch "bad practice". Vypadá to jako implementace robota Karla, který se dívá, zda je před ním stěna, zda je stěna vpravo nebo vlevo. Během toho se musí otočit, ale pak musí stát v původním směru.

no to je jedno, tendle kod, jak se zda, jde absolutne mimo moje chapani ... odevzdat mi to nekdo v praci na review, tak ho asi umlatim klavesnici a pak to udelam sam :D

tak vám děkuji za pomoc, na něco jsme přišel..

 #28 lekejs
tak doufam ze je to tady:

        for (int i = 0; i < numberOfSteps; i++) {
            while(!isOnBeeper()) {
                wallOnRight();
                followWallRight();

                while(isOnBeeper()){
                    wallOnLeft();
                    pickBeeper();
                    followWallLeft();
                }
            }
        }

Presne tak mam tam podminku a v ni dalsi podminku...

Zasláno z mobilního telefonu.