foreach内と外の動作、if内のbreakについて

Question

前提・実現したいこと

＜ソースコードとして前提＞
用意している配列において、同じ要素の数をカウントして、その数を出力したい。（５を出力）
＜自分として実現したいこと＞
ただこのコードの
var cnt = 0;の位置がどこにあるべきかの理由を知りたいだけです。

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

自分では以下の様なコードを入力しました。
　　　　var cnt = 0;
　　　　foreach (var pattern in array)
        {
            foreach (var word in array)
            {
                if (pattern == word)
                {
                    cnt++;
                }
            }
調べた結果、正解は
　　　　　foreach (var pattern in array)
        {
            var cnt = 0;
            foreach (var word in array)
            {
                if (pattern == word)
                {
                    cnt++;
                }
            }
で、foreachの中に以上のようにvar cnt=0;を置くのが正解らしいが、foreachの中にvar cnt = 0;が入っているということは、毎回、var cnt = 0;が作られていませんか。foreachの中の状況がいまいちわかりません。外に置くとなぜ間違いかわからない。

該当のソースコード

//これは自分がうった間違いのソースコードです。
using System;

class paiza
{
    static void Main()
    {
        string[] array = {"HND", "NRT", "KIX", "NGO", "NGO", "NGO", "NGO", "NGO"};
        var duplicate = 0;
        
　　　　var cnt = 0;
        foreach (var pattern in array)
        {
            foreach (var word in array)
            {
                if (pattern == word)
                {
                    cnt++;
                }
            }

            
                duplicate = cnt;
                
            
        }

        Console.WriteLine(duplicate);
    }
}

試したこと

正解を調べたところ、発生している問題・エラーメッセージのようにするのが正解のようです。
また、duplicate = cnt;
break;
を、if (cnt != 1)
{
duplicate = cnt;
break;
}
とするそうです。

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

なお、このサイトでの質問が初めての為、マルチポストがダメなのを知らず、知恵袋の方に先に質問しています。知恵袋では心配でしたのでこちらにも質問しました。なお、知恵袋には正しいソースコードを、こちらでは自分が入力した間違いのソースコードを記入しています。知恵袋の方でも教えていただいているので、そちらの方もご参照下さい。https://detail.chiebukuro.yahoo.co.jp/qa/question_detail/q13214577238
また、こちらの問題はpaizaラーニングというこちらのサイトの
https://paiza.jp/works
レベルアップ問題集、Bランクレベルアップセット、C#編、文字と整数の組のソート2、STEP３の問題になります。

Comments

[退会済みユーザー]

質問文からは何が聞きたいのかが読めないのですが。

[YAmaGNZ]

paizaの環境のみで学習されるのではなく、VisualStudio等のデバッガを使用できるものを利用して学習されることをお勧めします。

[Zuishin]

マルチポストです。
https://detail.chiebukuro.yahoo.co.jp/qa/question_detail/q13214577238
ヘルプのこの部分を読んで適切に対処してください。
https://teratail.com/help#posted-otherservice

また、問題と正しいソースの入手先をリンクしてください。

[Zuishin]

深い……？

[Zuishin]

削除

[herumettokoala]

そんなにひどいコードなんですね。
どのように編集すれば、ひどいコードから正せますでしょうか？
よろしくお願い致します。

[Zuishin]

いえ、私の勘違いです。標準入力からデータが与えられるのを配列に直す問題かと思ったので。もっとスマートな書き方はありますが、そこまでひどくはありません。

Answer 1

var cnt = 0;の位置がどこにあるべきかの理由を知りたいだけです。

cntが何をカウントするものなのか考えてください。
それが分かれば、どこでcntをリセットすべきなのかが理解できるはずです。

質問へのコメントでも記載しましたが、paizaでは開発環境をインストールせずともコードの動作をブラウザ上で確認でき便利なのですが、ステップ実行や実行中の変数の中身の確認などデバッガーで出来ることが行えません。
ですので、VisualStudio等の開発環境をインストールし、デバッガーも使用できる環境を整えて学習することをお勧めします。

Answer 2

前提。すみませんが他サイトは見てません。リンクになってないです。paizaはキモなんで見ようとしましたがログインが必要そうだったので見れません。

質問者様のコードが誤りで、そのコードが正解になるなら、問題の理解が間違っていないか確認することをおすすめしたいです。
質問者様のコードは、arrayには重複は必ず1パターンしかないなど特定の条件があれば正解であり得ると思います。
つまり、

array = {"HND", "NRT", "NRT", "NGO", "NGO", "NGO", "NGO", "NGO"};

このようなパターンは出現しないことが保証されていれば問題ないと思います。
あるいはこういうパターンはあり得るが、ユニークでないもの（かぶりのあるもの）は何と重複していてもカウントする、という目的であれば問題ないと思います。

これを正解だと称する問題の制約を正しく把握できない限り、なぜ質問者様のコードが正しくないのか、もまた正しく説明できないです。

Comments

[Zuishin]

重複するものは一種類だけという制約がありました。

[papinianus]

ありがとうございます。
重複が一種類という制約なら最後にsqrtした結果を出せば良いように考えました。

が、おそらく私はケースの考慮が足りてないと思います。私ならこの問いはGroupByで解決する問いだと考えるので。

Answer 3

まずは最初のforeach文にブレークポイントを設定しておいて、そこから変数のナカミを見ながら１行づつ実行していきましょう。
それでコードの動作がわかると思います。

それを一通りやって、なおもわからないところがあるというなら改めて聞きましょう