Как вам такой code-style в PHP?

Question

[Тарас Сереванн]

Привет! Нашел в одной книге по PHP вот такой код:

<?php
$lst = array();
$k = “prostoe”;
for($i = 2; $i<100; $i++) {
for($j = 2; $j < $i; $j++) {
if( ($i % $j) == 0) {
$k=”ne prostoe”;
}
}
if ($k == “prostoe”)
$lst[] = $i;
else
$k = “prostoe”;
}
foreach ($lst as $list) echo $list.”</br>”;
?>

Как вам такой code-style?

Я бы оформил код вот так:

<?php
$lst = array();
$k = 'prostoe'; // я бы использовал true/false, но оставим так, как в оригинале

for($i = 2; $i < 100; $i++) {
    for($j = 2; $j < $i; $j++) {
        if( ($i % $j) == 0) {
            $k = 'ne prostoe';
        }
}

if ($k == 'prostoe') {
    $lst[] = $i;
} else {
    $k = 'prostoe';
}

foreach ($lst as $list) {
    echo $list . '</br>';
}
?>

Как вам мой code-style?

Answer 1

[Сергей Протько]

Я пожалуй просто оставлю это тут: PSR-1, PSR-2

Answer 2

[FanatPHP]

"В одной книге" форматирование просто съелось при верстке.
А в "в твоем коде" нет ничего исключительного, кроме пропущенных скобок.

Comments

[Тарас Сереванн]

а вы автор той самой книги?

Answer 3

[Александр]

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

Comments

[pi314]

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

[Александр]

pi314: да, этим не только школьники балуются в говнокоде

Answer 4

[Александр Кубинцев]

Codestyle формируется не только формальными стандартами, типа PSR, но и благодаря толковому именованию функций, переменных, объектов и т.д.
Поэтому, такие названия как $lst (да ещё и с элементом $list, OMG!), $k - это ужасно.

Answer 5

[65536]

Спросите здесь govnokod.ru/php

Answer 6

[Павел Соловьёв]

думаю в google возьмут даже без собеседований