JavaRush/Помощь по Java задачам/Помогите, не могу понять что не так

2 уровень

01.11.2022
667просмотров
13комментариев

Помогите, не могу понять что не так

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

Обсуждается

Ой уже перечитал все лекции, кучу доп материала и все в кашу... Самое более или менее приближенное, что смог сделать, можете подсказать? А я спать :3 Спасибо заранее

Напиши программу, которая считывает с консоли путь к файлу1 и путь к файлу2. Далее все байты из файла1 записывает в файл2, но при этом меняет их местами по такому принципу: первый со вторым, третий с четвертым, и т.д.
Если последний байт в файле1 нечетный, то пишем его в файл2 как есть. Для чтения и записи файлов используй методы newInputStream и newOutputStream класса Files.

Требования:

Программа должна считать с консоли пути к файлам.
Программа должна переписывать байты из одного файла в другой согласно условию.
Для чтения и записи файлов должны быть использованы методы newInputStream и newOutputStream класса Files.
Потоки для чтения и записи должны быть закрыты.
Не используй следующие классы File, FileInputStream, FileOutputStream, FileReader, FileWriter пакета java.io.

package com.javarush.task.pro.task15.task1504; import java.io.*; import java.nio.file.Files; import java.nio.file.Path; import java.nio.file.Paths; import java.util.Scanner; /* Перепутанные байты */ public class Solution { public static void main(String[] args) throws IOException { //напишите тут ваш код Scanner sc = new Scanner(System.in); String src = sc.nextLine(); String dest = sc.nextLine(); try (InputStream inputStream = Files.newInputStream(Path.of(src)); OutputStream outputStream = Files.newOutputStream(Path.of(dest))) { byte[] buffer = new byte[65536]; // 64 kb while (inputStream.available() > 0) { int real = inputStream.read(buffer); for (int i = 0; i < buffer.length; i++) { if (real % 2 != 0) { int j = 1; buffer[i] = buffer[j]; buffer[j] = buffer[i]; j++; outputStream.write(buffer, 0, real - 1); } } } } } }

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

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

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

Гофф

Уровень 3

Master

1 ноября 2022, 09:58

полезный

Видать, запутался. Если во сне озарение не нашло, то вот: 1. Строка 24. Его надо крутить не до buffer.length, а до real, потому что байтов в последней считанной пачке может оказаться меньше, чем buffer.length 2. Если числа меняются попарно, то цикл, наверное, лучше сделать именно по парам, то-есть с шагом в два (i += 2).

Dimash

Уровень 2

2 ноября 2022, 08:41

Спасибо, вообще в итоге решил другим путем, но сейчас переделал код по вашему совету и в итоге валидатор принял, но выбивает ошибку, в момент создания файла и с кодом, который прошёл через проверку и валидатор принял то же самое... java.nio.file.AccessDeniedException вот код

public static void main(String[] args) throws IOException {
        Scanner sc = new Scanner(System.in);
        String src = sc.nextLine();
        String dest = sc.nextLine();
        try (InputStream inputStream = Files.newInputStream(Path.of(src));
             OutputStream outputStream = Files.newOutputStream(Path.of(dest))) {
            byte[] buffer = new byte[65536]; // 64 kb
            while (inputStream.available() > 0) {
                int real = inputStream.read(buffer);
                for (int i = 0; i < real; i+=2) {
                    if (real % 2 != 0) {
                        int j = 1;
                        buffer[i] = buffer[j];
                        buffer[j] = buffer[i];
                        j = j + 2;
                        outputStream.write(buffer, 0, real - 1);
                    }
                }
            }
        }
    }

Гофф

Уровень 3

Master

2 ноября 2022, 09:09

1. В сообщении об ошибке ещё указывается и строка, которая ошибку вызывает. Насколько я могу судить, какие-то проблемы с доступом к файлу. 2. Иф в строке 11 совершенно не нужен, Даже не понимаю, зачем ты его вставил. Более того - если в файле нечётное число байт, то последняя пачка этих байт из-за него не обработается. 3. ПЕерменная j не нужна, зачем? Она у тебя всегда равна i + 1. 4. В принципе всё равно, но в аналогичных случаях лучше сравнивать текущий элемент с предыдущим, нежели со соледующим. То-есть цикл лучше начать со второго элемента for (int i = 1; i < real; i+=2) и свопать buffer[i] и buffer[i - 1] 5. Свопаешь ты тоже неправильно, погугли, как это сделать грамотно. Сейчас ты в buffer[i] присваиваешь значение buffer[j], а потом в buffer[j] закидываешь значение buffer[i], которое в этот момент уже равно... да buffer[j] оно равно к этому моменту, ты же сам предыдущей командой туда buffer[j] загнал.

Гофф

Уровень 3

Master

2 ноября 2022, 09:11

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

Dimash

Уровень 2

2 ноября 2022, 09:56

Эх... извиняюсь, что заставил так долго ждать. В итоге-то решение, которое принял валидатор не работало с нечетным числом, потому решил переработать его, но там было это невозможно из-за логики, там не было создания массива и использования циклов для проверки. Потому решил переработать и этот вариант, с учётом ваших замечаний, но обнаружил, что файл2 не перезаписывался и решил поискать в чем ошибка и лишь переписав правильное решение, обнаружил скорее всего, что надо бы создать второй массив байт, и скорее всего ошибка в самой логике for В итоге открыл правильное решение и проработал с ним в дебаггере и закомментировал, но для меня загадкой все же остаётся содержание цикла на 15 строке зачем он?

Scanner sc = new Scanner(System.in);
    String src = sc.nextLine();
    String dest = sc.nextLine();
    try (InputStream inputStream = Files.newInputStream(Path.of(src));
         OutputStream outputStream = Files.newOutputStream(Path.of(dest))) {
        byte[] bytes = inputStream.readAllBytes(); // узнаем размер массива байт
        byte[] bytes1 = new byte[bytes.length]; // создаем новый массив для хранения данных
        for (int i = 0; i < bytes.length; i += 2) {
            // увеличиваем на два, чтобы позже узнать четность байт
            if (i < bytes.length - 1) {
                // если последний элемент нечетный
                bytes1[i] = bytes[i + 1];
                // bytes1[0] = bytes[0+1] => next step: bytes[2] = bytes[3]
                // меняем элемент четный с нечетным через i+1
                bytes1[i + 1] = bytes[i]; // bytes[1]
            } else {
                bytes1[i] = bytes[i]; // или же пишем один элемент
            }
        }
        outputStream.write(bytes1);
    } catch (IOException e) {
        System.out.println("Something went wrong");
    }
}

Dimash

Уровень 2

2 ноября 2022, 10:16

В итоге посмотрел и проработал ещё и с третьим вариантом xD вот если интересно, тут уже проверка более адекватная

public static void main(String[] args) throws IOException {
        //напишите тут ваш код
        Scanner sc = new Scanner(System.in);
        String src = sc.nextLine();
        String dest = sc.nextLine();
        try (InputStream inputStream = Files.newInputStream(Path.of(src));
             OutputStream outputStream = Files.newOutputStream(Path.of(dest))) {
            byte[] buffer = new byte[65536]; // 64 kb
            while (inputStream.available() > 0) {
                int read = inputStream.read(buffer);
                if (read % 2 == 0) { // для нечетных
                    for (int i = 0; i < read; i += 2) {
                        byte temp = buffer[i]; // временный для нуля
                        buffer[i] = buffer[i + 1]; // byte[0] = byte[1] => byte[1]
                        buffer[i + 1] = temp; // byte[1] = byte[0]
                    }
                } else {
                    for (int i = 0; i < read - 1; i += 2) { // для четных read-1=> чтоб записался последний нечетный символ
                        byte temp = buffer[i]; // временный для нуля
                        buffer[i] = buffer[i + 1];
                        buffer[i + 1] = temp;
                    }
                }
                outputStream.write(buffer, 0, read);
            }

Гофф

Уровень 3

Master

2 ноября 2022, 19:08

полезный

Годитсо )) Разве что если бы ты сравнивал с предыдущим, то проверки на чётность и ветвления не понадобилось бы:

try (InputStream inputStream = Files.newInputStream(Path.of(src));
    OutputStream outputStream = Files.newOutputStream(Path.of(dest))) {
    byte[] buffer = new byte[65536]; // 64 kb
    while (inputStream.available() > 0) {
        int read = inputStream.read(buffer);
        for (int i = 1; i < read; i += 2) {
            byte temp = buffer[i];
            buffer[i] = buffer[i - 1];
            buffer[i - 1] = temp;
        }
        outputStream.write(buffer, 0, read);
    }
}

Гофф

Уровень 3

Master

2 ноября 2022, 19:12

полезный

Кстати, свопить можно и без дополнительной переменной temp: a = a + b; b = a - b; a = a - b; правда, тут есть ограничения на размер суммы, она не должна превышать MAX_VALUE.

Dimash

Уровень 2

3 ноября 2022, 08:51

А тоже, если сравнивать с предыдущим даже без проверки на четность и нечетность, спасибо) А можно поконкретнее с этими способом свапа без доп переменной с примером?)

Гофф

Уровень 3

Master

3 ноября 2022, 08:58

Ну подставь сам любые значения в переменные и побудь немного компьютером, посмотри, что будет происходить, в трёх строчках не запутаешься :]

Dimash

Уровень 2

3 ноября 2022, 09:02

a = 5; b = 3; 5 = 5+3; (8) a = 8 3 = 8-3; (5) b = 5 5 = 8-5; (3) a = 3 Да хороший свап) Сначала идёт сумма переменных, затем отнимаем сумму поочередно от переменной пары) Спасибо, что помогли в принципе разобраться, наверное ещё так долго на уровне я не сидел)

Anonymous #3134856

Уровень 34

29 января 2023, 12:24

Я что-то совсем затупила с этой задачей(( поясните, пожалуйста, логику в цикле for. Не могу понять, почему проверка на четность не нужна((

Гофф

Уровень 3

Master

29 января 2023, 12:30

Потому что цикл перебирает только нечётные числа. В цикле for (int i = 1; i < read; i += 2) счётчик i будет 1, 3, 5, 7, 9, 11 ... Нам остаётся только поменять местами [i] и [i - 1]