Visual Studio C# 戻り値に関して

Question

---現在、Visual Studio 2017でmacアドレスを取得し、文字列で戻り値として扱おうとしております。
コードは以下です

        static String GetMca()
        {
            // アダプタリストを取得する
            NetworkInterface[] adapters = NetworkInterface.GetAllNetworkInterfaces();

            foreach (NetworkInterface adapter in adapters)
            {
                // 物理（MAC）アドレスの取得
                PhysicalAddress phy = adapter.GetPhysicalAddress();
                String macAddress = phy.ToString();
            }
            return xxxxxxx;
        }

return xxxxxxx;
の部分で戻り値に　macAddress　を入れたいと考えているのですがうまくいきません。
少し調べてみたのですが、macAddress がローカル変数になってしまっているのでしょうか？

Answer 1

複数の値を返すよう書き直してみました。
検証はしていません。

static IEnumerable<String> GetMca()
{
    // アダプタリストを取得する
    NetworkInterface[] adapters = NetworkInterface.GetAllNetworkInterfaces();
    return adapters.Select(a => a.GetPhysicalAddress().ToString());
}

最初の値のみ欲しければ

string s = GetMca().FirstOrDefault();

のようにします。

Comments

[hanimaru11]

ご回答ありがとうございます。
ご教授いただきましたコードでも無事macアドレスを戻り値で取得することができました。
サンプルコードを自分で弄っていましたので、ここまでシンプルにできるものなのかと驚いております。
現状ではご教授頂いたコードでなぜできているのかわかりませんのでこの後調べて理解したいと思っております。
ありがとうざいました！

Answer 2

foreachの大括弧の中で「string macAddress」と宣言してしまっているので、macAddress変数はforeachのカッコの中でしか参照できません。

戻り値の行でも使いたいならば、foreachの前で「string macAddress」と書けば大丈夫です。

ただ、別の方の指摘にもあるようにこのままだと

1. foreachで複数のアダプター情報の1つを対象にする
2. そのmacアドレスをmacAddressに入れる。
3. 次のアダプター情報を対象にする。
4. そのmacアドレスをmacAddressに上書きする……

上記の3、4を繰り返し、複数のアダプターがあると最後の1つのmacアドレスしか取れません。それでよいのなら、前述のstring宣言の場所を変えるだけでうまくいくと思いますよ。

Comments

[hanimaru11]

ご回答ありがとうございます
今回、１つのアダプタのmacアドレスを取得することが目的でしたのでforeach前で宣言を追加しました。
ですが確かに今後複数アダプタの情報を取得することも考え、改良していきたいと思います。
ありがとうございました！

Answer 3

ブロックの中｛｝で定義した変数は、そのブロックの外では参照できません。
その外で定義しておきましょう。

って、foreach のループしてるなら、その最後の値になってしまうけど、そんでいいの？

Comments

[hanimaru11]

ご回答ありがとうございます、以下のようにforeachの外側で変数を定義することで自分の行おうとしてたことはできました！

static String GetMca()
{
string macAddress = null;

// アダプタリストを取得する
NetworkInterface[] adapters = NetworkInterface.GetAllNetworkInterfaces();

foreach (NetworkInterface adapter in adapters)
{
// 物理（MAC）アドレスの取得
PhysicalAddress phy = adapter.GetPhysicalAddress();
macAddress = phy.ToString();
return macAddress ;
}
return macAddress ;
}

foreachに関しまして、今回macアドレスのみ使用したくサンプルを見ながら不要な部分を削ったりしていました。元々はIPなども取得するようになっておりましたのでそのせいかと思います。
もう少しきちんとできるよう試行してみます。
本当にありがとうございました！

[y_waiwai]

いろいろやって動作を試して、不都合あれば修正して試して、を繰り返すことです。
がんばってください。

で、ちなみに、そのコードだと、foreach の最初の値を返すってことになりますね。
まあ、ネットワークアダプタがない場合はループに入らずnullを返すのでそんでいいか。

Answer 4

static String GetMca()
{
**string macAddress = null;**

// アダプタリストを取得する
NetworkInterface[] adapters = NetworkInterface.GetAllNetworkInterfaces();

foreach (NetworkInterface adapter in adapters)
{
// 物理（MAC）アドレスの取得
PhysicalAddress phy = adapter.GetPhysicalAddress();
macAddress = phy.ToString();
return macAddress ;
}
return macAddress ;
}

foreach前で宣言することで解決しました。
ですが回答いただきました「複数のアダプタへの対応」ができておらず、現状ではforeachの結果で複数あるアダプタの最後のものが変数にセットされている状態です。

Comments

[hanimaru11]

回答いただきました方々、申し訳ありません。
自分の解決方法をまとめたのですが、ベストアンサーになってしまいました。ごめんなさい…