PHP: А вы можете оценить "профессионализм" программиста взглянув на код?

программирование php html perl

Например, мой код...

<?php
echo set_title("Регистрация пользователя");

include MODEL_PATH."register_model.php";


if ( isset($_POST['_regsubmit']) )
{
if (!check_for_empty( $_POST['_realname'], $_POST['_username'], $_POST['_pass1'], $_POST['_answer'] ))
$er_text = "Все поля должны быть заполнены!";
else
{
if ( user_exists($_POST['_username']) )
$er_text = "Такой пользователь уже существует!";
else
{
if ( strcasecmp($_POST['_pass1'], $_POST['_pass2']) != 0 )
$er_text = "Пароли не совпадают!";
else
{
save_new_user($_POST['_realname'], $_POST['_username'], $_POST['_pass1'], $_POST['_answer']);
setcookie("login_id", get_user_id($_POST['_username']));
redirect_to("index.php?module=main");
}
}
}
}


include VIEW_PATH."auth/register.tpl";
?>

Примечание:
<?php
include "db.php";

define("FUNC_PATH", $_SERVER['DOCUMENT_ROOT']."/mybudget_multi/config/" );

define("CONTROL_PATH", $_SERVER['DOCUMENT_ROOT']."/mybudget_multi/application/controller/");
define("MODEL_PATH", $_SERVER['DOCUMENT_ROOT']."/mybudget_multi/application/model/");
define("VIEW_PATH", $_SERVER['DOCUMENT_ROOT']."/mybudget_multi/application/view/");

function connect_to_db( $config )
{
if ( mysql_connect( $config['host'], $config['user'], $config['pass'] ) )
{
mysql_select_db( $config['dbname'] );
mysql_query("SET NAMES `UTF8`");
}
}

function connect_close()
{
mysql_close();
}

?>

Примечание:
это два разных файла, один register_controller.php, а другой вообще config.php
Ответы:
Несложный код - нечего оценивать. Разве что семанику - с ней все ОК.
новичек, но... на верном пути)
А где код?
Оценивать нечего, да и отступы сбились.
А вообще, было более кошерно (раз уж модели MVC привлекаются) выносить строки с сообщениями об ошибках в отдельный файл.
Создаётся файл, где через define определяются всякие MSG_ERR_FIELDS_NOT_FILLED, и в файле с логикой отдаются уже такие константы. Плюсы, например -- несложно добавить поддержку нескольких языков, если потребуется, да и другие.


14 лет назад

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

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

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