JavaRush/Помощь по Java задачам/Неизвестная ошибка проверки

24 уровень

11.02.2022
746просмотров
20комментариев

Неизвестная ошибка проверки

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

Решен

Неизвестная ошибка во время проверки, подскажите правильно сделал? В созданный временный файл что то записывается 😀 Объясните как эту ошибку можно исправить?

Напиши код, который будет считывать с клавиатуры ссылку на файл в интернете, скачивать его и сохранять во временный файл.
Используй методы createTempFile(null, null) и write(Path, byte[]) класса Files, а также метод openStream() класса URL.

Требования:

Программа должна считать из консоли ссылку на файл в интернете (строку).
Программа должна скачать файл по ссылке и сохранить его во временный файл.
В программе должны быть использованы методы классов Files и URL согласно условию.
Не используй следующие классы: File, FileInputStream, FileOutputStream, FileReader, FileWriter пакета java.io.

//package com.javarush.task.pro.task15.task1521; // //import java.io.IOException; //import java.io.InputStream; //import java.net.URL; //import java.nio.file.Files; //import java.nio.file.Path; //import java.util.Scanner; // ///* //Временное сохранение файла //*/ // //public class Solution { // public static void main(String[] args) throws IOException { // Scanner scanner = new Scanner(System.in); // String line = scanner.nextLine(); // //напишите тут ваш код // } //} package com.javarush.task.pro.task15.task1521; import java.io.File; import java.io.IOException; import java.io.InputStream; import java.net.URL; import java.nio.file.Files; import java.nio.file.Path; import java.util.Scanner; /* Временное сохранение файла */ public class Solution { public static void main(String[] args) throws IOException { Scanner scanner = new Scanner(System.in); String line = scanner.nextLine(); URL url = new URL(line); // URL url = new URL("https://javarush.ru"); try (InputStream input = url.openStream()){ byte[]buffer=input.readAllBytes(); Path tmpFile = Files.createTempFile(null,null); Files.write(tmpFile,buffer); } } }

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

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

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

OlunaSoftware Developer в MaibornWolff

11 февраля 2022, 13:28

С толку тебя сбили, readAllBytes() волне рабочий метод и должен применяться. И вообще всё у тебя в решении правильно, только тупой валидатор к этой задаче не заточен на try-with-resourсes и все, как эти ни печально. В исходном коде просто убери try И задача решится. Сверила со своим кодом (мне тоже пришлось убирать try)

Ksenia VolkovaJava Developer в DXC

Master

11 февраля 2022, 13:39

Решения с try принимаются валидатором. Например, тут

OlunaSoftware Developer в MaibornWolff

11 февраля 2022, 13:51

Бывает. Конкретно у этой задачи именно из-за try решение не принималось. И "правильное решение" именно такое, как Dmytro нарешал, только без трая. Увы, это технически безграмотно, но в данном случае это так. Вот оно, канонически "правильное", хехе, скопировала вам сюда прямо из задачи из вкладки "правильное решение", не моё.

public class Solution {
    public static void main(String[] args) throws IOException {
        Scanner scanner = new Scanner(System.in);
        String line = scanner.nextLine();
        URL url = new URL(line);
        InputStream input = url.openStream();

        Path tempFile = Files.createTempFile(null, null);
        Files.write(tempFile, input.readAllBytes());
    }
}

Ksenia VolkovaJava Developer в DXC

Master

11 февраля 2022, 13:58

"Правильное решение" - это вовсе не единственное решение, принимаемое валидатором. И то, что "правильные решения" не всегда оптимальные - давно известный факт (за последний год, кстати, стали лучше). Но по ссылке в моем предыдущем комменте решение с try, которое тем не менее было принято валидатором.

OlunaSoftware Developer в MaibornWolff

11 февраля 2022, 14:07

Именно так, поэтому я беру слово "правильное" в кавычки, это сарказм, и поэтому пишу что без try это безграмотно и жалко что не принимается.

Ksenia VolkovaJava Developer в DXC

Master

11 февраля 2022, 14:32

Да принимается с try, принимается, не надо так переживать )

OlunaSoftware Developer в MaibornWolff

11 февраля 2022, 15:47

Значит, починили, молодцы :) P.S. И не надо мне приписывать посторонних эмоций, я ни о чем не переживала. Констатирую факт и помогаю в комментариях, не более того.

Dmytro

Уровень 24

11 февраля 2022, 13:23

решение

Этот вариант ответа полностью правильный. Даже скопировавши правильный ответ, выдает ошибку "Неизвестная ошибка во время проверки задания". В итоге поэкспериментировав ему не понравился комментарий в начале кода. Не знаю почему так(

Ksenia VolkovaJava Developer в DXC

Master

11 февраля 2022, 09:23

Не используй readAllBytes() Он появился в 9-й версии, а в этой части курса скорее всего используется 8-я.

Dmytro

Уровень 24

11 февраля 2022, 10:06

Избавившись readAllBytes() проблему не решило, ошибка та же.

package com.javarush.task.pro.task15.task1521;

        import java.io.File;
        import java.io.IOException;
        import java.io.InputStream;
        import java.net.URL;
        import java.nio.file.Files;
        import java.nio.file.Path;
        import java.util.Scanner;

/*
Временное сохранение файла
*/

public class Solution {
    public static void main(String[] args) throws IOException {
        Scanner scanner = new Scanner(System.in);
        String line = scanner.nextLine();
        URL url = new URL(line);
//        URL url = new URL("https://javarush.com");
        try (InputStream input = url.openStream()){
            byte[]buffer=new byte [1024];
            int temp=input.read(buffer);
            Path tmpFile = Files.createTempFile(null,null);
            while (temp!=-1){
                Files.write(tmpFile,buffer);
                temp=input.read(buffer);
            }
        }
    }
}

Ksenia VolkovaJava Developer в DXC

Master

11 февраля 2022, 10:08

Во-первых, хорошо бы убрать этот импорт

import java.io.File;

Во-вторых, сколько байт ты запишешь вот тут, если в исходном файле было, скажем, 5 байт?

Files.write(tmpFile,buffer);

Dmytro

Уровень 24

11 февраля 2022, 10:20

для этого я создал цикл

while (temp!=-1){
               Files.write(tmpFile,buffer);
               temp=input.read(buffer);

заполнил буфер, записал в файл. И так пока поток не закончится.

Ksenia VolkovaJava Developer в DXC

Master

11 февраля 2022, 10:31

Только ты не ответил на вопрос - сколько байт все же запишется в твой файл, если в исходном файле их 5? ))

Dmytro

Уровень 24

11 февраля 2022, 12:08

первой записью я так понимаю будет размер массива buffer. т.е ты хочешь сказать чтр надо размер buffer делать меньше?

Ksenia VolkovaJava Developer в DXC

Master

11 февраля 2022, 12:24

Я только хочу сказать, что итоговый файл должен быть такого же размера, что и исходный. Каким образом ты этого добьешься - решай сам.

Dmytro

Уровень 24

11 февраля 2022, 12:28

Уменьшил размер массива , но неизвестная ошибка так и осталась. Даже не знаю в какую сторону смотреть :(

Ksenia VolkovaJava Developer в DXC

Master

11 февраля 2022, 12:32

Покажи актуальный код

Dmytro

Уровень 24

11 февраля 2022, 12:37

package com.javarush.task.pro.task15.task1521;

        import java.io.IOException;
        import java.io.InputStream;
        import java.net.URL;
        import java.nio.file.Files;
        import java.nio.file.Path;
        import java.util.Scanner;

/*
Временное сохранение файла
*/

public class Solution {
    public static void main(String[] args) throws IOException {
        Scanner scanner = new Scanner(System.in);
        String line = scanner.nextLine();
        URL url = new URL(line);
//        URL url = new URL("https://javarush.com");
        try (InputStream input = url.openStream()){
            byte[]buffer=new byte [5];
            int temp=input.read(buffer);
            Path tmpFile = Files.createTempFile(null,null);
            while (temp!=-1){
                Files.write(tmpFile,buffer);
                temp=input.read(buffer);
            }
        }
    }
}

Ksenia VolkovaJava Developer в DXC

Master

11 февраля 2022, 12:54

Во-первых, это не решает проблему размера. А если в файле 3 байта? или 1? И еще один момент. Когда ты вот так записываешь байты в файл

Files.write(tmpFile,buffer);

то на каждом витке цикла ты будешь перезаписывать файл. То есть конечный файл всегда будет равен размеру буфера. Я сейчас посмотрела предыдущие вопросы - вроде бы поскольку эта задача относится к части Pro, то в ней должна быть версия java повыше 8-й. То есть вероятно, что readAllBytes() тут будет работать. Попробуй вернуть ту часть кода из начальной версии. Но с потоками ввода/вывода в любом случае стоит разобраться получше.

OlunaSoftware Developer в MaibornWolff

11 февраля 2022, 13:30

Нет, это новый курс, readAllBytes() вполне работает, а проблема тупее, чем можно было ожидать: к этой задаче не ожидалось применение try