Нашли или выдавили из себя код, который нельзя назвать нормальным,
на который без улыбки не взглянешь?
Не торопитесь его удалять или рефакторить, — запостите его на
говнокод.ру, посмеёмся вместе!
<?
function flevel($exp)
{
include_once "config/mysql.php";
$querylevel = "SELECT maxexp FROM level";
$levelquery = mysql_query($querylevel);
while($rowslvl = mysql_fetch_row($levelquery))
{
$levelarr[] = $rowslvl[0];
}
switch(TRUE)
{
case ($exp <= $levelarr[0]):
return $level = "1";
break;
case ($exp <= $levelarr[1]):
return $level = "2";
break;
case ($exp <= $levelarr[2]):
return $level = "3";
break;
case ($exp <= $levelarr[3]):
return $level = "4";
break;
case ($exp <= $levelarr[4]):
return $level = "5";
break;
case ($exp <= $levelarr[5]):
return $level = "6";
break;
case ($exp <= $levelarr[6]):
return $level = "7";
break;
case ($exp <= $levelarr[7]):
return $level = "8";
break;
case ($exp <= $levelarr[8]):
return $level = "9";
break;
case ($exp <= $levelarr[9]):
return $level = "10";
break;
case ($exp <= $levelarr[10]):
return $level = "11";
break;
case ($exp <= $levelarr[11]):
return $level = "12";
break;
/* ... */
case ($exp <= $levelarr[42]):
return $level = "43";
break;
case ($exp <= $levelarr[43]):
return $level = "44";
break;
case ($exp <= $levelarr[44]):
return $level = "45";
break;
case ($exp <= $levelarr[45]):
return $level = "46";
break;
case ($exp <= $levelarr[46]):
return $level = "47";
break;
case ($exp <= $levelarr[47]):
return $level = "48";
break;
case ($exp <= $levelarr[48]):
return $level = "49";
break;
case ($exp > $levelarr[49]):
return $level = "50";
break;
}
}
?>
Пришел в проект по созданию веб игры, смотрю организацию проекта... Жопа там, что папки, что код, что база одно и тоже, каша. Вот нашел одну забавную функцию. Она возвращает уровень персонажа судя по кол-во XP. Вместо того что бы писать 'SELECT `level` FROM `level` WHERE `minexp` <= ' . $exp . ' AND `maxexp` > ' . $exp надо было switch .. case писать. Нету слов.
качество кода, конечно, ниже плинтуса - ни знаний php, ни понимания sql, ни вообще какого-либо мышления.
все выглядит так, будто писано школьником в духе "я в первый раз компы вижу, но я хочу сделать крутую игру! она станет хитом!"
> Вместо того что бы писать 'SELECT `level` FROM `level` WHERE `minexp` <= ' . $exp . ' AND `maxexp` > ' . $exp
ну во-первых, в SQL есть between
а во-вторых - нахера хранить по две границы в каждой строке и геморройно следить за тем, что [i] запись начинается ровно с того, чем закончилась [i-1] (тут и констрейнт то не напишешь), когда можно хранить лишь одно значение и искать
select max(l.level) from levels l where :1 < l.high
Да к так выглядит более читабильнее. К тому же, если есть имя столбца типа "text", "int" или другое зарезервированное слово (ну школьники...) придется писать:
SELECT один, два, `text` FROM ...
Как бы ломает читабельность. Может я не прав. Поправьте если не прав.
> выглядит более читабильнее
Ну вот лично меня они бесят 😉 Из-за них кода не видно, особенно если все это перемешано с ' и " (а в типичном унаследованном пыхокоде без подготовленных запросов они всяко перемешаны). А если еще и таблицы экранируют в стиле select `table`.`field`, `table`.`another_field` то у меня глаза истекают кровью...
> Как бы ломает читабельность.
Тут ты ломаешь читабельность на одном идентификаторе, а там - на всех подряд...
К слову, вот сколько видел кода для других СУБД - почти нигде не экранируют идентификаторы. А все почему? А, походу, потому, что только мускуль считает все типы и прочую муть зарезервированными словами, и не дает их использовать без экранов в качестве имен полей и таблиц. Еще один повод его не любить.
$db->exec("select `id` from `accounts` where `login` = '" . $db->quote($login) . "'");
// vs
$db->prepare("select id from accounts where login = :login");
все выглядит так, будто писано школьником в духе "я в первый раз компы вижу, но я хочу сделать крутую игру! она станет хитом!"
ну во-первых, в SQL есть between
а во-вторых - нахера хранить по две границы в каждой строке и геморройно следить за тем, что [i] запись начинается ровно с того, чем закончилась [i-1] (тут и констрейнт то не напишешь), когда можно хранить лишь одно значение и искать
Товарищи `мускулисты`, объясните мне, `пожалуйста`, зачем `вы` все `идентификаторы` оборачиваете в `волосатые` `кавычки`?
`Но` ведь `не` весь `подобный` код `сгенерен`?
Как бы ломает читабельность. Может я не прав. Поправьте если не прав.
Ну вот лично меня они бесят 😉 Из-за них кода не видно, особенно если все это перемешано с ' и " (а в типичном унаследованном пыхокоде без подготовленных запросов они всяко перемешаны). А если еще и таблицы экранируют в стиле select `table`.`field`, `table`.`another_field` то у меня глаза истекают кровью...
> Как бы ломает читабельность.
Тут ты ломаешь читабельность на одном идентификаторе, а там - на всех подряд...
К слову, вот сколько видел кода для других СУБД - почти нигде не экранируют идентификаторы. А все почему? А, походу, потому, что только мускуль считает все типы и прочую муть зарезервированными словами, и не дает их использовать без экранов в качестве имен полей и таблиц. Еще один повод его не любить.
Такой SQL выглядит читабельно, но AR лучше в смысле читабельности по моему мнению )
Active Record?