toshizooooooo 2012-08-24 10:46:16

[その他] そのプレフィックスいる? & 予備・・・ (SQL) このエントリーをはてなブックマークに追加

投稿者からのアピールポイント

ネーミングが日本語のローマ字っていうのも気になるが、いくらDB設計書のカラム名に「社員~」って書いてあったからって全てに「syain_」つけなくても・・・。 よほどローンチ後にDBスキーマ変更したくないのか、yobi1~yobi10をすべてのテーブルに必ずつけなければいけないルールだった。その後、実際テーブルによってはyobi3あたりまでは使われた。

create table syain (
syain_bango int not null primary key,
syain_kubun int not null,
syain_mei varchar(20) not null,
syain_seibetsu varchar(2) not null,
syain_seinengappi date not null,

/* ・・・以下"syain_~"が続く*/

yobi1 varchar(200) null,
yobi2 varchar(200) null,
yobi3 varchar(200) null,
yobi4 varchar(200) null,
yobi5 varchar(200) null,
yobi6 varchar(200) null,
yobi7 varchar(200) null,
yobi8 varchar(200) null,
yobi9 varchar(200) null,
yobi10 varchar(200) null
)

使い方ヒント: 「これは臭う」という行を見付けたら、各行のsmellをクリックしてマーキングしておきましょう(要Twitter OAuth認証)

コメント(9)

#1 math_neko 2012-08-24 10:48:01  

DB 設計のやり方から出直してまいれ、だな

#2 m_shige1979 2012-08-24 11:08:30  

規約によってはローマ字であることは許容できるが、この場合は、プレフィックスは項目名に付ける意図がわからんな

#4 Verna_Velna 2012-08-26 02:37:02  

SQLの中にsyain.syain_seibetsuとかが頻出してそうだな。

#5 sindoguest1 2012-08-27 01:56:18  

DB内でユニークな名前だとテーブル名を省略できる事があるってんで、こういう名前をつける人は居るが、yobi1~yobi10は別テーブルでいいだろ......

#7 koroharo 2012-08-28 23:13:19  

yobi に shain_ がついてない方が気になる。クソコードだとしても、一貫性をもったクソコードであって欲しい。

#8 michieru 2012-08-29 16:25:39  

もしかして、natural join対策? まさかね

#9 kokkekun 2012-11-01 06:04:27  

古臭いという意味で臭うのは確かだけど、ローマ字はいちいち予約語覚えなくていいように。カラムの接頭辞はJOINでいちいちエイリアス考えなくていいように、じゃないかな。yobiは知らんけど。

コメント投稿には、twitter認証が必要です。

Twitter認証

このウンコードに臭った人は、こちらのウンコードにも臭ってます

[その他] HOW TO ABC..

このエントリーをはてなブックマークに追加

実際にあった某システムの超重要なマスター...

create table item_master (
 A varchar(2...

鑑賞する »

[Java] フルHD推奨。

このエントリーをはてなブックマークに追加

画面の左側がわりと無駄な気がします。

public int hoge(){
    int result = 0;
...

鑑賞する »

[PHP] switch文の乱用

このエントリーをはてなブックマークに追加

臭うファイルがあったので開いてみたら・・...

switch($mode){
case "a":
case "b":
ca...

鑑賞する »