どう対処する？【Webシステムのデータ検索機能の改修】

Question

前提・実現したいこと

レベルが定められたデータがあり、
登録者のレベルから±2に該当するデータを
表示できるような検索機能を追加をしたい。

どのような手段が最善か議論したいです。
（質問の仕方も意見もらえると助かります）

起きている問題・自分の対処

■起きている問題
データのあるテーブルを見ると
レベルに応じてidが連番であてられていない。
（下記のような構成）

id|level_name
1|初級1-1
2|初級1-2
3|初級1-3
4|初級1-4
5|初級1-5
101|中級1-1
102|中級1-2
103|中級1-3
104|中級1-4
105|中級1-5

■自分の対処
そのレベルテーブルは今後変更されないと伺ったため
（されてもその場合は相当先）
以下のようなコードで乗り切ろうと思う。

if($levelID == 1){
  $minLevel=1;
  $maxLevel=3;
}elseif($levelID == 2){
  $minLevel=1;
  $maxLevel=101;  
}elseif($levelID == 4){
  $minLevel=3;
  $maxLevel=101;  
}elseif($levelID == 5){
  $minLevel=3;
  $maxLevel=102;  
}elseif($levelID == 101){
  $minLevel=4;
  $maxLevel=103;
}elseif($levelID == 102){
  $minLevel=5;
  $maxLevel=104;
}elseif($levelID == 104){
  $minLevel=102;
  $maxLevel=201;
}elseif($levelID == 105){
  $minLevel=103;
  $maxLevel=201;
}else{
  $minLevel=$levelID - 2;
  $maxLevel=$levelID + 2;
}

補足情報

ちなみに他手段としては以下だが
デメリットを考慮するとその手段を取らない決断に至った。

・idの振り直し
→他このような処理がないとも言えない。
古いシステムなので結構闇が深い。
id(101)などでファイル全件検索を掛けても良いが
それだけでは不安が拭えないため対応を見送った。

・連番の新規カラムを追加
→恥ずかしながら新規カラム追加を行ったことがない。
起こりうる不具合が把握できていなさすぎると感じため対応を見送った。
こういったコードが記載されていると不具合となる、
など見識のある方、ご教授いただけると幸いです。

Comments

[June666]

>レベルが定められたデータがあり、
登録者のレベルから±2に該当するデータを
表示できるような検索機能を追加をしたい。
・・・
データのあるテーブルを見るとレベルに応じてidが連番であてられていない。

私の頭ではいまいち理解できないので、そのテーブルを表で記載して頂くか、CREATE文の追記が欲しいです。

>ちなみに他手段としては以下だが
以下とはどれを指すのでしょうか？

[Schneider]

ご返信ありがとうございます。
不備を補完させていただきました。

[hentaiman]

プラマイ２しないピンポイントのデータは表示できるのか

[Schneider]

>hentaimanさん
はい、状況としては当授業（ピンポイントのデータ）に再受講というボタンを設けてピンポイントのデータ（id）はPOSTしようと思っています。質問のコードでいうと$levelIDがそれに該当します。

[hentaiman]

質問に書いてあるデータだけだとidとレベルは連番に見えるが、本当に連番ではないのか（連番の意味を勘違いしていないか

[q_sane_q]

少なくともレベルに沿ってidが昇順にはなっているのでしょうか？
それともそれすら定かではない？

[Schneider]

>hentaimanさん
> q_sane_qさん
正確に記載すると以下のテーブルです。
初級、中級毎では連番になっている認識です。
（あいまいになってしまいすみません。。）
はい、レベルに沿ってidは昇順です。

----
id|level_name
1|初級1-1
2|初級1-2
3|初級1-3
4|初級1-4
5|初級1-5
101|中級1-1
102|中級1-2
103|中級1-3
104|中級1-4
105|中級1-5
106|中級1-6
107|中級1-7
201|中級2-1
202|中級2-2
203|中級2-3
204|中級2-4
205|中級2-5
206|中級2-6
----

Answer 1

$levelID = 101;

$data = [1,2,3,4,5,101,102,103,104,105];
$cnt = count($data);

$idx = array_search($levelID, $data);
if($idx!==false){
    $minIdx = $idx>1 ? $idx - 2 : 0;
    $maxIdx = $idx<$cnt-2 ? $idx+2 : $cnt-1;
    
    echo $data[$minIdx]."_".$data[$maxIdx];
    //4_103
}else{
    echo "Unknown";
}

Answer 2

idがレベルに沿って昇順なら

id	min_level_id	max_level_id
1	1	3
5	3	102

こんなテーブルを作って組み合わせるとか（VIEWつくったり）
もっと複雑な組み合わせ対応が必要なら

id	link_level_id
5	3
5	4
5	5
5	101
5	102

こんなテーブルを作って組み合わせるとか

Answer 3

<?php

function getDisplayLevels($userLevel)
{
    $levels = [
        [
            'id' => 1,
            'name' => '初級1 - 1'
        ],
        [
            'id' => 2,
            'name' => '初級1 - 2'
        ],
        [
            'id' => 3,
            'name' => '初級1 - 3'
        ],
        [
            'id' => 4,
            'name' => '初級1 - 4'
        ],
        [
            'id' => 5,
            'name' => '初級1 - 5'
        ],
        [
            'id' => 101,
            'name' => '中級1 - 1'
        ],
        [
            'id' => 102,
            'name' => '中級1 - 2'
        ],
        [
            'id' => 103,
            'name' => '中級1 - 3'
        ],
        [
            'id' => 104,
            'name' => '中級1 - 4'
        ],
        [
            'id' => 105,
            'name' => '中級1 - 5'
        ]
    ];
    $index = 0;
    foreach ($levels as $i => $level) {
        if ($level['id'] === $userLevel) {
            $index = $i;
            break;
        }
    }
    $res = [];
    for ($ite = $index - 2; $ite <= $index + 2; $ite++) {
        if (isset($levels[$ite])) {
            $res[] = $levels[$ite];
        }
    }
    return $res;
}

$userLevel = 5;
$levels = getDisplayLevels($userLevel);
var_dump($levels);

Answer 4

$levels = [1, 2, 3, 4, 5, 101, 102, 103, 104, 105, ...];
$found = -1;
for ($i = 0; $i < count($levels); $i++)
if ($levels[$i] == $levelID) {
$found = $i; break;
}
$minLevel = $levels[$found>=2 ? $found-2 : 0];
$maxLevel = $levels[$found+2 >= count($levels) ? count($levels) : $found+2];

Comments

[Schneider]

ご返信ありがとうございます。
はい、
WHERE id BETWEEN $minLevel AND $maxLeve
のような実装をしようと思っています。
ただ、下記のようなidで直接判別するのが果たして良いコードと
言えるのか、と思い質問させていただきました。
idで直接判別するような方法以外で
良い実装方法があればご教授いただきたいです。
----
if($levelID == 1){
$minLevel=1;
$maxLevel=3;
}elseif($levelID == 2){
$minLevel=1;
$maxLevel=101;
}elseif($levelID == 4){
$minLevel=3;
$maxLevel=101;
}elseif($levelID == 5){
$minLevel=3;
$maxLevel=102;
}elseif($levelID == 101){
$minLevel=4;
$maxLevel=103;
}elseif($levelID == 102){
$minLevel=5;
$maxLevel=104;
}elseif($levelID == 104){
$minLevel=102;
$maxLevel=201;
}elseif($levelID == 105){
$minLevel=103;
$maxLevel=201;
}else{
$minLevel=$levelID - 2;
$maxLevel=$levelID + 2;
}
----