WordPressの独自テーブルで、取得はできるが保存ができない問題

Question

###実現したいこと
以下はphpMyAdminで作成した「dokuzi」というテーブルへの保存として書いたソースコードです。

これが保存されず特に何も起こらないのですが、どうしたら保存できるようになるか教えていただけませんでしょうか。

###該当のソースコード

// 保存
$table_name = $wpdb->prefix . 'dokuzi'; 
$yasai_arr = ['yasai_2',0];
$ins_values = "('". $yasai_arr[0]. "',".$yasai_arr[1] ."),";
$sql ="insert into ".$wpdb->prefix.$table_name. " (yasai,price) VALUES ".$ins_values.";";
$wpdb->query($sql);

###試したこと（テーブルの存在を確認）
まず「dokuzi」が$wpdbで使えることを確認できました。下記「結果」の最後の行で出力されているので問題ないかと存じます。

// テーブル確認
global $wpdb;
foreach ($wpdb->tables as $key => $value) {
  echo '<p>[' . $key . ']' . $value . '</p>';
}

↓「結果」

<p>[0]posts</p>
<p>[1]comments</p>
<p>[2]links</p>
<p>[3]options</p>
<p>[4]postmeta</p>
<p>[5]terms</p>
<p>[6]term_taxonomy</p>
<p>[7]term_relationships</p>
<p>[8]termmeta</p>
<p>[9]commentmeta</p>
<p>[10]dokuzi</p>

###試したこと（取得の確認）
そして保存はできないなら取得はどうかと思い下記実行すると、これも「結果」にあるように取得できました。

// 取得
$product_name = 'yasai_1';
$sql = $wpdb->prepare("SELECT p.yasai, p.price FROM $wpdb->dokuzi p WHERE p.yasai LIKE %s", '%'.$product_name.'%' );
# -> "SELECT p.yasai, p.price FROM wp_products p WHERE p.yasai LIKE '%検索キーワード%'"
$rows = $wpdb->get_results($sql);
if($rows){
	foreach ($rows as $row) {
		echo "<p>【".$row->yasai."】価格：".$row->price."円</p>";
	}
}

↓「結果」

<p>【yasai_1】価格：0円</p>

Answer 1

$wpdb->prefix.$table_name
このテーブル名正しいんでしょうか？
見る限りwp_wp_dokuziみたいになっちゃうと思うのですが

一応wpではinsert処理は
$wpdb->insertメソッド使ってやった方が確実・簡単と思います
それなりにインジェクション対策にもなりますし

Comments

[marusankakun]

ありがとうございます。以下でできました。しかしインジェクション対策は以下prepareだとマズイでしょうか？

$id = 123;
$yasai = 'yasai_3';
$price = 100;
$sql = $wpdb->prepare("INSERT INTO $wpdb->dokuzi (id, yasai, price) values (%d, %s, %d)", $id, $yasai, $price);
$wpdb->query($sql);

[KazuhiroHatano]

もちろんprepareでもいいです
insertメソッド使えばSQL文法ミスとかの
ケアレスミスのリスクが少ないってだけです

[KazuhiroHatano]

いや、カラム追加された時とかにSQL文を編集しなくて済む様にもできて
保守性が良くなるというメリットもありますかね

[marusankakun]

なるほど、insertメソッドの方がよさそうですね。ちなみになんですが、ご回答のリンクで取得をするget_resultsでは成功と失敗で分岐する例がありますけど、取得でなく保存の場合にはそのような分岐はないのでしょうか？分岐がないと万が一insertに失敗してもwp_errorとか返らないわけですよね？その心配はないのでしょうか？

[marusankakun]

でもこちら（https://one-ap-engineer.com/wpdb）には成功と失敗の戻り値があるみたいなことが書いてありますね。先程私が書いた下記コードについていえば、成功と失敗はどのように処理するのがよさそうでしょうか？（その他不適切な点あればご教授願えませんでしょうか。）

$id = 123;
$yasai = 'yasai_3';
$price = 100;
$sql = $wpdb->insert("INSERT INTO $wpdb->dokuzi (id, yasai, price) values (%d, %s, %d)", $id, $yasai, $price);
$wpdb->query($sql);

[KazuhiroHatano]

失敗した場合エラーにはなりませんがfalseが返ります
成功した場合は処理した行数が返って来るはず

https://core.trac.wordpress.org/browser/tags/5.3/src/wp-includes/wp-db.php

コメのinsertメソッドの使い方が違います

[marusankakun]

すみません、あと一つだけ教えていただけませんでしょうか。

まずデータベースのyasaiの値に「ナス トマト」という値が入り、priceには「500」が入ります。

このとき、以下$yasaiの「トマト ナス」という値でも「500」を検索結果として取得したいのですが、そのような順不同的な検索のためには、$yasaiは（または$sqlは）どのように書けばよろしいのでしょうか？

$yasai = 'トマト ナス';
$sql = $wpdb->prepare("SELECT p.yasai, p.priceFROM $wpdb->dokuzi p WHERE p.search_words LIKE %s", '%'.$yasai.'%' );

[KazuhiroHatano]

そんなことをする必要性が出て来ること自体に多少問題を感じますが

普通にやるなら
WHERE yasai LIKE %トマト% AND yasai LIKE %ナス%
みたいな感じにLIKEを繋げるしかないんじゃないでしょうかね

あるいはDBにはyasaiの値は必ずソートしてから入れる様にして
検索の時もyasaiの値は渡す前にソートしてから使う様にするかですかね

[marusankakun]

LIKEしかないかと思っていたのですが、ソートしてから入れるという考え、すごいですね。お聞きして本当によかったです。ありがとうございました。