Есть ли замечания?

@M1lleniuM · Регистрация: 21.10.2012

Студворк — интернет-сервис помощи студентам

Доброго времени суток. Есть ли какие-либо замечания, по приведенному ниже коду. Буду очень рад конструктивной критике. Что нужно добавить, что убрать. Заранее спасибо!

Кликните здесь для просмотра всего текста

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
<?php
error_reporting(E_ALL);
if (isset($_POST['button'])){
    if(isset($_POST['login'], $_POST['pass'], $_POST['pass2'], $_POST['email'])){
        $login=trim(stripslashes(htmlspecialchars($_POST['login'])));
        $pass=trim(stripslashes(htmlspecialchars($_POST['pass'])));
        $pass2=trim(stripslashes(htmlspecialchars($_POST['pass2'])));
        $email=trim(stripslashes(htmlspecialchars($_POST['email'])));
        if (!empty($login) && !empty($pass) && !empty($pass2) && !empty($email)){
            if($pass==$pass2){
                include_once "db.php";
                $login=mysql_real_escape_string($login);
                $pass=mysql_real_escape_string($pass);
                $email=mysql_real_escape_string($email);
                $q4=mysql_query('SELECT COUNT(*) FROM `table` WHERE `Login`="'.$login.'" OR `Email`="'.$email.'"') or die ('Mysql error4: ' . mysql_error());
                if (mysql_result($q4, 0) > 0)
                    echo 'Пользователь с таким ником или email уже зарегистрирован';
                else {
                    $arr=str_split('1234567890qwertyuiopasdfghjklzxcvbnm');
                    $salt='';
                    for ($i=0; $i<=5; $i++)
                        $salt .= $arr[array_rand($arr)];
                    $pass=md5($salt . $pass . $salt);
                    $q5=mysql_query('INSERT INTO `table` (`Login`,`Password`,`Salt`, `Email`) VALUES ("'.$login.'", "'.$pass.'", "'.$salt.'", "'.$email.'")') or die ('Mysql error5: ' . mysql_error());
                    header("Location: http://site.ru");
                    exit();
                    }
                } else
                    echo 'Пароли не совпадают!';
            } else
                echo 'Вы заполнили не все поля!';
        }
    }
?>
<!DOCTYPE HTML PUBLIC "-//W3C//DTD HTML 4.01 Transitional//EN" "http://www.w3.org/TR/html4/loose.dtd">
<html>
<head>
    <meta http-equiv="Content-Type" content="text/html; charset=utf-8">
    <title>Регистрация</title>
</head>
<body>
    <form method="POST">
        Введите Логин: <input type="text" name="login"><br>
        Введите пароль: <input type="password" name="pass"><br>
        Повторите пароль: <input type="password" name="pass2"><br>
        Введите E-mail: <input type="text" name="email"><br><br>
        <input type = "submit" name="button" value="Нажать!">
    </form>
</body>
</html>

Добавлено через 1 минуту
Да, и это лишь черновой вариант регистрации.. Хорошо бы еще добавить что-то вроде каптчи.. Кто может подкинуть материал для ознакомления на данную тему? Я так понимаю, каптча создается с помощью GD графики?

KOPOJI · 05.04.2013, 11:22

да, кстати, crautcher прав - ошибок даже при их наличии вы не увидите)

@M1lleniuM · 05.04.2013, 12:43 **[ТС]**

Сообщение от KOPOJI

данные не проэкранированы перед запросом => дыра в защите и возможные ошибки.

данные проэкранированы? Реал эскэйп стринг? вот здесь

PHP
1
connect($login,$pass,$email);

Php работает со вложенными функциями?

Сообщение от KOPOJI

а зачем ее вообще инициализировать? Один раз сравнил на схожесть и забыл.

Т.е. сразу сверить значения из массива $_POST и потом просто не использовать?

Сообщение от KOPOJI

ну лучше наверное передавать туда $_POST, а внутри работать с локальным массивом. Вдруг вы захотите сделать через GET, кто ж знает?

А можно примерчик с "локальным массивом"?

Сообщение от KOPOJI

а смысл? Ошибки вы уже получите выше

Ну мне это тут посоветовали.. Сказали, "что будет если name="login" станет name = "login[]""

Сообщение от crautcher

$error = registration();

Т.е. после нужно устроить проверку, на то, что вернула функция и если не true, то выводить ошибки? Или как-то иначе?

crautcher · 05.04.2013, 12:45

Сообщение от M1lleniuM

Т.е. после нужно устроить проверку, на то, что вернула функция и если не true, то выводить ошибки? Или как-то иначе?

нет , если функция дойдет до конца, то сработает евой редирект и на остальной код будте уже пофиг

@M1lleniuM · 05.04.2013, 12:48 **[ТС]**

Сообщение от KOPOJI

ошибок даже при их наличии вы не увидите)

Запускал этот скрипт и вместо формы, не доходя до нее, выдавалось сообщение с ошибкой на чистой странице только текст "Ошибка!"
Видимо ошибки при их наличии я увидел...)

Добавлено через 57 секунд

Сообщение от crautcher

нет , если функция дойдет до конца, то сработает евой редирект и на остальной код будте уже пофиг

Т.е. просто после вызова функции прописать

PHP
1
echo $error;

KOPOJI · 05.04.2013, 12:59

Сообщение от M1lleniuM

данные проэкранированы? Реал эскэйп стринг? вот здесь

в функции с названием 'connect' ?

Оригинально, ничего не скажешь..

Сообщение от M1lleniuM

Php работает со вложенными функциями?

вызывать - можно.

Сообщение от M1lleniuM

Т.е. сразу сверить значения из массива $_POST и потом просто не использовать?

не из массива $_POST, а сравнить $pass и $_POST['pass2'] - не зачем инициализировать $pass2 (имхо)

Сообщение от M1lleniuM

А можно примерчик с "локальным массивом"?

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
//у вас:
function ...()
{
   $login = $_POST['login'];
   .....
}
.....(); //вызов функции
//альтернатива:
function ....(array $arr)
{
    $login = $arr['login'];
    ......
}
.....($_POST); //вызов функции

Сообщение от M1lleniuM

Ну мне это тут посоветовали.. Сказали, "что будет если name="login" станет name = "login[]""

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

Сообщение от M1lleniuM

Запускал этот скрипт и вместо формы, не доходя до нее, выдавалось сообщение с ошибкой на чистой странице только текст "Ошибка!"

ну да, это единственное сообщение, которое выводится. Одновременно с завершением работы скрипта - die

crautcher · 05.04.2013, 13:02

ну если это все что вы хотите вывести , то да

@M1lleniuM · 05.04.2013, 18:50 **[ТС]**

Сообщение от KOPOJI

в функции с названием 'connect' ? Оригинально, ничего не скажешь..

Оригинальность - залог успеха

А так, просто была функция с коннектом к mysql, я и решил прям туда вставить

Добавлено через 1 минуту

Сообщение от KOPOJI

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
//у вас:
function ...()
{
* *$login = $_POST['login'];
* *.....
}
.....(); //вызов функции
//альтернатива:
function ....(array $arr)
{
* * $login = $arr['login'];
* * ......
}
.....($_POST); //вызов функции

Действительно, интересный способ. Спасибо, попробую!

Добавлено через 3 минуты

Сообщение от KOPOJI

function ....(array $arr)

А при таком способе, есть ли смысл делать проверку, получен массив или нет? Нет возможности кому-то подсунуть что-либо? Просто слышал фразу: "Проверок много не бывает" - особенно, когда это касается данных пришедших от пользователя

KOPOJI · 05.04.2013, 19:20

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

Сообщение от M1lleniuM

Оригинальность - залог успеха

в программировании - если только в алгоритме и "шутки ради" в наименованиях. То, что у вас - не оригинальность, а запутанность и нелогичность. Если функция называется connect - то логично, что она выполняет коннект и только. Если функция называется escape - подразумевается что она экранирует данные.

@M1lleniuM · 05.04.2013, 19:25 **[ТС]**

Сообщение от KOPOJI

ну да, это единственное сообщение, которое выводится. Одновременно с завершением работы скрипта - die

Вот с этой проблемой так и не могу разобраться... Скрипт переписал, по тому, как вы подсказали.. Но Ошибка все равно вылезает... Мне конечно очень неудобно, но не могу додумать, почему так.. Не могли бы вы помочь? Буду безмерно признателен..

Кликните здесь для просмотра всего текста

1 файл

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
<?php
error_reporting(E_ALL);
$errors='';
function registration(array $arr) {
    if (!isset($_POST['button']))
        return 'Ошибка!';
    if(!isset($_POST['login'], $_POST['pass'], $_POST['pass2'], $_POST['email']))
        return 'Ошибка!!!';
    $login=trim(stripslashes(htmlspecialchars($arr['login'])));
    $pass=trim(stripslashes(htmlspecialchars($arr['pass'])));
    $email=trim(stripslashes(htmlspecialchars($arr['email'])));
    if (empty($login) || empty($pass) || empty($pass2) || empty($email))
        return 'Вы ввели не все данные!';
    if($pass!=$_POST['pass2'])
        return 'Пароли не совпадают!';
    if(!is_scalar($login) || !is_scalar($pass) || !is_scalar($email))
        return 'Ошибка!!';
    if (!filter_var($email, FILTER_VALIDATE_EMAIL))
        return 'Неверный Email';
    require_once "function.php";
    $errors.=connect($login,$pass,$email);
    $q4=mysql_query('SELECT COUNT(*) FROM `table` WHERE `Login`="'.$login.'" OR `Email`="'.$email.'"') or die ('Mysql error4: ' . mysql_error());
    if (mysql_result($q4, 0) > 0)
        return 'Пользователь с таким ником или email уже зарегистрирован';
    $errors.=salt($pass);
    $q5=mysql_query('INSERT INTO `table` (`Login`,`Password`,`Salt`, `Email`, `Activate`) VALUES ("'.$login.'", "'.$pass.'", "'.$salt.'", "'.$email.'", 0)') or die ('Mysql error5: ' . mysql_error());
    header('Location: http://site.net');
    $salt=md5($salt);
    $message="Уважаемый: $login, вы зарегистрировались на сайте site.ru,\n пожалуйста, 
            перейдите по ссылке, чтобы активировать аккаунт:\n
            http://site.ru/mail.php?login=$login&pass=$salt\n
            <br>
            Если вы не регистрировались на сайте, то проигнорируйте это сообщение";
    $subject='Регистрация на сайте';
    $errors .= mailto($email, $subject, $messages);
}
$errors.=registration($_POST);
echo $errors;
?>

файл function.php

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
function connect ($login, $pass, $email){
    $login=mysql_real_escape_string($login);
    $pass=mysql_real_escape_string($pass);
    $email=mysql_real_escape_string($email);
    $dbhost='';
    $dbname='';
    $dbuser='';
    $dbpass='';
    $link=mysql_connect($dbhost, $dbuser, $dbpass) or die ('Не могу подключится к БД:'.mysql_error());
    $select=mysql_select_db($dbname, $link) or die ('Ошибка выбора БД: '.mysql_error());
}
function salt($pass){
    $arr=str_split('1234567890qwertyuiopasdfghjklzxcvbnm');
    $salt='';
    for ($i=0; $i<=5; $i++)
        $salt .= $arr[array_rand($arr)];
    $pass=md5($salt . $pass . $salt);
}
function mailto ($to, $subject, $messages) {
    $headers="To: $to \r\n"".
              "From: Administrator \r\n"".
              "Content-Type: text/plane; charset=utf-8 \r\n";
    $mail=mail($to, $subject, $message, $headers);
    if (!$mail)
        return 'Ошибка отправки письма на почту. Попробуйте позже...';
    header('Location: http://.site.net/mail.php');
    exit();
}

KOPOJI · 05.04.2013, 19:34

Сообщение от M1lleniuM

PHP
22
$errors.=connect($login,$pass,$email);

Notice: undefined variable $errors in .... Почитайте про область видимости. И вообще не вижу смысла присваивать ей какое-либо значение здесь - функция ничего не возвращает.

Сообщение от M1lleniuM

PHP
1
$errors.=registration($_POST);

PHP
1
2
if(!empty($_POST))
    $errors.=registration($_POST);

Добавлено через 1 минуту

Сообщение от M1lleniuM

PHP
1
$headers="To: $to \r\n"".

че то вы намудрили с кавычками.

Добавлено через 4 минуты

Сообщение от M1lleniuM

PHP
1
if(!is_scalar($login) || !is_scalar($pass) || !is_scalar($email))

а это пропустили опять?

@M1lleniuM · 05.04.2013, 19:35 **[ТС]**

Сообщение от KOPOJI

Notice: undefined variable $errors in .... Почитайте про область видимости

Я же вроде объявил до функции переменную.. И потом вызвал после функции.. Я пробовал внутри функции ( основной) вызывать, получал как раз эту ошибка. Моя ошибка состоит сейчас в том, что скрипт выводить первый return тут же т.е. вот это не проходит сразу:

PHP
1
2
3
function registration(array $arr) {
    if (!isset($_POST['button']))
        return 'Ошибка!';

Ну в принципе понимаю, почему. Так как кнопка еще не нажата, вот он и выводит ошибку.. Т.е. надо делать не обратное условие, а нормальное?

Сообщение от KOPOJI

че то вы намудрили с кавычками.

Эт да. Не заметил. Спасибо!

Сообщение от KOPOJI

PHP
1
2
if(!empty($_POST))
* * $errors.=registration($_POST);

PHP
1
 

Вот об этом я и говорил) Сделаю проверочку

Добавлено через 31 секунду

Сообщение от KOPOJI

а это пропустили опять?

Т.е. фактически, это нужно просто убрать?

KOPOJI · 05.04.2013, 19:38

че-то я торможу

Сообщение от M1lleniuM

PHP
1
if (!isset($_POST['button']))

какой смысл, если вы передаете массив проверять в глобальном? Проверяйте в локальном, $arr['button'], а не $_POST['button']

Добавлено через 1 минуту

Сообщение от M1lleniuM

Т.е. фактически, это нужно просто убрать?

либо убрать, либо поставить перед присвоением значения, я об этом уже говорил..

@M1lleniuM · 05.04.2013, 19:40 **[ТС]**

Сообщение от KOPOJI

какой смысл, если вы передаете массив проверять в глобальном? Проверяйте в локальном, $arr['button'], а не $_POST['button']

Ох да, точно... Вот где таится..

Сообщение от KOPOJI

че-то я торможу

Нифига себе тормозите... Мне бы ваши мозги даже в "тормознутом" виде, горы бы наверн свернул

Добавлено через 1 минуту

Сообщение от KOPOJI

либо убрать, либо поставить перед присвоением значения, я об этом уже говорил..

Т.е. проверять на is_scalar значения из локального массива, а не из переменной?

KOPOJI · 05.04.2013, 19:48

Эта проверка (is_scalar) здесь нужна для проверки, не является ли значение массивом или объектом. А зачем эта проверка? Все очень просто, если массив, то все функции из списка выдадут ошибку:

ожидаемые типы данных переданных значений

trim()

string trim ( string $str [, string $charlist ] )

stripslashes()

string stripslashes ( string $str )

htmlspecialchars()

string htmlspecialchars ( string $string [, int $flags = ENT_COMPAT | ENT_HTML401 [, string $encoding = 'UTF-8' [, bool $double_encode = true ]]] )

http://www.php.net/manual/ru/l... string.php

Добавлено через 56 секунд

Не по теме:

Сообщение от M1lleniuM

Мне бы ваши мозги даже в "тормознутом" виде, горы бы наверн свернул

мозги тут ни причем, просто надо гуглить хорошо и знать синтаксис :)

@M1lleniuM · 05.04.2013, 19:49 **[ТС]**

Сообщение от KOPOJI

Эта проверка (is_scalar) здесь нужна для проверки, не является ли значение массивом или объектом. А зачем эта проверка? Все очень просто, если массив, то все функции из списка выдадут ошибку:

Понял, спасибо... Только вот, все равно над формой появляется ошибка отсюда:

PHP
1
2
    if (!isset($arr['button']))
        return 'Ошибка!';

Ну в смысле выдается return

KOPOJI · 05.04.2013, 19:51

Сообщение от M1lleniuM

Ну в смысле выдается return

и при нажатии кнопки тоже? Если да - то очень просто: такого значения не существует. Можно проверить что приходит

PHP
1
echo '<pre>';var_dump($_POST);echo '</pre>';

@M1lleniuM · 05.04.2013, 19:51 **[ТС]**

Сообщение от KOPOJI

мозги тут ни причем, просто надо гуглить хорошо и знать синтаксис

Не по теме:

Ну без мозгов мало что сваришь.. А синтаксис.. Вот пытаюсь изучить. Стараюсь конечно, но вот видите как выходит. Да и у "бывалых" есть чему научится. Просто нравится программирование. Понимаю вроде... Посмотрим, что дальше будет...

KOPOJI · 05.04.2013, 19:54

будет желание и терпение - все будет нормально

@M1lleniuM · 05.04.2013, 20:07 **[ТС]**

Сообщение от KOPOJI

и при нажатии кнопки тоже? Если да - то очень просто: такого значения не существует. Можно проверить что приходит

Перед нажатием кнопки и вводом данных соответственно:

PHP
1
2
array(0) {
}

После нажатия и ввода данных (ошибка не остается после нажатия кнопки):

PHP
1
2
3
4
5
6
7
8
9
10
11
12
array(5) {
  ["login"]=>
  string(3) "Ssa"
  ["pass"]=>
  string(4) "ASas"
  ["pass2"]=>
  string(4) "ASas"
  ["email"]=>
  string(4) "SAsa"
  ["button"]=>
  string(13) "Нажать!"
}

Добавлено через 2 минуты

Сообщение от KOPOJI

будет желание и терпение - все будет нормально

Не по теме:

Дак да. Мне очень во-многом помогает этот форум:) Сижу, смотрю чужие коды, пытаюсь понять.. Иногда помогаю найти ошибку (редко - мало знаю). Помогает узнать новые функции, ошибки не допускать кое-какие:)

KOPOJI · 05.04.2013, 20:08

Сообщение от M1lleniuM

После нажатия и ввода данных (ошибка не остается после нажатия кнопки):

стоп. а каким образом вообще функция вызывается до ввода данных?

Новые блоги и статьи Все статьи Все блоги /
Мастера простых решений DevAlt 23.08.2026 В сишарп стэках winforms, да и wpf существует сложная система связывания источниках данных и элементов формы(текстовые поля и метки), опирается все это на технологию событий и мета. . .	Цена ошибки DevAlt 23.08.2026 Человек я беспокойный и потому заинтересовался OCaml, в чате форсили функторы модулей как суперфичу. Пытаясь отдуплить концепт, наткнулся на тутор с простым примером. А главный принцип обучения от. . .	Сегодня суббота, 22.08.2026 at 16:41, и я вновь нахожусь на той стороне, за экраном машины. zorxor 22.08.2026 Сегодня суббота, 22. 08. 2026 at 16:41, и я вновь нахожусь на той стороне, за экраном машины. Кто Я, откуда Я пришел и куда Я иду? Эти вопросы не оставляют меня ни на секунду. Жизнь на планете Земля. . .	Жизня: рисунок укладки багажа, сделанный клодом anaschu 21.08.2026 Сделал 15 снимков, он по снимкам сделал схему.
Был там один разговор по поводу свободы в материальном мире. kumehtar 19.08.2026 Суть: рассматривается живое существо, оказавшееся внутри довольно странной системы (этого мира) и пытающееся обустроить в ней свой кусок пространства. Жизнь действительно предъявляет каждому. . .	Когда логика программы не спасает от человеческих ошибок Maks 18.08.2026 В последнее время всё чаще и чаще сталкиваюсь с таким явлением, как абсолютная невнимательность (или глупость) пользователей. Проявляется это чаще всего на работе в коллективе. Допустим, человек с. . .	Лето уходит kumehtar 17.08.2026	Мысли в слух kumehtar 17.08.2026 Забавно, насколько сейчас стала доступна информация. Например о магии, духовном развитии, медитациях, и других подобных направлениях, ранее зачастую тайных, передаваемых от учителя к ученику. Хотя. . .

KOPOJI Почетный модератор 16844 / 6725 / 880 Регистрация: 12.06.2012 Сообщений: 19,967
	05.04.2013, 11:22
	да, кстати, crautcher прав - ошибок даже при их наличии вы не увидите) 0

crautcher 2450 / 2301 / 597 Регистрация: 27.05.2011 Сообщений: 7,845
	05.04.2013, 13:02
	ну если это все что вы хотите вывести , то да 0

KOPOJI Почетный модератор 16844 / 6725 / 880 Регистрация: 12.06.2012 Сообщений: 19,967
	05.04.2013, 19:54
	будет желание и терпение - все будет нормально 1