비 객체의 fetchAll() 멤버 함수 호출

Question

왜 _sth 객체에서 fetchAll 메소드를 사용하기 때문에 Call to a member function fetchAll() on a non-object 오류를 반환하는지 알지 못합니다.

아무도 왜 이것을하는지 알 수 있습니까?

FormValidator PHP

$users = $this->_db->query("SELECT * FROM $ruleValue WHERE $input = ?", array($inputValue)); 
if($users->count()) { 
    $this->addError($input, $rule); 
} 

방법

public function query($sql, $params = array()) { 
    $this->_sth = $this->_dbh->prepare($sql); 

    if($this->_sth = $this->_sth->execute($params)) { 
     return $this->_sth->fetchAll(PDO::FETCH_OBJ); 
    } 

    return false; 
} 

Answer 1

if($this->_sth = $this->_sth->execute($params)) { 

단일 = 할당 연산자이다. execute()은 부울 TRUE 또는 FALSE를 반환하고이를 $this->_sth에 할당합니다. 도 TRUE FALSE 어느

return $this->_sth->fetchAll(PDO::FETCH_OBJ); 

개체이며, 더 fetchAll() 방법이 없다.

@RocketHazmat의 주석이 정확하므로 execute()의 반환 값을없이 _sth에 할당하여 테스트 할 수 있습니다. 예 :

if($this->_sth->execute($params)) { 
    return $this->_sth->fetchAll(PDO::FETCH_OBJ); 
} 

Answer 2

이미 말했듯이 두 가지 큰 결함이 있습니다.

1. 수업 중에 상태를 소개하지 마십시오. 지역 변수를 사용하고 결과를 리턴하십시오.
2. Ther은 쿼리에서 단일 변수가 아닌 이어야합니다.

이

public function query($sql, $params = array()) { 
    $sth = $this->_dbh->prepare($sql); 
    $sth->execute($params); 
    return $sth->fetchAll(PDO::FETCH_OBJ); 
} 

의 상태를 소개하지 마십시오 당신이 필요로하는 모두이다.

게으르지 말고 전체 쿼리를 작성하십시오. [다시 ** 그리고 당신이 당신의 잘못된 기능을 사용하려는 **]

$user = $this->_db->query("SELECT * FROM users WHERE param = ?", [$inputValue]); 
if($user) { 
    $this->addError($input, $rule); 
}