UnityではじめるC#、引数について(string messenger)

Question

現在、「UnityではじめるC#」という書籍を読んで勉強中です。

引数について分かりかけてきたところで、急に突き放されてしまったページがあって、混乱しています。
##問題のコード(後半です。)

using System.Collections;
using System.Collections.Generic;
using UnityEngine;

public class Lesson1 : MonoBehaviour
{
    public string PlayerName;

    // Start is called before the first frame update
    void Start()
	{
        
    }

    // Update is called once per frame
    void Update()
    {
        string message = GetMessage(PlayerName);
        UnityEngine.Debug.Log(message);
    }

    string GetMessage(string messenger)
	{
        string curtime = System.DateTime.Now.ToLongTimeString();
        return PlayerName + "「今は" + curtime + "」";
    }
}

上記を実行すると、コンソールに、**名前「今は12:35:37 PM」**とでます。

##理解していること
void Updateというメソッドの中で、messageを定義して、
次の行で、それをコンソールに表示してね！としている。
(Unityのコンソール)

じゃあそのmessageに代入されたGetMessageってなに？というのが、
さらに下の行にあって、メソッドが作られている。

つまりmessageを呼び出すと、このstring GetMessageが呼び出されて、
returnによって名前と、時間を含む文字が返ってくる。

という感じだと理解しました。
##分からないこと
最後のメソッドの、string GetMassage(string messenger)のカッコ内、
string messengerですが、この意味が分かりません。
引数ということは分かるのですが、PlayerNameのように、定義もされてないし、
なんのためなのか分かりません。
なので、いらないのかな？と思って、（）と空の状態で実行したところエラーになってしまいました。
全く別の名前なら上手く行ったので、何でもいいの！？とますます混乱しています。

##知りたいこと
このstring messengerがなぜ必要なのか、一体何をしているのか、もし教えていただけたら嬉しいです！

Comments

[Zuishin]

これがこのまま本に書いてあったのであれば、別の本をお勧めします。
初心者の作ったものにしか見えません。

[matomaru]

ご回答ありがとうございます！
初心者のための入門書とありますし、まだ最初の数ページのことなので、
簡単に書いてあるということではないのですか？

[Zuishin]

簡単に書いてあるということではなく、アンチパターンが複数含まれています。

[退会済みユーザー]

質問のコードは本の通り書いてないのでは？　転記ミスをする質問者さんはここ Teratail によくいます。実は以下のようになっているとか？

string GetMessage(string messenger)
{
string curtime = System.DateTime.Now.ToLongTimeString();
return messenger + "「今は" + curtime + "」";
}

[matomaru]

本に誤りがあったようです。
SurferOnWwwさんのように自身で考えられたら良かったのですが‥。精進します。
ご意見ありがとうございました！

[退会済みユーザー]

返答をありがとうございました。

そうでしたか、本なのに・・・

Answer 1

GetMessageメソッド内のPlayerNameはmessengerの誤植でしょう。

string GetMessage(string messenger)
{
    string curtime = System.DateTime.Now.ToLongTimeString();
    return messenger + "「今は" + curtime + "」";
}

こう書かれていれば「任意の messenger から時間が伝えられる」という文脈に一致します。

Comments

[matomaru]

ご回答ありがとうございます！
誤植でした、、messengerならばスッキリとします！
自分で気付けるようになりたいです。

Answer 2

void Updateというメソッドの中で、messageを定義して、
次の行で、それをコンソールに表示してね！としている。
(Unityのコンソール)

はい

じゃあそのmessageに代入されたGetMessageってなに？というのが、
さらに下の行にあって、メソッドが作られている。

はい
正確にはmessageにGetMessageの返り値が代入されています。

つまりmessageを呼び出すと、このstring GetMessageが呼び出されて、
returnによって名前と、時間を含む文字が返ってくる。

messageは変数です。呼び出すものじゃないです。

最後のメソッドの、string GetMassage(string messenger)のカッコ内、
string messengerですが、この意味が分かりません。
引数ということは分かるのですが、PlayerNameのように、定義もされてないし、

引数として書いてある時点で定義です。

なんのためなのか分かりません。

プログラムは目的がなくても文法さえあっていれば動きます。

なので、いらないのかな？と思って、（）と空の状態で実行したところエラーになってしまいました。

エラーメッセージを読みましょう。

全く別の名前なら上手く行ったので、何でもいいの！？とますます混乱しています。

はい、引数名はなんでもいいです。

---

string GetMassage(string messenger)とmessengerを引数として定義しておきながら、
GetMessage()内で全くmessengerを使っていないから、意味がわからず混乱しておられるのかと思います。

混乱のもとである、定義した引数を全く使わないというのは文法上正しく、エラーになりません。
おそらくですけどその先のページで利用するコードを書くのではないでしょうか?

(またはただ単に質問者さんが書き間違えたかです。)

Comments

[matomaru]

丁寧にありがとうございます！
まだまだ変数や定義について、完全に理解できていないということがわかりました。
混乱の理由はおっしゃる通りで、書き換えても上手くいきました。勉強します！

Answer 3

質問に書いてあるコードですが

    void Update()
    {
        string message = GetMessage(PlayerName);
        UnityEngine.Debug.Log(message);
    }

    string GetMessage(string messenger)
    {
        string curtime = System.DateTime.Now.ToLongTimeString();
        return PlayerName + "「今は" + curtime + "」";
    }

ではなく

    void Update()
    {
        string message = GetMessage(PlayerName);
        UnityEngine.Debug.Log(message);
    }

    string GetMessage(string messenger)
    {
        string curtime = System.DateTime.Now.ToLongTimeString();
        return messenger + "「今は" + curtime + "」";
    }

とあるべきです。

string GetMessage(string messenger)のstring messengerの部分は
GetMessage(PlayerName)といった感じでこの関数が呼ばれたときに渡されたPlayerNameがmessengerという変数に格納されるよといった意味になります。

前者のコードですと引数として定義したmessengerを使用していないために引数の説明になっていないソースになります。
後者のソースですと引数とは何なのかが分かるのではないでしょうか。

Comments

[matomaru]

ご回答ありがとうございました！
とても分かりやすいです！確かにmessengerであれば理解できます！
安心しました。

Answer 4

変数ですから、呼び出し元と呼び出し先で型さえ一致していれば問題ありません（大抵の言語で同じ考え方と思います）。
もちろんあまりに意味がなかったり離れるようなネーミングはメンメナンス性に欠けるのでやめたほうが良いですが。

Answer 5

string message = GetMessage(PlayerName);

ここで、GetMessage関数が実行されてます
その実行結果を、
string型の messageという名前の変数に代入してます

string message; という記述で、string型のmessageという名前の変数を定義している、というのはわかるでしょうか