2012-02-01 11 views
1

私は1つのスタックオーバーフローの用途に似たタグシステムを作成しています。タグのデータベースに対して新しく入力されたタグをチェックし、既存のタグのキーを使用する一致が見つかるとアイテムに接続するためのものです。私はこの検証プロセスの問題に取り組んでいます。私はそれについてどうやって行くのか分からない。これは私がこれまで持っていたものです。タグシステムの構築と検証

$sql = $dbRead->quoteInto("SELECT * FROM item_tag WHERE tag IN (?)", $tag_where); 

$tag_result = $dbRead->fetchAll($sql); 

    $i = 0; 
    while ($i < count($tags)) { 

if (in_array($tags[$i], $tag_result)) { 

    $array_key = array_search($tags[$i], $tag_result); 

    } 
else { 
    $data = array ('tag' => $tags[$i]); 
$dbWrite->insert('item_tag', $data); 
$tag_ID = $dbWrite->lastInsertId(); } 

$data = array('itemID' => $item_ID, 
       'tagID' => $tag_ID); 
$dbWrite->insert('item_tag_connection', $data); 
++$i; 
    } 

この問題はif elseの一部です。

if (in_array($tags[$i], $tag_result)) { 

    $array_key = array_search($tags[$i], $tag_result); 

    } 

ユーザが入力したタグでいっぱいの配列でタグテーブルを照会しました。私はタグとタグの両方の主キーを持つ配列に結果をパッケージ化しました。私はその後、各ユーザーが入力したタグを調べ、アレイに対してチェックしました。私が問題に遭遇した部分は、タグにヒットしたときにそのプライマリキーを取得しています。

私はこの特別なやり方で設定されていません。それほど複雑ではない同じ結果を達成する別の方法があるなら、私はそれを聞いてみたいと思う。

答えて

1

これは、アレイ$tag_resultの使用方法です。 $tag_resultはタグの配列ではありませんが、行の配列です。各行contais、限り私はあなたのコードからtagIDtagを伝えることができます。これを行うための最善の方法ではなく、(非常に非常に非効率的である)データベースからすべてのタグを尋ねると、あなたがこのような何かを照会、ある

$tag_result = [ {tagID: 1, tag: "php"} , {tagID: 2, tag: "mysql"}, {tagID: 3, tag: "tags"} ] 

$sql = $dbRead->quoteInto("SELECT * FROM item_tag WHERE tag IN (?)", "'" . join("','", $tags) . "'"); 
だからあなた $tag_result配列は次のようになります
4

私はストアドプロシージャを使用して、PHPコードからMySQLに1回の呼び出しを行います。ストアドプロシージャが醜い少し:(

例を見えるかもしれませんので、残念ながら、MySQLは配列型のparamsとdoesntの支持台の戻り値の型を扱うに素晴らしいではありませんコール:

-- usage: call insert_question(<user_id>,<question>,<tags>,<separator>); 

call insert_question(1,'why are stored procs useful ?', 'database,mysql,stored-procedures,kiss,performance',','); 

$sql = sprintf("call insert_question(%d,'%s','%s','%s')", $userID,$ques,$tags,$separator); 

出力:

call insert_question(1,'why are stored procs useful ?', 'database,mysql,stored-procedures,kiss,performance',','); 

mysql> select * from tags order by tag_id; 
+--------+-------------------+ 
| tag_id | tag    | 
+--------+-------------------+ 
|  1 | database   | 
|  2 | mysql    | 
|  3 | stored-procedures | 
|  4 | kiss    | 
|  5 | performance  | 
+--------+-------------------+ 
5 rows in set (0.00 sec) 

mysql> select * from questions order by question_id; 
+-------------+---------+-------------------------------+---------------------+ 
| question_id | user_id | question      | created_date  | 
+-------------+---------+-------------------------------+---------------------+ 
|   1 |  1 | why are stored procs useful ? | 2012-02-02 00:54:26 | 
+-------------+---------+-------------------------------+---------------------+ 
1 row in set (0.00 sec) 

mysql> select * from question_tags order by tag_id, question_id; 
+--------+-------------+ 
| tag_id | question_id | 
+--------+-------------+ 
|  1 |   1 | 
|  2 |   1 | 
|  3 |   1 | 
|  4 |   1 | 
|  5 |   1 | 
+--------+-------------+ 
5 rows in set (0.00 sec) 
mysql> select * from tags order by tag_id; 
+--------+-------------------+ 
| tag_id | tag    | 
+--------+-------------------+ 
|  1 | database   | 
|  2 | mysql    | 
|  3 | stored-procedures | 
+--------+-------------------+ 
3 rows in set (0.00 sec) 

mysql> select * from questions order by question_id; 
Empty set (0.00 sec) 

mysql> select * from question_tags order by tag_id, question_id; 
Empty set (0.00 sec) 

今はSPROCを呼ぶことにします

スクリプト:ここ

フルスクリプト:http://pastie.org/3299425

-- TABLES 

drop table if exists users; 
create table users 
(
user_id int unsigned not null auto_increment primary key, 
username varchar(32) unique not null 
) 
engine=innodb; 

drop table if exists tags; 
create table tags 
(
tag_id smallint unsigned not null auto_increment primary key, 
tag varchar(255) unique not null 
) 
engine=innodb; 

drop table if exists questions; 
create table questions 
(
question_id int unsigned not null auto_increment primary key, 
user_id int unsigned not null, 
question varchar(512) not null, 
created_date datetime not null 
) 
engine=innodb; 

drop table if exists question_tags; 
create table question_tags 
(
tag_id smallint unsigned not null, 
question_id int unsigned not null, 
primary key (tag_id, question_id) -- clustered composite pk 
) 
engine=innodb; 

-- PROCS 

drop procedure if exists insert_question; 

delimiter # 
create procedure insert_question 
(
in p_user_id int unsigned, 
in p_question varchar(512), 
in p_tags_csv mediumtext, -- comma separated plz 
in p_separator char(1) 
) 
/* 
usage: 
call insert_question(1,'why are stored procs useful ?', 'database,mysql,stored-procedures,kiss,performance',','); 

$sql = sprintf("call insert_question(%d,'%s','%s','%s)", $userID,$ques,$tags,$separator); 

*/ 

proc_main:begin 

declare v_question_id int unsigned default 0; 
declare v_done tinyint unsigned default 0; 
declare v_idx int unsigned default 1; 
declare v_tag varchar(255) default null; 

-- validate input params 

if p_separator is null or length(p_separator) <= 0 then 
    set p_separator = ','; 
end if; 
if p_question is null or length(p_question) <= 0 then 
    leave proc_main; 
end if; 
if p_tags_csv is null or length(p_tags_csv) <= 0 then 
    leave proc_main; 
end if; 

-- split the tags into a memory table (ugly bit as mysql doesnt support table types) 

drop temporary table if exists tmp_tags; 

create temporary table tmp_tags(
tag_id smallint unsigned null, 
tag varchar(255) 
)engine = memory; 

while not v_done do 

    set v_tag = trim(substring(p_tags_csv, v_idx, 
    if(locate(p_separator, p_tags_csv, v_idx) > 0, 
     locate(p_separator, p_tags_csv, v_idx) - v_idx, length(p_tags_csv)))); 

    if length(v_tag) > 0 then 
    set v_idx = v_idx + length(v_tag) + 1; 
    insert into tmp_tags(tag) values(v_tag); 
    else 
    set v_done = 1; 
    end if; 
end while; 

-- which tags do we already have ? 

update tmp_tags tt 
inner join tags t on t.tag = tt.tag 
set tt.tag_id = t.tag_id; 

-- insert tags 

insert into tags (tag) select tag from tmp_tags where tag_id is null; 

update tmp_tags tt 
inner join tags t on t.tag = tt.tag 
set tt.tag_id = t.tag_id 
where 
tt.tag_id is null; 

-- insert question and question_tags 

insert into questions (user_id, question, created_date) values (p_user_id, p_question, now()); 

set v_question_id = last_insert_id(); 

insert into question_tags 
select distinct tag_id, v_question_id from tmp_tags; 

-- return output 

select 
p_question, 
u.user_id, 
u.username, 
v_question_id, 
tt.* 
from 
tmp_tags tt 
inner join users u on u.user_id = p_user_id 
order by 
tt.tag_id; 

-- cleanup 

drop temporary table if exists tmp_tags; 

end proc_main # 

delimiter ; 

-- TEST DATA 

insert into users (username) values ('f00'); 
insert into tags (tag) values ('database'),('mysql'),('stored-procedures'); 

-- TESTING 

select * from users order by user_id; 
select * from tags order by tag_id; 
select * from questions order by question_id; 
select * from question_tags order by tag_id, question_id; 

call insert_question(1,'why are stored procs useful ?', 'database,mysql,stored-procedures,kiss,performance',','); 

select * from tags order by tag_id; 
select * from questions order by question_id; 
select * from question_tags order by tag_id, question_id; 

・ホープ、このことができます:)

編集:

OFC、他のRDBMSで(SQLサーバ、オラクル)sprocは簡単です:)

+0

うわー、いい仕事。おそらくこのシステムでは少し複雑すぎるかもしれませんが、ストアドプロシージャの追加の露出は良いことです。多くのシステムはそれらを無視して複雑な作業を困難にしています。 – Aatch

+0

残念ながら、このサーバー側のソリューションを複雑にするのはMySQLです。たとえば、SQLサーバにはsprocをいくつかのステートメントに単純化するテーブルタイプなどがありますが、感謝します:) –

関連する問題