Подскажите, что изменить

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

Обсуждается

Добрый день. При отправке на проверку программа ничего не выводит, заканчивается время. При простой компиляции ошибок нет, но вероятно, что что-то не так?! Прошу подсказать.

Считать с консоли три имени файла: файл1, файл2, файл3.
Разделить файл1 по следующему критерию:
Первую половину байт записать в файл2, вторую половину байт записать в файл3.
Если в файл1 количество байт нечетное, то файл2 должен содержать большую часть.
Закрыть потоки.

Требования:

Программа должна три раза считать имена файлов с консоли.
Для чтения из файла используй поток FileInputStream, для записи в файлы - FileOutputStream
Первую половину байт из первого файла нужно записать во второй файл.
Вторую половину байт из первого файла нужно записать в третий файл.
Потоки FileInputStream и FileOutputStream должны быть закрыты.

package com.javarush.task.task18.task1808; import java.io.*; /* Разделение файла */ public class Solution { public static void main(String[] args) throws IOException { BufferedReader reader = new BufferedReader(new InputStreamReader(System.in)); String file1Name = reader.readLine(); String file2Name = reader.readLine(); String file3Name = reader.readLine(); FileInputStream fileInputStream = new FileInputStream(file1Name); byte[] file1Array = new byte[fileInputStream.available()]; while (fileInputStream.available() > 0) { fileInputStream.read(file1Array); } FileOutputStream fileOutputStream1 = new FileOutputStream(file2Name); FileOutputStream fileOutputStream2 = new FileOutputStream(file3Name); if (file1Array.length % 2 == 0) { while (fileInputStream.available() >= file1Array.length / 2) { fileOutputStream1.write(file1Array, 0, file1Array.length / 2); } while (fileInputStream.available() > 0 && fileInputStream.available() < file1Array.length / 2) { fileOutputStream2.write(file1Array, (file1Array.length / 2) + 1, file1Array.length / 2); } } else { while (fileInputStream.available() >= (file1Array.length / 2) - 1) { fileOutputStream1.write(file1Array, 0, (file1Array.length / 2) + 1); } while (fileInputStream.available() > 0 && fileInputStream.available() < (file1Array.length / 2) - 1) { fileOutputStream2.write(file1Array, (file1Array.length / 2) + 1, (file1Array.length / 2) - 1); } } fileInputStream.close(); fileOutputStream1.close(); fileOutputStream2.close(); } }

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

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

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

Alexandr

Уровень 36

31 октября 2021, 11:49

Сделай все в одном цикле

while (fileInputStream.available() > 0) {
      int amountOfBytes = fileInputStream.read(file1Array);

      if(amountOfBytes % 2 != 0){
            // тут записываешь согласно условию в иф
      }
      if(если четное){
           // тут записываешь согласно условию в иф
      }
}

Ksenia VolkovaJava Developer в DXC

Master

31 октября 2021, 11:32

А файлы-то записываются? )) Что ты проверяешь этим условием цикла (и для чего он нужен)?

while (fileInputStream.available() >= file1Array.length / 2)

Hello

Уровень 17

31 октября 2021, 12:33

Написал по-другому. Результат проверки стал выходить, но кроме первого условия никакие не проходят, к сожалению.

public class Solution {
    public static void main(String[] args) throws IOException {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        String file1Name = reader.readLine();
        String file2Name = reader.readLine();
        String file3Name = reader.readLine();

        FileInputStream fileInputStream = new FileInputStream(file1Name);
        byte[] file1Array = new byte[fileInputStream.available()];
        while (fileInputStream.available() > 0) {
            fileInputStream.read(file1Array);
        }

        FileOutputStream fileOutputStream1 = new FileOutputStream(file2Name);
        FileOutputStream fileOutputStream2 = new FileOutputStream(file3Name);
        if (file1Array.length % 2 == 0) {
            for (int i = 0; i < file1Array.length / 2; i++) {
                fileOutputStream1.write(file1Array.length / 2);
            }
            for (int i = file1Array[(file1Array.length / 2)]; i < file1Array.length; i++) {
                fileOutputStream2.write(file1Array.length / 2);
            }
        } else {
            for (int i = 0; i < (file1Array.length / 2) + 1; i++) {
                fileOutputStream1.write((file1Array.length / 2) + 1);
            }
            for (int i = file1Array[(file1Array.length / 2) + 1]; i < file1Array.length; i++) {
                fileOutputStream2.write(file1Array.length / 2);
            }
        }
        fileInputStream.close();
        fileOutputStream1.close();
        fileOutputStream2.close();
    }
}

Ksenia VolkovaJava Developer в DXC

Master

31 октября 2021, 13:23

fileOutputStream1.write(file1Array.length / 2);

Что должна делать эта строчка? Что такое file1Array.length/2?

Hello

Уровень 17

31 октября 2021, 15:56

public class Solution {
    public static void main(String[] args) throws IOException {
        BufferedReader reader = new BufferedReader(new InputStreamReader(System.in));
        String file1Name = reader.readLine();
        String file2Name = reader.readLine();
        String file3Name = reader.readLine();

        FileInputStream fileInputStream = new FileInputStream(file1Name);
        byte[] file1Array = new byte[fileInputStream.available()];
        while (fileInputStream.available() > 0) {
            fileInputStream.read(file1Array);
        }
        fileInputStream.close();
        FileOutputStream fileOutputStream1 = new FileOutputStream(file2Name);
        FileOutputStream fileOutputStream2 = new FileOutputStream(file3Name);
        if (file1Array.length % 2 == 0) {
            for (int i = 0; i < file1Array.length / 2; i++) {
                fileOutputStream1.write(file1Array[i]);
?
            }
            for (int i = file1Array[(file1Array.length / 2)]; i < file1Array.length; i++) {
                fileOutputStream2.write(file1Array[i]);
            }
        } else {
            for (int i = 0; i < (file1Array.length / 2) + 1; i++) {
                fileOutputStream1.write(file1Array[i]);
            }
            for (int i = file1Array[(file1Array.length / 2) + 1]; i < file1Array.length; i++) {
                fileOutputStream2.write(file1Array[i]);
            }
        }
        fileOutputStream1.close();
        fileOutputStream2.close();
    }
}

Ksenia VolkovaJava Developer в DXC

Master

31 октября 2021, 16:08

for (int i = file1Array[(file1Array.length / 2)]; i < file1Array.length; i++)

Какое начальное значение ты присваиваешь i ? Вообще, хорошо бы перед сдачей валидатору запускать и проверять свой код самостоятельно. Чтобы не отправлять на проверку явно нерабочий код.

Hello

Уровень 17

31 октября 2021, 16:13

Всегда. Конечно. Результат Process finished with exit code 0 ничего не подсказывает.

Ksenia VolkovaJava Developer в DXC

Master

31 октября 2021, 16:14

Что значит не подсказывает? Результат-то какой? А что у тебя в исходном файле? и что получается в двух итоговых?