Q&A
> やらない方が良いのでしょうか?
「やらない方が良い」というような記事や意見をどこかで見聞きされたでしょうか。それともコードを書き進めていて心配になってしまっただけ、とか。
オブジェクト指向を勉強中の初心者になります。
何度も再利用するコードは関数やメソットにして使用した方が良いのは分かるのですが
コードを見やすくするという目的で一度だけしか使用しない処理を関数やメソットにしてコーディングする方法はやらない方が良いのでしょうか?
補足として一度きりしか使わない処理は一つの目的としてまとまっています
過去の質問をほぼちゃんとクローズしてない荒らし質問者なので回答するだけ無駄
あれこれ手を出し過ぎて気が散りまくっていて全く進んでない自覚はないようですね。
どうも自分のやり方に不安を抱いての質問だったようですね。まず、ひとつのプログラミング言語を自分が困らない程度まで習熟すればいいのにと常々思います。
回答5件
0
ベストアンサー
一度きりしか使わない処理は一つの目的としてまとまっています
そういうものを関数にすると，（その処理を使う場所にべた書きする場合と比較して）
- 処理に名前をつけることができます
- その処理の IN/OUT が明確になります
- その処理でしか使わないものが，その処理の外側に漏れ出なくて済むようになります
- 処理の責務が明確になります
あなたも
- コードが見やすくなる
と考えていますし，わりと良い事ばかりに見えます．
しかしながら
やらない方が良い
と考えている理由（何を念頭におかれているのかわかりませんが）もある様子ですから，
そこは状況を鑑みてやるかどうかを決定すればよい（：上記に列挙した利点を上回る懸念点等がある場合には，「やらない方が良い」）という話にしかならないでしょう．
まぁ，ふつーは，見やすくなると思うならやっておけばいいのではないかな，と思います．
投稿2021/10/23 01:56
総合スコア11996
> コードを見やすくするという目的
であるのだから，その処理を使う回数はどうでもいいこと．
行動の理由と無関係（再利用性を考えている場面ではない）なのだから，そもそも考慮事項ではない．
むしろ，その関数を他の場所から呼ばせない（呼べない）ようにすることが必要な場面であろう．
オブジェクト指向をあまり理解できていなかったので
てっきり絶対に再利用するメソッドなどしかだめだと思っていました、、
見やすさという観点もオブジェクト指向という事がわかったので
基本的には現状一度しか使わない処理もメソッド化していこうと思います
一番わかりますかったのでベストアンサーにさせて頂きます
> オブジェクト指向
という概念とはあまり（というか，ほぼ）関係ない話題な気がします．
この話の中にはオブジェクト間の相互作用だとかそういう話もありませんし．
実装箇所がクラスのメソッドなのだとしても，
話の内容としては
「Ｃ言語で全部 main 関数に書いてたんだけど，処理を関数に切り分けて main 関数はそれらを順番に１回ずつ呼ぶような形にした方が良いだろうか？」
…的な事柄側でしょう．
コードの切り分け先が， 関数 ではなくて メソッド である場合，
「メソッドはメンバを好き勝手に読み書きするかもしれない」ということが，かえってコードを読みにくくしてしまわないように注意が必要です．
本件の目的としては，private な staticメソッド を作る形が良いでしょう．
実装の遮蔽はOOP以前の問題ではありますね。
0
一度だけしか使用しない処理
内容によりけり。
きちんと基盤を作り込むことができたら拡張性も再利用性も高くなります。
あとあと絶対に使われないとは限りませんし、結局粒度の問題に思います。
役割分担をより細分化すれば様々な場所から使いやすい内容になるはずです。
「一度しか使わないから」ではなく
「様々な用途で使えるようにするにはどうしたら良いか」を考えてはどうでしょう。
あまりに粒度が荒かったり役割をもたせすぎると、可能な限り疎結合にすべきというオブジェクト指向の本質から外れます。
（実際のコードのサンプルでいいから出したほうがより具体的な意見が集まるでしょうけどね）
投稿2021/10/23 01:58
総合スコア80875
0
namespace的な使い方(いわゆるユーティリティクラス)は良くないとされていますが、ちゃんとOOP的な考え方であればいいのではないでしょうか。
それと、メソットではなくメソッド(method)です。
投稿2021/10/22 22:59
総合スコア4962
0
ぱっとみーのコードのわかりやすさ、というのが至上命題です。
どしどしやるべきものです。
オブジェクト指向というのは、何をおいてもわかりやすさ、というのを優先します
投稿2021/10/22 21:57
総合スコア88042
0
どんどんやりましょう。OOPは構造化プログラミングの上に立つものです。
なおC++に関して言えばOOPに囚われすぎず、フリー関数(メンバー関数と対比した言い方)を使うほうが物事がうまくいきます。
投稿2021/10/22 19:13
総合スコア5852
ありがとうございます！
どんどん、使って行こうと思います
C++についても勉強していたのである程度学習したらオブジェクト指向の勉強の一環として
フリー関数についても学習しようと思います
まあフリー関数ってただの普通の関数ですね。
あなたの回答
tips
プレビュー
