JavaRush/Помощь по Java задачам/Как такое может быть?

29 уровень

14.06.2019
602просмотров
10комментариев

Как такое может быть?

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

Решен

Если удалить реализацию сортировки и подсчета,

package com.javarush.task.task18.task1821;

/*
Встречаемость символов
*/

import java.io.FileInputStream;
import java.io.IOException;
import java.util.ArrayList;
import java.util.Collections;

public class Solution {
    public static void main(String[] args) throws IOException {
        try (FileInputStream fileInputStream = new FileInputStream(args[0])) {
            ArrayList listFromFile = new ArrayList();
            while (fileInputStream.available() > 0) {
                listFromFile.add(fileInputStream.read());
            }
            fileInputStream.close();
        }
    }
}

то требования Создай поток для чтения из файла, который приходит первым параметром в main. Поток для чтения из файла должен быть закрыт. ВЫПОЛНЯЮТСЯ!!! Как объяснить и исправить данную трагедь?

Программа запускается с одним параметром - именем файла, который содержит английский текст.
Посчитать частоту встречания каждого символа.
Отсортировать результат по возрастанию кода ASCII (почитать в инете).

Пример:
','=44, 's'=115, 't'=116.

Вывести на консоль отсортированный результат:
[символ1] частота1
[символ2] частота2
Закрыть потоки.

Пример вывода:
, 19 - 7 f 361

Требования:

Считывать с консоли ничего не нужно.
Создай поток для чтения из файла, который приходит первым параметром в main.
В файле необходимо посчитать частоту встречания каждого символа и вывести результат.
Выведенный в консоль результат должен быть отсортирован по возрастанию кода ASCII.
Поток для чтения из файла должен быть закрыт.

package com.javarush.task.task18.task1821; /* Встречаемость символов */ import java.io.FileInputStream; import java.io.IOException; import java.util.ArrayList; import java.util.Collections; public class Solution { public static void main(String[] args) throws IOException { try (FileInputStream fileInputStream = new FileInputStream(args[0])) { ArrayList listFromFile = new ArrayList(); while (fileInputStream.available() > 0) { listFromFile.add(fileInputStream.read()); } Collections.sort(listFromFile); int countUnic = 1; for (int i = 0; i < listFromFile.size() - 1; i++) { if (listFromFile.get(i).equals(listFromFile.get(i + 1))) { countUnic++; } else { System.out.println((String.valueOf(Character.toChars((Integer) listFromFile.get(i))) + " " + countUnic)); countUnic = 1; } } System.out.println((String.valueOf(Character.toChars((Integer) listFromFile.get(listFromFile.size() - 1))) + " " + countUnic)); fileInputStream.close(); } } }

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

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

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

Анастасия Льюис

Уровень 29

14 июня 2019, 22:00

пришлось отказаться от своего решения,валидатор принял решение с использованием TreeMap<Byte, Integer>

JustinianJudge в Mega City One

Master

15 июня 2019, 10:56

валидатор эту задачу принимает по разному, в первоначальном решении были логические ошибки работы с коллекциями. Но вариант с тримап в принципе здесь более уместен, так что забыть и идти дальше не забывая про восстановление нервных клеток и отдых )

Riccio

Уровень 35

Master

14 июня 2019, 19:35

Вы используете try-with-resources в обоих случаях. Вам не надо закрывать поток вручную командой fileInputStream.close(); Где-то встречал особенность валидатора, что поток должен быть закрыть сразу после чтения. А не в последней строчкой кода. Попробуйте вынести из блока try всё, что не касается чтения файла.

Анастасия Льюис

Уровень 29

14 июня 2019, 19:49

исправила с учетом ваших замечаний,но увы, на результат проверки это не повлияло. может быть есть еще мысли?

public class Solution {
    public static void main(String[] args) throws IOException {
        ArrayList listFromFile = new ArrayList();
        try (FileInputStream fileInputStream = new FileInputStream(args[0])) {
            while (fileInputStream.available() > 0) {
                listFromFile.add(fileInputStream.read());
            }
        }
        Collections.sort(listFromFile);
        int countUnic = 1;
        for (int i = 0; i < listFromFile.size() - 1; i++) {
            if (listFromFile.get(i).equals(listFromFile.get(i + 1))) {
                countUnic++;
            } else {
                System.out.println((String.valueOf(Character.toChars((Integer) listFromFile.get(i))) + " " + countUnic));
                countUnic = 1;
            }
        }
        System.out.println((String.valueOf(Character.toChars((Integer) listFromFile.get(listFromFile.size() - 1))) + " " + countUnic));
        }
    }

JustinianJudge в Mega City One

Master

14 июня 2019, 19:51

попробуй вариант без try with resources. Просто создавай поток вначале. И закрывай его в конце кода. Может валидатор вредничает

Анастасия Льюис

Уровень 29

14 июня 2019, 19:55

Увы, этот вариант тоже не прошел

JustinianJudge в Mega City One

Master

14 июня 2019, 19:58

ошибки те же? сбрось код варианта, который не прошел последним

Анастасия Льюис

Уровень 29

14 июня 2019, 20:06

и с fileInputStream.close(); в конце кода такая же история

public class Solution {
    public static void main(String[] args) throws IOException {
        ArrayList listFromFile = new ArrayList();
        FileInputStream fileInputStream = new FileInputStream(args[0]);
            while (fileInputStream.available() > 0) {
                listFromFile.add(fileInputStream.read());
            }
        fileInputStream.close();
        Collections.sort(listFromFile);
        int countUnic = 1;
        for (int i = 0; i < listFromFile.size() - 1; i++) {
            if (listFromFile.get(i).equals(listFromFile.get(i + 1))) {
                countUnic++;
            } else {
                System.out.println((String.valueOf(Character.toChars((Integer) listFromFile.get(i))) + " " + countUnic));
                countUnic = 1;
            }
        }
        System.out.println((String.valueOf(Character.toChars((Integer) listFromFile.get(listFromFile.size() - 1))) + " " + countUnic));
        }
    }

JustinianJudge в Mega City One

Master

14 июня 2019, 20:18

хм, вроде нашлось что-то, сейчас пару мин проверю

JustinianJudge в Mega City One

Master

14 июня 2019, 20:23

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