関数のクラス化において、何が適切か分かりません

Question

前提

「おすすめの本にタグをつけて投稿する掲示板」を作成しています。

現状は400個ほどの関数のみで構成されているのですが、クラス化に挑戦することにしました。

どの関数をクラス化するのが適切かが分からないので、ひとまず「投稿コンテンツ（本やタグ）にまつわる処理」をクラス化していこうと考えました。

発生している問題

しかし、本テーブルへのINSERTの過程で、タグテーブルとリレーションテーブルもINSERTするのですが、

クラス化にあたって リレーションテーブルへの保存処理を class Book の中に書くか class Tag の中に書くか、分からず躓いております。

（リレーションテーブルへの保存処理は、関数 insert_my_relations が担っています。）

知りたいこと

どちらが良いのか、または別の書き方が良いのかを知りたいです。アドバイス頂戴できましたら幸いです。

ご回答を頂戴する身として大変僭越ではございますが、「その関数群（該当のソースコード）なら、このようにクラス化したらいいだろう」といった内容でご教示頂くと大変ありがたく思います。

該当のソースコード

現状の関数はこのようになっています。（エラー処理とセキュリティ面は無視してください。）

// パスワードを確認
function check_thread_password( $thread_id, $password ){
	return ['status'=>'ok'];
}

// 本テーブルを INSERT 
function insert_my_books( $book_data ){
	return ['status'=>'ok','book_id'=>1];
}

// タグテーブルを INSERT 
function insert_my_tags( $tag_data ){
	return ['status'=>'ok','tag_ids'=>[1,2]];
}

// リレーションテーブルを INSERT 
function insert_my_relations( $book_id, $tag_ids ){
	return ['status'=>'ok'];
}

// おすすめの本を投稿
function ajax_post_recommend_book(){
	// post された値
	$password  = $_POST['password'];
	$book_data = $_POST['book_data'];
	$thread_id = $_POST['thread_id'];

	// パスワードを確認
	$checked = check_thread_password( $thread_id, $password );
	if ( $checked['status'] === 'ok' ) {
		// テーブルロックして INSERT 
		$inserted = insert_my_books_with_table_lock( $book_data );
		if ( $inserted['status'] === 'ok' ) {
			$book_id = $inserted['book_id'];
		}		
	}
	
	echo ['status'=>'ok','book_id'=>$book_id];
}

// テーブルロックして INSERT 
function insert_my_books_with_table_lock( $book_data ){
	
	$pdo = new PDO();
	$pdo->query("LOCK TABLES my_books WRITE, my_tags WRITE, my_relations WRITE");
	
	// book テーブルを INSERT 
	$inserted = insert_my_books( $book_data );
	if ( $inserted['status'] === 'ok' ) {
		$book_id = $inserted['book_id'];

		// tag テーブルを INSERT 
		$inserted = insert_my_tags( $book_data['tag_data'] );
		if ( $inserted['status'] === 'ok' ) {
			$tag_ids = $inserted['tag_ids'];

			// リレーションテーブルを INSERT  
			insert_my_relations( $book_id, $tag_ids );
		}

	}

	$pdo->query("COMMIT");
	return ['status'=>'ok','book_id'=>$book_id];
}

試したこと

上記の関数群について、class Bookと class Tag に分けて次のようにクラス化してみました。

ここで上述の問題が生じまして、つまり、関数 insert_my_relations を下記は class Book に入れていますが、これを class Tag に入れた方がいいのかで悩んでいる次第です。

もしくは何か全く別の書き方で適切なクラス化の方向性があるでしょうか…

// パスワードを確認
function check_thread_password( $thread_id, $password ){
	return ['status'=>'ok'];
}

// タグにまつわる処理
class Tag{

	// タグテーブルを INSERT 
	public function insert_my_tags( $tag_data ){
		return ['status'=>'ok','tag_ids'=>[1,2]];
	}

}

// 本にまつわる処理
class Book{

	// 本テーブルを INSERT 
	public function insert_my_books( $book_data ){
		return ['status'=>'ok','book_id'=>1];
	}

	// リレーションテーブルを INSERT 
	public function insert_my_relations( $book_id, $tag_ids ){
		return ['status'=>'ok'];
	}
	
	// テーブルロックして INSERT 
	public function insert_my_books_with_table_lock( $book_data ){
		
		$pdo = new PDO();
		$pdo->query("LOCK TABLES my_books WRITE, my_tags WRITE, my_relations WRITE");

		// book テーブルを INSERT 
		$inserted = $this->insert_my_books( $book_data );
		if ( $inserted['status'] === 'ok' ) {
			$book_id = $inserted['book_id'];

			// tag テーブルを INSERT
			$tag = new Tag();
			$inserted = $tag->insert_my_tags( $book_data['tag_data'] );
			if ( $inserted['status'] === 'ok' ) {
				$tag_ids = $inserted['tag_ids'];

				// リレーションテーブルを INSERT  
				$this->insert_my_relations( $book_id, $tag_ids );
			}

		}

		$pdo->query("COMMIT");
		return ['status'=>'ok','book_id'=>$book_id];
	}
	
}

// おすすめの本を投稿
function ajax_post_recommend_book(){
	// post された値
	$password  = $_POST['password'];
	$book_data = $_POST['book_data'];
	$thread_id = $_POST['thread_id'];
	
	// パスワードを確認
	$checked = check_thread_password( $thread_id, $password );
	if ( $checked['status'] === 'ok' ) {
		// テーブルロックして INSERT 
		$book = new Book();
		$inserted = $book->insert_my_books_with_table_lock( $book_data );
		if ( $inserted['status'] === 'ok' ) {
			$book_id = $inserted['book_id'];
		}		
	}
	
	echo ['status'=>'ok','book_id'=>$book_id];
}

補足情報（FW/ツールのバージョンなど）

PHPは7.2ですが、8.0にアプデ可能です。

Comments

[yambejp]

タグにまつわる/本にまつわる・・・どういうくくりで何をしたいのか伝わってきません
なぜクラスを分けているか要件定義すればどちらがベターか見えてくるかもしれませんが
どちらかをベースにもう片方をextendするという考え方もあります

[nikuatsu]

「どういうくくりで何をしたいのか」が決められないからですね。すみません。

今はひとまず「投稿コンテンツ（本とタグ）にまつわる処理をクラス化してみよう」と進めています。取り組んでいくうちに「どういうくくりで何をしたいのか」を決める能力が身についていくだろうという考えです。

ご回答を頂戴する身としましては「その関数群なら、試したことにある方向性よりも、このようにクラス化したらいいだろう」といった内容でご教示頂くと大変ありがたく思います。

[yambejp]

「分ける必要や利便性がないなら分けない」で回答になっていますか？

[nikuatsu]

「必要性や利便性がない選択を行わない」という常識的な内容なので異論はございませんが、質問への回答としてそのような常識をご提示いただくことにつきましては、質問者の立場からするとやや思うところはございます。

[yambejp]

回答者の立場からすると相当思うところがあってああいう補足(回答)になっているんですけどね。

[nikuatsu]

ここまでのやりとりで「相当思うところ」が生じるような何かがあったのですね…。なんだか申し訳ございません。

[nikuatsu]

「相当思うところ」が何か考えてみたのですが、もしかして前回のご質問（ https://teratail.com/questions/p48zgc0ac22qmg ）で低評価が付けられてしまったことでしょうか？
それでしたら私は低評価をしていないので、どうかご容赦頂ければと思います。