Два массива измерений перезаписывают

Question

Эй, этот код работает отлично, но его трудно прочитать. Я ищу способ сделать это лучше.

public void printArray(String[/*row*/][/*column*/] twoDiArray) { 
    if (twoDiArray.length == 2) { 
     for (int i = 0; i < twoDiArray[0].length; i++) { 
      //prints attribute name and value 
      attributeNameAndValue(twoDiArray[0][i],twoDiArray[1][i]); 
     } 
    } else { 
     System.out.println("Does not fit format standards :: 2d array :: two rows max :: first row name :: second row value"); 
    } 
} 

Часть, которую мне очень не нравится, - это вызовы длины в выражении if и цикле for. Есть ли лучший способ сделать это или это просто неаккуратный раздел Java-языка.

Answer 1

У вас есть пар имя-значение, если ваши имена уникальны, вместо этого вы должны использовать Map<String, Integer>. В противном случае создайте свой собственный класс, например. Attribute и использовать List<Attribute>:

public class Attribute { 

    private final String name; 
    private final int value; 

    public Attribute(String name, int value) { 
     this.name = name; 
     this.value = value; 
    } 

    public String getName() { 
     return name; 
    } 

    public int getValue() { 
     return value; 
    } 

} 

Это дает вам время компиляции безопасности для второго измерения. Ваш код будет выглядеть следующим образом:

public void printArray(List<Attribute> attributes) { 
    for (Attribute attribute : attributes) { 
     attributeNameAndValue(attribute.getName(), attribute.getValue()); 
    } 
} 

Answer 2

Вы можете использовать новые переменные, чтобы сохранить значение twoDiArray.length и использовать новый вар ниже.

Answer 3

Это, безусловно, можно немного почистить. Другой подход, о котором говорили другие люди, будет иметь гораздо более чистый код. Это немного странно, что у вас есть строка и столбец. Реверсивный порядок будет также сделать вещи чище:

public void printArray(String[/*row*/][/*column*/] twoDiArray){ 

    if(twoDiArray.length!=2){ 
     System.out.println("Does not fit format standards :: 2d array :: two rows max :: first row name :: second row value"); 
     return; 
    } 
    int len = twoDiArray[0].length; 
    for(int i = 0; i<len.length; i++){ 
     //prints attribute name then value 
     attributeNameAndValue(twoDiArray[0][i],twoDiArray[1][i]); 
    } 
} 

Другое дело было бы изменить attributeNameAndValue принять i и twoDiArray для «аккуратным» вызова.

Answer 4

другой способ проверить массив

public void printArray(String[/*row*/][/*column*/] twoDiArray){ 
    assert twoDiArray.length == 2; 
    for(int i=0; i<twoDiArray[0].length; i++){ 
    attributeNameAndValue(kvp[i][0], kvp[i][1]); 
    } 
} 

Answer 5

Чтобы улучшить читабельность кода применяется условной кода Java (http://www.oracle.com/technetwork/java/codeconvtoc-136057.html). Введем переменные строк и столбцов (по количеству строк и число столбцов)

public void printArray(String[/*row*/][/*column*/] twoDiArray) { 
    int rows = twoDiArray.length; 
    if (rows != 2) { 
     System.out.println("Does not fit format standards :: 2d array :: two rows max :: first row name :: second row value"); 
     return; 
    } 
    int columns = twoDiArray[0].length; 
    for (int column = 0; column < columns; column++) { 
     //prints attribute name then value 
     attributeNameAndValue(twoDiArray[0][column], twoDiArray[1][column]); 
    } 
}