Java façon plus élégante d'écrire des instructions avec des images

Question

Existe-t-il un moyen plus élégant/plus court/organisé d'écrire ce morceau de code?

for (int i = 0; i < SCREENSIZE; i++) { 
     for (int j = 0; j < SCREENSIZE; j++) { 
      if (map[y + i][x + j] == '@') 
       g.drawImage(item, j * TILESIZE,i * TILESIZE, null); 
      else if (map[y + i][x + j] == ' ') 
       g.drawImage(ground, j * TILESIZE,i * TILESIZE, null); 
      else if (map[y + i][x + j] == 'i') 
       g.drawImage(bush, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == '~') 
       g.drawImage(ocean, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == '=') 
       g.drawImage(fence, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == '#') 
       g.drawImage(grass, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == 'Y') 
       g.drawImage(townsPerson, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == '/') 
       g.drawImage(house01, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == '¯') 
       g.drawImage(house02, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == '\\') 
       g.drawImage(house03, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == '[') 
       g.drawImage(house04, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == 'n') 
       g.drawImage(house05, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == '_') 
       g.drawImage(house06, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == ']') 
       g.drawImage(house07, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == '`') 
       g.drawImage(cground, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == 'O') 
       g.drawImage(boulder, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == 'Ÿ') 
       g.drawImage(alien, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == '.') 
       g.drawImage(tree01, j * TILESIZE, i * TILESIZE, null); 
      else if (map[y + i][x + j] == 'T') 
       g.drawImage(tree02, j * TILESIZE, i * TILESIZE, null); 
     } 
    } 

Answer 1

Quelque part (? Dans le constructeur), enregistrer un Map comme variable membre:

images = new HashMap<Character, Image>(); 
images.put('@', item); 
images.put(' ', ground); 

Ensuite, votre dessin ressemblera:

for (int i = 0; i < SCREENSIZE; i++) { 
    for (int j = 0; j < SCREENSIZE; j++) { 
     g.drawImage(images.get(map[y+i][x+j]), j * TILESIZE, i * TILESIZE, null) 
    } 
} 

Answer 2

La première amélioration pourrait être d'utiliser une structure switch/case, mais dans votre cas, une carte simple (Map<Char,Image>) sera encore mieux.

Allant encore plus loin, vous pouvez utiliser un ENUM au lieu de caractères pour identifier les objets qui vous aideront à éviter les fautes de frappe, mais à tout le moins, vous devriez utiliser des constantes de caractères, comme

public static final char MAP_ITEM = '@'; 
public static final char MAP_GROUND = ' '; 

et ainsi sur.

Answer 3

Puisque vous fondez de caractère , vous pouvez utiliser une instruction switch. L'autre chose que vous pouvez faire est, puisque vous utilisez g.drawImage(SOMETHING, j * TILESIZE, i * TILESIZE, null) dans tous les cas, vous pouvez extraire tout ce qui est la même chose après que le commutateur et juste assigner une variable et l'utiliser pour les changements qui

EX:

Object graphic; 
switch (map[y + i][x + j]) { 
case '@': 
    graphic = item; 
    break; 
case '#': 
    graphic = grass; 
    break; 
// etc.... 
} 
g.drawImage(graphic, j * TILESIZE, i * TILESIZE, null); 

Answer 4

utiliser un commutateur/cas (pas de carte) car le compilateur aurait plus de place pour optimiser, car il connaît l'ensemble exact des valeurs char que vous passez sur:

... 
switch(map[y+i][x+j]) { 
    case: '@': image = item; break; 
    case: ' ': image = ground; break; 
    ... 
} 
g.drawImage(image, j * TILESIZE, i * TILESIZE, null); 
... 

Answer 5

vous pouvez utiliser un commutateur avec une valeur char si tha Cela pourrait être plus clair. De plus, les paramètres 2, 3 et 4 sont toujours les mêmes, alors peut-être définir un var dans le commutateur et appeler la méthode à l'extérieur. Un petit code pseudo:

for() { 
    for() { 
    Image obj; 
    switch (map[y+i][x +j]) { 
     case '@': 
     Obj = item; 
     break; 
    // other cases 
     default: 
     //default or error handling 
    } 
    g.drawImage(obj, j * TILESIZE, i * TILESIZE, null); 
} 
}