ログイン
i-mobile
記事の一覧
4.<I <3 u | [はてな][Firefox]は...>6.

2009-03-09

[][]Firefox 3 Hacks にあるSQLがあまりに長い件 Add Star

Firefox 3 Hacks ―Mozillaテクノロジ徹底活用テクニック
江村秀之,池田譲治,下田洋志,松澤太郎,dynamis
オライリージャパン

今更ながらじっくり読み始めた。で、やはり読み応えのあるのは「3章 Firefox3向けの拡張機能開発テクニック」であるが、SQLがあまりに長い。解析に一苦労である。

例えば、

P.153 2つのタグいずれか(和集合)を持つブックマークの照会

SELECT DISTINCT
       (SELECT title FROM moz_bookmarks
          WHERE fk = p.id AND title IS NOT NULL),
       p.title, p.url
  FROM (SELECT *
          FROM moz_bookmarks b
            JOIN moz_bookmarks parent ON b.parent = parent.id
          WHERE parent id =
            (SELECT id FROM moz_bookmarks
               WHERE title = ?1 AND parent =
                 (SELECT folder_id FROM moz_bookmarks_roots
                    WHERE root_name = 'tags'))
            OR parent.id =
            (SELECT id FROM moz_bookmarks
               WHERE title = ?2 AND parent =
                 (SELECT folder_id FROM moz_bookmarks_roots
                    WHERE root_name = 'tags'))
       ) tagged
       JOIN moz_places p ON tagged.fk = p.id

長すぎである。もちろんPlacesデータベースがtagsテーブルを持たない構造になっているからだが、それにしても複雑で長すぎる。良く見てみると

            (SELECT id FROM moz_bookmarks
               WHERE title = ?1 AND parent =
                 (SELECT folder_id FROM moz_bookmarks_roots
                    WHERE root_name = 'tags'))

がほとんど同じ形で2回出てくる。これなんか無駄であろう。最適化するとどうなるかは知らないが、2回同じように照会しているようで気持ち悪い。

ってことで、書き直し。

SELECT
  b2.title, p.title, p.url
FROM
  moz_places p, moz_bookmarks b1, moz_bookmarks b2
WHERE
  p.id = b1.fk
  AND b1.fk = b2.fk
  AND b2.title IS NOT NULL
  AND b1.parent in (
    SELECT id FROM moz_bookmarks
      WHERE parent = (SELECT folder_id FROM moz_bookmarks_roots WHERE root_name = 'tags')
        AND title in (?1, ?2)
  )

これで良いんじゃね?

ついでに、次ページにある2つのタグを持つ(積集合)ブックマークの照会では

SELECT
  b2.title, p.title, p.url
FROM
  moz_places p, moz_bookmarks b1, moz_bookmarks b2
WHERE
  p.id = b1.fk
  AND b1.fk = b2.fk
  AND b2.title IS NOT NULL
  AND b1.parent in (
    SELECT id FROM moz_bookmarks
      WHERE parent = (SELECT folder_id FROM moz_bookmarks_roots WHERE root_name = 'tags')
        AND title in (?1, ?2)
  )
  GROUP BY b2.fk HAVING count(b2.fk) = 2

GROUP BY b2.fk HAVING count(b2.fk) = 2で良くなるし、3つになっても少しの変化で対応できる。

まぁいまさら指摘しても遅いし、SQLの本じゃないからどうでも良いことか(ぉ

piro_or ●2009/03/09 23:58
SQLのことをよく分からないまま書いたのがモロバレですね><
teramako ●2009/03/10 15:05
最後に書きましたが、SQLの本じゃないので気にしなくて良いと思いますよ。
teramako ●2009/03/10 20:43
SQL間違っていたので修正
▼名前
▼内容
▼画像認証

4.<I <3 u | [はてな][Firefox]は...>6.
●ウェブ検索●