質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.13%

テーブル設計と要件を満たすカラム指定の問題

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,085

Kei227

score 36

・火曜日2限に開講されている授業一覧
を表示させるために下記のテーブルを設定し、

create table classes (
  id int primary key auto_increment,
  name text
);

insert into classes (id,name) values (1,'IT');
insert into classes (id,name) values (2,'Math');

create table weekdays (
  weekday_id int primary key auto_increment,
  weekday_name text,
  id int,
  foreign key (id) references classes (id)
);

insert into weekdays (weekday_id,weekday_name) values (1,'Sunday');
insert into weekdays (weekday_id,weekday_name) values (2,'Monday');
insert into weekdays (weekday_id,weekday_name) values (3,'Tuesday');

create table classtimes (
  classtime_id int,
  id int,
  foreign key (id) references classes (id),
  weekday_id int,
  foreign key (weekday_id) references weekdays (weekday_id)
);

insert into classtimes (id,classtime_id,weekday_id) values (1,1,1);
insert into classtimes (id,classtime_id,weekday_id) values (1,2,1);
insert into classtimes (id,classtime_id,weekday_id) values (1,3,1);
insert into classtimes (id,classtime_id,weekday_id) values (2,1,2);
insert into classtimes (id,classtime_id,weekday_id) values (2,2,2);
insert into classtimes (id,classtime_id,weekday_id) values (2,3,2);
insert into classtimes (id,classtime_id,weekday_id) values (2,1,3);
insert into classtimes (id,classtime_id,weekday_id) values (2,2,3);
insert into classtimes (id,classtime_id,weekday_id) values (2,3,3);

下記のsql文を書きましたが、

select classes.name from classtimes
inner join classes on classtimes.classtime_id = classes.id
inner join weekdays on classtimes.classtime_id = weekdays.weekday_id
where weekdays.weekday_id = 3;

下記のようにエラーが発生してしまいます。

+------+
| name |
+------+
| Math |
| Math |
| Math |
+------+

うまく出力されない理由として、

・classtimeテーブルのinsert文がおかしい

ということは考えついたのですが、具体的にどう直せばいいのか分かりません。訂正していただけないでしょうか?

よろしくお願いお願いいたします。


=======
修正:
weekdaysテーブルとclasstimesテーブルの順番がおかしかったので訂正しました。
エラー内容はEmpty setです。
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • KiyoshiMotoki

    2015/11/06 16:44

    本当にこの順番でSQLを実行しましたか?
    classtimesテーブルはweekdaysテーブルを参照しているので、
    weekdaysテーブルより先にclasstimesテーブルを作成することはできないはずです。

    また、単に
    「うまく出力されません」
    ではなく、どのような現象が起きる(または、起きない)のかを詳しく記載して下さい。
    エラーメッセージが表示されるのであれば、それも添付して下さい。

    キャンセル

  • Kei227

    2015/11/06 17:58

    編集し直しました。

    キャンセル

回答 3

checkベストアンサー

0

ご質問の内容からは2限を表現するデータがどれなのかが読み取れませんでしたが、
とりあえずclasstimesテーブルのclasstime_idカラムだと仮定して回答させていただきます。

正しいSQL文は
SELECT classes.name 
FROM classes 
INNER JOIN classtimes 
    ON classes.id = classtimes.id 
WHERE classtimes.weekday_id = 3 
    AND classtimes.classtime_id = 2;
または
SELECT classes.name 
FROM classes 
INNER JOIN classtimes 
    ON classes.id = classtimes.id 
INNER JOIN weekdays 
    ON classtimes.weekday_id = weekdays.weekday_id 
WHERE weekdays.weekday_name = 'Tuesday' 
    AND classtimes.classtime_id = 2;
です。

Kei227様のSQL文が意図通りの結果を返さない原因は、テーブル結合(INNER JOINの部分のことです)の条件付けを間違えているからです。
1つ目の結合条件ですが、classesテーブルで"授業"を一意に識別するカラムはidで、classtimesテーブルでclasses.idを参照しているカラムもidなので、
on classtimes.id = classes.id
とすべきでした。

2つ目の結合条件については、そもそもweekdaysテーブルの結合条件に同テーブルを使用していないですね。
weekdaysテーブルで"曜日"を一意に識別するカラムはweekday_idなので、1つ目の結合条件と同様に
on classtimes.weekday_id = weekdays.weekday_id
とすべきでした。

結果的に意図するデータを取得できているとしても、これではデータの内容や条件が変更されると、再び意図通りに動かなくなりますよ。


あと、weekdaysテーブルにidカラムは不要です。
"曜日"と"授業"を紐づける役割はclasstimesテーブルが担っているので、
weekdaysテーブルが直接、classesテーブルを参照する必要はないためです。

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/11/06 22:36

    詳しい解説のほど、ありがとうございます!!

    キャンセル

  • 2015/11/06 22:38

    https://teratail.com/questions/19527?complete=
    よろしかったらこちらの問題も確認していただけないでしょうか?

    キャンセル

  • 2015/11/06 23:04

    嫌ですw

    理由は
    https://teratail.com/questions/19527
    の追加・修正依頼に記載させていただきました。

    キャンセル

0

classtimesのテーブルで、classtime_id int primary key auto_incrementとプライマリーキーとして使ってしまっているので、これをclasses.idと結びつけようとしてもまったく無関係なものとなってしまい、うまくいきません。

classtimes.idclasses.idと外部キーで結ばれているので、INSERT時にclasstimes.idに適切な値を入れて、こちらでJOINしましょう。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/11/06 17:18

    classtime_idのprimary keyを外し、insert文を下記のように編集したのですがEmpty setと出てしまいます。

    ==========
    insert into classtimes (classtime_id,weekday_id) values (1,1);
    insert into classtimes (classtime_id,weekday_id) values (2,1);
    insert into classtimes (classtime_id,weekday_id) values (3,1);
    insert into classtimes (classtime_id,weekday_id) values (4,1);
    insert into classtimes (classtime_id,weekday_id) values (5,1);
    insert into classtimes (classtime_id,weekday_id) values (1,2);
    insert into classtimes (classtime_id,weekday_id) values (2,2);
    insert into classtimes (classtime_id,weekday_id) values (3,2);
    insert into classtimes (classtime_id,weekday_id) values (4,2);
    insert into classtimes (classtime_id,weekday_id) values (5,2);
    insert into classtimes (classtime_id,weekday_id) values (1,3);
    insert into classtimes (classtime_id,weekday_id) values (2,3);
    insert into classtimes (classtime_id,weekday_id) values (3,3);
    insert into classtimes (classtime_id,weekday_id) values (4,3);
    insert into classtimes (classtime_id,weekday_id) values (5,3);
    ==========

    *月曜日の1~5限、火曜日の1~5限のようなことを意図して書いています。

    キャンセル

  • 2015/11/06 18:11

    1コマの授業を定義する上で、曜日・時間・科目の3つ組が必要だと思うのですが…上のINSERTで「科目」は指定もしていないですよね?

    キャンセル

0

最後に【limit1】をつけることで解決できました。 

select classes.name from classtimes
inner join classes on classtimes.classtime_id = classes.id
inner join weekdays on classtimes.classtime_id = classes.id
where weekdays.weekday_id = 3 and classtime_id = 2 limit 1;

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2015/11/06 22:22 編集

    このSQL文は誤っています。
    回答に詳しい説明をさせていただきますので、ご確認ください。

    キャンセル

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.13%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る