Goのif文について

Question

まだGo初心者なので教えていただきたいです。

以下のメソッドでbool値を返したいのですが

func main(){
    result := result(message string)
    print(result)
}

func result(m string) string {
    if string == "hello" {
          back := "done"
    } else {
          back := "undone"
    }
    return back
}

としたいのですが、このようにしてgo run すると

 undifined:back

とでます。どのようなgo言語の使用でこんなエラーが出ているのでしょうか？

また

func result(m string) string {
    if string == "hello" {
          back := "done"
          return back
    } else {
          back := "undone"
          return back
    }
}

としても、エラーがでます。
どのようにしたら、綺麗なコードが枯れるのでしょうか。
よろしくお願いします。

Answer 1

以下のように修正していただくと動くと思われます。
2つ目のコードのif string ==の部分を変えました。

func main() {
    var m = "hello"
    fmt.Println(result(m))
}
func result(m string) string {
    if m == "hello" {  //if stringとするとダメです
        back := "done"
        return back
    } else {
        back := "undone"
        return back
    }
}

追記

ifの中で変数を初期化すると、スコープがif文内だけになってしまいます。
なので

func result(m string) string {
    var back string //ifの外でbackを初期化
    if m == "hello" {
        back = "done"
    } else {
        back = "undone"
    }
    return back
}

このようにするとreturnが一つですみます。

Comments

[退会済みユーザー]

returnが二つあることに違和感を持ってしまい、if文をぬけたあとにreturnをしたいのですが、どのようにすればいいでしょうか？

[退会済みユーザー]

追記しました

[退会済みユーザー]

ありがとうございます！

Answer 2

質問者さんのコードに1点、DDxlk さんのコードですが、1点修正があります。

質問者さんのコードはおそらく、result という関数を呼び出した結果を表示したいのだと思います。であれば以下の様に呼び出す必要があります。

func main() {
	var m = "hello"
	fmt.Println(result(m))
}

また DDxlk さんのコードですが、Go のスタイルでは if/else 両方で return するのは、最後の return を忘れてしまう可能性がある為、あまり良いコードではないという事になっています。試しに golint というツールを入れて

golint xxx.go

※xxx.go はファイル名

を実行してみて下さい。

bad.go:13:9: if block ends with a return statement, so drop this else and outdent its block

この様に、else で return する場合には以下の様に書くのが Go のスタイルとされています。

package main

import "fmt"

func main() {
	var m = "hello"
	fmt.Println(result(m))
}

func result(m string) string {
	if m == "hello" {
		return "done"
	}
	return "undone"
}

どうしても back という変数を使われたいならば、以下の様に宣言を行うと良いです。

package main

import "fmt"

func main() {
	var m = "hello"
	fmt.Println(result(m))
}

func result(m string) string {
	var back string
	if m == "hello" {
		back = "done"
	} else {
		back = "undone"
	}
	return back
}

Comments

[退会済みユーザー]

あああああ
main関数の
var m = "hello"
fmt.Println(result)
の部分は完全に打ち間違えだと思います！
ご指摘ありがとうございます。修正しておきます！