所以,简单地说,我觉得这个代码应该起作用。从字面上看,我只是在尝试创建一个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";
}
}
?>
我正在使用Composer和Medoo来执行此数据库条目。我知道数据库代码有效,因为我已经运行了它并且运行良好。
我正在为此而苦苦挣扎的是代码中的变量$database
。如果我从混音中删除该变量,则功能调用有效。我觉得我只是不明白我应该在哪里声明变量/如何从功能内部/外部引用它。谢谢。
如上一个示例所建议,您应该使用类似的东西并将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
}
这里的问题是使用全局范围。而不是:
global $database;
if($database->has("customer",
使用
if($this->database->has("customer",
您还可以考虑在构造函数中实例化$数据库,即
private $database;
public function __construct() {
$this->database = new medoo([args....