LINUX.ORG.RU

Как вам такой Coding Standard?

 


0

0
if (!$result)
    return false;
  
for ($i = 0; $i < 17; $i++)
    if ($myArray[$i] == $value)
    {
        $result[] = $myArray[$i];
        return $result;
    }
    else
        $failed++;

Это я не предлагаю) Это требования заказчика.

★★

Последнее исправление: webmak (всего исправлений: 2)

предвкушая вопрос «как не ужасно» - отвечаю )

if (!$result) {
    return false;
}
  
for ($i = 0; $i < 17; $i++) {
    if ($myArray[$i] == $value) {
        $result[] = $myArray[$i];
        return $result;
    } else {
        $failed++;
    }
}

aol ★★★★★
()
Ответ на: комментарий от aol

задолбали уже со своими веселыми коддинг стандартами) я предпочитаю так-же

webmak ★★
() автор топика

Ужасно, слишком много мутабельности, потом не поймёшь что здесь происходит.

holuiitipun
()
Ответ на: комментарий от post-factum

на какой строке скобочки - дело десятое!
а вот за отсутствие скобочек - да, я бы отрывал не только руки.

aol ★★★★★
()
Ответ на: комментарий от aol

на какой строке скобочки - дело десятое!

2 штуки на одной строке трудноуловимы.

UVV ★★★★★
()
Ответ на: комментарий от aol

предвкушая вопрос «как не ужасно» - отвечаю )

[...]

удваиваю!

но только вместо

$result[] = $myArray[$i];

я бы написал бы

$result []= $myArray[$i];

так как смысл ``[]=`` тянет на один оператор (оператор: «добавить элемент», по аналогии с ``+=`` или с ``>=``)

# P.S.: ну ещё не понятно — откуда у людей такая любовь к ``$i++`` (читаю «Ай — Увеличить!») .. ведь ОЧЕВИДНО что более приятно видеть глазу запись ``++$i`` (читаю «Увеличить Ай!»)

user_id_68054 ★★★★★
()
Последнее исправление: user_id_68054 (всего исправлений: 5)
Ответ на: комментарий от post-factum

Твои половые проблемы здесь мало кого волнуют.

равным счетом, как всем насрать на твои предпочтения относительно стиля

Давай другой пример, если этот для тебя сложный. Чуть нагляднее:

А я бы за скобочки в одну строку руки отрывал. Несмотря на какой-то-там-стандарт.

А я бы за желтые чехлы для телефонов руки отрывал. Несмотря на чьи-то там предпочтения.

reprimand ★★★★★
()
Ответ на: комментарий от post-factum

Всё правильно там. Вырывал бы руки тем, кто не выделяет блоки подобным образом. Если бы все так писали, никакого бы goto fail; не было.

EXL ★★★★★
()
Ответ на: комментарий от EXL

Желаю твоим коллегам бесконечных патронов.

anonymous
()

codding

Это от какого слова образовано? От «source codde?»?

Доколе же в программисты будут идти люди, не осилившие ни русского, ни английского?

Alve ★★★★★
()
Ответ на: комментарий от post-factum

В чём прикол ставить открывающую скобку на отдельной строке? По моему ерунда какая-то. И почему «user_get_rating» а не «get_user_rating»? Нэймспэйсы на коленке?

MrClon ★★★★★
()
Ответ на: комментарий от MrClon

ставить открывающую скобку на отдельной строке

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

Нэймспэйсы на коленке?

Скорее да, чем нет, но тред не об этом.

post-factum ★★★★★
()
Ответ на: комментарий от user_id_68054

$i++ канонiчнее. Не знаю кто как, а я читаю $i++ как «увеличить i на 1», потому-что привычно и узнаваемо, воспринимается как отдельное «слово», а не как конструкция которую надо как-то распарсить.

MrClon ★★★★★
()
Вы не можете добавлять комментарии в эту тему. Тема перемещена в архив.