C# と VB.NET の質問掲示板

C#プログラムでC++のDLLからコマンドを呼び出して動作させようとしております。
C++DLLをC#で呼び出すところは全くの初心者です。ご承知置き下さい。

DLLのコマンド Connect は2つのPC間を通信させる内容になっており、
Connect () の中身は 2つのPCそれぞれのIPアドレスを32ビット形式、そしてそれぞれのポート番号を
入れるようになっています。

C++ DLL の実行コマンド↓

Connect(struct ConnectParameter)

struct ConnectParameter
 char Addr1[32]　　例 192.168.1.1
 char Addr2[32]　　例 192.168.1.2
 unsigned int Port1　　例 9999
 unsigned int Port2　　例 9999

C++DLLのコマンドを実行させるための32ビット形式のchar型に対して
C#ではどの型で対応していいかよく分からなかったので
単純にstring形式でIPアドレスを入力、int形式でポート番号を入力、
としてしまいましたが、本来であればどのようにしたら良いのでしょうか？
どなたかご助言、お願い致します。


C# プログラム↓

public string Addr1;
public string Addr2;
public int Port1;
public int Port2;

//入力
Addr1 = "192.168.1.1".ToString();
Addr2 = "192.168.1.2".ToString();
Port1 = 9999;
Port2 = 9999;

//実行
Connect(Addr1, Addr2, Port1, Port2);

> 32ビット形式のchar型

char Addr1[32]は、要素数32のcharの静的配列です。
静的配列はMarshalAs属性でSizeConstを指定します。

[StructLayout(LayoutKind.Sequential, CharSet=CharSet.Ansi)]
public struct ConnectParameter
{
    [MarshalAs(UnmanagedType.ByValTStr, SizeConst=32)]
    public string Addr1;
    [MarshalAs(UnmanagedType.ByValTStr, SizeConst=32)]
    public string Addr2;
    public int Port1;
    public int Port2;
}

ざっとこんな感じになります。

ところでC++の
Connect(struct ConnectParameter)
は本当にこのままですか？
* とか & とかが付いていませんか？

Hongliangさんの回答を参考に下記プログラムを作成してみましたが、
うまく通信ができませんでした。↓
要因が今のところ分かりません。

//構造体の宣言
ConnectParameter CP = new ConnectParameter();

//DLL実行コマンドのインポート
[DllImport("ConnectionDLL.dll", CallingConvention = CallingConvention.Cdecl)]
extern static int Connect(ConnectParameter CP);

//CPの中身 入力
CP.Addr1 = "192.168.1.1";
CP.Addr2 = "192.168.1.2";
CP.Port1 = 9999;
CP.Port2 = 9999;

//通信コマンド実行
Connect(CP);



もともと、C++の実行プログラム(↓)でしたら問題なく動作するので
C#プログラム(↑)のところがおかしいような気がするのですが、
どなたか気付かる点、ありましたら教えてください。


	WSADATA wsadata;
	int     rc;

	rc = WSAStartup( 0x0101, &wsadata );
	if( rc != 0 )
	{ return; }

	sprintf_s( CP.Addr1, "%s", "192.168.1.1" );
	CP.Port1 = 9999;
	sprintf_s( CP.Addr2, "%s", "192.168.1.2" );
	CP.Port2 = 9999;

	//ここで入力するとDLLでのエラーを回避できる(DLL内ではコピーするだけ)
	try
	{
		LPHOSTENT PC_lpHost = gethostbyname(CP.Addr1 );
                CP.my.sin_family = AF_INET;    	 
		CP.my.sin_addr = *((LPIN_ADDR*PC_lpHost->h_addr_list);		        CP.my.sin_port = htons( CP.Port1 );

		LPHOSTENT MP_lpHost = gethostbyname(CP.Addr2 );
                CP.dst.sin_family = AF_INET;    	 
		CP.dst.sin_addr = *((LPIN_ADDR*PC_lpHost->h_addr_list);		        CP.dst.sin_port = htons( CP.Port2 );
	}
	catch(...)
	{
		MessageBox::Show( "エラー" );
		return;
	}

	//Open
	int Ans;

	try
	{
		Ans = Connect(CP);
	}
	catch(...)
	{
		Ans = -1;
	}

	//Status
	if( Ans == 0 ) 
	{ 
		MessageBox::Show( "接続 OK" );
	}
	else  
	{ 
		MessageBox::Show( "接続 NG" );
	}

分かりにくくて申し訳ありませんでした。

C++のConnectParameter構造体とConnect関数の定義はこちらになります。

struct ConnectParameter
{
   char  Addr1[32];
   char  Addr2[32];
   unsigned int Port1;
   unsigned int Port2;
   struct sockaddr_in my;
   struct sockaddr_in dst;
   int PLC_Type;
};

Connect(struct ConnectParameter CP);


もともと外部で作製されたC++のDLLだけ提供されており、
それをC#で実行させたいという目的であるため、DLLの中身は上記記載の内容しか
把握できていない状態です。

ただ、C++で動作させるソースコードは提供されている。

Hongliangさん、ご丁寧にご対応頂き感謝しております。


■No93010 (Hongliang さん) に返信
> 以前の
>>C++ のところ、＊が付いていました。
> という回答が正しいのであれば、
>>Ans = Connect(CP);
> はコンパイルエラーになるはずですが。
> 
>>CP.my.sin_addr = *((LPIN_ADDR*PC_lpHost->h_addr_list);
> この辺のコードも、
>>struct ConnectParameter
>>char Addr1[32]　　例 192.168.1.1
>>char Addr2[32]　　例 192.168.1.2
>>unsigned int Port1　　例 9999
>>unsigned int Port2　　例 9999
> と全く相容れません。
> 
> C++のConnectParameter構造体とConnect関数の定義をコピー＆ペーストで正しく提示してください。

> struct ConnectParameter
> {
>    char  Addr1[32];
>    char  Addr2[32];
>    unsigned int Port1;
>    unsigned int Port2;
>    struct sockaddr_in my;
>    struct sockaddr_in dst;
>    int PLC_Type;
> };
わざわざmy/dstを公開してるということは、
それらも呼び出し側が設定する必要があるということなんでしょうね。
なのでC#でもこれらを定義してConnectParameterに追加する必要があります。
sockaddr_inは以下のような定義になるでしょう。

struct sockaddr_in {
    public ushort sin_family, sin_port;
    [MarshalAs(UnmanagedType.ByValArray, SizeConst=4)]
    public byte[] sin_addr;
    [MarshalAs(UnmanagedType.ByValArray, SizeConst=8)]
    public byte[] sin_zero;
}

sin_addrに相当するバイト配列は、IPアドレス限定であれば、
IPAddress.Parse(address).GetAddressBytes()
で取得できます。
名前解決も必要なら、
(1) IPAddress.TryParseして成功ならそのIPAddressを使う
(2) 1に失敗したらDns.GetHostEntryを呼び出し、AddressListの中から、
　　AddressFamilyがInterNetworkのIPAddressを使う
という手順が良いかと思います。

// PLC_Typeは設定しなくていいのかな…。

> Connect(struct ConnectParameter CP);
結局
> C++ のところ、＊が付いていました。
は勘違いということで良いんですかね？
付いているかどうかで、C#のDllImportするConnect関数の定義も変わってきますが。

Hongliangさん、ご丁寧に回答頂き誠に有難う御座います。
C++ DLL作成者に問い合わせたところ、コマンド実行のために my / dst の指定は必要とのことで、
Hongliangさんご察しの通りでした。

struct sockaddr_inの定義コード有難うございます。
参考にさせていただきます。 ↓下記のようなコードでとりあえず試してみます。

        //ConnectParameterの定義
        [StructLayout(LayoutKind.Sequential, CharSet = CharSet.Ansi)]
        public struct ConnectParameter
        {
            [MarshalAs(UnmanagedType.ByValTStr, SizeConst = 32)]
            public string Addr1;
            public int Port1;
            [MarshalAs(UnmanagedType.ByValTStr, SizeConst = 32)]
            public string Addr2;
            public int Port2;
        }

        //sockaddr_inの定義
        struct sockaddr_in
        {
            public ushort sin_family, sin_port;
            [MarshalAs(UnmanagedType.ByValArray, SizeConst = 4)]
            public byte[] sin_addr;
            [MarshalAs(UnmanagedType.ByValArray, SizeConst = 8)]
            public byte[] sin_zero;
        }

　　　　//コマンド実行
        private void button1_Click(object sender, EventArgs e)
        {
                CP.Addr1 = "192.168.1.2";
                CP.Port1 = 9999;
                CP.Addr2 = "192.168.1.1";
                CP.Port2 = 9999;

                my.sin_addr = IPAddress.Parse(CP.Addr1).GetAddressBytes();
                dst.sin_addr = IPAddress.Parse(CP.Addr2).GetAddressBytes();

                Connect(CP);
        }

struct sockaddr_in の中身としてsin_family / sin_port / sin_addr / sin_zero とありますが、
sin_addr だけでなく、sin_family や sin_port も指定してあげないと
うまく動作しない感じでしょうか。。。

>>Connect(struct ConnectParameter CP);
> 結局
>>C++ のところ、＊が付いていました。
> は勘違いということで良いんですかね？

Connect(struct ConnectParameter CP)に関しては、*付いていませんでした。
勘違いでした。。。　混乱させて申し訳ありません。