У меня не "быдлокод" ?

i80586, 90 просмотров

<?php

include_once "db.inc";
$db = NULL;

function set_title( $title )
{
echo "<title>".$title."</title>";
}

function echo_br( $count = 1 )
{
for ($i = 1; $i <= $count; $i++) echo "<br>";
}

function redirect_to( $link )
{
header("Location: ".$link);
}

function connect_to_db()
{
global $db;
$db = mysql_connect( "localhost", "root", "vertrigo" );
mysql_select_db ( $dbname, $db );
}

function is_login()
{
if ( ! $_SESSION['login_id'] || $_SESSION['login_id'] != 1 )
{
redirect_to ( "index.php" );
exit;
}
}

function register_new_user($uname, $upass, $urealname)
{
global $db;
$sql = "Insert Into `users` (`username`, `password`, `realname`) Values ('".$uname."', '".$upass."', ".$uname."', '".$urealname."')";
if ( mysql_query( $sql, $db ) )
return true;
else
return false;
}

function is_user_exists( $username )
{
global $db;
$username = lowcase( $username );
$sql = "Select Count(`id`) From `users` Where Lower(`username`) = '".$username."'";
$res = mysql_query($sql, $db);
if ( mysql_num_rows($res) > 0 )
return true;
else
return false;
}

?>

Примечание:
И где именно быдлость?

Ответы:

abtop-a

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

Анатолий К

классический быдлокод

abtop-a

Быдлость появляется в начале, и заканчивается в конце.

vir2alex

Быдлокод - это код, который пьёт, курит, матерится, а твой код так делает?

стоит иметь ввиду, что все плюсы могут быть перечеркнуты дешевым объективом.

1. начни с форматирования кода. отступы, они нужны.
2. вы используете несколько различных баз на странице? если нет, то пропишите один раз mysql_select_db и не тяните идентификатор соединения по всему коду. (http://php.net/manual/ru/function.mysql-select-db.php)
3. конструкции вида if a return true else return false; лучше заменить на return a; так короче и понятнее. к примеру:
if ( mysql_num_rows($res) > 0 )return true;
else return false;
можно заменить на return mysql_num_rows($res) > 0;
4. зачем нужно вводить функции, состоящие из вызова одной функции? (set_title, redirect_to)
5. вы что-то позиционируете на странице с помощью вывода n-ного количество <br>-ов?! если да, то вам стоит почитать про css. если нет, то зачем вам echo_br?
6. функция переписывается намного короче
function is_login(){
if ( ! $_SESSION['login_id'] || $_SESSION['login_id'] != 1 ) // оставим на вашей совести вот эту проверку.
header("location:index.php" );
}

Bu-Bu

а у меня кот нормальный

Epsiloncool

Существует ряд правил, которые делают код удобным и лаконичным. В вашем случае многие из этих правил были нарушены и не соблюдены.

Артём Шегеда

Чтобы код не стал быдлокодом, начни с того, что программа состоит, как минимум, из логики и представления. Логика - это подключение к бд, проверка прав, создание/удаление пользователей. Представление - это формирование результатов работы той самой логики в виде, удобном для пользователя. Отдели одно от другого, лучше - с использованием классов (PHP это позволяет). Как только ты это сделаешь, код станет менее говняным

14 лет назад

RPI.su - самая большая русскоязычная база вопросов и ответов. Наш проект был реализован как продолжение популярного сервиса otvety.google.ru, который был закрыт и удален 30 апреля 2015 года. Мы решили воскресить полезный сервис Ответы Гугл, чтобы любой человек смог публично узнать ответ на свой вопрос у интернет сообщества.

Все вопросы, добавленные на сайт ответов Google, мы скопировали и сохранили здесь. Имена старых пользователей также отображены в том виде, в котором они существовали ранее. Только нужно заново пройти регистрацию, чтобы иметь возможность задавать вопросы, или отвечать другим.

Чтобы связаться с нами по любому вопросу О САЙТЕ (реклама, сотрудничество, отзыв о сервисе), пишите на почту [email protected]. Только все общие вопросы размещайте на сайте, на них ответ по почте не предоставляется.