Help us understand the problem. What is going on with this article?

[C#] Stackを複製する時の罠

More than 1 year has passed since last update.

C# でコピーコンストラクタを書いてて、 Stack の複製の落とし穴に嵌ったのでメモ。

下記のコード、一見何も問題ないように思いますよね? 

Stack<T> CopyStack<T>(Stack<T> from){
   return new Stack<T>(from);
}

でも、実はこうすると中身が逆順になってしまうんです。
fromの中身が [1,2,3] だと、複製したスタックは [3,2,1]になっちゃいます。

正しく複製するには、引数を逆転させないといけなかった。

Stack<T> CopyStack<T>(Stack<T> from){
   return new Stack<T>(from.Reverse());
}

Stack のコンストラクタに Stack を渡すと IEnumerable に変換されますが、
この時の変換処理は、順次 Pop して取り出していく実装になってます。
ところが、コンストラクタでは IEnumerable から取り出した順に Push していく実装になってます。
このミスマッチによって中身の逆転した Stack が生成されてしまうようです。

[1,2,3] >> []
[1,2] >> [3]
[1] >> [3,2]
[] >> [3,2,1]

Why not register and get more from Qiita?
  1. We will deliver articles that match you
    By following users and tags, you can catch up information on technical fields that you are interested in as a whole
  2. you can read useful information later efficiently
    By "stocking" the articles you like, you can search right away
Comments
No comments
Sign up for free and join this conversation.
If you already have a Qiita account
Why do not you register as a user and use Qiita more conveniently?
You need to log in to use this function. Qiita can be used more conveniently after logging in.
You seem to be reading articles frequently this month. Qiita can be used more conveniently after logging in.
  1. We will deliver articles that match you
    By following users and tags, you can catch up information on technical fields that you are interested in as a whole
  2. you can read useful information later efficiently
    By "stocking" the articles you like, you can search right away
ユーザーは見つかりませんでした