Форма регистрации и логина по видео урокам - все равно не работает

@unicum · Регистрация: 25.03.2012

Братва помогите по форме логина , не могу найти ошибку выходить это

Fatal error: Can't use function return value in write context in C:\xampp\htdocs\logreg\login.php on line 16
то есть на строку $errors() = 'You need to enter a username and password';

мои файлы
это логин файл

PHP

<?php
include 'core/init.php';
 
/*
if (user_exists('alex') === true) {
    echo 'Exists';
}
die ();
*/
 
if (empty($_POST) === false) {
    $username = $_POST['username'];
    $password = $_POST['password'];
    
    if (empty($username) === true || empty($password) === true){
        $errors() = 'You need to enter a username and password';
    } else if (user_exists($username) === false) {
        $errors() = 'We can\'t find that username. Have you registered?';
    }
}
?>

init.php

PHP

<?php
session_start();
error_reporting();
 
require 'database/connect.php';
require 'functions/general.php';
require 'functions/users.php';
 
$errors = array();
?>

connect.php

PHP

<?php
$connect_error = 'Sorry, we\'re experiencing connection problems';
mysql_connect('localhost','root','') or die($connect_error);
mysql_select_db('lr');
?>

general.php

PHP

<?php
function sanatize($data){
    return mysql_real_escape_string($data);
}
?>

users.php

PHP

<?php
function user_exists($username) {
    $username = sanatize($username);
    $query = mysql_query("SELECT COUNT ('user_id') FROM 'users' WHERE 'username' = '$username'");
    return (mysql_result($query, 0) == 1) ? true : false;
}
 
 
?>

за любую помощь буду благодарен!

KOPOJI · 22.07.2012, 03:01

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

$errors() = 'You need to enter a username and password';

я так понимаю там нужно квадратные скобки [] а не круглые

Добавлено через 1 минуту
или вообще без них - смотря как вы хотите, в массив или в обычную переменную

@pyramida · 22.07.2012, 04:04

А я вообще не опнимаю зачем заносить текст в массив если его нужно просто выдать пользователю?

@pyramida · 22.07.2012, 04:04

А я вообще не понимаю зачем заносить текст в массив если его нужно просто выдать пользователю?

KOPOJI · 22.07.2012, 12:47

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

А я вообще не понимаю зачем заносить текст в массив если его нужно просто выдать пользователю?

1. использование сессий/кук/хедеров - аналог "частичного" накапливания вывода в переменную.
2. можно проверить - если массив пустой, то значит все ок, иначе - выводим не пустые значения.
3. удобнее - все ошибки при наличии хранятся в одном месте, не надо их искать

Добавлено через 10 минут
и сказать что этот скрипт слегка уязвим - это еще очень мягко. скрипт просто сплошная дыра %)
p.s. кстати, функция empty() решается одним пробелом в поле ввода - это если переменную проверять, а сам массив $_POST при нажатии на кнопку формы (независимо от того, писали что в полях ввода или нет) сразу станет не пустым...

@pyramida · 22.07.2012, 16:17

KOPOJI, 1,2,3 это все относительно. Конечно не спорю у меня маловато опыта в серьезных проектах, может быть там такое и применяется. Сами знаете что лучше посмотреть ТЗ чтобы понять что больше половины кода просто не нужны

Вам знаком метод KISS надеюсь?

KOPOJI · 22.07.2012, 17:04

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

1,2,3 это все относительно.

я и не говорил что это обязаловка

просто если допустим вместо

PHP

1	echo "<html> .... ";

накапливать в переменную этот вывод то потом не будет проблем с отправкой заголовков в середине документа (или ob_start() прописать). ну это, опять же, кому как удобнее.
а по поводу кода - там вообще все лишнее, толку то с такой регистрации все равно нет

пробелов понаставь в форме - и ты уже зарегился)

Не по теме:

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

Вам знаком метод KISS надеюсь?

к сожалению, нет.. можно поподробнее о нем?)

Vovan-VE · 22.07.2012, 17:07

Не по теме:

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

Вам знаком метод KISS надеюсь?

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

к сожалению, нет.. можно поподробнее о нем?)

Keep It Simple (http://ru.wikipedia.org/wiki/K... %B8%D0%BF))

KOPOJI · 22.07.2012, 17:22

Не по теме:

ахахахх :D жестко и правдиво :D

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

PHP

<?php
 session_start();
 if(isset($_POST['go'])) {
//connect to bd
    $login = mysql_real_escape_string(stripslashes(trim($_POST['login'])));
    $pass = mysql_real_escape_string(stripslashes(trim($_POST['pass'])));
if(!empty($login)&&!empty($pass)) {
    $query = sprintf("SELECT id FROM `users` Where `login`='%s' and `pass`='%s'",$login,$pass);
    $res = mysql_query($query);
if($res) {
if(mysql_num_rows($res)) {
// все ок, есть такой - ставим сессию
}
else echo "Пользователь с таким логином не найден, проверьте правильность введенных данных";
}
else echo "В данный момент сервер не доступен. Попробуйте позднее";
}
else echo "Вы заполнили не все поля!";
}
?>
<form method=post>
Login: <input type=text name=login />
Pass:  <input type=password name=pass />
<input type=submit name=go value="Поехали!" />
</form>

Добавлено через 1 минуту
если по простому - то нормально, без всяких лишних ваших функций с одной строчкой в виде возврата обработанного значения

и коду всего 25-30 строчек..

KOPOJI Почетный модератор 16842 / 6720 / 880 Регистрация: 12.06.2012 Сообщений: 19,967
	22.07.2012, 03:01	2
	Сообщение от unicum $errors() = 'You need to enter a username and password'; я так понимаю там нужно квадратные скобки [] а не круглые Добавлено через 1 минуту или вообще без них - смотря как вы хотите, в массив или в обычную переменную 0

@pyramida 91 / 91 / 13 Регистрация: 14.07.2012 Сообщений: 539
	22.07.2012, 04:04	3
	А я вообще не опнимаю зачем заносить текст в массив если его нужно просто выдать пользователю? 0

KOPOJI Почетный модератор 16842 / 6720 / 880 Регистрация: 12.06.2012 Сообщений: 19,967
	22.07.2012, 12:47	5
	Сообщение от pyramida А я вообще не понимаю зачем заносить текст в массив если его нужно просто выдать пользователю? 1. использование сессий/кук/хедеров - аналог "частичного" накапливания вывода в переменную. 2. можно проверить - если массив пустой, то значит все ок, иначе - выводим не пустые значения. 3. удобнее - все ошибки при наличии хранятся в одном месте, не надо их искать Добавлено через 10 минут и сказать что этот скрипт слегка уязвим - это еще очень мягко. скрипт просто сплошная дыра %) p.s. кстати, функция empty() решается одним пробелом в поле ввода - это если переменную проверять, а сам массив $_POST при нажатии на кнопку формы (независимо от того, писали что в полях ввода или нет) сразу станет не пустым... 0

@pyramida 91 / 91 / 13 Регистрация: 14.07.2012 Сообщений: 539
	22.07.2012, 16:17	6
	KOPOJI, 1,2,3 это все относительно. Конечно не спорю у меня маловато опыта в серьезных проектах, может быть там такое и применяется. Сами знаете что лучше посмотреть ТЗ чтобы понять что больше половины кода просто не нужны Вам знаком метод KISS надеюсь? 0

Vovan-VE
	22.07.2012, 17:07 #8
	Не по теме: Сообщение от pyramida Вам знаком метод KISS надеюсь? Сообщение от KOPOJI к сожалению, нет.. можно поподробнее о нем?) Keep It Simple (http://ru.wikipedia.org/wiki/K... %B8%D0%BF)) 1

Опции темы