C++で書かれたコードをC#へ移植

Question

###質問
現在C++で作成されたコードをC#へ置き換えることをしています。
C++のコードでは、関数で内部に保持しているオブジェクトの取得／設定を行っています。
このままC#へ置き換えても問題はないかと思うのですが、取得したいオブジェクト内の一部の変数を取得したいときは、プロパティによるアクセスにできないかなと考えています。
下記C#コードしか思い浮かばないのですがこれだと
・内部にクラスが増えてしまう
・プロパティによるアクセスはできたがオブジェクト自体を設定／取得したい場合、オブジェクト内に専用関数を用意しないといけない
などからいまいちな気がします。
いい方法はないでしょうか。

C++(MFC)

例では内部に保持している構造体は一つですが、実際には複数存在します。
class SampleClass
{
public:
	struct TagSample {
		int Value1;
		int Value2;
	};
private:
	TagSample* m_TagSample;
	CRITICAL_SECTION m_Critical;
public:
	bool GetTagSample(TagSample* tagSample);
	bool SetTagSample(TagSample* tagSample);
	int GetValue1();

};

----------ソース
#include "SampleClass.h"

bool SampleClass::GetTagSample(TagSample* tagSample)
{
    ::EnterCriticalSection(&m_Critical);
    memcpy(tagSample, m_TagSample, sizeof(TagSample));
    ::LeaveCriticalSection(&m_Critical);
    return true;
}

bool SampleClass::SetTagSample(TagSample* tagSample)
{
    ::EnterCriticalSection(&m_Critical);
    memcpy(m_TagSample, tagSample, sizeof(TagSample));
    ::LeaveCriticalSection(&m_Critical);
    return true;
}

int SampleClass::GetValue1()
{
    int ret = 0;
    ::EnterCriticalSection(&m_Critical);
    ret = m_TagSample->Value1;
    ::LeaveCriticalSection(&m_Critical);
    return ret;
}
---------------------------
C#
---------------------------
    class SampleClass
    {
        public class TagSample
        {
            public class Inner
            {

                public int Value1 { get; set; }
                public int Value2 { get; set; }
            }

            private Inner inner = new Inner();

            private object m_Critical = new object();

            public int Value1
            {
                get { lock (this.m_Critical) return this.inner.Value1; }
                set { lock (this.m_Critical) this.inner.Value1 = value; }
            }
            public int Value2
            {
                get { lock (this.m_Critical) return this.inner.Value2; }
                set { lock (this.m_Critical) this.inner.Value2 = value; }
            }

        }

        private TagSample m_TagSample = new TagSample();

        public TagSample Sample => m_TagSample;
    }

Answer 1

現在C++で作成されたコードをC#へ置き換えることをしています。

という話の意味次第で，何が良い形か？というのが変わるのではないだろうか：

「C++で作られたやつがどんな動作をするか？というアプリケーションの仕様が分かっていて → C#で作り直すよ」
みたいな話ならば，細部の実装方法はどうでもよく，動作仕様を満たすものをC#で好きな形で書けばよいわけですが，

そうではなくて，
「C++ から C# に翻訳する」ような話ならば，なるべく「直訳」の形にしておいた方が何かあったときに対応が把握しやすくて良いのではないでしょうか？
C++とC#とでメソッドの個数が変わったとか，内部型だったものが別の場所に移ったとか，そういう変更が施されるほど，元のC++との対応を見るのが面倒になるように思います．
C++でメソッドだったものをプロパティに置き換えれば，メソッドの呼び出し側の字面もまた相応に変わってしまいますし．

Comments

[fana]

…といった感じで，私なら
（メソッドの中身の実装はともかくとして）少なくともインタフェースの形を無暗にいじりたくないなぁ，と思います．
そこによほどの利点が無い限りは．

[otake21]

意見ありがとうございます。
確かにこの部分を変えることによるほかの部分の修正を考慮していませんでした。

Answer 2

仕様を変更してもいいなら、私なら以下のようにやりますね。

まず、内部クラスは外部クラスとして切り出す。
(私はあまり内部クラスは使わないので、以降の処理ができるかどうか不明なため。
可能であれば内部クラスでもok)

この切り出したクラスをもとのままclass Innerとする。

そして、質問文でいうclass SampleClassのプロパティはInnerのオブジェクト自体にする。

そうすれば、

// 一応、引数として渡してみる
SampleClass sampleObject1 = new SampleClass( new Inner() );

// オブジェクトの設定
sampleObject1.InnerObj = new Inner();

// オブジェクトの取り出し
Inner inner = sampleObject1.InnerObj;

// あるいは直接
int vaue1 = sampleObject1.InnerObj.Value1;

のようにアクセスできます。

Comments

[otake21]

意見ありがとうございます。
内部クラスが増えるとごちゃつきやすいので、、参考になります。