[その他] そのプレフィックスいる？　＆　予備・・・　（SQL）-ウンコード・マニア

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...

鑑賞する »