Bymas.ru | Оцените начинания

Вот второй день учу php и mysql. Оцените регистрацию, ткните если что то по колхозному (не хочу как RUS пилить дополнения к кх :D)

<?php

require "db.php";

$data = $_POST;

if( isset($data['reg'])) 

{

	

	$errors = array();

	if( trim($data['login']) == '') {

		$errors[] = 'Вы ввели пустой логин';

	}

		if( trim($data['email']) == '') {

		$errors[] = 'Вы ввели пустой email';

	}

		if( $data['password'] == '') {

		$errors[] = 'Вы ввели пустой пароль';

	}

	if (R::count('user', "login = ?", array($data['login'])) > 0 ) {

		$errors[] = 'Данный логин уже используется</div>';

	}

	if (R::count('user', "email = ?", array($data['email'])) > 0 ) {

		$errors[] = 'Данный email уже используется</div>';

	}

	if( empty($errors) )

	{

		$user = R::dispense('user');

		$user->login = $data['login'];

		$user->email = $data['email'];

		$user->password = $data['password'];

		R::store($user);

		echo '<div style="color: green;">Успешная регистрация</div><hr>';

	} else 

	{

		echo '<div style="color: red;">'.array_shift($errors).'</div><hr>';

	}

}

include ('regad.html');

?>

Изображение

------

57946_2.png (164.5 Kb)
Скачиваний: 100

________
посл. ред. 30.07.2018 в 19:42; всего 1 раз(а); by Tocix

Ну и сам regad.html

<!DOCTYPE html>

<meta charset="utf-8">

<html>

<style>

body {

	background-color: #E7EFF0;

	font-size: 14px ;

	margin: 3

	max-width:650px;

	#width: 100%;

	word-wrap: break-word;



</style>

<body><form name="login" method="post" action="reg.php">

<p>

	<p><strong>Придумайте логин</strong></p>

	<input type="text" name="login" size="16" value="<?php echo @$data['login']; ?>">

</p>

<p>

	<p><strong>Введите свой email</strong></p>

	<input type="email" name="email" size="16" value="<?php echo @$data['email']; ?>">

</p>

<p>

	<p><strong>Создайте надежный пароль</strong></p>

	<input type="password" name="password" size="16">

</p>

<p>

	<button type="submit" name="reg">Регистрация</button>

</p>

</form>

</body>

</html>

Честно, бред , начиная с вестки, зачем css пихать в html? <?php echo @$data['email']; ?> - что это? зачем эхо тут? <?=$data['email']?> , много лишних if. Ну это моё мнение.

пытался зашифровать пароль через password_hash но что то траблы были

Byte, Это же всего лишь простая регистрация, я вообще не понимаю что вы пишите.
условия, лишь что бы избежать и вывести ошибки

все тут норм :)
Удачи!

[Brony] pimnik98 (30.07.2018 в 20:01)
все тут норм :)
Удачи!

Спасибо. Я понимаю, что есть над чем работать)) много условий прописал)))

Работает - не трогай:) так надо делать, а качество кода и т.д. до одного места если пилишь для себя!

Tocix (30.07.2018 в 20:07)
[Brony] pimnik98 (30.07.2018 в 20:01)
все тут норм :)
Удачи!

Спасибо. Я понимаю, что есть над чем работать)) много условий прописал)))

у меня на дня было было больше из-за 3х проверок. :D

LDeV Prod, Спасибо