回答編集履歴

3

推敲

2022/09/08 05:01

投稿

sazi
sazi

スコア25195

test CHANGED
@@ -1,7 +1,8 @@
1
1
  > ・マジックナンバーを利用してはいけないとのご指摘を頂いたのですが、どのように修正するかわからない
2
2
  COD_CODEの数値しか該当するものは無さそうなので、それが該当するとします。
3
3
  この場合、定数などの名称付けられたもので置き換えるという事になりますけど、マスターテーブルの値を定数化するとか意味不明です。
4
- やるならマスターテーブルは使用せずに、コード値を定数として定義してそれを使用する、若しくは、コード値を数値ではなく文字にするという事になるかと思います。
4
+ やるならマスターテーブルは使用せずに、コード値を定数として定義してそれを使用する、
5
+ 若しくは、コード値を数値ではなく文字にする(COD_CODEをCOD_KBNのような値にする)という事になるかと思います。
5
6
 
6
7
  > ・SQLをもう少し分かりやすくしたい
7
8
 

2

追記

2022/09/08 04:59

投稿

sazi
sazi

スコア25195

test CHANGED
@@ -1,7 +1,7 @@
1
1
  > ・マジックナンバーを利用してはいけないとのご指摘を頂いたのですが、どのように修正するかわからない
2
2
  COD_CODEの数値しか該当するものは無さそうなので、それが該当するとします。
3
3
  この場合、定数などの名称付けられたもので置き換えるという事になりますけど、マスターテーブルの値を定数化するとか意味不明です。
4
- やるならマスターテーブルは使用せずに、コード値を定数として定義してそれを使用するという事になるかと思います。
4
+ やるならマスターテーブルは使用せずに、コード値を定数として定義してそれを使用する、若しくは、コード値を数値ではなく文字にするという事になるかと思います。
5
5
 
6
6
  > ・SQLをもう少し分かりやすくしたい
7
7
 

1

推敲

2022/09/08 04:49

投稿

sazi
sazi

スコア25195

test CHANGED
@@ -1,5 +1,5 @@
1
1
  > ・マジックナンバーを利用してはいけないとのご指摘を頂いたのですが、どのように修正するかわからない
2
- この場合COD_CODEの数値しか該当するものは無さそうなので、それが該当するとします。
2
+ COD_CODEの数値しか該当するものは無さそうなので、それが該当するとします。
3
3
  この場合、定数などの名称付けられたもので置き換えるという事になりますけど、マスターテーブルの値を定数化するとか意味不明です。
4
4
  やるならマスターテーブルは使用せずに、コード値を定数として定義してそれを使用するという事になるかと思います。
5
5