Это говнокод? Если да, то в чем именно я провинился?

Question

[PO6OT]

Смотрите папку /app/system/load:
http://woonem.tk/download/3.1.2_stable.rar ~ссылка недействительна

Смотрите файл index.html и папку /.inc:
http://woonem.tk/download/2.0.0_stable.rar ~ссылка недействительна

Comments

[Александр Мелеховец]

Вы б его на гитхаб какой залили, а то неудобно ковырять архив, особенно в метро с мобилки. Можете считать пунктом номер 1 к "в чём я провинился" :)

[PO6OT]

Александр: да я сам с мобилки сижу, домой приду, залью

[Сергей delphinpro]

ламповый зип-архивчик :)))
вам только вчера интернет подключили? :))

[PO6OT]

не понял)

[Алексей Уколов]

Ламповый, это .rar :)

[Денис Инешин]

Вот еще, ваш зип-архив с вирусами качать)

[PO6OT]

ок, выложу код так

[PO6OT]

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

[Сергей delphinpro]

Вадим Егоров: github универсальный ), на худой конец pastebin

[PO6OT]

Сергей: ок

[Vitaly Sivkov]

Чувак, не задалбывай людей своими вопросами, почитай книжки для сначала.
А потом иди на github, и смотри как люди пишут код.

[Виталий Хоменко]

Ох черт, давно я такого не видел! C:

[PO6OT]

Виталий Хоменко: такого говнокода или такого фреймворка

[Виталий Хоменко]

Вадим Егоров: То, что вы показали, нельзя назвать фреймворком. Даже говнокодом его назвать сложно, т.к. у вас попросту нет понимания, как именно пишется код. Мой вам совет: приостановите свои работы и почитайте книг.

[PO6OT]

Виталий Хоменко: хорошо

Answer 1

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

Да, это говнокод. А чем провинились я устану перечислять

- $GLOBALS и использоваие суперглобальных массивов
- использование собачки для подавления ошибок
- for($count=0;isset($block[$count]);$count++) и стремные бесконечные циклы вместо foreach
- переменные вида $ppfile и $pppfile, ваш код очень "удобно" читать.
- смешивание php и html и css
- по сути код состоит преимущественно из канкатенации строк... это грустно
- использование закрывающего тега ?>

Comments

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

Вадим Егоров: ну, это как пойти в загул в инязовскую общагу и не взять презервативы.

[PO6OT]

Сергей Протько: я вас не понимаю, честно
ну хотя бы $_GET можно использовать?

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

Вадим Егоров: нет, не напрямую. Обычно из $_GET/$_POST/etc формируют массивы с данными, валидируют их и фильтруют а уже потом юзают. Но не напрямую. Еще неплохой вариант - обертки вроде symfony/http foundation, но тогда уже можно взять какой микро фреймворк на вроде slim или silex.

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

Вадим Егоров: какая разница если выходит куча. Разграничивайте отвественность, выносите html в шаблоны а css в стили, обезопасьте себя от XSS, работы с базой я у вас там не увидел... попробуйте уменьшить вложенность всех этих массивов...

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

я в принципе считаю что глобальные штуки это плохо.

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

Вадим Егоров: да везде, for(;;) - бесконечный цикл с ручным выходом по условию, я уверен что можно сделать чище и проще, но мне лень вникать что вы делаете.

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

Вадим Егоров: ну загуглите...

[Oxoron]

Сергей Протько: Мой наставник говорил: не изучай больше, чем одну вещь за раз, без особой необходимости.
Вадим Егоров: Для ответа на вопрос, чем плох глобалс, отлично подойдет гугл. "Чем плохи глобальные переменные?" Если вкратце - в малых проектах это нормально, в больших можно изменить переменную и не учесть при этом всех последствий.

Answer 2

[trevoga_su]

for($count=0; isset($block[$count]); $count++){
это что за код такой? что он делает? О_о

$app[server][ip]=$_SERVER[REMOTE_ADDR];
почему ты не соблюдаешь синтаксис языка? включи error_reporting(E_ALL) и посмотри сколько у тебя будет ошибок и предупреждений.

$app[server][get]=$_GET;
$app[server][post]=$_POST;
$app[server][cookie]=$_COOKIE;
$app[server][session]=$_SESSION;

ты делаешь сам не понимая что. в этом коде нет никакого смысла. достаточно открыть документация и прочитать 1 раз, что эти массивы - суперглобальны и их НЕ надо никуда присваивать

if(!is_array($block)){
list($name, $path)=explode('=', $block);
$href='http://'.$host.$path;
$return='<a href="'.$href.'"';
if($page==$path) $return.=' style="color: #d42;"';
$return.='>'.$name.'</a>';
}else{

про форматирование кода ты никогда не слышал? почитай, что это такое

в общем типичный говнокод новичка. надо больше мануал читать на первых порах, а писать - меньше.

Comments

[PO6OT]

$app[server][get]=$_GET;
$app[server][post]=$_POST;
$app[server][cookie]=$_COOKIE;
$app[server][session]=$_SESSION;

это на будущее, чтоб когда нужно будет форматировать эти даныые, не пришлось переписывать весь код

[PO6OT]

for($count=0; isset($block[$count]); $count++){
мне это кажется приличным

[PO6OT]

...
if(!is_array($block))
{
list($name, $path)=explode('=', $block);
$href='http://'.$host.$path;
$return=''.$name.'';
}
else
{
...

так нормально?

[thatside]

Вадим Егоров: между переменными, вызовами функций и знаками операторов тоже принято ставить пробелы
не везде, конечно
но хотя бы так:
list($name, $path) = explode('=', $block);
$href = 'http://' . $host . $path;

[PO6OT]

thatside: между знаками конкатенации неудобно ставить пробелы

[thatside]

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

[Евгений]

Вадим Егоров:
вместо for($count=0; isset($block[$count]); $count++)
правильней будет использовать foreach
<?php
foreach($block as $aBlock){
// и т.д.
}

?>

[PO6OT]

Евгений: я знаю. когда я писал этот код, я не знал про foreach. я учился на каких-то электронных говнокнигах на j2me (телефоны такие, по тому, что компа у меня не было, а pdf-ридеры на j2me никто писать не собирался) которые вероятно писали говнокодеры тупее меня и кроме конкатенации, операторов +, -, /, *, конструкций for, if, else, echo и нескольких функций для работы со строками и файлами я знать не мог
это был мой 2012
а как вы проводили этот год?

[Евгений]

Вадим Егоров: мой 2012 год ознаменовался постройкой теплицы на саду и удачным поиском работы в вебстудиях

Answer 3

[Евгений Елчев]

www.ozon.ru/context/detail/id/3159814 - ответит на все ваши по говнокоду сегодня, завтра и через 10.

Comments

[PO6OT]

название напишите

[Евгений Елчев]

Вадим Егоров: Совершенный код. Там же по ссылке есть и описание. Эту книгу нужно читать, прямо обязательно. Причем это не только мое мнение, она во всех списках в топе. Сам купил бумажную, не пожалел ни разу.

[PO6OT]

Евгений Елчев: ссылка на андроиде ведёт на пустую страницу

[PO6OT]

Евгений Елчев: