Переход от внутреннего l oop и go к внешнему l oop после выполнения действия - PullRequest
Новая
       4

Переход от внутреннего l oop и go к внешнему l oop после выполнения действия

1 голос
/ 29 мая 2020

Есть код для класса BigestCountries.

Он состоит из 2 массивов: 

private String[][] biggestCountries; - holds country name and the continent, e.g. biggestCountries[CHINA][COUNTRY_NAME] = "China"; biggestCountries[CHINA][COUNTRY_CONTINENT] = "Asia";

private int[][] countryData; - holds populations and year founded, e.g. countryData[CHINA][COUNTRY_POPULATION] = 1433783686; countryData[CHINA][COUNTRY_AGE_FOUNDED] = 1949;

public String[] getCountriesFoundedBetween(int min, int max){
    int countriesMatched;
    countriesMatched = 0;  
    String[] countriesFoundedBetween;

    if(biggestCountries == null || biggestCountries.length == 0){
         return null;
    }

    for(int i = 0; i < biggestCountries.length; i++){
        if(countryData[i][COUNTRY_AGE_FOUNDED] >= min && countryData[i][COUNTRY_AGE_FOUNDED] <= max){

            System.out.println(String.format("%d %s", countryData[i][COUNTRY_AGE_FOUNDED], biggestCountries[i][COUNTRY_NAME]));
            countriesMatched++;
        }
    }

    if(countriesMatched > 0){
        countriesFoundedBetween = new String[countriesMatched];
    } else {
        return null;
    }

    for(int i = 0; i < biggestCountries.length; i++) { // outer loop for countries array length of NUMBER_OF_COUNTRIES

        String countryMatched = null;
        System.out.println("biggestCountries[i] " + biggestCountries[i][COUNTRY_NAME]);

        if(countryData[i][COUNTRY_AGE_FOUNDED] >= min && countryData[i][COUNTRY_AGE_FOUNDED] <= max){
            for(int j = 0; j < countriesFoundedBetween.length; j++){ // how to escape inner loop?

                countryMatched =  biggestCountries[i][COUNTRY_NAME];
                countriesFoundedBetween[j] = countryMatched;
                System.out.println("countriesFoundedBetween: " + countriesFoundedBetween[j] + "; biggestCountries[i][COUNTRY_NAME]: " + biggestCountries[i][COUNTRY_NAME]);

            }      
        }      
    }

    return countriesFoundedBetween;
}

К сожалению, он не может выйти из внутреннего l oop и повторно - записывает соответствующую страну во все строки вновь созданного массива. Result

Ответы [ 2 ]

1 голос
/ 29 мая 2020

Метод getCountriesFoundedBetween() может быть реализован по-другому, без необходимости использования вложенных циклов, как показано ниже. 

private static String[] getCountriesFoundedBetween(int min, int max) {
    if (max < min) {
        throw new IllegalArgumentException("'max' less than 'min'");
    }
    String[] countriesFoundedBetween;
    int countriesMatched = 0;
    int[] indexes = new int[biggestCountries.length];
    if (biggestCountries != null && biggestCountries.length > 0) {
        for (int i = 0; i < biggestCountries.length; i++) {
            if(countryData[i][COUNTRY_AGE_FOUNDED] >= min &&
               countryData[i][COUNTRY_AGE_FOUNDED] <= max) {
                indexes[countriesMatched++] = i;
            }
        }
        countriesFoundedBetween = new String[countriesMatched];
        for (int i = 0; i < countriesMatched; i++) {
            countriesFoundedBetween[i] = biggestCountries[indexes[i]][COUNTRY_NAME];
        }
    }
    else {
        countriesFoundedBetween = new String[0];
    }
    return countriesFoundedBetween;
}

Приведенный выше код также возвращает пустой массив, а не null, что предпочтительнее для методов, возвращающих массивы.

Вот полный пример использования населения для определения крупнейших стран. 

public class Countrys {
    private static final int  CHINA      = 0;
    private static final int  INDIA      = 1;
    private static final int  U_S_A      = 2;
    private static final int  INDONESIA  = 3;
    private static final int  PAKISTAN   = 4;
    private static final int  BRAZIL     = 5;
    private static final int  NIGERIA    = 6;
    private static final int  BANGLADESH = 7;
    private static final int  RUSSIA     = 8;
    private static final int  MEXICO     = 9;

    private static final int  COUNTRY_NAME = 0;
    private static final int  COUNTRY_CONTINENT = 1;
    private static final int  COUNTRY_POPULATION = 0;
    private static final int  COUNTRY_AGE_FOUNDED = 1;

    private static int[][] countryData = new int[][]{{1_427_647_786, 1949},
                                                     {1_352_642_280, 1950},
                                                     {  328_239_523, 1776},
                                                     {  273_523_615, 1945},
                                                     {  220_892_340, 1947},
                                                     {  210_147_125, 1889},
                                                     {  206_139_589, 1960},
                                                     {  164_689_383, 1971},
                                                     {  144_384_244, 1991},
                                                     {  128_932_753, 1810}};
    private static String[][] biggestCountries = new String[][]{{"China"     , "Asia"},
                                                                {"India"     , "Asia"},
                                                                {"U.S.A."    , "North America"},
                                                                {"Indonesia" , "Asia"},
                                                                {"Pakistan"  , "Asia"},
                                                                {"Brazil"    , "South America"},
                                                                {"Nigeria"   , "Africa"},
                                                                {"Bangladesh", "Asia"},
                                                                {"Russia"    , "Europe"},
                                                                {"Mexico"    , "North America"}};

    private static String[] getCountriesFoundedBetween(int min, int max) {
        if (max < min) {
            throw new IllegalArgumentException("'max' less than 'min'");
        }
        String[] countriesFoundedBetween;
        int countriesMatched = 0;
        int[] indexes = new int[biggestCountries.length];
        if (biggestCountries != null && biggestCountries.length > 0) {
            for (int i = 0; i < biggestCountries.length; i++) {
                if(countryData[i][COUNTRY_AGE_FOUNDED] >= min &&
                   countryData[i][COUNTRY_AGE_FOUNDED] <= max) {
                    indexes[countriesMatched++] = i;
                }
            }
            countriesFoundedBetween = new String[countriesMatched];
            for (int i = 0; i < countriesMatched; i++) {
                countriesFoundedBetween[i] = biggestCountries[indexes[i]][COUNTRY_NAME];
            }
        }
        else {
            countriesFoundedBetween = new String[0];
        }
        return countriesFoundedBetween;
    }

    public static void main(String[] args) {
        String[] result = getCountriesFoundedBetween(1950, 1980);
        System.out.println(Arrays.toString(result));
    }
}

Выполнение приведенного выше кода дает следующий результат: 

[India, Nigeria, Bangladesh]
0 голосов
/ 29 мая 2020
  1. Если вы назовете класс BigestCountries, он все равно будет иметь атрибут biggestCountries, который представляет собой массив, содержащий информацию о крупнейших странах, я думаю, вы не используете потенциал OO.
  2. Вы можете использовать Class для представления данных Country. С помощью инкапсуляции данных вы можете избавиться от вложенного массива, таким образом, вложенного l oop и оператора потока управления (break, continue et c), константы индекса поиска, keep syn c - индекса между biggestCountries и countryData et c.
  3. Ваш код выполняет одну и ту же задачу дважды. Первые l oop подсчитывают совпавшую страну и инициализируют массив. Второй l oop фактически помещает совпадающее название страны в массив.
  4. Java структура сбора предоставляет структуру данных с размером c. Я использую ArrayList здесь
  5. Я думаю, что ageFound следует называть yearFound?

Рефакторинг вашего кода

Страна. java

public class Country {

    private String name;
    private int population;
    private int yearFound;
    private String continent;

    public Country(String name, String continent, int year, int population) {
        this.name = name;
        this.population = population;
        this.continent = continent;
        this.yearFound = year;
    }

    public String getName() {
        return name;
    }

    public void setName(String name) {
        this.name = name;
    }

    public int getPopulation() {
        return population;
    }

    public void setPopulation(int population) {
        this.population = population;
    }

    public String getContinent() {
        return continent;
    }

    public void setContinent(String continent) {
        this.continent = continent;
    }

    public int getYearFound() {
        return yearFound;
    }

    public void setYearFound(int yearFound) {
        this.yearFound = yearFound;
    }

    @Override
    public String toString() {
        return "Country [name=" + name + ", population=" + population + ", yearFound=" + yearFound + ", continent="
                + continent + "]";
    }

}

Крупнейшие страны. java

import java.util.ArrayList;
import java.util.Iterator;
import java.util.List;

public class BiggestCountries {

    private List<Country> countries = new ArrayList<Country>() {
        {
            add(new Country("China", "Asia", 1949, 1433783686));
            add(new Country("Canada", "North America", 1867, 37590000));
            add(new Country("United States", "North America", 1776, 328200000));
        }
    };

    public List<Country> getCountriesFoundedBetween(int min, int max) {
        List<Country> matchedCountry = new ArrayList<Country>();
        Iterator<Country> itrCoutnry = countries.iterator();

        while (itrCoutnry.hasNext()) {
            Country country = itrCoutnry.next();
            int yearFound = country.getYearFound();
            if (min < yearFound && max > yearFound) {
                matchedCountry.add(country);
            }
        }
        return matchedCountry;
    }

}

Тестовый запуск 

public static void main(String[] args) {

    List<Country> matchedCountries = new BiggestCountries().getCountriesFoundedBetween(1700, 1899);
    System.out.println(matchedCountries);
}

Результат 

[Country [name=Canada, population=37590000, yearFound=1867, continent=North America], Country [name=United States, population=328200000, yearFound=1776, continent=North America]]
...