В чем разница?

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

Решен

Добрый день! Почему вот так валидатор принимает метод close():

public void close() throws IOException{
        stream.flush();
        stream.write("JavaRush © All rights reserved.".getBytes());
        stream.close();
    }
а вот так, выдает ошибку? пишет, что не ту строку дозаписываю
public class AmigoOutputStream extends FileOutputStream {
   ....
    public String to_write = "JavaRush © All rights reserved.";
    public byte[] to_write_array = to_write.getBytes();
    ....
    public void close() throws IOException{
        stream.flush();
        stream.write(to_write_array);
        stream.close();
    }
....

Просто же разбиваю одну операцию на несколько шагов и все

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

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

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

Артем Divertitto Senior Android-разработчи в United Tech

Master

13 июня 2019, 11:34

Скинь весь код

Take_it_easy

Уровень 24

Expert

13 июня 2019, 11:54

package com.javarush.task.task18.task1813; import java.io.*; import java.util.*; public class AmigoOutputStream extends FileOutputStream { private FileOutputStream stream; public String to_write = "JavaRush © All rights reserved."; public byte[] to_write_array = to_write.getBytes(); public static String fileName = "C:/tmp/result.txt"; public AmigoOutputStream(FileOutputStream stream) throws FileNotFoundException{ super(fileName); this.stream = stream; } public void close() throws IOException{ stream.flush(); stream.write(to_write_array); stream.close(); } public void write(byte[] b) throws IOException{ stream.write(b); } public void write(byte[] b, int off, int len) throws IOException{ stream.write(b,off,len); } public void write(int b) throws IOException{ stream.write(b); } public void flush() throws IOException{ stream.flush(); } public static void main(String[] args) throws FileNotFoundException { new AmigoOutputStream(new FileOutputStream(fileName)); } }

Take_it_easy

Уровень 24

Expert

13 июня 2019, 11:55

решение

Валидатор выдавал ошибку и писал - "проверьте, какую строку дозаписываете". Поменял одну строку в close() и все заработало

Санек

Уровень 35

Expert

13 июня 2019, 10:04

не по теме, но все-же:

public byte[] to_write_array = to_write.getBytes();

слышали про кэмэлкейс?

Take_it_easy

Уровень 24

Expert

13 июня 2019, 10:40

Привет ДаЯСлышалПроКэмелкейс, только он тут при чем?

Justinian Judge в Mega City One

Master

13 июня 2019, 10:52

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