関数は機能ごとに分けるべきか？機能を引数に渡して関数はまとめるべきか？

Question

ご質問

関数を作るとき、機能ごとに関数を分けますか？機能を引数で渡して分岐し関数はまとめますか？

次のような例でふと思いました。

例

【例１】
INSERT、DELETE、UPDATE の場合

<?php
// Ａ：機能ごとに関数を分ける
function insert_row( $args ){}
function delete_row( $args ){}
function update_row( $args ){}

// Ｂ：機能を引数で渡して分岐し関数はまとめる
function apply_row( $type, $args ){
	if( $type === 'insert' ){}
	else if( $type === 'delete' ){}
	else if( $type === 'update' ){}
}
?>

【例２】
ローカルストレージの値を取得する場合

// Ａ：機能ごとに関数を分ける
function getFontFamily(){
	return localStorage.getItem('font-family') || 'serif';
}
function getMode(){
	return localStorage.getItem('mode') || 'normal';
}

// Ｂ：機能を引数で渡して分岐し関数はまとめる
function getStorage( type ){
	if( type === 'font-family' ){
		return localStorage.getItem('font-family') || 'serif';
	}else if( type === 'mode' ){
		return localStorage.getItem('mode') || 'normal';
	}
}   

いかがでしょうか。

「どういうときにＡのように分けて、どういうときにＢのようにまとめる」など、みなさんがユルめに従いがちな、慣習的なものがあれば知りたいです。

私の場合

私は独学で素人で（なのでここでやり方を述べるのは恐縮ですが）、これまで「基本はＢでまとめるけど、関数が長くなりすぎたらＡで分ける」というなんとなくな方法でした。

または次のような感じで、Ａのように分けると、引数のキーが$args = ['id'=>1',name'=>'a','age'=>20,,,,] などと多いときに同じ展開処理が複数の関数で重複するため、これを嫌ってＢでまとめること多いです。（PHPならextractがありますが、どの変数がextractで展開されたものなのかわかりにくいので使いたくない派です。）

// Ａ：機能ごとに関数を分ける
function insert_row( $args ){
	// ここでも展開
	$id = $args['id'];
	$name = $args['name'];
	$age = $args['age'];
}
function delete_row( $args ){
	// ここでも展開
	$id = $args['id'];
	$name = $args['name'];
	$age = $args['age'];
}
function update_row( $args ){
	// ここでも展開
	$id = $args['id'];
	$name = $args['name'];
	$age = $args['age'];
}

// Ｂ：機能を引数で渡して分岐し関数はまとめる
function apply_row( $type, $args ){
	// 展開がこの１回で済む
	$id = $args['id'];
	$name = $args['name'];
	$age = $args['age'];

	if( $type === 'insert' ){}
	else if( $type === 'delete' ){}
	else if( $type === 'update' ){}
}

ＡとＢの使い分けについて、統一的なルールや一般的な慣習や個人的なご意見など、幅広くお聞かせいただければ幸いです。よろしくお願い致します。

Answer 1

PHPよくわからないので例２について（JavaScriptもすごくわかっているわけではないですが）。

個人的には、これはAかと思います。
Bはそれぞれがローカルストレージに保存しているという仕様に依存しているので、低レベルな関数名になっており機能がわかりにくく、変更に弱そうで好みではないです。

どうしても記述が面倒ということであれば、高レベルな関数から低レベルな関数を呼び出すような形で、こうするような気がします。

function getFontFamily(){
	return getStorage('font-family');
}
function getMode(){
	return getStorage('mode');
}

function getStorage( type ){
	if( type === 'font-family' ){
		return localStorage.getItem('font-family') || 'serif';
	}else if( type === 'mode' ){
		return localStorage.getItem('mode') || 'normal';
	}
}   

これであれば、font-family だけローカルストレージを参照しないという変更があっても、getFontFamily 関数を変更すればよいと思います。

Comments

[nikuatsu]

ありがとうございます。まとめるよりも変更への柔軟性を重視するという発想になるわけですね。わかってきました。ありがとうございます。

[nikuatsu]

ちなみにこれはクラスを使うような機能ではないでしょうか？
クラス入門者でいまいち理解が及んでいませんが、クラスにするならこう↓だとして、こうせず関数にすべき機能でしょうか？

class localStoage{

constructor(font, mode) {
this.font = font
this.mode = mode
}
calc( type ) {
return getStorage( type );
}

}

const localStoage = new localStoage();
localStoage.calc = 'sans';

[Lhankor_Mhy]

クラスにするかどうかは全体の構成によると思いますが、当方の回答は高レベルなものと低レベルなものを区別した方がいいという、言ってみればレイヤーの話です。抽象度の低い関数は、一見して何をしてるのかわからないので可読性が下がったりプログラマの負担が増えたりする、という話でもあります。

例示のコードでも（何をしているコードなのかわかりませんが）別に構わないと思いますが、結局のところ抽象度が低くなっているので抽象度の高い関数でラップしたくなるなあ、というのが個人的な感想です。

[nikuatsu]

抽象度の高いものでどんどんラップしていく感じですか。ありがとうございます。練習していきます。

[Lhankor_Mhy]

いえ、クラスを作るにしても抽象度の高いクラスにしては？ ということです。

[kokemaru3]

参考になりました。

Answer 2

機能を引数に渡して関数はまとめる

と、忘れてしまって、同じ機能の関数を作る羽目になります

Comments

[Zuishin]

普通に考えて忘れるわけないと思いますが、それでも忘れる人はどうやったって忘れます。

[nikuatsu]

ありがとうございます。たくさん書くと忘れることもあるかもしれませんね。気を付けたいと思います。

Answer 3

命題のようにまとめるならクラスでメソッドを使うのが現実的だと思います

Comments

[nikuatsu]

ありがとうございます。恐れ入りますが、本例でのクラスを少しご回答頂けませんでしょうか。クラス未経験で本例での応用のイメージが掴めません…

ＡやＢがクラスでどう便利になるのかというところがよく理解できていないのです。