Не пропускает валидатор. Всё работает.

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

Решен

package com.javarush.task.task19.task1920;

/*
Самый богатый
*/
import java.io.BufferedReader;
import java.io.FileReader;
import java.io.IOException;
import java.io.InputStreamReader;
import java.util.*;

public class Solution {
    public static void main(String[] args) throws IOException {
        //BufferedReader file = new BufferedReader(new FileReader(args[0]));
        BufferedReader file = new BufferedReader(new FileReader("D:\\projects\\java\\JavaRushTasks\\2.JavaCore\\src\\com\\javarush\\task\\task19\\task1919\\1.txt"));

        Map<String, Double> map = new HashMap<>();
        double max = 0;

        while(file.ready()){
            String str = file.readLine();
            String name = str.split(" ")[0];
            Double value = Double.parseDouble(str.split(" ")[1]);
            if(value > max) max = value;
            boolean t = false;
            for(Map.Entry<String, Double> pair : map.entrySet()){
                if(name.equals(pair.getKey())){
                    if(pair.getValue() + value > max) max = pair.getValue() + value;
                    map.put(name, pair.getValue() + value);
                    t = true;
                }
            }
            if(!t) map.put(name, value);
        }
        file.close();
        List<String> list = new ArrayList<>();

        TreeMap<String, Double> sortedMap = new TreeMap<>(map);

        for(Map.Entry<String, Double> pair : sortedMap.entrySet()){
            if(pair.getValue().equals(max))  list.add(pair.getKey());
        }
        Collections.sort(list);
        for(String s : list) System.out.println(s);
    }
}

В метод 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.io.InputStreamReader; import java.util.*; public class Solution { public static void main(String[] args) throws IOException { BufferedReader file = new BufferedReader(new FileReader(args[0])); //BufferedReader file = new BufferedReader(new FileReader("D:\\projects\\java\\JavaRushTasks\\2.JavaCore\\src\\com\\javarush\\task\\task19\\task1919\\1.txt")); Map<String, Double> map = new HashMap<>(); double max = 0; while(file.ready()){ String str = file.readLine(); String name = str.split(" ")[0]; Double value = Double.parseDouble(str.split(" ")[1]); if(value > max) max = value; boolean t = false; for(Map.Entry<String, Double> pair : map.entrySet()){ if(name.equals(pair.getKey())){ if(pair.getValue() + value > max) max = pair.getValue() + value; map.put(name, pair.getValue() + value); t = true; } } if(!t) map.put(name, value); } file.close(); List<String> list = new ArrayList<>(); TreeMap<String, Double> sortedMap = new TreeMap<>(map); for(Map.Entry<String, Double> pair : sortedMap.entrySet()){ if(pair.getValue().equals(max)) list.add(pair.getKey()); } Collections.sort(list); for(String s : list) System.out.println(s); } }

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

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

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

Justinian Judge в Mega City One

Master

4 января 2020, 02:02

полезный

BufferedReader file = new BufferedReader(new FileReader("D:\\projects\\java\\JavaRushTasks\\2.JavaCore\\src\\com\\javarush\\task\\task19\\task1919\\1.txt"));

С таким программа и не должна проходить валидацию, твой код посылается на сервер, где проходит валидацию, имя файла должно приходить аргументом запуска. Я настоятельно советую тестировать в условиях максимально приближенных к действительности, создаешь в корне проекта JavaRushTask (только создаешь именно через Идею) пару файлов, называешь удобно, File1, File2, SomeFile И после этого в любой задаче проекта ты сможешь вот так:

String fileName = "File1";
FileReader fileReader = new FileReader(fileName);

Если имя файла приходит аргументом, то я либо просто вытирал args[0] в скобках и писал "File1" либо , что есть более правильно, в Edit configuration пишешь File1 в аргументах запуска. Быстро, удобно, эффективно. Никаких путей трехэтажных, никакого экранирования и максимально близко к работе валидатора. А то что нам теперь думать, только остается гадать, менял ли ты во время валидации это или нет, что там было, дополнительные траблы и тебе и нам. И кстати удобно работать тем, что прямо в Идее есть файл с которым удобно работать, консоль, код, файл в одном окне. Который показывает изменения в режиме реального времени и удобно править

И читай Clean code Роберта Мартина, потому что вот это if(!t) map.put(name, value); и BufferedReader file это мягко говоря не очень хорошо. Твой уровень гораздо выше такого кода, не знание правил написания кода на джаве сдерживает твой прогресс. И CTRL+ALT+

Mark Solncev

Уровень 37

4 января 2020, 10:44

Это, конечно, всё прекрасно, но я специально сделал комментарием строку с путем до файла. Валидатор не пропускает решение. За CTRL + ALT + L - отдельное спасибо)

Justinian Judge в Mega City One

Master

4 января 2020, 11:11

решение

Я комментирую что вижу. ты привел код, он нерабочий по причине озвученной мной. Если ты спрашиваешь совета то должен такое либо оговаривать, но еще лучше - убирать скотч и изоленту, чтобы отвечающие могли сконцентрироваться на решении. Ты не поверишь сколько раз правильные решения не проходили валидацию из-за того что люди забывали снимать комментирование или наоборот ставить. Ты у себя один, а для отвечающих это 100 вопрос по одной и той же задаче :) Поэтому учитывай этот момент в будущем. Даешь код, давай его в чистовом виде. Искать ошибки не всегда легко, и ты сильно усложнишь этот процесс подобными черновиками, вызывая лишние вопросы и мысли. И фигурные скобки! вот такие строки

if pair.getValue() + value > max max = pair.getValue() + value;

практически не читаются, нужно турборежим включать в голове чтобы прочитать что где кому value и что делается

boolean t

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

if (!t) {
   map.put(name, value);
}

только карты Таро остается достать. Что такое т... Правила клин код для того и придумали, когда программист пишет шифрограмму и ему кажется что все под контролем - это самообман. По этой задаче типичный кейс, и он скорее всего и у тебя тоже есть - это не учитывание отрицательных значений. Второй вариант, условие цикла file.ready но...насколько я помню такой вариант прокатывает. Исправь момент с отрицательными числами. Попробуй сдать на валидацию - не пройдет, выложи в комменты последний вариант

Mark Solncev

Уровень 37

4 января 2020, 11:32

Учёл отрицательные числа, всё заработало, Большое Вам спасибо за то, что навели на решение и рассказали про чистый код)

public class Solution {
    public static void main(String[] args) throws IOException {

        BufferedReader file = new BufferedReader(new FileReader(args[0]));

        Map<String, Double> map = new HashMap<>();
        double max = 0;

        while (file.ready()) {
            String str = file.readLine();
            String name = str.split(" ")[0];
            Double value = Double.parseDouble(str.split(" ")[1]);
            if (value > max) max = value;

            boolean hasBeenInArray = false;

            for (Map.Entry<String, Double> pair : map.entrySet()) {
                if (name.equals(pair.getKey())) {
                    if (pair.getValue() + value > max) {
                        max = pair.getValue() + value;
                    }
                    map.put(name, pair.getValue() + value);
                    hasBeenInArray = true;
                }
            }
            if (!hasBeenInArray) {
                map.put(name, value);
            }
        }

        file.close();

        List<String> list = new ArrayList<>();

        TreeMap<String, Double> sortedMap = new TreeMap<>(map);
        max = 0;

        //Find Max
        for (Map.Entry<String, Double> pair : sortedMap.entrySet()) {

            if (pair.getValue() > max) {
                max = pair.getValue();
            }
        }

        //Find Name with Max
        for (Map.Entry<String, Double> pair : sortedMap.entrySet()) {

            if (pair.getValue().equals(max)) {
                list.add(pair.getKey());
            }
        }

        Collections.sort(list);

        for (String s : list) {
            System.out.println(s);
        }
    }
}

Justinian Judge в Mega City One

Master

4 января 2020, 11:52

о, вот это дело, красота :) хорошая новость, поздравляю :) На здоровье, успехов!