Size: a a a

2021 March 03

ПГ

Павел Г. in symfony
Ivan
Там изначально кто-то говорил что 30+ сетеров это плохо. Вот и возник вопрос почему человек так думает. Если данные неотделимы друг от друга то хоть 255 если того требует логика
Просто нет тут логики.
источник

ПГ

Павел Г. in symfony
В основном когда поднимается это разговор, у людей зачастую есть бизнес процессы. А в вашем примере их нет
источник

ЕР

Евгений Ромашкан... in symfony
Ivan
Там изначально кто-то говорил что 30+ сетеров это плохо. Вот и возник вопрос почему человек так думает. Если данные неотделимы друг от друга то хоть 255 если того требует логика
Во первых логика обычно таки не требует править 30 полей одновременно
Во вторых, скорее всего это неудачная логика, которую нужно декомпозировать
источник

JK

Jeka Kovtun in symfony
Ivan
Например есть UserProfile где есть рост,вес, цвет глаз и т.д. Какой смысл такие данные разделять на несколько сущностей?
Например есть смысл их вынести чтоб не вычитывать этот мусор из БД каждый раз когда приходит запрос и тебе нужно его авторизовать, тебе нужно только ID и признак не забанен ли пользователь. Но нет, тащим 50+ полей.

А если у тебя 100K rps,  и 10 лямов пользователей, то дальше сам понимаешь как это круто всё будет работать.
источник

ЕР

Евгений Ромашкан... in symfony
Ivan
Там изначально кто-то говорил что 30+ сетеров это плохо. Вот и возник вопрос почему человек так думает. Если данные неотделимы друг от друга то хоть 255 если того требует логика
А, ну и в третьих - никакая логика не требует сеттеры
источник

ПГ

Павел Г. in symfony
Евгений Ромашкан
Во первых логика обычно таки не требует править 30 полей одновременно
Во вторых, скорее всего это неудачная логика, которую нужно декомпозировать
А вот смысл декомпозироваться ради декомпозиции? Чтобы потом join тащить?
источник

ЕР

Евгений Ромашкан... in symfony
Павел Г.
А вот смысл декомпозироваться ради декомпозиции? Чтобы потом join тащить?
Чтобы связность кода уменьшить
источник

ЕР

Евгений Ромашкан... in symfony
Если ты декомпозировал так что тебе джоин нужен, то ты не декомпозировал, поздравляю
источник

ПГ

Павел Г. in symfony
Евгений Ромашкан
Чтобы связность кода уменьшить
Связность какого кода, если это "Свойства пользователя", у которого нет логики. Если эти части требуются в разных местах - то согласен. А если нет, то не вижу смысла
источник

ЕР

Евгений Ромашкан... in symfony
Павел Г.
Связность какого кода, если это "Свойства пользователя", у которого нет логики. Если эти части требуются в разных местах - то согласен. А если нет, то не вижу смысла
Смотря какие свойства. Я уже писал выше - данные для аутентификации/авторизации обычно удобно таки вынести отдельно
источник

ЕР

Евгений Ромашкан... in symfony
Контактные данные - тоже
источник

ЕР

Евгений Ромашкан... in symfony
Если у тебя прям 30 полей для которых ничего кроме crud формочки не нужно, можно и одной сущностью, чего б нет
источник

ПГ

Павел Г. in symfony
Евгений Ромашкан
Смотря какие свойства. Я уже писал выше - данные для аутентификации/авторизации обычно удобно таки вынести отдельно
Ну я думаю мы поняли друг друга, об одном и том же скорее всего говорим.
источник

ПГ

Павел Г. in symfony
Евгений Ромашкан
Если у тебя прям 30 полей для которых ничего кроме crud формочки не нужно, можно и одной сущностью, чего б нет
Вот я про это
источник

ПГ

Павел Г. in symfony
ПРичем если в одном месте
источник

ПГ

Павел Г. in symfony
Если разные места - скорее всего уже разное по смыслу, и надо делить
источник

I

Ivan in symfony
Евгений Ромашкан
Смотря какие свойства. Я уже писал выше - данные для аутентификации/авторизации обычно удобно таки вынести отдельно
Фразу "Если данные неотделимы друг от друга" ты понимаешь? Зачем пытаешь придумать плохие варианты и на них сделать пример?
источник

ЕР

Евгений Ромашкан... in symfony
Ivan
Фразу "Если данные неотделимы друг от друга" ты понимаешь? Зачем пытаешь придумать плохие варианты и на них сделать пример?
Частенько видно людей для которых "все что связано с пользователем не отделимо от User"
источник

ЕР

Евгений Ромашкан... in symfony
В этом проблема
источник

ЕР

Евгений Ромашкан... in symfony
Если у тебя там не crud, а есть логика, опять же, её можно пересмотреть
источник