«phpClub» — архив тем ("тредов"), посвящённых изучению PHP и веб-технологий.
Аноним 2018/10/24 07:36:22  №1283708 1
Омг, Анон, я смог написать эту ебучую функцию с 3 банками. может кто-нибудь глянуть и сказать, насколько код ущербен? https://ideone.com/B0PT91
Ответы: >>1283718 >>1285491
Аноним 2018/10/24 08:09:52  №1283718 2
>>1283708
Зачем ты присваиваешь результат выполнения функции переменным и пытаешься его вывести, если у тебя весь вывод информации осуществляется в функции? Алсо, ставь пробелы между параметрами функции, после управляющих конструкций и перед кавычками, изучи psr-2 хотя бы в рамках описания уже известных тебе языковых конструкций, если хочешь чтобы твой код можно было нормально прочесть.
Ответы: >>1283720
Аноним 2018/10/24 08:18:23  №1283720 3
>>1283718
>Алсо, ставь пробелы между параметрами функции, после управляющих конструкций и перед кавычками, изучи psr-2 хотя бы в рамках описания уже известных тебе языковых конструкций, если хочешь чтобы твой код можно было нормально прочесть.
>
Благодарочка, исправлюсь. А присваиваю, потому что иначе почему-то ideone не хочет выводить ответ, и output будет пуст
Аноним 2018/10/27 16:31:12  №1285491 4
>>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 5
>>1285491
>Ты по одному продукту делаешь вывод обо всех.
Они все дерьмовые. CMS - это неюзабельная параша.