Не проходит проверку

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

Обсуждается

здравствуйте, я не могу понять в предыдущих задачах делегировал выполнение именно таким образом, почему не принимает

1 Измени класс AmigoOutputStream так, чтобы он стал Wrapper-ом для класса FileOutputStream. Используй наследование.
2 При вызове метода close() должны выполняться следующая последовательность действий:
2.1 Вызвать метод flush().
2.2 Записать в конец файла фразу "JavaRush © All rights reserved.", используй метод getBytes().
2.3 Закрыть поток методом close().

Требования:

Метод main изменять нельзя.
Класс AmigoOutputStream должен наследоваться от класса FileOutputStream.
Класс AmigoOutputStream должен принимать в конструкторе объект типа FileOutputStream.
Все методы write(), flush(), close() в классе AmigoOutputStream должны делегировать свое выполнение объекту FileOutputStream.
Метод close() должен сначала вызвать метод flush(), затем записать в конец файла текст, затем закрыть поток.

package com.javarush.task.task18.task1813; import java.io.*; import java.nio.channels.FileChannel; import java.nio.charset.StandardCharsets; import java.util.Scanner; /* AmigoOutputStream */ public class AmigoOutputStream extends FileOutputStream { public static String fileName = "C:/tmp/result.txt"; private FileOutputStream original; String s = "JavaRush © All rights reserved."; public AmigoOutputStream(FileOutputStream fileOutputStream) throws FileNotFoundException { super(fileName); this.original = fileOutputStream; } public static void main(String[] args) throws FileNotFoundException { new AmigoOutputStream(new FileOutputStream(fileName)); } @Override public void write(int b) throws IOException { original.write(b); } @Override public void write(byte[] b) throws IOException { original.write(b); } @Override public void write(byte[] b, int off, int len) throws IOException { original.write(b, off, len); } @Override public void close() throws IOException { original.close(); original.flush(); FileOutputStream f = new FileOutputStream(fileName, true); byte[] byttte ; byttte = s.getBytes(StandardCharsets.UTF_8); for (int i = 0; i < byttte.length; i++) { f.write(byttte[i]); f.close(); } } @Override public FileChannel getChannel() { return original.getChannel(); } @Override protected void finalize() throws IOException { super.finalize(); } @Override public void flush() throws IOException { original.flush(); } }

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

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

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

Кирилл Java Developer

10 декабря 2021, 04:31

1. Используй outputStream родителя, а не поле. 2. В close() не соблюдена последовательность условия задачи:

flush();
write(s.getBytes());
close();

3. По-моему тут close() излишен, при каждом считывании байта ты закрываешь поток.

for (int i = 0; i < byttte.length; i++) {
    f.write(byttte[i]);
    f.close();
}

Поток надо закрыть один раз после того, как он сделал всю работу.

Zuzu

Уровень 25

10 декабря 2021, 17:15

тот close что был я случайно как то вклеил, на самм деле суть вопроса в другом. Тот порядок действий в метде close() неверно отображен. У меня это выглядит вот так.

@Override
 public void close() throws IOException {
    original.flush();
     FileOutputStream f = new FileOutputStream(fileName, true);
     byte[] byttte ;
     byttte = s.getBytes(StandardCharsets.UTF_8);
     for (int i = 0; i < byttte.length; i++) {
         f.write(byttte[i]);
         f.close();
     }
 }

Метод close() не делегирует свое выполнение классу FileOutputStream. Метод close() не вызывает flush() у объекта FileOutputStream. Я не понимаю претензий, разве я не делегировал выполнение мтода close() обьекту типа FileOutputStream? Хорошо в услвии написано: "Все методы write(), flush(), close() в классе AmigoOutputStream должны делегировать свое выполнение объекту FileOutputStream." валидатор говорит нужно делегировать непосредственно классу, я попробовал вот такую запись и ничего.

super.flush();

валидатор пишет одно и то же. Можете кто то помочь хотя бы с пунктом делигации метода flush()? Еще вы написали использовать outputStream родителя, а я даже эти методы не оверрайдил, спасибо я попробую

Кирилл Java Developer

11 декабря 2021, 05:35

Попробуй упростить close(). Напиши так:

@Override
public void close() throws IOException {
    flush();
    String s = "JavaRush © All rights reserved.";
    write(s.getBytes());
    fileOutputStream.close();
}

Тут получается, что к flush() и write() мы обращаемся напрямую (без fileOutputStream) так как в самом методе происходит делегация, а вот к close() обращаемся через переменную fileOutputStream, потому что мы в данный момент находимся в close() и нужно сделать делегирование. Если не поможет, то кинь в коммент весь код.

Дмитрий Ижевск

Уровень 22

12 декабря 2021, 14:59

А зачем ты открываешь новый файловый поток? FileOutputStream f = new FileOutputStream(fileName, true) В задаче задание записать и закрыть