В чем ошибки?

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

Решен

Временный файл создан с помощью метода Files.createTempFile - валидатор говорит, что нет URL скопирован во временный файл с помощью метода Files.copy - валидатор, говорит, что нет Файл перемещен с помощью метода Files.move - валидатор, говорит, что нет Ну что я делаю не так?

Реализуй метод downloadFile(String urlString, Path downloadDirectory), на вход которого подается ссылка для скачивания файла и папка, куда нужно закачать файл.
Все ссылки имеют вид:
https://yastatic.net/morda-logo/i/citylogos/yandex19-logo-ru.png
http://toogle.com/files/rules.txt
https://pacemook.com/photos/image1.jpg

Метод должен создать объект URL и загрузить содержимое файла на локальный диск.
Выкачивай сначала во временную директорию, чтобы в случае неуспешной загрузки в твоей директории не оставались недокачанные файлы.
Затем перемести файл в пользовательскую директорию. Имя для файла возьми из ссылки.
Используй только классы и методы из пакета java.nio.

Требования:

Метод downloadFile должен создавать объект URL для переданной ссылки.
Метод downloadFile должен создать временный файл с помощью метода Files.createTempFile.
Метод downloadFile должен скачать файл по ссылке во временный файл, используя метод Files.copy.
Метод downloadFile должен переместить файл из временной директории в пользовательскую, используя метод Files.move.
Имя сохраненного файла должно быть таким же, как в URL-ссылке.

package com.javarush.task.task31.task3112; 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.nio.file.Paths; import java.nio.file.StandardCopyOption; /* Загрузчик файлов */ public class Solution { public static void main(String[] args) throws IOException { Path passwords = downloadFile("https://javarush.ru/testdata/secretPasswords.txt", Paths.get("c:\\Users\\MiK\\Downloads\\")); for (String line : Files.readAllLines(passwords)) { System.out.println(line); } } public static Path downloadFile(String urlString, Path downloadDirectory) throws IOException { // Открыть URL URL url = new URL(urlString); InputStream inputStream = url.openStream(); // Скачать URL во временный файл Path tempfile = Files.createTempFile(downloadDirectory, "tmp_", ".tmp"); Files.copy(inputStream, tempfile, StandardCopyOption.REPLACE_EXISTING); // Закрыть URL //inputStream.close(); // Скопировать временный файл в конечный файл String filename = urlString.substring(urlString.lastIndexOf("/") + 1); Path result = Paths.get(downloadDirectory.toString(), filename); Files.move(tempfile, result, StandardCopyOption.REPLACE_EXISTING); return result; } }

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

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

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

art

Уровень 27

16 июня 2019, 19:03

inputStream.close(); закомментирован нет проверки существует ли конечный файл, если нет создать Path result = Paths.get(downloadDirectory.toString(), filename); директория и файл должны быть разделены символом /

Mikhail

Уровень 40

17 июня 2019, 17:12

// Создаем downloadDirectory, если не существует
        if(!Files.exists(downloadDirectory))
            Files.createDirectories(downloadDirectory);

        //Скачиваем URL
        URL url = new URL(urlString);
        InputStream inputStream = url.openStream();

        Path tmpPath = Files.createTempFile(downloadDirectory, "tmp", "tmp");
        Files.copy(inputStream, tmpPath, StandardCopyOption.REPLACE_EXISTING);
        inputStream.close();

        String filename = urlString.substring(urlString.lastIndexOf("/") + 1, urlString.length());
        Path result = FileSystems.getDefault().getPath(downloadDirectory.toString(), filename);
        return Files.move(tmpPath, result);

Переписал так - все равно не проходит валидатор. Зачем создавать конечный файл?

art

Уровень 27

17 июня 2019, 17:44

Path resultName = Paths.get(downloadDirectory.toString() + "/" + nameFile.toString()); если нет разделителя путь ищется в виде папкафайл.txt if (Files.notExists(downloadDirectory)) Files.createDirectories(downloadDirectory); если файл не существует куда сохранять будешь?

art

Уровень 27

17 июня 2019, 17:50

package com.javarush.task.task31.task3112;

import java.io.IOException;
import java.io.InputStream;
import java.net.URL;
import java.nio.file.Files;
import java.nio.file.Path;
import java.nio.file.Paths;
import java.nio.file.StandardCopyOption;

/*
Загрузчик файлов
*/
public class Solution {

    public static void main(String[] args) throws IOException {
        Path passwords = downloadFile("https://javarush.com/testdata/secretPasswords.txt",
                Paths.get("D:/MyDownloads"));    //TODO     D:/MyDownloads

        for (String line : Files.readAllLines(passwords)) {
            System.out.println(line);
        }
    }

    public static Path downloadFile(String urlString, Path downloadDirectory) throws IOException {
        // implement this method
        URL url = new URL(urlString);
        InputStream inputStream = url.openStream();

        String urlName = url.getPath();

        Path namePath = Paths.get(urlName);

        Path nameFile = namePath.getFileName();

        Path tempFile = Files.createTempFile( "temp-" , ".tmp");
        Files.copy(inputStream, tempFile, StandardCopyOption.REPLACE_EXISTING);
        inputStream.close();

        Path resultName = Paths.get(downloadDirectory.toString() + "/" + nameFile.toString());
        //System.out.println(resultName);

        if (Files.notExists(downloadDirectory)) Files.createDirectories(downloadDirectory);

//        if (resultName.toFile().isFile())
//            Files.delete(resultName);
//        Path result = Files.createFile(resultName);

        Files.move(tempFile, resultName, StandardCopyOption.REPLACE_EXISTING);

        return resultName;
    }
}