JavaRush/Помощь по Java задачам/Ваще в ауте от задачки

4 уровень

02.07.2019
554просмотров
7комментариев

Ваще в ауте от задачки

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

Архивный

Подскажите плз, как поправить с минимум потерь? Выводит все четко. Только четные числа по возрастанию. Остальное чето не понял.

В этой задаче тебе нужно:

Ввести имя файла с консоли.
Прочитать из него набор чисел.
Вывести в консоли только четные, отсортированные по возрастанию.

Пример ввода:
5 8 -2 11 3 -5 2 10

Пример вывода:
-2 2 8 10

Требования:

Программа должна считывать данные с консоли.
Программа должна создавать FileInputStream для введенной с консоли строки.
Программа должна выводить данные на экран.
Программа должна вывести на экран все четные числа, считанные из файла, отсортированные по возрастанию.
Программа должна закрывать поток чтения из файла — FileInputStream.

package com.javarush.task.task13.task1326; import java.util.*;import java.io.*; /* Сортировка четных чисел из файла */ public class Solution { public static void main(String[] args) throws Exception { ArrayList <Integer> list = new ArrayList<>(); BufferedReader br = new BufferedReader(new InputStreamReader(System.in)); String nameFile = br.readLine(); String MyFile = "file"; if (nameFile.equals(MyFile)) { try { InputStream inStream = new FileInputStream("D:/file.txt"); BufferedReader reader = new BufferedReader(new InputStreamReader(inStream)); String str; while ((str = reader.readLine()).matches("^-?\\d+$")) { list.add(Integer.parseInt(str)); } inStream.close(); } catch (Exception e) { } ArrayList<Integer> even = new ArrayList<>(); for (int in : list) { if (in % 2 == 0) { even.add(in); } } Collections.sort(even); for (int num : even) System.out.println(num); } else {System.out.println("Неверное имя файла");} } }

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

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

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

JustinianJudge в Mega City One

Master

2 июля 2019, 12:41

полезный

Для начала убрать

String MyFile = "file";
 if (nameFile.equals(MyFile)) {
         InputStream inStream = new FileInputStream("D:/file.txt");

должно быть так:

BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
String fileName = reader.readLine();
InputStream inputStream = new FileInputStream(fileName);
BufferedReader reader = new BufferedReader(new InputStreamReader(inputStream));

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

AlexeyV

Уровень 4

Expert

2 июля 2019, 13:00

Спасибо, немного поправилось. Но осталось одно не выполненное условие : "Программа должна вывести на экран все четные числа считанные из файла отсортированные по возрастанию." - У меня сортировка то рабочая. Но я не догоняю почему она не принимается.

public static void main(String[] args) throws Exception {
        ArrayList <Integer> list = new ArrayList<>();
        BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
        String fileName = br.readLine();
        InputStream inputStream = new FileInputStream(fileName);
        BufferedReader reader = new BufferedReader(new InputStreamReader(inputStream));
        try {
            while ((fileName = reader.readLine()).matches("^-?\\d+$")) {
                list.add(Integer.parseInt(fileName));
                inputStream.close();
            }

        } catch (Exception e) {
        }
        ArrayList<Integer> even = new ArrayList<>();
        for (int in : list) {
            if (in % 2 == 0) {
                even.add(in);
            }

            Collections.sort(even);
            for (int num : even)
                System.out.println(num);
        }
    }
}

JustinianJudge в Mega City One

Master

2 июля 2019, 13:04

ок, а что у тебя выводит для файла с примера:

JustinianJudge в Mega City One

Master

2 июля 2019, 13:07

решение

да, там один знак перенести, вроде все ок

AlexeyV

Уровень 4

Expert

2 июля 2019, 13:26

Спасибо! Прошел проверку) Но все равно надо разбираться как оно там принимает, закрывает. Пока все не очень понятно.

package com.javarush.task.task13.task1326;
import java.util.*;import java.io.*;
/*Сортировка четных чисел из файла*/

public class Solution {
    public static void main(String[] args) throws Exception {
        ArrayList <Integer> list = new ArrayList<>();
        BufferedReader br = new BufferedReader(new InputStreamReader(System.in));
        String fileName = br.readLine();
        InputStream inputStream = new FileInputStream(fileName);
        BufferedReader reader = new BufferedReader(new InputStreamReader(inputStream));
          try {
                while ((fileName = reader.readLine()).matches("^-?\\d+$")) {
                    list.add(Integer.parseInt(fileName));
                    inputStream.close();
                }

            } catch (Exception e) {
            }
            ArrayList<Integer> even = new ArrayList<>();
            for (int in : list) {
                if (in % 2 == 0) {
                    even.add(in);
                }
            }
         Collections.sort(even);
            for (int num : even)
                System.out.println(num);
    }
}

JustinianJudge в Mega City One

Master

2 июля 2019, 13:31

полезный

Со временем разберешься. Дам на будущее совет - не придумывай своих авторских сокращений, аббревиатур и тд. Суть в том, что код мы пишем для других. Мы почти никогда не будем писать его для себя. А другие должны прочитать, у одного integer это in, у другого intr, у третьего еще как... Больше половины рабочего времени программист тратит на чтение чужого кода. Обращай внимание кто как пишет, называет и тд. Временную переменную интеджер можно обозвать n, для стринг s, str. А вот это уже очень плохо: fileName = reader.readLine()).matches("^-?\\d+$") непонятные имена и сокращения еще ладно, но это уже введение в заблуждение, ведь ты не читаешь имя файлов, ты просто выгружаешь строки. Предложу свой вариант, он не идеален, не буду тебя грузить стилистикой джава кода, но для ориентира пусть будет, и обращай внимание как в лекциях на джава раше пишут.

public static void main(String[] args) throws Exception {
        ArrayList<Integer> numbers = new ArrayList<>();

        BufferedReader consoleReader = new BufferedReader(new InputStreamReader(System.in));
        String fileName = consoleReader.readLine();

        InputStream inputStream = new FileInputStream(fileName);
        BufferedReader reader = new BufferedReader(new InputStreamReader(inputStream));

        try {
            String str;
            while ((str = reader.readLine()) != null) {
                if (str.matches("^-?\\d+$")) {
                    numbers.add(Integer.parseInt(str));
                }
            }
            reader.close();

        } catch (Exception e) {
        }

        ArrayList<Integer> evenNumbers = new ArrayList<>();
        for (int number : numbers) {
            if (number % 2 == 0) {
                evenNumbers.add(number);
            }
        }

        Collections.sort(evenNumbers);

        for (int number : evenNumbers)
            System.out.println(number);
       }
}

AlexeyV

Уровень 4

Expert

2 июля 2019, 13:46

Спасибо большое! Мне это очень полезно!