Форумы портала PHP.SU :: Версия для печати

Warning: Cannot use a scalar value as an array in /home/admin/public_html/forum/include/fm.class.php on line 757

Warning: Invalid argument supplied for foreach() in /home/admin/public_html/forum/include/fm.class.php on line 770
Форумы портала PHP.SU :: Версия для печати :: что неправильно

Форумы портала PHP.SU » » Вопросы новичков » что неправильно

Страниц (2): [1] 2 »

1. Каролина - 11 Мая, 2012 - 20:23:29 - перейти к сообщению

помогите исправить код

PHP:
скопировать код в буфер обмена

<form action="<?PHP  $_SERVER['PHP_SELF'] ?> method="post">
Email<inpyt type="text" name="email">
Пароль<inpyt type="password" name="pass">
</form>
<?PHP 
$pass=$_POST['pass'];
$Email=$_POST['Email'];
if(empty($Email) ||  empty($pass))
{
echo"вы не ввели Пароль или Email";
}
else
{
if($_SESSION['Email']=$Email or $_SESSION['pass']=$pass)
echo"вы вошли";
else 
echo"вы не правельно чот ввели";
}
?>

2. Bio man - 11 Мая, 2012 - 20:43:17 - перейти к сообщению

= это присваивание. Сравнение == или ===

3. Symby - 11 Мая, 2012 - 20:46:54 - перейти к сообщению

А не лучше так?

CODE (htmlphp):
скопировать код в буфер обмена

 
<form action="<?PHP $_SERVER['PHP_SELF'] ?> method="post">
Email<inpyt type="text" name="email">
Пароль<inpyt type="password" name="pass">
</form>
<?php
 
$pass = $_POST['pass'];
$Email = $_POST['Email'];
 
if (empty($Email) || empty($pass)) exit "вы не ввели Пароль или Email";
if($_SESSION['Email'] == $Email || $_SESSION['pass'] == $pass) echo"вы вошли";
else echo"вы не правельно чот ввели";
 
?>
 

И вообще, где вы сессию открываете? Вид вы проверяете в глобальном массиве сессий, там вид пусто.

4. Каролина - 11 Мая, 2012 - 20:51:30 - перейти к сообщению

у тя ошибка в 9строке

5. Green - 11 Мая, 2012 - 20:54:57 - перейти к сообщению

Вы, возможно, не поверите... Но тут все не правильно. Не понял

Начнем с того, что:

CODE (htmlphp):
скопировать код в буфер обмена

 
<form action="<?PHP $_SERVER['PHP_SELF'] ?>
 

Надо заменить на:

CODE (htmlphp):
скопировать код в буфер обмена

 
<form action="<?PHP echo($_SERVER['PHP_SELF']); ?>"
 

Далее, после HTML формы, всё действие надо заключить в условие, которое проверяет, например, нажата ли вообще кнопка в форме. Читайте про isset().

CODE (htmlphp):
скопировать код в буфер обмена

 
if($_SESSION['Email']=$Email or $_SESSION['pass']=$pass)
 

Вообще не пойми что. Не понял

Следует заменить на:

CODE (htmlphp):
скопировать код в буфер обмена

 
if($_SESSION['Email'] == $Email or $_SESSION['pass'] == $pass)
 

Но, "вы вошли" не отобразится никогда. Ибо таких сессий не существует вовсе.
Вы сделайте, чтобы если не пусты $pass и $Email, то в сессии записываются их значения.

Работать у Вас все равно ничего не будет. Все переписывать надо. Логически вообще все не верно построено.

И еще, не inpyt, а input.

6. Bio man - 11 Мая, 2012 - 20:55:17 - перейти к сообщению

Symby пишет:

А не лучше так?

Нет, так ничем не лучше.

Каролина пишет:

у тя ошибка в 9строке

Извините меня модераторы, но меня бесят такие посты. Данное сообщение не несет никакой смысловой нагрузки, так почему бы его не оформить как оффтоп или лучше вообще не писать?

7. sKaa - 11 Мая, 2012 - 21:02:03 - перейти к сообщению

Спойлер (Отобразить)

PHP:
скопировать код в буфер обмена

<?
error_reporting(-1); // all errors can be display
session_start();     // session start
 
/*
 & Check current session key, if key isset - auto auth user :) sorry it's as-is. 
*/ 
if(isset($_SESSION['password']) && isset($_SESSION['email'])){
      $authStep++;
}
 
 
/*
 && it's email and password for you script debuging. 
 && Enter it!
*/
$test_mail = 'qwe@qw.ru';
$test_pass = 'password';
 
// & Declare local var's
$messageArray = array();
$authStep = 0;
$authErrlvl = 0;
 
if(isset($_POST['email']) && isset($_POST['password']) && !empty($_POST['email']) && !empty($_POST['password'])){
        if($_POST['password'] == $test_pass)
                $_SESSION['password'] = $_POST['password'];
        else {
                $authErrLvl++;
                $messageArray[] = 'Вы ввели неверный пароль.'; 
        }
        
        if($_POST['email'] == $test_mail)
                $_SESSION['email'] = $_POST['email'];
        else {
                $authErrLvl++;
                $messageArray[] = 'Вы ввели неверный логин.'; 
        }
        
        if($authErrlvl){
                unset($_SESSION['email']);
                unset($_SESSION['email']);
        }
        else {
                $messageArray[] = 'Вы успешно авторизованы на сайт, форма входа скрыта от вас ибо нехера говнокодить.';
                $authStep = 1;
        }
                
} else if(empty($_POST['email']) || empty($_POST['password'])){
        $messageArray[] = 'Поля формы помеченные (*) обязательны к заполнению!';
        $authErrlvl++;
        
} 
 
if($authStep==0){
        if($authErrlvl){
                // non-auth or not has post data
                if(count($messageArray)){
                        // has auth error messages ? print this;
                        foreach($messageArray as $key => $message){
                                echo '<div style="color: red;" >' . $message . ' </div>';
                        }
                }
        ?>
    <form action="<?='http://' . $_SERVER['HTTP_HOST'] . '/' . current(explode('?', $_SERVER['PHP_SELF']))?>" method="post">
        <table>
            <tr> <td> Email </td> <td>: <input type="text" name="email"></td> </tr>
            <tr> <td> Password </td> <td>: <input type="text" name="password"></td> </tr>
        </table>
        <center>
            <input type="submit" value="Вход...">
        </center>
        </form>
        <?
 
        }
        return; // now game over :)
}
else {
        // it's auth user ... 
        if(count($messageArray)){
                foreach($messageArray as $key => $message){
                        echo '<div style="color: green;" >' . $message . ' </div>';
                }
        }
        
}

8. Symby - 11 Мая, 2012 - 21:02:55 - перейти к сообщению

Каролина пишет:

у тя ошибка в 9строке

Ошибка в том что не существуют такие данные в массиве пост, может сами найдете ошибку?

9. avtor.fox - 11 Мая, 2012 - 21:05:05 - перейти к сообщению

Bio man не кипятись Улыбка

Вообще что бы отправить форму саму на себя, достаточно

CODE (html):
скопировать код в буфер обмена

<form method='POST'>

sKaa, не чуди. Ты тоже решил <?PHP $_SERVER///;?> Однако

Просто ужасная проверка формы Ниндзя

10. Green - 11 Мая, 2012 - 21:06:05 - перейти к сообщению

CODE (htmlphp):
скопировать код в буфер обмена

 
<?PHP
        if(isset($_SESSION["email"]) == TRUE AND isset($_SESSION["pass"]) == TRUE) {
           exit("Вы уже выполниили вход в аккаунт!");
        }
    
    if(isset($_POST["go"]) == TRUE) {
        $email = $_POST["email"];
        $pass  = $_POST["pass"];
        
        if(empty($email) == TRUE OR empty($pass) == TRUE) {
            exit("Вы ничего не ввели!");
        } else {
            $_SESSION["email"] = $email;
            $_SESSION["pass"] = $pass;
        }
    }
?>
<form action="<?PHP echo($_SERVER["PHP_SELF"]); ?> method="POST">
    <b>Email:</b> <br /> <input type="text" name="email">     <b>Пароль:</b> <br /> <input type="password" name="pass"> <br />     <button name="go">Войти >></button>
</form>
 

Вот накидал вариант. Однако

Печально. Форум испортил код чуток. Подмигивание

11. sKaa - 11 Мая, 2012 - 21:07:29 - перейти к сообщению

avtor.fox, че вы меня морозите, я бухой, даже внимания не обращаю на такие мелочи, и так да здравствует моя совесть, что я соизволил дримвейвер запустить, а не на коленке писать Улыбка

(Добавление)
Green, фигня, я предложил более гибкий вариант Улыбка

Жаль ооп для каролины пока непостижим Улыбка

12. avtor.fox - 11 Мая, 2012 - 21:09:10 - перейти к сообщению

sKaa. Пьяным?! Да ещё и в "погоде-мечте"?! Да Вы батенька извращенец Улыбка

13. sKaa - 11 Мая, 2012 - 21:12:36 - перейти к сообщению

Я религиозный фанатик, меня редко обламывает скоротать вечерок за парой строчек кода, это только в последнее время я стал подкреплять мотивацию такого времяпрепровождения баночками пива, что часто пропорциональны строчечкам кода Улыбка

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

Каролина ! Я те все волосы повыдергаю если ты мне спасибо не нажмешь!!!! ))) И всем остальным тоже кто проявлял внимание к твоим непонятным кусочкам полу-кода бл....ь!!!

14. Green - 11 Мая, 2012 - 21:18:31 - перейти к сообщению

sKaa пишет:

(Добавление)
Green, фигня, я предложил более гибкий вариант Улыбка

Жаль ооп для каролины пока непостижим Улыбка

Ну так я поэтому такой простой компактный вариант накидал, дабы ТС поняла как вообще это должно работать.

Я конечно же могу предоставить вариант, в котором учтены любые мелочи)
Но в таком случае, я уверен, ТС ничего не поймет в коде, и опять попытается свое написать)))) Улыбка

Я как бы написал на понятном ей стиле программирования, если можно так сказать. Улыбка

15. sKaa - 11 Мая, 2012 - 21:26:38 - перейти к сообщению

Green, мое мнение - плохие примеры кода еще больше подталкивают новичков к стилю кода "говнокод" или "индусский код" тут еще важно учитывать склад ума новичка по скольку оба эти стиля совершенно разные, и зависят не столько от задачи приложения, а от количества строк в нем или несуразных решений простых задач, но ничего хорошего в них обоих нету.
В том время как красивые и в тоже время простые примеры без лишних плясок с бубном мотивируют новичков на совершенствование навыков! - имхо!
(Добавление)
Немного дополнил свой код проверкой состояния сессий Улыбка

(Добавление)
И <? echo $_SERVER['PHP_SELF']; ?> легко можно заменить на <?=$_SERVER['php_self']?>, конечно это функция требует включенной директивы в пхп.ини (short_open_tag но могу ошибаться), обычно она включена если всякие денверы стоят.
(Добавление)
И вообще пока не пришла ТС и не занялась обыденным копи-пастом я заменил :
form action="<?=$_SERVER['php_self']?>"
на : так наверное будет самое лучшее решение, но необходимо всё-же тестить, у меня не установлены сейчас ни апач ни модуль пхп ессно к нему, пишу на холстах так сказать Улыбка

form action="<?='http://' . $_SERVER['HTTP_HOST'] . '/' . current(explode('?', $_SERVER['PHP_SELF']))?>"