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

@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
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
<?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
1
2
3
4
5
6
7
8
9
10
<?php
session_start();
error_reporting();
 
require 'database/connect.php';
require 'functions/general.php';
require 'functions/users.php';
 
$errors = array();
?>

connect.php

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

general.php

PHP
1
2
3
4
5
<?php
function sanatize($data){
    return mysql_real_escape_string($data);
}
?>

users.php

PHP
1
2
3
4
5
6
7
8
9
<?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
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
<?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 строчек..

Новые блоги и статьи Все статьи Все блоги /
Символьное дифференцирование igorrr37 13.02.2026 / * Логарифм записывается как: (x-2)log(x^2+2) - означает логарифм (x^2+2) по основанию (x-2). Унарный минус обозначается как ! */ #include <iostream> #include <stack> #include <cctype>. . .	Камера Toupcam IUA500KMA Eddy_Em 12.02.2026 Т. к. у всяких "хикроботов" слишком уж мелкий пиксель, для подсмотра в ESPriF они вообще плохо годятся: уже 14 величину можно рассмотреть еле-еле лишь на экспозициях под 3 секунды (а то и больше),. . .	И ясному Солнцу zbw 12.02.2026 И ясному Солнцу, и светлой Луне. В мире покоя нет и люди не могут жить в тишине. А жить им немного лет.	«Знание-Сила» zbw 12.02.2026 «Знание-Сила» «Время-Деньги» «Деньги -Пуля»
SDL3 для Web (WebAssembly): Подключение Box2D v3, физика и отрисовка коллайдеров 8Observer8 12.02.2026 Содержание блога Box2D - это библиотека для 2D физики для анимаций и игр. С её помощью можно определять были ли коллизии между конкретными объектами и вызывать обработчики событий столкновения. . . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL_LoadPNG (без SDL3_image) 8Observer8 11.02.2026 Содержание блога Библиотека SDL3 содержит встроенные инструменты для базовой работы с изображениями - без использования библиотеки SDL3_image. Пошагово создадим проект для загрузки изображения. . .	SDL3 для Web (WebAssembly): Загрузка PNG с прозрачным фоном с помощью SDL3_image 8Observer8 10.02.2026 Содержание блога Библиотека SDL3_image содержит инструменты для расширенной работы с изображениями. Пошагово создадим проект для загрузки изображения формата PNG с альфа-каналом (с прозрачным. . .	Установка Qt-версии Lazarus IDE в Debian Trixie Xfce volvo 10.02.2026 В общем, достали меня глюки IDE Лазаруса, собранной с использованием набора виджетов Gtk2 (конкретно: если набирать текст в редакторе и вызвать подсказку через Ctrl+Space, то после закрытия окошка. . .

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

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