Question

我是PHP的新手，我正在为一个小课程做OOP登录/注册教程，所以它不适用于生病的网站。一小时或15小时后，我停留在一段似乎不起作用的代码上。我试图与我的一位朋友解决这个问题，他对PHP有更多的了解，我搜索了谷歌，检查了多个论坛和所有YouTube评论的解决方案，但我卡住了。我知道所有其他代码都是正确的，因为它完全可以解决这个问题：

所以我的qeustion是：什么是正确的PHP代码，使独特的案例工作。

            case 'unique'     
                $check =$this->_db->get($rule_value,array($item,"=",
                $value));  
    line 40>    if($check->count()){ 
                $this->addError("{$item} bestaat al.");

                with this error:

！）致命错误：在第40行调用堆栈中的C：\ wamp \ www \ websitegroop \ classes \ validate.php中的非对象上调用成员函数count（）

验证课程：

class Validate  {
private $_passed = false,
        $_errors = array(),
        $_db = null;
public function __construct() {
    $this->_db = Database::getInstance();
}

public function check($source, $items = array()) {
    foreach($items as $item => $rules) {
        foreach($rules as $rule => $rule_value) {

            $value = trim($source[$item]);



            if($rule === 'required' && empty($value)) {
                $this->addError("{$item} is required");
            } else if(!empty($value)) {
                switch($rule) {
                    case 'min':
                        if(strlen($value) < $rule_value) {
                            $this->addError("{$item} moet minimaal {$rule_value} letters of cijfers zijn.");
                        }
                    break;
                    case 'max':
                        if(strlen($value) > $rule_value) {
                            $this->addError("{$item} kan maximaal {$rule_value} letters of cijfers zijn.");
                        }
                    break;
                    case 'matches':
                        if($value != $source[$rule_value]) {
                            $this->addError("{$rule_value} moet hetzelfde zijn als {$item}.");
                        }
                    break;
                    case 'unique':
                        $check = $this->_db->get($rule_value, array($item, "=", $value));
                        if($check->count()){ 
                            $this->addError("{$item} bestaat al.");
                        }
                    break;


                }
        }
    }
}

if(empty($this->_errors)) {
    $this->_passed = true;
}

return $this;
}

private function addError($error) {
    $this->_errors[] = $error;
}

public function errors() {
    return $this->_errors;
}

public function passed() {
    return $this->_passed;
}

}

register.php：

               <?php
                require_once 'core/init.php';

                 if(Input::exists()) {
                 if(Token::check(Input::get('token'))) {

$validate = new Validate();
$validation = $validate->check($_POST, array(
    'gebruikersnaam' => array(
            'required' => true,
            'min' => 3,
            'max' => 30,
            'unique' => 'groopklanten'
    ),
    'paswoord' => array(
            'required' => true,
            'min' => 6,
    ),
    'paswoord_nogmaals' => array(
            'required' => true,
            'matches' => 'paswoord'
    ),
    'bedrijfsnaam' => array(
            'required' => true,
            'min' => 3,
            'max' => 30,
    ),
    'kvknr' => array(
            'required' => true,
            'min' => 8,
            'max' => 8,
            'unique' => 'groopklanten'
    ),
    'contactpersoon' => array(
            'required' => true,
            'min' => 4,
            'max' => 30,
    ),
    'functie' => array(
            'required' => true,
            'min' => 4,
            'max' =>30,
    ),
    'telbedrijf' => array(
            'required' => true,
            'min' => 10,
            'max' => 12,
            'unique' => 'groopklanten'
    ),
    'adres' => array(
            'required' => true,
            'min' => 4,
            'max' => 40,
            'unique' => 'groopklanten'
    ),
    'afleveradres' => array(
            'min' => 4,
            'max' => 40,
    ),
    'postcode' => array(
            'required' => true,
            'min' => 6,
            'max' => 7,
    ),
    'woonplaats' => array(
            'required' => true,
            'min' => 3,
            'max' => 30,
    ),
    'emailadres' => array(
            'required' => true,
            'min' => 5,
            'max' => 30,
            'unique' => 'groopklanten'
    )
)); 

if($validate->passed()) {
    echo 'geregistreerd';
} else {
    foreach($validation->errors() as $error) {
        echo $error, '<br>';
        }
        }
        }
        }

        ?>

        <form action="" method="post">
<div class="field">
    <label for="gebruikersnaam">Gebruikersnaam:</label>
    <input type="text" name="gebruikersnaam" id="gebruikersnaam">
</div>

<div class="field">
    <label for="paswoord">Kies een paswoord:</label>
    <input type="password" name="paswoord" id="paswoord">
</div>

<div class="field">
    <label for="paswoord_nogmaals">Nogmaals uw paswoord:</label>
    <input type="password" name="paswoord_nogmaals" id="paswoord_nogmaals">
</div>

<div class="field">
    <label for="bedrijfsnaam">Bedrijfsnaam:</label>
    <input type="text" name="bedrijfsnaam" id="bedrijfsnaam">
</div>

<div class="field">
    <label for="kvknr">KvK nummer:</label>
    <input type="text" name="kvknr" id="kvknr">
</div>

<div class="field">
    <label for="contactpersoon">Naam contactpersoon:</label>
    <input type="text" name="contactpersoon" id="contactpersoon">
</div>

<div class="field">
    <label for="functie">Functie:</label>
    <input type="text" name="functie" id="functie">
</div>

<div class="field">
    <label for="telbedrijf">Telefoonnummer bedrijf:</label>
    <input type="tell" name="telbedrijf" id="telbedrijf">
</div>

<div class="field">
    <label for="adres">Adres:</label>
    <input type="text" name="adres" id="adres">
</div>

<div class="field">
    <label for="afleveradres"> Afwijkend aflever adres:</label>
    <input type="text" name="afleveradres" id="afleveradres">
</div>

<div class="field">
    <label for="postcode">Postcode:</label>
    <input type="text" name="postcode" id="postcode">
</div>

<div class="field">
    <label for="woonplaats">Woonplaats:</label>
    <input type="text" name="woonplaats" id="woonplaats">
</div>

<div class="field">
    <label for="emailadres">E-mail adres:</label>
    <input type="email" name="emailadres" id="emailadres">
</div>

<input type="hidden" name="token" value="<?php echo Token::generate();   ?>">
<input type ="submit" value="Register">
<input type="reset" name="reset" value="Reset Velden">

<p> Bij het verzenden gaat u automatisch akoord met de <a href="algemenevoorwaarden.html"> algemene voorwaarden!</a></p>

输入类：

        <?php
        class input {
        public static function exists($type = 'post') {
        switch($type) {
        case 'post':
            return (!empty($_POST)) ? true : false;
        break;
        case 'get':
            return (!empty($_GET)) ? true : false;
        break;
        default:
            return false;
        break;
    }
}

public static function get($item) {
    if(isset($_POST[$item])) {
        return $_POST[$item];
    } else if(isset($_GET[$item])) {
        return $_GET[$item];
    }
    return '';
}
}

数据库类：

       <?php
        class Database {
        private static $_instance = null;
        private $_pdo, 
        $_qeury, 
        $_error = false, 
        $_result, 
        $_count = 0;

    private function __construct() {
    try {
        $this->_pdo = new PDO('mysql:host=' . Config::get('mysql/host')  . ';dbname=' . Config::get('mysql/db'), Config::get('mysql/username'),   Config::get('mysql/password'));

    } catch(PDOException $e) {
        die($e->getMessage());
    }
}

public static function getInstance() {
    if(!isset(self::$_instance)) {
     self::$_instance = new Database();
    }
    return self::$_instance;
}

public function query($sql, $params = array()) {
    $this->_error = false;
    if($this->_query = $this->_pdo->prepare($sql)) {
        $x = 1;
        if(count($params)) {
            foreach($params as $param) {
            $this->_query->bindValue($x, $param);
            $x++;

            }
        }

        if($this->_query->execute()) {
            $this->_results = $this->_query->fetchALL(PDO::FETCH_OBJ);
            $this->_count = $this->_query->rowCount();
        } else {
            $this->_error = true;

        }

    }

    return $this;
}

public function action($action, $table, $where = array()) {
    if(count($where) === 3) {
        $operators = array('=', '>', '<', '>=', '<=');

        $field      = $where[0];
        $operator   = $where[1];
        $value      = $where[2];

        if(in_array($operator, $operators)) {
            $sql = "{$action} FROM {$table} WHERE {$field} {$operator} {value} ?";

            if(!$this->query($sql, array($value))->error()) {
                return $this;
            }
        }
    }
    return false;
}

public function get($table, $where) {
    return $this->action('SELECT *', $table, $where);

}

public function delete($table, $where) {
    return $this->action('DELETE *', $table, $where);
}

public function insert($table, $fields =array()) {
    if(count($fields)) {
        $keys = array_keys($fields);
        $values = null;
        $x = 1;

        foreach($fields as $field) {
            $values .= '?';
            if ($x < count($fields)) {
                $values .= ', ';
            }
            $x++;
        }

        $sql = "INSERT INTO groopklanten (`" . implode('`, `', $keys)."`) VALUES ({$values})";

        if(!$this->query($sql, $fields)->error()) {
            return true;
        }
    }
        return false;
}

public function update($table, $id, $fields) {
    $set = '';
    $x = 1; 

    foreach($fields as $name => $value) {
        $set .= "{$name} = ?";  
        if($x < count($fields)) {
            $set .= ', ';
        }
        $x++;
    }

    $sql = "UPDATE {$table} SET {$set} WHERE id = {$id}";

    if(!$this->query($sql, $fields)->error()) {
        return true;
    }

    return false;
}

public function results() {
    return $this->_results; 
}

public function first() {
    return $this->_results[0];
}

public function error() {
    return $this->_error;
}

public function count() {
    return $this->_count;
  }
   }

的init.php

         <?php
          session_start();

     $GLOBALS['config'] = array(
    'mysql' => array(
        'host' => 'localhost',
        'username' => 'nick16061983',
        'password' => 'N1sn0p1!A',
        'db' => 'websitegroop'
    ),
    'remember' => array(
        'cookie_name' => 'hash',
        'cookie_expiry' => 604800
    ),  
    'session' => array(
        'session_name' => 'groopklanten',
        'token_name' => 'token'

    )
   );

     spl_autoload_register(function($class) {
     require_once 'classes/' .$class. '.php';
     });

到目前为止，我尝试过的所有其他事情告诉我，我总是可以注册现有的用户名，或者总是说它已经注册了。当用户名不在我的数据库中时。

提前致谢，

尼克

Answer 1

您收到此错误的原因是因为失败时action()方法的返回值是布尔值。因此，如果您尝试在布尔值上调用count()（这是无意义的，因为布尔值不能有方法），它会抛出该错误。相反，您应该首先检查返回值是否是对象，如下所示：

if ($check) { ... }

只要$check是一个对象，它就应该传递这个if语句。然后你可以照常进行。

您可以尝试以下方法：

// Make sure it's not false
if ($check) {
    // Then count the errors
    if ($check->count()) { ... }
}

随机说明：

if ($check && $check->count()) { ... }

如果第一个参数等于false，如果PHP没有检查&&运算符的第二个参数，则此代码可以也可以工作。只是一个随机的注释来保存线条。

Answer 2

致命错误：在第40行调用堆栈中的C：\ wamp \ www \ websitegroop \ classes \ validate.php中的非对象上调用成员函数count（）

那是因为action()方法返回false而不是当前对象，这意味着您的查询无法执行。这有两个主要原因：

首先，请查看以下代码段。这里有一个小的语法错误，

class Database {
    private static $_instance = null;
    private $_pdo, 
    $_qeury, // syntax error

    ...

应该是，

class Database {
    private static $_instance = null;
    private $_pdo, 
    $_query,

    ...

其次，您在$sql = "{$action} FROM {$table} WHERE {$field} {$operator} {value} ?";方法中的陈述action()是错误的，它应该是这样的：

$sql = "{$action} FROM {$table} WHERE {$field} {$operator} ?";

所以你的整个Database课应该是这样的：

<?php

    class Database {
        private static $_instance = null;
        private $_pdo, 
        $_query, 
        $_error = false, 
        $_result, 
        $_count = 0;

        private function __construct() {
            try {
                $this->_pdo = new PDO('mysql:host=' . Config::get('mysql/host')  . ';dbname=' . Config::get('mysql/db'), Config::get('mysql/username'),   Config::get('mysql/password'));

            } catch(PDOException $e) {
                die($e->getMessage());
            }
        }

        public static function getInstance() {
            if(!isset(self::$_instance)) {
             self::$_instance = new Database();
            }
            return self::$_instance;
        }

        public function query($sql, $params = array()) {
            $this->_error = false;
            if($this->_query = $this->_pdo->prepare($sql)) {
                $x = 1;
                if(count($params)) {
                    foreach($params as $param) {
                        $this->_query->bindValue($x, $param);
                        $x++;
                    }
                }

                if($this->_query->execute()) {
                    $this->_results = $this->_query->fetchALL(PDO::FETCH_OBJ);
                    $this->_count = $this->_query->rowCount();
                } else {
                    $this->_error = true;
                }

            }

            return $this;
        }

        public function action($action, $table, $where = array()) {
            if(count($where) === 3) {
                $operators = array('=', '>', '<', '>=', '<=');

                $field      = $where[0];
                $operator   = $where[1];
                $value      = $where[2];

                if(in_array($operator, $operators)) {
                    $sql = "{$action} FROM {$table} WHERE {$field} {$operator} ?";

                    if(!$this->query($sql, array($value))->error()) {
                        return $this;
                    }
                }
            }
            return false;
        }

        public function get($table, $where) {
            return $this->action('SELECT *', $table, $where);

        }

        public function delete($table, $where) {
            return $this->action('DELETE *', $table, $where);
        }

        public function insert($table, $fields =array()) {
            if(count($fields)) {
                $keys = array_keys($fields);
                $values = null;
                $x = 1;

                foreach($fields as $field) {
                    $values .= '?';
                    if ($x < count($fields)) {
                        $values .= ', ';
                    }
                    $x++;
                }

                $sql = "INSERT INTO groopklanten (`" . implode('`, `', $keys)."`) VALUES ({$values})";

                if(!$this->query($sql, $fields)->error()) {
                    return true;
                }
            }
                return false;
        }

        public function update($table, $id, $fields) {
            $set = '';
            $x = 1; 

            foreach($fields as $name => $value) {
                $set .= "{$name} = ?";  
                if($x < count($fields)) {
                    $set .= ', ';
                }
                $x++;
            }

            $sql = "UPDATE {$table} SET {$set} WHERE id = {$id}";

            if(!$this->query($sql, $fields)->error()) {
                return true;
            }

            return false;
        }

        public function results() {
            return $this->_results; 
        }

        public function first() {
            return $this->_results[0];
        }

        public function error() {
            return $this->_error;
        }

        public function count() {
            return $this->_count;
        }
    }

?>

OOP登录/注册

2 个答案: