これらの文字列の正しいチェック構文は何でしょうか？

Question

私はフォルダK：/ Comicsをチェックし、それぞれの名前+番号をデータベースのテーブル名=コミックに挿入するスクリプトを書いています。今私がしたいのは、挿入クエリを実行する前にこのコミックが既に存在するかどうかを確認することです。

テーブルの構造：

CREATE TABLE IF NOT EXISTS `comics` (
    `id` int(100) NOT NULL AUTO_INCREMENT, 
    `name` varchar(255) NOT NULL, 
    `issue` varchar(4) DEFAULT NULL, 
    `bio` longtext NOT NULL, 
    `pages` int(10) NOT NULL, 
    `size` varchar(100) NOT NULL, 
    `price` varchar(11) NOT NULL, 
    PRIMARY KEY (`id`) 
) ENGINE=MyISAM DEFAULT CHARSET=latin1 ; 

コード：

$query = 'INSERT IGNORE INTO comics (name, issue) VALUES '.implode(',', $values); 
$result = mysql_query($query); 
echo ($result) ? 'Inserted successfully' : 'Failed to insert the values'; 

：私は仕事だろうと思ったがない（まだ、すでに存在しているデシベルに漫画を追加します）どういう

<?php 
    $main_folder = 'K:/Comics/'; // should be K:\Comics\ but I changed it because of the highlighting issue 
    $folders = glob($main_folder.'* [0-9]*', GLOB_ONLYDIR); 

    $comics_series = array(); 
    foreach($folders as $folder){ 
     $comics_series[] = preg_split('/(.+)\s(\d+)/', str_replace($main_folder, '', $folder), -1, PREG_SPLIT_DELIM_CAPTURE|PREG_SPLIT_NO_EMPTY); 
    } 

    $values = array(); 
    foreach($comics_series as $pair){ 
     $values[] = "('".mysql_real_escape_string($pair[0])."', '".((int) $pair[1])."')"; 
    } 

    $query = 'INSERT IGNORE INTO comics (name, issue) VALUES '.implode(',', $values); 
    $result = mysql_query($query); 
    echo ($result) ? 'Inserted successfully' : 'Failed to insert the values'; 
?> 

私は何を忘れましたか？ドキュメントにはIGNOREを追加するだけでいいです...

Answer 1

<?php 
$main_folder = 'K:/Comics/'; 
$folders = glob($main_folder.'* [0-9]*', GLOB_ONLYDIR); 

$comics_series = array(); 
foreach($folders as $folder){ 
    $comics_series[] = preg_split('/(.+)\s(\d+)/', str_replace($main_folder, '', $folder), -1, PREG_SPLIT_DELIM_CAPTURE|PREG_SPLIT_NO_EMPTY); 
} 

$values = array(); 
foreach($comics_series as $pair){ 
    // clean the values to protect against SQL injection 
    $pair = array(
     mysql_real_escape_string($pair[0]), 
     mysql_real_escape_string($pair[1]) 
    ); 
    // add it to the values array, for insert 
    $values[] = "('".$pair[0]."', '".$pair[1]."')"; 
} 

$query = 'INSERT INTO comics (name, issue) VALUES '.implode(',', $values).' '. 
     'ON DUPLICATE KEY UPDATE `issue` = VALUES(`issue`)'; 
$result = mysql_query($query); 
echo ($result) ? 'Inserted successfully' : 'Failed to insert the values'; 
?> 

Answer 2

mysql_query()結果のリソースではなく、整数や値のセットなどを返します。しかし

if (mysql_num_rows($check_query) == 0){ 

に

if ($check_query == '0'){ 

を変更してみてください、私はおそらくちょうど使用します。

INSERT IGNORE INTO ... 

それはおそらく問題ではありませんので。詳細については、MySQL Insert Syntaxをご覧ください。

Answer 3

mysql_queryは、クエリが成功した場合はステートメントハンドルを返し、クエリが失敗した場合はブール型のFALSEを返します。

$check_query_result = mysql_query(...) or die(mysql_error()); 

if(mysql_num_rows($check_query_result) == 0) { 
    ... comic doesn't exist ... 
} 

ローを返さないクエリは失敗条件ではありません。行を持たないだけの結果です。

Answer 4

doesn't workあまりにも有益ではありません。

• 利用代わり== '0'のmysql_num_rowsは、それは本当に無意味だ
• $values周りに囲む引用符がありません。
• Iは、テーブル内の別個であることが、これらの値の組み合わせを強制するテーブルのnameとissue列にcombined unique indexを置くことになる括弧

Answer 5

で値をラップ。

あなたは、その後、二重の記録を心配以下としませんでした：

$query = 'INSERT INTO comics (name, issue) VALUES '.implode(',', $values).' 
      ON DUPLICATE KEY UPDATE id=id'; //makes sure you don't get an error 

（@Dereleasedで述べたように）また重複の挿入を避けるためにをINSERT IGNORE声明を使用することができます（前で指定されたunique index）。

ON DUPLICATE KEY UPDATEステートメントのドキュメントも参照してください。挿入中に複製として見つかったレコードを更新する場合に役立ちます。