仕様の変更に強いコードを書きたいよねって話 - Qiita

この記事は NIJIBOX Advent Calendar2019の13日目の投稿です。 背景 何かしらのロジックを作る際に、仕様変更に強いコードを書きたいぞい！ってエンジニアだったら思いませんか。今の仕様なら動くけど、もし仕様が変わり、そのために関数全書き直しとかしんどみが深すぎます。今回はこのしんどみを少しでも回避できるように柔軟なコードを書くぞい！って記事です。 ページネーションコンポーネントを例にしますが、なぜページネーションなのかというと僕が最近業務でページネーションを作り、かつ仕様の変更に強いコードの大切さを実感したからです。 そもそもページネーションとは ページネーション（pagination）とは、日本語で丁付け、ページ割りという意味で、Web制作においては、検索結果一覧など、内容の多いページを複数のWebページに分割し、各ページへのリンクを並べてアクセスしやすくするために

[sdkd6611]

結局は、今作ってるものがどこまで変わる可能性あるのかをしっかり見つめることだよな。可能性考慮しすぎてもダメ、しなさすぎてもダメ。

[strow0343]

こういうのは設計そのものより設計根拠を何かしらの形で残す方を重視した方がいい。仕様変更が入った際に変える箇所・不要になって捨てる箇所が明確になるように

[megumin1]

悪記事。可読性は相当悪くなっている。特にrange(), getPageNums()等が一見何をやっているのかさっぱりわからない。関数名が曖昧、さらに中でr,s,eとか一文字変数を使っていたり、なにをするコードなのかさっぱりわからない。

[fuji_haruka]

start = Math.max(0, Math.min(total - size, s)) にするとさらに循環的複雑度が減るし可読性が上がりますね／コメントに書いた

[lacucaracha]

そしていざ仕様変更が生じたときに、引き継いだ人が意図に気づける設計でないといけないもんなあ。エンジニアは難しい商売だ。

[ledsun]

変更を予測して抽象レイヤーを入れて、予測が外れて抽象レイヤーが邪魔になって、予測を諦めて、やっぱり予測してを繰り返していい塩梅を見つけることを「経験を積む」というので、やっていこう！

[ka2jun8]

議論するのにいい題材なんだろうな。

[buhoho]

これを履き違えて不必要に重厚なレイヤー、複雑な実装になってて、小回り効かずむしろ変更に手間がかかる。みたいなの見かける/記事例もちょっとやり過ぎ感ある…

[gabari]

「※ sizeは表示するリンクの数、totalは総ページ数のこと」という分が説明のために書かれている時点で負けている（リーダブルではない）

[shikiarai]

こういう姿勢だけでも弊社的にはすごく価値があるけど伝わるかな……

[masayoshinym]

ちょっとした例題くらいの気持ちで書いただろうコードがブコメでタコ殴りにされてるのを見るにつけ、怖くてQiitaに投稿なんかできねぇよって気持ちになる。

[Error401]

申し訳ないけど、このメソッドを書き始める時点で、current, total, size(変数名は原文ママ)を仮引数にしない時点で、プログラマに向いてないんじゃないのまで思っちゃう

[Makeneko]

「仕様変更に強いコードを書きたいぞい！ってエンジニアだったら思いませんか。」昔は私もそう思った時期がありました。皆が通る道なのだろうか。

[diveintounlimit]

当人には申し訳ないけど、そういう事言う以前のコードだな。。という感想。絶対にあり得ない可能性のために無駄に重厚で動作も重たいコードを書くやつなら居た。最終的にゴリっと削られたけど。YAGNI大切。

[Phenomenon]

テストは始めに書いた方が楽だよ

[ene0kcal]

「仕様の変更に強いコード」という欲に負けてはいけません。問題なのは、場当たり的にコーティングすること。関数名、変数名が的確でないこと。ページ数なのにsizeってダメダメ。一文字変数もダメです。

[kazuau]

rangeのところ、自分の学習結果で他意はない　return new Array(size).fill(Math.floor(start) + 1).map((v, i) => v + i);

[versatile]

実は、元のコードのほうが意図が伝わりやすく、読みやすい。でも変更には弱い。残念

[nakag0711]

ページネーションに付随する処理ではあるが…

[magnoliak]

端的に大事なことが全部まとまってて、分かりやすい！

[nyop]

書きたいよね。

[n314]

あれ？良い題材だなと思ったけどなんか違うな。ページネーションって合計行数と最大リンク表示数と1ページ表示行数を渡してリンク用データを得るやつじゃないのか。

[atsushifx]

分かっていることだけど。TDD,リファクタリング、デザインパターンという神器があってこそ

[ryuichi1208]

大事。

[n_231]

僕ならrange関数はstartとsizeを受け取ってsize回くりかえすようにするかな。そのfor文怖くない？まあいいんだけど。

[L-Tip]

大事なことだと思います。[Program][コード]

[yosuke_furukawa]

最近技術顧問でペアプロした時にコメントしたページネーションの話ですー！