покритикуйте код

[Духовность™]

$a = 1;

без шуток.

 

[akd]

говнокод детектед

 

[Ragazzo]

Как-то некрасиво, надо так

namespace a;

class A
{
    private static $a;

    public static function a($a)
    {
         self::$a = $a;
    }
}

a/A::a(1);

Вот как-то так надо...

 

[akd]

Ragazzo, прокомментировать/задокументировать еще надо бы

 

[zerkms]

Ragazzo, прокомментировать/задокументировать еще надо бы

И тестами покрыть

 

[флоппик]

Имя переменной должно быть осмысленным. Рядом желателен комментарий, объясняющий, почему по-умолчанию там единица. А так — в целом нормально.

 

[Духовность™]

Имя переменной должно быть осмысленным. Рядом желателен комментарий, объясняющий, почему по-умолчанию там единица. А так — в целом нормально.

молодец =)

 

[Ярослав]

a/A::a(1);

Вот как-то так надо...

Не, вызывать нужно так

\a\A::a(1);

 

[Absinthe]

Волшебные значения не доставляют.


$numberOfTits = Tits::TITS_ARCHERY_SURGICAL_INTERVENTION;

 

[Krishna]

Волшебные значения не доставляют.
$numberOfTits = Tits::TITS_ARCHERY_SURGICAL_INTERVENTION;

+1

Я бы ещё предложил юзать венгернскую нотацию (что сам всегда делаю на пыхе)

$intNumberOfTits = Tits::TITS_ARCHERY_SURGICAL_INTERVENTION;

 

[fixxxer]

А название константы, что, никого не смущает? Нифига ж не ясно, что это число.

 

[Вурдалак]

Венгерская нотация сосёт.

 

[Krishna]

Венгерская нотация сосёт.

Аргументируй уж тогда.

 

[melo]

Согласен, чем так провинилась Венгерская нотация?

 

[Вурдалак]

Венгерская нотация нужна тем, кто не в состоянии держать в голове типы локальных переменных функции, а таким людям вообще противопоказано прикасаться к коду. Для остального есть phpDoc и type hinting. Да и потом, для автокомплита нужно будет сначала набирать тип переменной — это тоже минус.

 

[флоппик]

А еще в пхп все значения приходящие извне, из базы, из урла — строки.

 

[Absinthe]

Аргументируй уж тогда.

Сосет, т.к. дублирование кода: тип и в названии, и в хинте IDE.

из базы

Чё?

 

[alekciy]

Аргументируй уж тогда.

Но до кучи можно добавить классическое "PHP не строго типизированный язык". Исходя из чего уже по определению нет большого смысла выносить в имя переменной её тип.

 

[Krishna]

Но до кучи можно добавить классическое "PHP не строго типизированный язык". Исходя из чего уже по определению нет большого смысла выносить в имя переменной её тип.

Именно для этого и надо выносить. Т.к. IDE его не контролирует, в отличие от строготипизированных - так меньше шансов налажать.

В Java я венгерскую нотацию не использую, т.к. нет необходимости, IDE всё подсвечивает и, главное, типом ошибиться не даст компилятор.

 

[Krishna]

А еще в пхп все значения приходящие извне, из базы, из урла — строки.

Их как раз при присваивании конечным переменным, именованным в венгерской нотации мы приводим к нужному типу. С базой ты, конечно, попутал что-то.