PHPページの空白 - スコープを理解していない

Question

このコードがうまくいくはずです。文字通り、この時点で私はいくつかの情報を取り、データベースに対してコマンドを実行するPHPクラスを作成しようとしています。コマンドが機能するので、それは私の変数の範囲と関係があるとは限りません。

私はPHPが初めてです。これは面白いことです。

<?php 
require __DIR__ . '/../bin/composer/vendor/autoload.php'; 

$cx = new Customer(); 
$cx->WriteCxToDB(); 

class Customer { 
    public $database = new medoo([ 
    'database_type'=>'mysql', 
    'database_name'=>'dbname', 
    'server'=>'localhost', 
    'username'=>'dbusername', 
    'password'=>'dbpassword', 
    'charset'=>'utf8' 
    ]); 

    public function WriteCxToDB(){ 
    global $database; 
    if($database->has("customer", [ 
     "OR"=>[ 
     "username"=>"cxusername", 
     "email"=>"email@gmail.com" 
     ] 
     ])) 
     { 
     echo "User already exists"; 
    }else{ 
     $database->insert("customer", [ 
     "username"=>"username", 
     "keyword"=>"keyword", 
     "email"=>"email@gmail.com", 
     "phone"=>"444-444-4444", 
     "first_name"=>"First", 
     "last_name"=>"Last" 
    ]); 
    echo "User added"; 
    } 
    echo "Done"; 
    } 
} 
?> 

私はこのデータベースエントリを行うために作曲家とmedooを使用しています。私はそれを自分の上で走らせて、うまく動作するので、データベースコードが動作することを知っています。

私が苦労しているのは、コード内の変数$databaseのようです。その変数をミックスから削除すると、関数呼び出しが機能します。私は、変数の宣言がどこにあるのか/関数の内部/外部から参照する方法を理解していないような気がします。ありがとう。

Answer 1

、コンストラクタで$データベースをインスタンス化する検討することもでき、使用

if($this->database->has("customer", 

db接続をクラスに渡すと、ベースクラスを拡張するとdb接続の再利用が可能になります。

private $database; 

public function __construct($db_connection = null){ 
    //do stuff or set db 
    $this->database = $this->db_connect; 
} 

やエラーをキャッチ考える確認するためにそれを

private function db_connect(){ 
     return new medoo([ 
     // required 
     'database_type' => 'mysql', 
     'database_name' => 'name', 
     'server' => 'localhost', 
     'username' => 'your_username', 
     'password' => 'your_password', 
     'charset' => 'utf8',  
     ]); 

    } 

を行うには、クラスのメソッドを作ります。 DB上で一意または主キーを使用する方が安全な方法です。そうでない場合は、DBで検証と検索を行う必要があります。キーを追加し、重複エラーがないか確認します。

if($database->error()){ 
    //deal with return or pass to logging 
} 

Answer 2

ここでの問題は、グローバルスコープの使用です。代わりに：

global $database; 
    if($database->has("customer", 

あなたはこのようなものを使用する必要があり、前の例で示唆したように、あなたはまた、すなわち

private $database; 

public function __construct() { 
    $this->database = new medoo([args....