2011-01-29 8 views

答えて

1

byte[] p = new byte[m_pRecvBuffer.Length - 5]; 
Array.Copy(m_precvBuffer, p, m_pRecvBuffer.Length - 5); 
uint noncelen1 = p[0] * 256 + p[1]; 

に似何かしかし、その場合には、私はあなたが実際には配列のコピーを使用する必要はないと思います。ちょうど

uint noncelen1 = p[5] * 256 + p[6]; 

で十分でしょう。

+0

私はすでに書いたコードに似ているので、私はこのルートを行った。 –

+0

@hokkaido:このような新しいバイト配列を宣言してデータをコピーする場合、これは私が投稿した答えよりはるかに効率が悪いです。メモリを割り当てたり、配列をコピーしたりする理由はありません。 –

+0

@ジョナサン、私は同じことを言います。しかし、コードの残りの部分がどのように見えるかはわかりません。おそらくポインタを取得してそこから読み始め、おそらくそれを配列+オフセットにリファクタリングすることが実現できないことがあるかもしれません。 OPの小さなコードスニペットから、それを伝えるのは難しいです。 – Joey

2

Hmm私は、このコードではリファクタリングが必要なのだろうかと思います。はい、pointers in C#を使用できます。しかし、そのスニペットに基づいてより良い選択肢があるかもしれません。入ってくるストリームの一部を読み込もうとしているようです。たぶんC#のstream libraryはあなたのコードのこの部分でうまくいくでしょうか?

2

コードは実際に何を行い、コードを分析するのではなく、動作を変換します。 unsafeメソッドとポインタ演算をC#で使うことができますが、これはおそらく解決するよりも多くの問題を引き起こします。

+0

はい、私はこれを認識しています。しかし、私はC++よりもC#をよく知っているので、与えられたコードをデバッグすることはできません。私の頭をこの周りで包み込むための第一歩は、C#で動作させることです。 –

+0

ああ大丈夫です。私にとっても同じ問題ですが、配列索引作成という考えがありましたが、私は指を置くことができませんでした。 – Femaref

0

[] RecvBufferは、バイトとして宣言されていると仮定すると、あなたはこのようなものだろう:

int noncelen1 = RecvBuffer[5] * 256 + RecvBuffer[6]; 
関連する問題