Dcms-Social v2.2

Оброк 9 Ноября 2017
kantry, Полностью с тобой согласен.
Отсюда собственно и "каша мнений" вместо дельного совета...))
Как положенно...
Это неправильно(пояснение почему)
Я бы сделал вот так(тоже пояснение)
И все довольны, ибо идёт приятный, конструктивный диалог
А не набор отчаянных выкриков...))

kantry 9 Ноября 2017
olexiy, у всех так, только не ясно чем, и кому это мешает?:-D
А вот то что php теги по всему движку короткие встречаются, никого не задело))

Оброк 9 Ноября 2017
Engine, Ага, а теперь скажи...
Как это всё что ты накатал, связать
С вопросом человека, вокруг которого собственно и завертелся спор...
// ---
olexiy 26 Окт 2017
Это у меня такое отображение или у всех так?
// ---
Даже снимок человека прикреплю , если тебе лень
идти в начало темы, и вместо понять о чём спор
Писать что ты собственно и написал - не к селу не к городу...
И кто тут теперь смешон...))
Внимательнее как то надо...
Снимок_вопрос (16.9Kb)
Рейтинг:  0/0  | Скачано: 64 раз(а)

Zed 8 Ноября 2017
Ребят, хватит флудить пожалуйста

Modules 8 Ноября 2017
Это сообщение ушло париться вместе с автором в баню!

Engine 1 Ноября 2017
Ты смешон, чувак))
1. Зацепился за русский язык, но сам наделал в тексте кучу ошибок.
2. Ты не видел мой код, но уже определил меня в ранг говнокодеров. Ну и кто тут пустослов?)

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

Примеры плохой читабельности кода. Не буду вставлять куски кода, дам ориентиры.

1. Табуляция вместо пробелов. Можно сколько угодно спорить что лучше, но бывали случаи, когда табы ломали мой код.

2.
/info.php строка 16.
Писать условия и их тела в одну строку - признак дурного тона. С первого взгляда я даже не сразу понял, что там 2 условия на одной строке.

3. Тот же файл, строки 79-97
Непонятная каша вообще без форматирования

4. При присвоении значения переменной, необходимо знак присвоения отделать пробелами
Хорошо
$a = 10;
Плохо
$a=10;

В коде в некоторых случаях это игнорируется и даже смешивается.

5. Смешение php и html

Теперь что касается организации алгаритма.
1. В некоторых случаях из таблицы выбираются все поля, когда используется только одно, два

2. Описание обработчиков (форм или других действий клиента) находится как правило в начале файла и выполняется благодаря условию.

Логика должна быть отделена от UI в другой файл. Это позволит работать независимо верстальщику и программисту.
Отчасти это относится и к читабельности кода

3.
/sys/inc/fnc.php
Здесь у меня нет ни каких претензий к автору поста. Все таки этот код писал не он.

В этом файле находится сборник всех функций движка. Так делать нельзя.
Нужно сгруппировать функции по назначению (не знаю как правильно это описать) в класс. Например функции для работы с данными юзера.

И не просто написать
class User {
И сюда запихнуть копипастом функции юзера
}
А создать класс по всем правилам ООП, с зависимостями, внутренними переменными. Желательно использовать namespace. И затем использовать сущности объектов

4. Отсутствие юнит тестов
[i]Добавлено:[

Скиталец 29 Окт 2017
а запросы то нафига переносами порвали они так выполняются дольше можите проверить