ReSQL.ru
     
powered by simpleCommunicator - 2.0.30     © 2024 Programmizd 02
Мобильная версия    Контакт    Правила    FAQ    Помощь
Map
Доб. в избранное | Игнор. тему | Прикреп. тему | Пометить прочит. / непрочит. | Фильтр
Форумы / Java [игнор отключен] [закрыт для гостей] / Код в конструкторе
25 сообщений из 72, страница 1 из 3
  1  все
Код в конструкторе
    #40009842
faustgreen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
faustgreen
Участник
Как правильно реализовать класс, который должен выполнить определенные предварительные действия?

Абстрактный пример:
Допустим, нужно создать класс, которые будет получать через параметр конструктора имя файла. При его создании парсить этот файл и сохранять строки в мапу. После чего пользователь, методом getLine() сможет дергать эти строки.

Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
public class Reader{

   privete final Map<Integer, String> sentences;

   // Конструктор
   public Reader(Path file){
   }

   public String getLine(int index){
      return sentences.get(index);
   }
}



1). Писать код в конструкторе - это же плохо?
2). Т.е. что делать если нужно провести определенную инициализацию класса (но не простое присвоение значений полям, а определенная логика)?
...
Рейтинг: 0 / 0
19.10.2020, 14:23
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40009845
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
faustgreen,
Общие правила.
- инициализировать лучше в конструкторе если это не обращение к внешним ресурсам, внешним ИС, либам и не будет райзе 99,9%
...
Рейтинг: 0 / 0
19.10.2020, 14:31
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40009846
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
faustgreen,
Так как файл - внешний ресурс, то нужно сделать доп метод напр. open() или init()
...
Рейтинг: 0 / 0
19.10.2020, 14:34
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40009847
mad_nazgul
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
mad_nazgul
Участник
faustgreen
Как правильно реализовать класс, который должен выполнить определенные предварительные действия?

1). Писать код в конструкторе - это же плохо?
2). Т.е. что делать если нужно провести определенную инициализацию класса (но не простое присвоение значений полям, а определенная логика)?


1) Ну почему плохо?!
Кто запрещает?

Ну если не хотите писать тело в конструкторе, напишите в отдельной функции.
Которую потом вызовите в конструкторе.
...
Рейтинг: 0 / 0
19.10.2020, 14:35
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40009850
faustgreen
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
faustgreen
Участник
PetroNotC Sharp, т.е. будет выглядеть так:
Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
13.
14.
15.
16.
17.
public class Reader{

   privete final Map<Integer, String> sentences;

   // Конструктор
   public Reader(Path file){
      open(file);
   }

   public String getLine(int index){
      return sentences.get(index);
   }

   private void open(Path file){
       // some logic;
   }
}


?

Если это не является bad practise, то ок. Вообщем то это и был основной вопрос.
...
Рейтинг: 0 / 0
19.10.2020, 14:44
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40009856
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
faustgreen,
Не так. Нельзя открывать или обращатся в конструкторе
...
Рейтинг: 0 / 0
19.10.2020, 14:52
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40009873
Leonid Kudryavtsev
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Leonid Kudryavtsev
Участник
PetroNotC Sharp

Не так. Нельзя открывать или обращатся в конструкторе

mad_nazgul
1) Ну почему плохо?!
Кто запрещает?


сами авторы Java о таком запрещении почему-то не знают.

https://docs.oracle.com/javase/7/docs/api/java/io/FileInputStream.html#FileInputStream(java.io.File)
...
Рейтинг: 0 / 0
19.10.2020, 15:21
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40009874
Leonid Kudryavtsev
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Leonid Kudryavtsev
Участник
faustgreen

Если это не является bad practise, то ок. Вообщем то это и был основной вопрос.


beauty is in the eye of the beholder
...
Рейтинг: 0 / 0
19.10.2020, 15:22
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40009989
забыл ник
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
забыл ник
Участник
Правильнее всего сделать статический метод инициализатор, который возвращает option, ну в джаве в принципе можно и исключение кинуть.

Код: java
1.
2.
3.
4.
5.
6.
7.
8.
9.
10.
11.
12.
Class Builder (

Public static Builder makeBuilder(path)(
 //Init resource
 New Builder()
)





)
...
Рейтинг: 0 / 0
19.10.2020, 19:58
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40009995
kealon(Ruslan)
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
kealon(Ruslan)
Участник
Leonid Kudryavtsev,

так, то - боги
...
Рейтинг: 0 / 0
19.10.2020, 20:06
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010002
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
забыл ник,
Щаз мы его фабрикой классов или паттерном Строитель добьем)))
...
Рейтинг: 0 / 0
19.10.2020, 20:42
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010006
Sergunka
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
Sergunka
Участник
faustgreen
2). Т.е. что делать если нужно провести определенную инициализацию класса (но не простое присвоение значений полям, а определенная логика)?


Обычно для этой цели создают метод init с кодом возврата или ексепшином в зависимости от того чего там написано.
Это сложившиеся практика еще с прошлого века.

Вот вполне обоснованные дискуссии по этому поводу

https://softwareengineering.stackexchange.com/questions/334970/are-init-methods-a-code-smell

https://gamedev.stackexchange.com/questions/48446/why-should-i-use-separate-initialization-and-clean-up-methods-instead-of-putting

Основное правило это целесобразность таокого подхода в каждом конкретном случае. В общем случае это просто подозрительный код или неудачный дизайн.
...
Рейтинг: 0 / 0
19.10.2020, 21:06
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010025
kealon(Ruslan)
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
kealon(Ruslan)
Участник
faustgreen,

да на самом деле та же проблема, что и в плюсах, и в шарпе - отсутствие поддержки виртуальных методов в конструкторе
как говорится: проблема в языке by design - приходится вот так костылять с помощью init-методов

натягивают целесообразность такого подхода возможностью оптимизации на уровне компиляции
...
Рейтинг: 0 / 0
19.10.2020, 23:43
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010033
graycode
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
graycode
Гость
faustgreen
Абстрактный пример:
Допустим, нужно создать класс, которые будет получать через параметр конструктора имя файла. При его создании парсить этот файл и сохранять строки в мапу. После чего пользователь, методом getLine() сможет дергать эти строки.

Можно при вызове getLine() проверять есть ли экземпляр мапы и если нет, считывать и парсить файл (вызов соответствующего метода). Можно написать класс Reader с парсингом файла в конструкторе и сделать к нему виртуальную проксю LazyReader, которая при инициализации записывает имя файла в приватную переменную, также имеет приватное поле Reader, при вызове getLine() проверяет есть ли экземпляр Reader если нет то создает его и вызывает getLine() у экземпляра Reader.
...
Рейтинг: 0 / 0
20.10.2020, 00:50
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010070
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
graycode,

авторМожно написать класс Reader с парсингом файла в конструкторе и сделать к нему виртуальную проксю LazyReader, которая при инициализации записывает имя файла в приватную переменную, также имеет приватное поле Reader, при вызове getLine() проверяет есть ли экземпляр Reader если нет то создает его и вызывает getLine() у экземпляра Reader.
Круто)))
...
Рейтинг: 0 / 0
20.10.2020, 08:23
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010074
Фотография SQL2008
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
SQL2008
Участник
Выскажу свою точку зрения.
Конечно код в конструкторе писать можно, но есть одно большое НО.
Конструктор это процедура создания объекта и получается, что в случае ошибок выполнения кода сам объект не будет создан.
Мне думается правильно вначале создать объект, пусть практически пустой, а потом уже выполнять некоторые действия с ним.
...
Рейтинг: 0 / 0
20.10.2020, 08:38
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010077
kealon(Ruslan)
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
kealon(Ruslan)
Участник
SQL2008
Выскажу свою точку зрения.
Конечно код в конструкторе писать можно, но есть одно большое НО.
Конструктор это процедура создания объекта и получается, что в случае ошибок выполнения кода сам объект не будет создан.
Мне думается правильно вначале создать объект, пусть практически пустой, а потом уже выполнять некоторые действия с ним.
и какой вам смысл от объекта который не может на данный момент сделать что надо? :-)
...
Рейтинг: 0 / 0
20.10.2020, 08:59
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010080
mad_nazgul
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
mad_nazgul
Участник
kealon(Ruslan)
SQL2008
Выскажу свою точку зрения.
Конечно код в конструкторе писать можно, но есть одно большое НО.
Конструктор это процедура создания объекта и получается, что в случае ошибок выполнения кода сам объект не будет создан.
Мне думается правильно вначале создать объект, пусть практически пустой, а потом уже выполнять некоторые действия с ним.
и какой вам смысл от объекта который не может на данный момент сделать что надо? :-)


Fast Fail однако!

<:o)
...
Рейтинг: 0 / 0
20.10.2020, 09:11
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010089
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
kealon(Ruslan),
Объекта и не будет. Будет инициализация переменной в памяти.
А потом можно рожать полноценный 1000 способов.
У меня в коде противопоказания только на исключение и зависание.
Ну и раздельно удобно переинициализацию делать
ob.init(file1)
ob.init(file2)
...
Рейтинг: 0 / 0
20.10.2020, 09:45
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010091
andreykaT
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
andreykaT
Участник
кажется тут речь о том что вроде как, в спрингбинах и не только спринг в конструктор код не засунуть кроме this.whateverField=whateverField
и юзать можно-нужно типа постконструкт или как он там.
...
Рейтинг: 0 / 0
20.10.2020, 09:50
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010104
kealon(Ruslan)
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
kealon(Ruslan)
Участник
PetroNotC Sharp
kealon(Ruslan),
Объекта и не будет. Будет инициализация переменной в памяти.
А потом можно рожать полноценный 1000 способов.
У меня в коде противопоказания только на исключение и зависание.
Ну и раздельно удобно переинициализацию делать
ob.init(file1)
ob.init(file2)
ну как не будет, память выделена, валидная ссылка на него есть
да и если "потом рожать" чем это отличается от "создать"?

я понимаю что при таком подходе, например в плюсах, можно на этапе копиляции создать объект
в java то какой профит?
...
Рейтинг: 0 / 0
20.10.2020, 10:40
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010105
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
andreykaT,
Да
...
Рейтинг: 0 / 0
20.10.2020, 10:53
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010107
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
kealon(Ruslan),
Дак и разговор про универсальный совет под все ЯП.
Универсальный - объект без райзе и гарантированно создаваемый. Это назначение конструктора.
В java тоже есть заморочки чтобы следовать этому правилу. Не прогер тут создает объект.))))
Тут new давно не пишут
))))
...
Рейтинг: 0 / 0
20.10.2020, 10:56
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010108
kealon(Ruslan)
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
kealon(Ruslan)
Участник
andreykaT
кажется тут речь о том что вроде как, в спрингбинах и не только спринг в конструктор код не засунуть кроме this.whateverField=whateverField
и юзать можно-нужно типа постконструкт или как он там.
вынужденое потокание архитектурному решению спринга понятно, но в чём смысл категорично распространять его на всё?


PetroNotC Sharp
...
Ну и раздельно удобно переинициализацию делать
ob.init(file1)
ob.init(file2)

что мешает сделать и так, и так?
...
Рейтинг: 0 / 0
20.10.2020, 10:57
| Ответить | Цитировать | Написать  
Код в конструкторе
    #40010109
PetroNotC Sharp
Скрыть профиль Поместить в игнор-лист Сообщения автора в теме
PetroNotC Sharp
Участник
kealon(Ruslan)
что мешает сделать и так, и так?
?
...
Рейтинг: 0 / 0
20.10.2020, 10:58
| Ответить | Цитировать | Написать  
25 сообщений из 72, страница 1 из 3
  1  все
Форумы / Java [игнор отключен] [закрыт для гостей] / Код в конструкторе
Целевая тема:
Создать новую тему:
Автор:
Закрыть
Цитировать
Найденые пользователи ...
Разблокировать пользователей ...
Читали тему (0):
Читали форум (0):
Пользователи онлайн (0):
Польз. соглашение
NoSQL.ru, Чат Кролика       Пашэ: Рисунки, Чат       Фотоальбом: Участники, Встречи
Закрыть
start [/forum/topic.php?fid=59&msg=40010077&tid=2120650]:
 0ms
get settings:
 3ms
get forum list:
 5ms
check forum access:
 1ms
check topic access:
 1ms
track hit:
 40ms
get topic data:
 2ms
get forum data:
 1ms
get page messages:
 471ms
get tp. blocked users:
 0ms
others: 6ms
total:  530ms
созд. / загр.: 530ms
x
x
Закрыть


Просмотр
0 / 0
Close
Debug Console [Select Text]