JavaRush/Помощь по задачам/Опять валидатор упорствует против моего решения.

27 уровень

25.02.2019
504просмотров
8комментариев

Опять валидатор упорствует против моего решения.

Вопрос по задаче

Решен

Проверил неоднократно, и суммы максимальные вычисляю, и сортирую в алфавитном порядке, но валидатор упорствует! Есть идеи как его уговорить?

В метод main первым параметром приходит имя файла.
В этом файле каждая строка имеет следующий вид:
имя значение
где [имя] - String, [значение] - double. [имя] и [значение] разделены пробелом.

Для каждого имени посчитать сумму всех его значений.
Вывести в консоль имена в алфавитном порядке, у которых максимальная сумма.
Имена разделять пробелом либо выводить с новой строки.
Закрыть потоки.

Пример входного файла:
Петров 0.501 Иванов 1.35 Петров 0.85

Пример вывода:
Петров

Требования:

Программа НЕ должна считывать данные с консоли.
Программа должна считывать содержимое файла (используй FileReader).
Поток чтения из файла (FileReader) должен быть закрыт.
Программа должна выводить в консоль имена, у которых максимальная сумма.

package com.javarush.task.task19.task1920; /* Самый богатый */ import java.io.BufferedReader; import java.io.FileReader; import java.io.IOException; import java.util.*; public class Solution { public static void main(String[] args) throws IOException { FileReader f = new FileReader(args[0]); Map<String, Double> human = new HashMap<>(); String name = ""; Double zp = 0.0; double max = 0.0; BufferedReader reader = new BufferedReader(f); while (true) { String s = reader.readLine(); if (s == null) break; name = s.split(" ")[0]; zp = Double.parseDouble(s.split(" ")[1]); if (max < zp) { max = zp; } if(human.containsKey(name)){ double plzp = human.get(name) + zp; if (max < plzp) { max = plzp; } human.put(name, plzp); }else human.put(name, zp); } reader.close(); f.close(); List<String> nameM = new ArrayList<>(); for (Map.Entry<String, Double> par : human.entrySet()){ if(par.getValue() == max) nameM.add(par.getKey()); } Collections.sort(nameM); for(String s : nameM){ System.out.println(s); } } }

Комментарии (8)

популярные
новые
старые

Для того, чтобы оставить комментарий Вы должны авторизоваться

Konstantin

Уровень 41

25 февраля 2019, 18:06

if(par.getValue() == max) Думаю тут проблема, попробуйте - if(par.getValue().equals(max))

VIKTOR NEZHELSKIY

Уровень 27

Expert

25 февраля 2019, 18:41

Увы, результат тот же. Мой код выводит фамилии по алфавиту у тех кто самый богатый, но валидатор продолжает упорно видеть ошибку там где её нет.

Konstantin

Уровень 41

25 февраля 2019, 19:11

Откровенно говоря теряюсь в догадках. Запустил у себя и вроде работает как надо. Попробуйте сократить последние строки и просто делать вывод из HashMap, без ввода фамилий в ArrayList - for (Map.Entry<String, Double> par : human.entrySet()){ if(par.getValue() == max) System.out.println(par.getKey()); } Валидатор зачастую хочет непонятного, при вроде как выполненных условиях. В итоге решение превращается в угадайку - чего надобно :).

Konstantin

Уровень 41

25 февраля 2019, 19:38

А, ещё. Чтобы сортировка ключевых полей проводилась сразу использовать TreeMap.

VIKTOR NEZHELSKIY

Уровень 27

Expert

25 февраля 2019, 20:10

TreeMap я пробовал. На валидатор это не повлияло. Пробовал и без ArrayList сразу из TreeMap имена выводить. Уже после этого пробовал Collections.sort

Konstantin

Уровень 41

25 февраля 2019, 20:36

полезный

Учитывая что задание вы в принципе выполнили, не думаю что будет большим спойлером как его сделал я, так что вот код, возможно он вам поможет найти проблему -

public class Solution {
    public static void main(String[] args) {
//        String[] args = {"sallary2.txt"};

        try (
                FileReader fr = new FileReader(args[0]);
                BufferedReader brFR = new BufferedReader(fr);
        ) {
            String s;
            ArrayList<Person> alP = new ArrayList<>();
            TreeMap<String, Double> tmSD = new TreeMap<>();

            while (( s = brFR.readLine()) != null) {
                String[] str = s.split(" ");
                alP.add(new Person(str[0].trim(), Double.parseDouble(str[1].trim())));
                tmSD.put(str[0].trim(), 0.0);
            }

            for (int i = 0; i < alP.size(); i++) {
                Person p = alP.get(i);
                Iterator<Map.Entry<String, Double>> hmiI = tmSD.entrySet().iterator();
                while (hmiI.hasNext()) {
                    Map.Entry<String, Double> next = hmiI.next();
                    String k = next.getKey();
                    Double v = next.getValue();

                    if (k.equals(p.getName())) {
                        tmSD.replace(k, v + p.getSallary());
                    }
                }
            }

            Double maxSallary = Collections.max(tmSD.values());

            for (Map.Entry<String, Double> entry : tmSD.entrySet()) {
                String k = entry.getKey();
                Double v = entry.getValue();
                if (v.equals(maxSallary)) {
                    System.out.println(k);
                }
            }
        } catch (FileNotFoundException e) {
            e.printStackTrace();
        } catch (IOException e) {
            e.printStackTrace();
        }
    }
}

Класс Person самописный на два поля.

Ольга Баскакова

Уровень 29

26 февраля 2019, 05:56

У меня один раз валидатор не принимал, пока не убрала строку с закрытием потока чтения с консоли. Оо Просто так её убрала, и всё принялось.

VIKTOR NEZHELSKIY

Уровень 27

Expert

26 февраля 2019, 06:38

решение

Благодарю за помощь. Оказалось, что при выводе надо было вместо вывода имён из отдельного списка, сделать вывод из мапи с именами и значениями сумм. Вместо строк 42-50 написал просто:

for (Map.Entry<String, Double> entry : human.entrySet()) {
    String k = entry.getKey();
    Double v = entry.getValue();
    if (v.equals(max)) {
        System.out.println(k);
    }
}

и прошел валидацию! Оказалось, что просто список имён в алфавитном порядке не то же самое, что список имён с максимальными суммами в алфавитном порядке.