-2

以下をプロジェクトに実装する前に、簡単なサンプル クラスを作成しようとしています。改善できる/改善すべき点があるかどうかを知りたいです。誰かが私が今どのようにやっているかについて何か意見をくれれば、とてもうれしいです。

簡単なクラスの例:

  // define class
  class User{
  private $UserID;
  private $UserName;
  private $Email;

  function SetUserID($NewUserID){
    $this -> UserID = $NewUserID;
    }

  function GetUserID(){
    return $this -> UserID;
    }

  function SetUserName($NewUserName){
    // update object
    $this -> UserName = $NewUserName;
    } 

  function GetUserName(){
    return $this -> UserName;
    }

  function SetEmail($NewEmail){
    $this -> Email = $NewEmail;          
    } 

  function GetEmail(){
    return $this -> Email;
    }

  public function SaveUser(){
    // check if user exists
    $User = new User();
    if ($User->UserExists($this->UserID)){
      // user exists - update him
      $Pdo = new PDO('mysql:host=localhost;dbname=cms;charset=utf8', 'xxx', 'xxx');  
      $Query = $Pdo->prepare("Update User set UserName = :UserName, Email = :Email, UserID = :UserID where UserID = :UserID");
      $Query->bindValue(':UserName', $this->UserName);
      $Query->bindValue(':Email', $this->Email);
      $Query->bindValue('UserID', $this->UserID);
      $Query->execute();
      $Pro = null;
      }
    else{
      // insert new
      $Pdo = new PDO('mysql:host=localhost;dbname=cms;charset=utf8', 'xxx', 'xxx');  
      $Query = $Pdo->prepare("Insert into User (UserName, Email) values (:UserName, :Email)");
      $Query->bindValue(':UserName', $this->UserName);
      $Query->bindValue(':Email', $this->Email);
      $Query->execute();
      // close connection
      $Pdo = null; 
      }
    }

  private function UserExists($UserID){
    // returns true if users exists, false if not
    $Pdo = new PDO('mysql:host=localhost;dbname=cms;charset=utf8', 'root', 'vertrigo');
    $Query = $Pdo->prepare("SELECT * FROM User WHERE UserID=:UserID"); 
    $Query->bindValue(':UserID', $UserID);
    $Query->execute();
    $Row = $Query->fetch(PDO::FETCH_ASSOC);
    $Pdo = null;

    // get user
    if ($Row){
      return true;
      }
    else{
      return false;
      }
    }        
  }

function GetUserInfo($UserID){
  // open pdo
  $Pdo = new PDO('mysql:host=localhost;dbname=cms;charset=utf8', 'root', 'vertrigo');  
  $Query = $Pdo->prepare("SELECT * FROM User WHERE UserID=:UserID");
  $Query->bindValue(':UserID', $UserID);
  $Query->execute();
  $Row = $Query->fetch(PDO::FETCH_ASSOC);
  $Pdo = null;

  // get UserInfo
  $UserInfo = new User();
  $UserInfo -> SetUserID($UserID);
  $UserInfo -> SetUserName($Row["UserName"]); 
  $UserInfo -> SetEmail($Row["Email"]); 

  // return UserInfo
  return $UserInfo;
  }

これは、私がどのように作業したいかの基本的な例にすぎません。何を改善すればよいと思いますか？

4

1 に答える 1

0

どの PHP アプリケーションも、データベースに一度だけ接続する必要があります。
したがって、クラスは毎回作成するのではなく、作成済みの接続を使用する必要があります。

class User{
  private $UserID;
  private $UserName;
  private $Email;
  private $db;

  function __construct($pdo) {
    $this->db = $pdo;
  }

  private function UserExists($UserID)
  {
    $sql = "SELECT 1 FROM User WHERE UserID=?";
    $stm = $this->db->prepare($sql);
    $stm->execute(array($UserID));
    return $stm->fetchColumn();
    }        
  }
  // the rest going to use the same approach
}

$pdo = //see PDO tag wiki for theproper  example

$user = new User($pdo);
if ($user->UserExists($UserID)) {
  // whatever
}
于 2013-07-18T09:36:27.593 に答える