（あなたの周りでも見かけるかもしれない）インスタンス変数の間違った使い方 - Qiita

（2021-8-28追記） この記事の改訂版を書いてみました。改訂版の方が易しい内容になっているので、プログラミング初心者の方はこちらを参考にしてみてください。 はじめに：「引数があるよりは、ない方が良い」？ 先日、同僚の西見さん（@mah_lab）がこんな技術ブログを書いていました。 インスタンスメソッドとクラスメソッドはどのようにして使い分けるべきか？（Rubyの場合） 同じ内容を僕だったらどういうふうに書くかな～？と思って、ちょっと書き始めてみたんですが、わかりやすく実践的な説明をするのは意外と難しく、内容も西見さんのブログとほぼ同じになりそうだったので、途中で断念しました。 というわけで、インスタンスメソッドとクラスメソッドの使い分けが未だにあやふやだという方は、ぜひ西見さんのブログを読んでみてください！ ・・・なんですが、1点だけ気になる点がありました。 それはインスタンスメソッ

[Rinta]

インスタンス変数を排除して、ステートレスなAPIになったけど、だったらインスタンス化に意味がない。いわゆるstaticメソッドでいいのでは？

[ykhmfst2012]

先に○○が必要、という使い方を要求するクラスは呼び出し側が混乱する元だよね。それがシングルトン乱発で複数クラスにまたがってると悲惨。

[michael26]

この手の記事ははてブコメントまで読んで色んな思想を見るところまでが勉強

[love0hate]

本質は「インスタンスメソッドvsクラスメソッド」ではなく「副作用あるvsなし」の話じゃないのかな。

[uxlayman]

「機械的にルールを適用する」ことを疑うべきかと思う。意味を考えないと。

[toshiwo]

これはわかる。どこからでも操作できる変数的に使うと(書いた本人とその瞬間以外は)混乱する

[auient]

色んな人がバラバラなこと言ってるコメント欄を見るに、銀の弾丸的設計ルールはないのだなと思う。/ 元のコードでも、ファイルパスを知ってる箇所と結果がほしい箇所が離れているのなら、分ける意味がある

[vanbraam]

"引数は少ない方がいいが少なすぎると害がある"と言いたいのだと思うが,例が極端;Rubyではなくmethod overloadingのある言語ならnew Order(Csv file)で解決しそうだし

[mfham]

[勉強]

[tomcha0079]

“このクラスの一番の問題は、処理の過程でしか必要とされない一時的なデータをインスタンス変数（@csv_data, @result）に格納してしまっていることです。”

[kotsan]

“インスタンスメソッドとクラスメソッドはどのようにして使い分けるべきか？（Rubyの場合）”

[mas-higa]

（あなたの周りでも見かけるかもしれない）めっちゃ見かける。いつもげんなりしてる。

[iR3]

ふむふむ

[takaesu]

インスタンスメソッド・クラスメソッド

[kasumani]

（あなたの周りでも見かけるかもしれない）インスタンス変数の間違った使い方 はじめに：「引数があるよりは、ない方が良い」？ 先日、同僚の西見さん（@mah_lab）がこんな技術ブログを書いていました。 インスタンスメ

[rin51]

おっと

[yojik]

状況によるとしか。。。

[ka2nn]

メソッド分けて作ってしまって外にはシンプルなメソッドだけ露出させるのが良いと思う。 処理を論理的に分けたメソッドをAPI的なメソッドで簡単に使えるようにしておく。分けたメソッドが肥大化したら移譲な。

[tech-kazuhisa]

なるほど。レビューするとき気をつけよう/ “メソッドを呼び出す順番を間違えるとエラーになる”

[ledsun]

「結果をインスタンス変数に入れる」は悪手。戻り値で返すのは良い改善策。/2つの関数は分けたままの方が保守性が高い。例えば「GUIからcsv_dataを入力したくなったら？」

[n314]

状況による。区切り文字や入力文字コード、出力文字コードを変えながら複数のファイルを処理するなら最初のダメな例のやり方でやるかも。

[s17er]

コンストラクタ引数にファイルパスを渡し、その時点でロード処理もするかな

[gfx]

あるある

[mizchi]

完全コンストラクタ信者はいいんだけど狂信者もいるからな

[a2ikm]

やってる気がする

[zyusou]

こういうの書いちゃいそうだなぁ。気をつけねば。

[cocodrips]

気をつけよう

[zz_sexy]

新人研修中の子がそんなコード書いてたんで直させた。何を受け取って何を返すかが明確じゃなくなるんで呼び出し側見てもデータの流れがわからないよね、あと処理混ざったりするよね、と。

[rightgo09]

よく分かる