Телеграмм чат группы prophp7 страница 9664

Всем привет, делал тестовое, зададача была сделать применяя патерны, тестовое сделал, в фидбеке получил ответ что "низкая гибкостью к изменениям, не продуманные названия классов и методов" кто поможет советом, не знаю как сделать еще гибче решение https://github.com/ldaniil/weather

GitHub

ldaniil/weather

Weather application. Contribute to ldaniil/weather development by creating an account on GitHub.

источник

19:53пожаловаться #7

V

Vyacheslav in PHP

Aleksandr Khristenko

Что плохого может быть в следующем подходе: не используем функции, вместо них используем статические методы классов?

Для этого придумали неймспейсы. А в целом много синглтонов усложняет переиспользование кода.

источник

19:54пожаловаться #8

BT

Bohdan Turchyk in PHP

Daniil

Всем привет, делал тестовое, зададача была сделать применяя патерны, тестовое сделал, в фидбеке получил ответ что "низкая гибкостью к изменениям, не продуманные названия классов и методов" кто поможет советом, не знаю как сделать еще гибче решение https://github.com/ldaniil/weather

GitHub

ldaniil/weather

Weather application. Contribute to ldaniil/weather development by creating an account on GitHub.

сущности на базе массивов с родительским классом - ужас ужас

источник

19:56пожаловаться #9

D

Daniil in PHP

Bohdan Turchyk

сущности на базе массивов с родительским классом - ужас ужас

как надо?

источник

19:59пожаловаться #10

Р

Руслан in PHP

Daniil

как надо?

Зачем пихать педальный ларавель стайл в тестовое?

источник

20:03пожаловаться #11

Р

Руслан in PHP

class Weather
{

источник

20:04пожаловаться #12

BT

Bohdan Turchyk in PHP

Daniil

Всем привет, делал тестовое, зададача была сделать применяя патерны, тестовое сделал, в фидбеке получил ответ что "низкая гибкостью к изменениям, не продуманные названия классов и методов" кто поможет советом, не знаю как сделать еще гибче решение https://github.com/ldaniil/weather

GitHub

ldaniil/weather

Weather application. Contribute to ldaniil/weather development by creating an account on GitHub.

в общем и целом ты слишком упоролся в паттерны
джавой пахнет

источник

20:05пожаловаться #13

D

Daniil in PHP

Bohdan Turchyk

в общем и целом ты слишком упоролся в паттерны
джавой пахнет

ну требуют ище гибче чтобы было

источник

20:06пожаловаться #14

A

Adel in PHP

шли нахер этих требователей и ищи адекватные вакансии

источник

20:06пожаловаться #15

Р

Руслан in PHP

Adel

шли нахер этих требователей и ищи адекватные вакансии

топчик коммент, согласен на 200% 🤣

источник

20:07пожаловаться #16

D

Daniil in PHP

Руслан

Зачем пихать педальный ларавель стайл в тестовое?

"Открывающие фигурные скобки классов НЕОБХОДИМО переносить на следующую строку" это из PSR-2. Стиль кодирования

источник

20:07пожаловаться #17

Р

Руслан in PHP

Daniil

"Открывающие фигурные скобки классов НЕОБХОДИМО переносить на следующую строку" это из PSR-2. Стиль кодирования

Причем здесь скобки? Я про говностайл в энтити

источник

20:08пожаловаться #18

D

Dmitry in PHP

Daniil

Всем привет, делал тестовое, зададача была сделать применяя патерны, тестовое сделал, в фидбеке получил ответ что "низкая гибкостью к изменениям, не продуманные названия классов и методов" кто поможет советом, не знаю как сделать еще гибче решение https://github.com/ldaniil/weather

GitHub

ldaniil/weather

Weather application. Contribute to ldaniil/weather development by creating an account on GitHub.

Мне понравилось, как ты сделал. Все гибко как по мне и норм именование

источник

20:08пожаловаться #19

NT

Nikolay Turskyi in PHP

Daniil

"Открывающие фигурные скобки классов НЕОБХОДИМО переносить на следующую строку" это из PSR-2. Стиль кодирования

Он про $_attributes = []

источник

20:08пожаловаться #20