【问题标题】:Call only one function at a time一次只调用一个函数
【发布时间】:2017-06-14 15:45:38
【问题描述】:

我正在使用带有 ajax 的 php OOP 方法创建登录和注册示例函数。

当我点击登录按钮时,它也会自动触发注册功能,当点击注册时会触发登录功能。我知道问题是当我创建一个对象并调用类下面的两个函数时。我想知道有什么方法可以一次只调用一个函数。代码如下:

Ajax

function login() {

    jQuery('#loginform').on('submit', (function(e) {
        e.preventDefault();
        jQuery.ajax({
            url: 'scripts/controller.php/login',
            type: 'POST',
            data: new FormData(this),
            processData: false,
            contentType: false,
            cache: false,
            beforeSend: function() {
                jQuery('#btn-login').html('<i class="fa fa-spinner fa-spin fa-fw"></i>');
            },
            success: function(data) {
                if(data == 'Logged in') {
                    jQuery('.result').show();
                    jQuery('.result').html(data);
                    jQuery('#btn-login').html('Login');
                }
                else {
                    jQuery('.result').html(data);
                    jQuery('.result').show();
                    jQuery('#btn-login').html('Login');
                }
            }
        });
    }));

}

function register() {

    jQuery('#signupform').on('submit', (function(e) {
        e.preventDefault();
        jQuery.ajax({
            url: 'scripts/controller.php/register',
            type: 'POST',
            data: new FormData(this),
            processData: false,
            contentType: false,
            cache: false,
            beforeSend: function() {
                jQuery('#btn-signup').html('<i class="fa fa-spinner fa-spin fa-fw"></i>');
            },
            success: function(data) {
                if(data === 'An email has been sent. Please verify your account with in 3 days.') {
                    jQuery('.result').show();
                    jQuery('.result').fadeOut(5000);
                    jQuery('.result').html(data);
                    jQuery('#btn-signup').html('Sign Up');
                    jQuery('.result').html(data);
                    jQuery('#signupform')[0].reset();
                }
                else {
                    jQuery('.result').show();
                    jQuery('.result').html(data);
                    jQuery('#btn-signup').html('Sign Up');
                }
            }
        });
    }));

}

PHP 代码

<?php

    require('model.php');

    class curd {

        /************************************************/
                /*** LOGIN **/    
        /************************************************/

        public function login() {

            $restricted = array('--', '#', "'--", '/*', '*/', '/**/', '/*', '1/0', '*/ 1', "'", ';', '1=1','true','false', 'BEGIN', '+', '||', '|', "' or 1=1/*", "') or '1'='1--", "') or ('1'='1--", '*', 'drop' );

            $userEmail = strip_tags(stripcslashes(htmlspecialchars($_POST['email'])));
            $password = strip_tags(stripcslashes(htmlspecialchars($_POST['password'])));

            if(in_array($userEmail, $restricted) or in_array($password, $restricted)) {
                echo 'Avoid SQL injection attacks.';
            }
            else if(!filter_var($userEmail, FILTER_VALIDATE_EMAIL)) {
                echo 'Invalid email address.';
            }
            else if(strlen(trim($userEmail)) < 5) {
                echo 'Minimum characters in email are 5.';
            }
            else if(strlen(trim($password)) < 5) {
                echo 'Minimum characters in password are 5.';
            }
            else {
                $model = new curd_model();
                echo $model -> login($userEmail, md5(sha1($password)));
            }

        }

        /************************************************/
                /*** REGISTER **/    
        /************************************************/

        public function register() {

            $restricted = array('--', '#', "'--", '/*', '*/', '/**/', '/*', '1/0', '*/ 1', "'", ';', '1=1','true','false', 'BEGIN', '+', '||', '|', "' or 1=1/*", "') or '1'='1--", "') or ('1'='1--", '*', 'drop' );

            $username = strip_tags(stripcslashes(htmlspecialchars($_POST['username'])));
            $userEmail = strip_tags(stripcslashes(htmlspecialchars($_POST['email'])));
            $password = strip_tags(stripcslashes(htmlspecialchars($_POST['password'])));
            $question = strip_tags(stripcslashes(htmlspecialchars($_POST['question'])));
            $answer = strip_tags(stripcslashes(htmlspecialchars($_POST['answer'])));

            if(in_array($userEmail, $restricted) or in_array($password, $restricted) or in_array($userEmail, $restricted) or in_array($question, $restricted) or in_array($answer, $restricted)) {
                echo 'Avoid SQL injection attacks.';
            }
            else if(!filter_var($userEmail, FILTER_VALIDATE_EMAIL)) {
                echo 'Invalid email address.';
            }
            else if(strlen(trim($userEmail)) < 5) {
                echo 'Minimum characters in email are 5.';
            }
            else if(strlen(trim($password)) < 5) {
                echo 'Minimum characters in password are 5.';
            }
            else {
                $model = new curd_model();
                echo $model -> register($username, $userEmail, md5(sha1($password)), $question, $answer);
            }

        }
    }

    $object = new curd();
    $object -> login();
    $object -> register();

?>

【问题讨论】:

  • 为什么你的 PHP 代码同时调用 login 和 register 方法?您可能应该为每个操作设置单独的控制器。请使用 PDO 或 Mysqli 和准备好的/参数化查询,以便您可以删除所有条带/标签等。请使用 PHP 密码库,md5 是可怕的可怕做法。
  • 感谢您的回复,我只想在一个控制器文件中工作。像codeigniter一样,我总是只为控制器制作一个文件,同样我只想要一个文件。关于我使用 mysqli 的版本。
  • 您至少需要一些逻辑来将请求路由到不同路由 (URL) 的正确操作。

标签: javascript php jquery oop


【解决方案1】:

任何时候加载文件都会运行以下行:

   $object = new curd();
    $object -> login();
    $object -> register();

因此,登录和注册功能都会运行。 您有 2 个选择:

  1. 将这些函数拆分为 2 个不同的文件。

  2. 在您的 Ajax 中,添加一个参数,该参数将告诉该文件运行哪个函数。

在 MVC 中你可能有一个路由机制(例如:/user/login -> Controller User -> Method login) 如果你不这样做,你可以简单地使用一个查询字符串,比如:/scripts/controller.php?do=login 并且在你的 php 文件中有一个条件:

   $object = new curd();

$do = $_GET['do'];
    if($do == 'login'){
         $object -> login();
    } else {
         $object -> register();
    }

并在您的 ajax 中,更新请求 url:

jQuery.ajax({
            url: 'scripts/controller.php?do=register',

(对于登录请求也是如此 .. ?do=login)

【讨论】:

  • 谢谢我没有尝试过,但希望这能完全解决问题
猜你喜欢
  • 1970-01-01
  • 2022-01-27
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 1970-01-01
  • 2011-05-09
  • 2018-07-13
  • 2012-08-23
相关资源
最近更新 更多