Size: a a a

2018 October 31

РИ

Роман Игумнов in Laravel UA
Ребята всем привет. У меня нюбский вопрос, но если его не решить то я думаю что нормально ничего не получится. Есть вот такой вот класс http://prntscr.com/lcm48i

Как я читал раньше все пишут что хорошая практика чтобы класс выполнял только что-то одно.

Так вот я делаю так http://prntscr.com/lcm56d
и вызываю потом так http://prntscr.com/lcm59d
Правильно ли я делаю?
источник

AA

Ann Ali in Laravel UA
А зачем писать свою имплементацию, если в ларавель есть guzzlehttp?
источник

РИ

Роман Игумнов in Laravel UA
Ann Ali
А зачем писать свою имплементацию, если в ларавель есть guzzlehttp?
Я понимаю что есть. Суть в другом. Я хочу попрактиковаться с нуля. Мне нужен ваш совет, как правильно делать
источник

В

Вячеслав in Laravel UA
По сути да, ты ответственность разделил. Но
1. Не знаю зачем тебе писать чистые curl запросы, когда есть guzzle и zttp
2. Не понятно почему метод называется set
3. Если ты уже делаешь класс Curl, то он должен отвечать за http запросы в принципе, а не за конкретное получение информации из новой почты.
4. Применяя принцип Dependency Invertion из SOLID, класс Nova не должен зависить от конкретной реализации Curl, а должен зависить от интерфейса.
источник

М

Митяй in Laravel UA
Ann Ali
А зачем писать свою имплементацию, если в ларавель есть guzzlehttp?
А есть документация ? А тоя так же костыляю (
источник

MA

Maksim Aleshchenko in Laravel UA
Митяй
А есть документация ? А тоя так же костыляю (
источник

М

Митяй in Laravel UA
спс
источник

РИ

Роман Игумнов in Laravel UA
Вячеслав
По сути да, ты ответственность разделил. Но
1. Не знаю зачем тебе писать чистые curl запросы, когда есть guzzle и zttp
2. Не понятно почему метод называется set
3. Если ты уже делаешь класс Curl, то он должен отвечать за http запросы в принципе, а не за конкретное получение информации из новой почты.
4. Применяя принцип Dependency Invertion из SOLID, класс Nova не должен зависить от конкретной реализации Curl, а должен зависить от интерфейса.
О ждал что ты ответишь))) Я по сути просто хочу попробовать как правильно. Т.е правильно ли всегда применять DI ?
источник

М

Митяй in Laravel UA
Роман Игумнов
О ждал что ты ответишь))) Я по сути просто хочу попробовать как правильно. Т.е правильно ли всегда применять DI ?
НЕТ, праильно всегда понимаеть когда ЭТО надо
источник

РИ

Роман Игумнов in Laravel UA
Вячеслав
По сути да, ты ответственность разделил. Но
1. Не знаю зачем тебе писать чистые curl запросы, когда есть guzzle и zttp
2. Не понятно почему метод называется set
3. Если ты уже делаешь класс Curl, то он должен отвечать за http запросы в принципе, а не за конкретное получение информации из новой почты.
4. Применяя принцип Dependency Invertion из SOLID, класс Nova не должен зависить от конкретной реализации Curl, а должен зависить от интерфейса.
По 3 вопросу хочется остановиться подробнее. По 4 ты мне говорил и я так и стараюсь делать.
источник

РИ

Роман Игумнов in Laravel UA
Митяй
НЕТ, праильно всегда понимаеть когда ЭТО надо
Так поясните плиз новичку)
источник

РИ

Роман Игумнов in Laravel UA
3. Если ты уже делаешь класс Curl, то он должен отвечать за http запросы в принципе, а не за конкретное получение информации из новой почты.

Ок. За получение информации должен отвечать как я понял другой класс? Верно?
источник

ИС

Илья Сакович in Laravel UA
Роман Игумнов
3. Если ты уже делаешь класс Curl, то он должен отвечать за http запросы в принципе, а не за конкретное получение информации из новой почты.

Ок. За получение информации должен отвечать как я понял другой класс? Верно?
да
источник

ИС

Илья Сакович in Laravel UA
например, NovaPoshta
источник

ИС

Илья Сакович in Laravel UA
или NovaPoshtaClient
источник

М

Митяй in Laravel UA
Роман Игумнов
Так поясните плиз новичку)
почитай статьи что ЭТО такое, DI.
ЗАчем мне на ломанном языке тебе в 2 строчки пытатьсяэ то объяснить. Умей самообучаться, а мы лишь даем советы
источник

ИС

Илья Сакович in Laravel UA
потому как этот класс будет работать с апи новой почты
источник

РИ

Роман Игумнов in Laravel UA
Митяй
почитай статьи что ЭТО такое, DI.
ЗАчем мне на ломанном языке тебе в 2 строчки пытатьсяэ то объяснить. Умей самообучаться, а мы лишь даем советы
К чему такие сообщение? Просто поговорить? Я и сам могу понять что нужно читать и самообучаться.
источник

РИ

Роман Игумнов in Laravel UA
Илья Сакович
потому как этот класс будет работать с апи новой почты
Спасибо! Как всегда по факту!
источник

ИС

Илья Сакович in Laravel UA
Роман Игумнов
Спасибо! Как всегда по факту!
вот есть либа для работы с апи новой почты
источник