C# エラーログ出力でのエラー

Question

前提・実現したいこと

FileOutputUtilクラスでファイルをコピーする共通関数を作成しております。
エラーログ出力の箇所でエラーが起こっており解決できません。
共通関数のためstaticにしていることが原因でしょうか

お力添えをお願い致します。

発生している問題・エラーメッセージ

thisは静的プロパティオブジェクトでは無効です

_classNameは静的でないフィールド、メソッド、またはプロパティ で、オブジェクト参照が必要です

該当のソースコード

namespace Utilities
{
    public class FileOutputUtil
    {
        /// <summary>ユーザ情報を格納しておく</summary>
        protected UserInformation UserInfo;
        protected readonly string _className;

        /// <summary>
        /// CSVファイルのコピー
        /// </summary>
        public  static  bool orderFileCopy(string fromPath, string toPath)
        {
            bool retVal = false;
            string[] shareInfo;
          
            try
            ｛
                System.IO.File.Copy(fromPath, toPath, true);
                retVal = true;
            }
            catch (Exception ex)
            {
                // エラーログ出力
                Logger.OutputLogError(this.UserInfo.UserId, _className, "", "コピー時に例外エラーが発生", ex, Common.Logging.LogLevel.Error);
                retVal = false;
            }
            return retVal;
    }
}}

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

VisualStudio2019 C#

Answer 1

static関数では、thisは使えません。参照する変数はstaticにしよう

Comments

[NewNiche]

UserInfoと_classNameは、ほかのstaticでない関数でも使用しており、変数をstaticにすると他の関数でErrorがおきてしまうのですが、どのように対応すればよろしいのでしょうか
_className = this.GetType().Name;　の_className で赤波線Error（ 静的読み取り専用フィールドへの割り当てはできません (静的コンストラクタまたは変数初期化子では可)。）が起きております

[y_waiwai]

なら、FileOutputUtilのインスタンスからアクセスしよう、ってはなしになりますね

ちなみに、thisってのは自分自身のインスタンスを指しますが、 static関数にはインスタンスってのがないのでエラーになるんですな。

[y_waiwai]

まあ、やり口としては、引数にインスタンスを渡してやるとかすればいいかと。
あるいはUserIDを渡してやる、とか

[Zuishin]

_className というフィールドはそもそも必要ありません。
nameof(FileOutputUtil) で同じ結果が得られます。

[Zuishin]

UserInfo については、静的プロパティにするのが無理であれば、ユーザーコード側でインスタンスをどこかに保存しておくか、シングルトン化を検討するのが良いかもしれません。

[NewNiche]

皆様ありがとうございます。
結局、public bool orderFileCopyに変更することで解決させました。提案していただいた案も試してみます。