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

групп:

каналов:

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

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

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

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

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

go.backupland.com

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

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

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

Size: a a a

Yii Framework 2

2417 membersпожаловаться на группу
2021 June 10

ДК

Дмитрий Кожанов... in Yii Framework 2
Я придираюсь только потому что он выкатил это в общественность. И как я считаю:
1. Код должен быть максимально описан, что делает функция, что ты получишь и что она может выкинуть
2. У кода должны быть тесты
3. Код должен соответствовать стандартам, принятым PHP сообществом
источник
19:04пожаловаться#1

GV

Gena Voituk in Yii Framework 2
никто тебя не просит это всё расписывать. Достаточно описания передаваемых параметров, что возвращает функция и какие бъёт ошибки. Для этого можешь в шторме после написания функции напечатать /** и нажать enter  - у тебя шторм сам всё заполнит
источник
19:04пожаловаться#2

Д

Дмитрий in Yii Framework 2
ну вот разве-что это..
а так я просто кучу кода перелопатил, и везде комменты врали. в 90% случаев.. даже те же @throws бывает были там, а по факту там уже ничего не thorw-илось
источник
19:04пожаловаться#3

ДК

Дмитрий Кожанов... in Yii Framework 2
Кидаешь issue что комменты кривые
источник
19:05пожаловаться#4

Д

Дмитрий in Yii Framework 2
спасибо, я в курсе.
источник
19:05пожаловаться#5

GV

Gena Voituk in Yii Framework 2
абсолютно согласен
источник
19:05пожаловаться#6

GV

Gena Voituk in Yii Framework 2
тогда почему, с твоих слов, ты это не используешь?
источник
19:05пожаловаться#7

Д

Дмитрий in Yii Framework 2
перечитай мои сообщения. я не то что не использую - я это выкидываю отовсюду, где это возможно.
источник
19:06пожаловаться#8

Д

Дмитрий in Yii Framework 2
это гнилой код по большей части.
источник
19:06пожаловаться#9

GV

Gena Voituk in Yii Framework 2
ещё раз - зачем? Такой код намного читабельнее
источник
19:06пожаловаться#10

ДК

Дмитрий Кожанов... in Yii Framework 2
щас бы симфу гнилой называть )
источник
19:06пожаловаться#11

Д

Дмитрий in Yii Framework 2
такой код часто врёт.
источник
19:08пожаловаться#12

Д

Дмитрий in Yii Framework 2
кто-то что-то написал, потом кто-то переделал. коммент остался
источник
19:08пожаловаться#13

ДК

Дмитрий Кожанов... in Yii Framework 2
Ах да, забыл что код должен стараться быть максимально универсальным. Если PHP  поддерживает Windows, должно быть сделано все чтобы функции работали одинаково везде. (на это и нужны тесты)
источник
19:08пожаловаться#14

Д

Дмитрий in Yii Framework 2
если код "читабельнее" становится благодаря комметариям - это плохой код.
источник
19:09пожаловаться#15

V

Victooor in Yii Framework 2
+ (с) Макконел "Совершенный код"
источник
19:10пожаловаться#16

ДК

Дмитрий Кожанов... in Yii Framework 2
простой пример
Комменты указывают на проблемы свзанные с особенностями работы. И для какой цели добавлен этот код
источник
19:12пожаловаться#17

Д

Дмитрий in Yii Framework 2
ну тут - да.
но то что генерит, например, шторм, повторая то что в заголовке ф-и и так написано - нафиг не нужно.
источник
19:17пожаловаться#18

S

Sergey in Yii Framework 2
Надо когда у тебя большой проект, ты пишешь функцию и шторм тебе сразу подсказывает аргументы этой функции и т.д.
источник
19:19пожаловаться#19

ДК

Дмитрий Кожанов... in Yii Framework 2
Опять же. Универсальность. Штормом пользуются не все.

Это знаешь, как выпустить машину в тираж. Без документации к ней. А херли, все же на машинах ездят - знают как починить.
источник
19:19пожаловаться#20