[Java] Konstruktory a ukladanie dat pre jednotlive objekty [vyriešené]

Miso122 · Napísal **Miso122**: 21.03.2018 12:37

Cauko,
zacinam s Javou a uz odzaciatku som narazil na problem s ktorym neviem pohnut ani po hladani na nete.

Mam classu state, kde mam ulozeny 2D array z integerov (reprezentacia hlavolamu), konstruktor a getter ktory dane pole vrati.

Kód:

public class state
{
    private int x, y;
    private int[][] status;

    state(int a, int b, int[][] array) //constructor
    {
        this.x = a;
        this.y = b;
        this.status = Arrays.copyOf(array, array.length);
    }

   int[][] getStatus()
    {
        return Arrays.copyOf(status, status.length);
    }
}

V maine si chcem vytvorit 2 objekty, zaciatocny a konecny stav hlavolamu, ktore zadava uzivatel. Nacitam prvy vstup ktory ulozim do inputarray, nasledne vytovrim objekt a vypisem do konzoly:

Kód:

state startstate = new state(x, y, inputarray);
System.out.println(Arrays.deepToString(startstate.getStatus()));

Doteraz vsetko funguje ako ma. Problem nastane ked vytvorim druhy objekt rovnakym sposobom (pole inputarray bolo medzitym znova nacitane zo vstupu):

Kód:

state endstate = new state(x, y, inputarray);

Ked ale po tomto chcem vypisat obsah arrayov v jednotlivych objektoch, obidve metody getStatus vratia rovnake hodnoty, konkretne rovnaky array ako bol nacitane do objektu endstate a array v objekte startstate sa prepisal.

Co robim zle, resp kde konrketne mam chybu? Ako docielim aby oba objekty mohli mat jedinecne a rozdielne udaje, ku ktorym by som mohol pristupovat a menit ich nezavisle na sebe?

jarrro · Napísal **jarrro**: 21.03.2018 13:12

Len tip. Skús vrátiť priamo status nie jeho kópiu resp aj v konštruktore priraď priamo dané pole k tomu statusu

Miso122 · Napísal autor témy **Miso122**: 21.03.2018 13:23

uz som skusal, nic

expresado · Napísal **expresado**: 21.03.2018 14:30

V konstruktore,

Kód:

status = new int[array.length][];
for (int i = 0; i < array.length; i++) {
    status[i] = Arrays.copyOf(array[i], array[i].length);
}

Miso122 · Napísal autor témy **Miso122**: 21.03.2018 15:00

vdaka, funguje presne ako ma.

este otazka o getStatus, moze to zostat tak ako to je alebo aj tu treba upravit?

expresado · Napísal **expresado**: 21.03.2018 17:11

Vytvor objekt, vytiahni status, vypis hodnotu, zmen v nom nieco a znovu ho vytiahni a vypis hodnotu. Hned zistis ci sa reflektuje zmena do objektu teda ci ti poslal novu instanciu alebo iba referenciu

Miso122 · Napísal autor témy **Miso122**: 22.03.2018 15:58

Odskúšané vypísané, všetko funguje

Vďaka

expresado · Napísal **expresado**: 22.03.2018 16:15

si si isty ?

odskusaj si toto:

Kód:

        int[][] status = new int[2][2];
        status[0][0]=5;
        status[1][1]=3;
        state start=  new state(1,1,status); //vytvor objekt
        int[][] out = start.getStatus();//vytiahni status
        System.out.println(start.getStatus()[0][0]+","+start.getStatus()[1][1]);//vypis hodnotu
        out[0][0]=99;//zmen v nom nieco
        out[1][1]=33;//zmen v nom nieco
        System.out.println(start.getStatus()[0][0]+","+start.getStatus()[1][1]);//znovu ho vytiahni a vypis hodnotu

Miso122 · Napísal autor témy **Miso122**: 22.03.2018 16:25

V algoritme vytiahnem pole, 4x ho zmením a vytvorím nové objekty. Po skončení algoritmu vypisem znova začiatočné pole, nerobi žiaden problém

expresado · Napísal **expresado**: 22.03.2018 16:37

Tak potom super, s tou povodnou metodou getStatus() by ti vsak malo zmenit hodnotu. Len aby sa ti nahodou nestalo niekde, ze by sa ti menili hodnoty a nevedel si odkial, ten kod co som postol by ti to ukazal. Ak by tam isla referencia, druhy vypis by ti zobrazil zmenene hodnoty.

Miso122 · Napísal autor témy **Miso122**: 26.03.2018 0:40

no tak na tvojom kode mi to meni povodne pole

zaujimave

//ale uz som asi prisiel na to preco to v mojom pripade nemeni udaje, mam to riesene takto, kde vlastne len vytiahnem 2d array, ktory skopirujem a pracujem len s danou kopiou:

Kód:

            int[][] editedarray = act.getStatus();
            int[][] leftarray = new int[x][y];

            for (int i = 0; i < editedarray.length; i++)
            {
                leftarray[i] = Arrays.copyOf(editedarray[i], editedarray[i].length);
            }

[Java] Konstruktory a ukladanie dat pre jednotlive objekty [vyriešené]

Podobné témy