Почему мой DAO читает файл .txt и не хранит все строки данных? - PullRequest
Новая
       6

Почему мой DAO читает файл .txt и не хранит все строки данных?

0 голосов
/ 26 ноября 2018

Я довольно новичок в Java и в настоящее время работаю над небольшим приложением, которое может хранить и манипулировать городскими данными, которые считываются из текстового файла.У меня есть класс DAOTextImpl, который должен читать из текстового файла, выбирать все строки данных и сохранять их в ArrayList для сортировки.Пока что я могу получить только 3 из 4 строк данных, которые он должен хранить.

Может кто-нибудь взглянуть, почему выбираются только первые 2 набора городских данных исохранено (2015 и 2016)?

Примечание: я впервые использую StackOverflow, поэтому, если я что-то пропустил, сообщите мне.

Код: 

public class DaoTextImpl implements DAOInterface {

static final char DELIMITER=',';

@Override
public Repository load(String filename) {

    Repository repository = new Repository();

    try (BufferedReader br = new BufferedReader(new FileReader(filename))) {
        String[] temp;
        String line;
        ArrayList<String> lines = new ArrayList<>();
        ArrayList<YearData> yearData = new ArrayList<>();
        while ((line = br.readLine()) != null){
            lines.add(line);
        }
        for (int i = 0; i < lines.size(); i++) {
            temp = lines.get(i).split(Character.toString(DELIMITER));
            if (temp.length == 4) {
                int id = Integer.valueOf(temp[0]);
                String cityName = stripQuotes(temp[1]);
                String country = stripQuotes(temp[2]);
                int noofyeardata = Integer.valueOf(3);

                for (int j = (i + 1); j < (i + noofyeardata); j++) {
                    String[] yearDataArray = lines.get(j).split(Character.toString(DELIMITER));
                    String year = stripQuotes(yearDataArray[0]);
                    float precipitation = Float.valueOf(yearDataArray[1]);
                    int maxtemp = Integer.valueOf(yearDataArray[2]);
                    int mintemp = Integer.valueOf(yearDataArray[3]);
                    int windspeed = Integer.valueOf(yearDataArray[4]);
                    String winddirection = stripQuotes(yearDataArray[5]);
                    yearData.add(new YearData(year, precipitation, maxtemp, mintemp, windspeed, winddirection));
                }
                repository.add(new City(id, cityName, country, yearData));
            }
        }

    } catch (IOException ex) {
        ex.printStackTrace();
    }
    return repository;
}

Текстовое содержимое файла: 

1,"Cartagena","Spain",3
"2015",0.2,33,26,6,"S"
"2016",0.0,33,24,8,"SSW"
"2017",0.0,32,25,6,"E"
2,"Glasgow","Scotland",3
"2015",0.0,19,8,3,"SE"
"2016",0.1,21,11,6,"SE"
"2017",2.1,19,11,9,"SW"
3,"Valencia","Spain",4
"2015",0.0,34,24,6,"SE"
"2016",0.0,39,23,5,"SSE"
"2017",0.0,32,24,5,"E"
"2014",0.0,29,20,6,"ESE"

1 Ответ

0 голосов
/ 26 ноября 2018

Пара проблем в вашем коде.

1) Вы один на один в проверке состояния

j < (i + noofyeardata) должно быть j <= (i + noofyeardata)

2)int noofyeardata = Integer.valueOf(3) должно быть int noofyeardata = Integer.valueOf(temp[3]).Вот почему вы всегда читаете noofyeardata как 3

Еще один общий момент 1 .После завершения внутреннего цикла вам не нужно продолжать внешний цикл (i counter) с того места, где вы остановились.Вы можете увеличить i на noofyeardata (i = i + noofyeardata + 1) внутри тела цикла для (поскольку вы уже обработали noofyeardata строки).При этом вы также можете снять флажок if (temp.length == 4) .

for (int i = 0; i < lines.size();) {
        temp = lines.get(i).split(Character.toString(DELIMITER));
        int id = Integer.valueOf(temp[0]);
        String cityName = stripQuotes(temp[1]);
        String country = stripQuotes(temp[2]);
        int noofyeardata = Integer.valueOf(temp[3]);

        for (int j = (i + 1); j <= (i + noofyeardata); j++) {
           //your existing code
        }
        repository.add(new City(id, cityName, country, yearData));
        i += noofyeardata + 1;
    }
}

1 Предполагается, что содержимое файла строго соответствует указанному вами формату данных. 

<id>, <cityName>, <country>, <noofyeardata1>
<noofyeardata1 rows>
<id>, <cityName>, <country>, <noofyeardata2>
<noofyeardata2 rows>
....
...