«phpClub» — архив тем ("тредов"), посвящённых изучению PHP и веб-технологий.
Задачка Аноним 2018/10/25 09:45:39  №1284215 1
682HwXy8mSU.jpg (89, 693x392)
392x693
Добрый вечер, аноны. Сделал задачку из шапки, появился вопрос: возможно ли было сделать этот код компактней? У меня чувство, что я где-то проебался. Извиняюсь, если вопрос глупый, язык только начал изучать. Прикреплю код пикчей потому что я долбаеб потому что я сейчас на работе, а ебаный пхп меня почему-то очень увлек и я вместо сна решаю разные задачки. Как убегал не нашел ничего более логичного, чем сделать скрин и перебросить на телефон. Спасибо заранее.
Аноним 2018/10/25 12:57:32  №1284307 2
>>1284215
>возможно ли было сделать этот код компактней?
Возможно. если использовать не array(), а []
Ответы: >>1284377
Аноним 2018/10/25 13:54:12  №1284333 3
>>1284215
Так как ты пишешь, обычно не делают. Есть (нестрогое) правило, что если в массиве больше 2-х элементов, его пишут в столбик, а не в строчку.
Ну и да, советую поставить phpstorm, настроить себе code style: psr-2, а потом вызвать автоформатирование проекта - увидишь как правильно.
Ответы: >>1284377
Аноним 2018/10/25 15:09:25  №1284377 4
>>1284333
Спасибо, попробую
>>1284307
Не знаю почему, но категорически не хочу учить себя этому упрощению, потому писал через array()
Аноним 2018/10/27 16:31:12  №1285491 5
>>1284215

Не, код довольно краток и сокращать дальше уже нечего.

Насчет форматирования - мне кажется, что тесно сгруппированные значения массивов читабельнее. В PSR-1 и 2 (рекомендации по оформлению кода) я не нашел требований к оформлению массивов.

>>1283722

Она позволяет избавиться от написания рутинных запросов, что полезно. Когда у тебя десятки сущностей, это быстро надоедает. Плюс, поддерживает связи между сущностями.

Ну и в сущностях не обязательно делать геттеры/сеттеры для всех полей. А смотреть по ситуации. Может, какие-то поля нельзя менять. Или можно менять, но только одновременно. И т.д.

>>1283713

Ты по одному продукту делаешь вывод обо всех.

>>1283708

Пока что неправильно. У тебя там получаются круглые цифры:

> 12-й месяц выплаты. Долг перед банком :-3576.4368783754.Всего выплачено:60000

А должно быть всего выплачено около 61270.

> if ($open == 0){
> $creditSumm = $creditSumm;
Это можно было не писать, эта строчка ничего не делает.

Функции лучше не выводить результат, а возвращать - тогда тот, кто вызвал, может использовать его как хочет.
Ответы: >>1285500
Аноним 2018/10/27 16:45:28  №1285500 6
>>1285491
>Ты по одному продукту делаешь вывод обо всех.
Они все дерьмовые. CMS - это неюзабельная параша.
Аноним 2018/10/27 17:19:21  №1285515 7
>>1284215

echo join(" ", array_map(function($val){
return $val[array_rand[$val]]
},$poema);

на строку короче
Ответы: >>1285521
Аноним 2018/10/27 17:40:09  №1285521 8
>>1285515

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