所以,简单地说,我觉得这段代码应该可行。从字面上看,此刻我只是想创建一个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
。如果我从混合中删除该变量,则函数调用有效。我觉得我只是不明白我应该在哪里声明变量/如何从函数内/外引用它。感谢。
答案 0 :(得分:2)
这里的问题是使用全局范围。而不是:
global $database;
if($database->has("customer",
使用
if($this->database->has("customer",
你也可以考虑在构造函数中实例化$ database,即
private $database;
public function __construct() {
$this->database = new medoo([args....
答案 1 :(得分:2)
正如前面的示例所示,您应该使用类似的东西并将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',
]);
}
检查是否捕获错误。在数据库上使用唯一或主键将是一种更安全的方式,否则您必须在数据库上进行验证和搜索。添加密钥并检查重复错误。
if($database->error()){
//deal with return or pass to logging
}