我的 php 知識相當有限，但我最近需要將一些網頁從舊版本的 php 5.2 更新到 php 7.3。我已經設法更新了對 mysqli 等的大多數 mysql 引用并讓事情正常工作，但是有一個頁面使用了日歷，我真的在這部分和 fetch_field 部分苦苦掙扎，特別是我的任何例子發現似乎沒有類似的格式。我需要更新的代碼如下；require_once('Connections/connAsh.php');mysql_select_db($database_connAsh, $connAsh);function selectonerow($fieldsarray, $table, $uniquefield, $uniquevalue){ //The required fields can be passed as an array with the field names or as a comma separated value string if (is_array($fieldsarray)) { $fields = implode(", ", $fieldsarray); } else { $fields = $fieldsarray; } //performs the query $result = mysql_query("SELECT $fields FROM $table WHERE $uniquefield = '$uniquevalue'") or die("Could not perform select query - " . mysql_error()); $num_rows = mysql_num_rows($result); //if query result is empty, returns NULL, otherwise, returns an array containing the selected fields and their values if ($num_rows == NULL) { return NULL; } else { $queryresult = array(); $num_fields = mysql_num_fields($result); $i = 0; while ($i < $num_fields) { $currfield = mysql_fetch_field($result, $i); $queryresult[$currfield->name] = mysql_result($result, 0, $currfield->name); $i++; } return $queryresult; }}我的編輯嘗試是；require_once('../Connections/connAsh.php')$connAsh->select_db($database_connAsh);function selectonerow($fieldsarray, $table, $uniquefield, $uniquevalue){ //The required fields can be passed as an array with the field names or as a comma separated value string if (is_array($fieldsarray)) { $fields = implode(", ", $fieldsarray); } else { $fields = $fieldsarray; }
2 回答
慕田峪4524236
TA貢獻1875條經驗 獲得超5個贊
原來的功能在很多層面上都是錯誤的。重新創建它的功能是沒有意義的。
基本上你在這里討價還價的只是幾個 SQL 關鍵字。但這些關鍵字有助于提高可讀性。
出于某種原因，您決定智取幾代對 SQL 語法非常滿意的程序員，并制造難以理解的胡言亂語
$row = selectonerow("some, foo, bar", "baz", "id", [$uniquevalue]);
而不是幾乎自然的英語
$row = selectonerow("SELECT some, foo, bar FROM baz WHERE id=?", [$uniquevalue]);
來吧。這不值得。
讓你的函數接受常規的 SQL 查詢，而不是有限的難以理解的混亂。
function selectonerow(mysqli $conn, string $sql, array $params = []): array
{
if ($params) {
$stmt = $conn->prepare($sql);
$stmt = $mysqli->prepare($sql);
$stmt->bind_param(str_repeat("s", count($params), ...$params);
$stmt->execute();
$result = $stmt->get_result()
} else {
$result = $conn->query($sql);
}
return $result->fetch_assoc();
}
此功能將允許您使用任何查詢。例如，需要一個最高價格的行嗎？
$row = selectonerow("SELECT * FROM baz ORDER BY price DESC LIMIT 1");
需要更復雜的條件？沒問題
$sql = "SELECT * FROM baz WHERE email=? AND activated > ?";
$row = selectonerow($sql, [$email, $date]);
等等。任何 SQL。任何條件。
小怪獸愛吃肉
TA貢獻1852條經驗 獲得超1個贊
我建議去掉這個函數或者用 YCS 建議的函數替換它。
如果您真的想繼續使用此功能，請考慮以下修復。您使里面的代碼變得非常復雜，并且忘記將連接變量傳遞給函數。我已經簡化了它：
// open the DB connection properly inside Connections/connAsh.php
mysqli_report(MYSQLI_REPORT_ERROR | MYSQLI_REPORT_STRICT);
$connAsh = new mysqli('host', 'user', 'pass', $database_connAsh);
$connAsh->set_charset('utf8mb4');
// your function:
function selectonerow(mysqli $connAsh, $fieldsarray, $table, $uniquefield, $uniquevalue): array
{
//The required fields can be passed as an array with the field names or as a comma separated value string
if (is_array($fieldsarray)) {
$fields = implode(", ", $fieldsarray);
} else {
$fields = $fieldsarray;
}
//performs the query
$stmt = $connAsh->prepare("SELECT $fields FROM $table WHERE $uniquefield = ?");
$stmt->bind_param('s', $uniquevalue);
$stmt->execute();
return $stmt->get_result()->fetch_assoc();
}
這是您的功能，但消除了很多噪音。我$connAsh在函數的簽名中添加了，所以你每次調用這個函數時都必須傳入它。該函數將始終返回一個數組；如果沒有獲取記錄，則數組將為空。這是推薦的方式。還要記住始終使用準備好的語句！
- 2 回答
- 0 關注
- 132 瀏覽
添加回答
舉報
0/150
提交
取消