関数の引数を記述するかどうか

Question

class hoge: SuperHoge{
    string _appId;

    private init(){
        _appId = SuperHoge.systemId;
    }
}

このようなクラスがあったとします。
文字の末尾から真偽を判断する関数を作成する際に
明示的にパラメーターとしてappIdを渡したほうがインとアウトが明確になり良いコードかなと考えました。

下記２つの関数は同じ動作をします。
慣習としてどちらが優れたものでしょうか？
独学者故、自信が持てません。
理由と合わせてご回答頂ければ幸いです。

        protected virtual bool isStudentApplication(string appId)
        {
            
                //IDがSで終わっていたらtrue
                if (appId.EndsWith("S") == true)
                {
                    return true;
                }

                return false;
            
        }

        protected virtual bool isStudentApplication()
        {
            
                //IDがSで終わっていたらtrue
                if (_appId.EndsWith("S") == true)
                {
                    return true;
                }

                return false;
            
        }

Answer 1

こんにちは。

前者の関数はメンバ変数にアクセスしていませんね。なので、そもそも仮想関数にする意味がないですし、仮想関数にすることで要らぬ憶測を呼びメンテナンス性は劣化すると思います。

isStudentApplication()を含んでいるクラスは何でしょうか？
Applicationのようなクラスでアプリ１つ１つにオブジェクトが存在し、そのアプリが学生用かそうでないかを判定するメソッドとしてisStudentApplication()を用意するのであれば、後者一択です。

isStudentApplication()を含んでいるクラスがApplicationListのようなアプリを管理するクラスでisStudentApplication()が普通の関数、もしくは、static関数ならば前者も有りと思います。

Answer 2

_appIdの扱い方にもよってくると思います。

_appIdが頻繁に変わる可能性があり、一連の処理の流れでtrue/falseを返却させたいのであれば前者ですし、そうでないのであれば後者になると思います。