В рейтинге участвует:

групп:

каналов:

Виртуальный сервер на SSD - недорого!

Аренда выделенных и виртуальных серверов (VDS/VPS), хостинг, аренда IP-адресов, администрирование, круглосуточная поддержка

qwarta.ru подробнее

Резервное копирование с проверкой на вирусы!!!

Удобный сервис создания резервных копий на любой сервер сети интернет. Отслеживайте изменения, проверяйте на вирусы. Надежно защитите свой бизнес!

go.backupland.com

Выбираете сервер? Любая конфигурация на заказ!

Аренда физических серверов любых конфигураций под любые запросы - 1С бухгалтерия, игровые сервера, нагруженные проекты, интернет-магазины!

qwarta.ru подробнее

Size: a a a

PHP

1623 membersпожаловаться на группу
2019 December 25

D

Daniil in PHP
$_attributes сделал чтобы не городить однотипные гетеры и сетеры
источник
20:09пожаловаться#1

Р

Руслан in PHP
Daniil
$_attributes сделал чтобы не городить однотипные гетеры и сетеры
Да, лучше через магию атрибуты дергать
источник
20:09пожаловаться#2

ЕР

Евгений Ромашкан in PHP
Vyacheslav
Для этого придумали неймспейсы. А в целом много синглтонов усложняет переиспользование кода.
Так речь не о синглтонах, а о функциях и проблеме отсутствия автолоадинга функций из неймспейса
источник
20:09пожаловаться#3

Р

Руслан in PHP
Подход топчик, решишь часть свойств вынести в value-object и все это говно рассыпется
источник
20:10пожаловаться#4

BT

Bohdan Turchyk in PHP
Daniil
ну требуют ище гибче чтобы было
получилось негибко
оверинжиниринг
источник
20:11пожаловаться#5

D

Daniil in PHP
Bohdan Turchyk
получилось негибко
оверинжиниринг
в чем конкретно не гибко?, сейчас сушности не зависят друг от друга
источник
20:12пожаловаться#6

BT

Bohdan Turchyk in PHP
Руслан
Да, лучше через магию атрибуты дергать
лучше не делать сеттеры там, где они не надо
источник
20:12пожаловаться#7

BT

Bohdan Turchyk in PHP
Daniil
в чем конкретно не гибко?, сейчас сушности не зависят друг от друга
у тебя получилось слишком много вещей, которые не нужны в задаче
в частности фабрики и билдеры
источник
20:13пожаловаться#8

КГ

Константин Грачев 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:13пожаловаться#9

NT

Nikolay Turskyi in PHP
Я думаю тут требовали типа, команду, адаптер, и что бы все на Dto ходило туда сюда. Сериалайзер. Да и все.
источник
20:13пожаловаться#10

Р

Руслан in PHP
Bohdan Turchyk
лучше не делать сеттеры там, где они не надо
сетеры не нужны, сделай приватные свойства все, создание сущности через конструктор
источник
20:13пожаловаться#11

Р

Руслан in PHP
Зачем тебе сетеры тут вообще не понятно
источник
20:14пожаловаться#12

NT

Nikolay Turskyi in PHP
Руслан
Зачем тебе сетеры тут вообще не понятно
В сеттерах преобразование идет
источник
20:14пожаловаться#13

D

Daniil in PHP
Руслан
сетеры не нужны, сделай приватные свойства все, создание сущности через конструктор
принимать все свойства через конструктор не лучшее решение
источник
20:15пожаловаться#14

КГ

Константин Грачев in PHP
Daniil
принимать все свойства через конструктор не лучшее решение
Кто сказал?
источник
20:15пожаловаться#15

D

Daniil in PHP
Константин Грачев
Кто сказал?
если их 10 или 20
источник
20:16пожаловаться#16

NT

Nikolay Turskyi in PHP
Daniil
принимать все свойства через конструктор не лучшее решение
Создай VO и кидай в констуктор
источник
20:16пожаловаться#17

D

Daniil in PHP
Nikolay Turskyi
Создай VO и кидай в констуктор
во это класс
источник
20:16пожаловаться#18

КГ

Константин Грачев in PHP
Daniil
если их 10 или 20
Если у тебя их 10 или 20 то у тебя уже другие проблемы
источник
20:17пожаловаться#19

D

Daniil in PHP
Константин Грачев
Если у тебя их 10 или 20 то у тебя уже другие проблемы
ммм тоже верно
источник
20:17пожаловаться#20