Új hozzászólás Aktív témák

• lanszelot addikt

  #12102

  #12101 axioma

  2024-09-10 18:44:50

  Új Válasz

  Új hozzászólás

  Összes hozzászólása itt Válaszok az összes hozzászólására itt Válaszok erre a hozzászólásra

  

  lanszelot

  addikt

  #12101 axioma

  #12101 axioma

  Jaaa, akkor felreertettelek. OK, nevezzuk imageNumber-nek, az imageId az alfanumerikusra utal inkabb.
  Szoval az altalad "one" id-val ellatott resource-hoz az R resource class hozzarendelt egy imageNumber-t, ami int, es ezt tarolja az R.drawable.one alatt. Igy mar talan ertheto.

  A teljes switch elhagyhato. Az ertelme annyi, mintha a teljes switch(r) {...} helyett azt irnad, hogy viewNr[v].setImageRescource(imgId[r])
  (mondjuk nincs default agad, igy annyi kulonbseg azert van, hogy ha a v nem [0..5], akkor a switch lefut csak semmit nem allit be, az egysoros meg hibat ad)

  Van tobb aprosag amit lehetne me'g javitani. Pl. a viewNr letrehozas miert kerult be a listenerbe, es miert ket lepesben kap erteket?
  ImageView imageView1 = findViewById(R.id.Egyes);
viewNr[0] = imageView1;
  ehelyett
  viewNr[0] = findViewById(R.id.Egyes);

  A randszam feleslegesen van inicializalva randomra, azt nem hasznalod, felulirja a for ciklusban - ez gondot nem okoz csak kod-olvashatosagot, mire gondolhatott a kolto alapon...

  Switch - valóban Azt el is kezdtem átnézni kell e még, de jött a postás, és huss elfelejtettem

  viewNr scope miatt. Az imageView nem érhető el kintről.

  For ciklus nem lesz. Egyesével lesznek meghivva if -hez kötve, többször, többféleképpen.
  Emiatt lehet kell majd a randSzam, így nem nyúltam hozzá.

Új hozzászólás Aktív témák