Add scope_common_mistake{1-3}.go

Author: devlights

examples/basic/scope/README.md

@@ -5,4 +5,7 @@
 |file|example name|note|
 |----|------------|----|
 |scope\_basic.go|scope\_basic|スコープについての基本的な事項についてのサンプルです.|
+|scope\_common\_mistake1.go|scope\_common\_mistake1|変数宣言のスコープによるよくやる間違いについてのサンプルです.|
+|scope\_common\_mistake2.go|scope\_common\_mistake2|CommonMistake1の間違い修正パターン (1)|
+|scope\_common\_mistake3.go|scope\_common\_mistake3|CommonMistake1の間違い修正パターン (2)|
 

examples/basic/scope/examples.go

@@ -16,4 +16,7 @@ func NewRegister() mappings.Register {
 // Regist -- 登録します.
 func (r *register) Regist(m mappings.ExampleMapping) {
 	m["scope_basic"] = Basic
+	m["scope_common_mistake1"] = CommonMistake1
+	m["scope_common_mistake2"] = CommonMistake2
+	m["scope_common_mistake3"] = CommonMistake3
 }

examples/basic/scope/scope_common_mistake1.go

@@ -0,0 +1,48 @@
+package scope
+
+import (
+	"os"
+
+	"github.com/devlights/gomy/output"
+)
+
+var (
+	// 現在の作業ディレクトリ. CommonMistake関数で使う前に初期化されている想定
+	_cwd1 string
+)
+
+func loadcwd() error {
+	// 一見、ちゃんと os.Getwd() の結果を パッケージ変数 _cwd1 に設定できているように見えるが・・
+	_cwd1, err := os.Getwd()
+	if err != nil {
+		return err
+	}
+
+	// ここでの結果はちゃんと表示される
+	output.Stdoutl("[loadcwd]", _cwd1)
+
+	return nil
+}
+
+// CommonMistake1 -- 変数宣言のスコープによるよくやる間違いについてのサンプルです.
+func CommonMistake1() error {
+	if err := loadcwd(); err != nil {
+		return err
+	}
+
+	output.Stdoutl("[main]", _cwd1)
+
+	// -------------------------------------------------------------------------
+	// 実行すると、上の _cwd1 の値は空文字で出力される.
+	// 理由は、loadcwd() で設定している _cwd1 は、省略変数宣言 := を使っているため
+	// ローカル変数 _cwd1 が新たに生成されてしまったため。
+	//
+	// loadcwd() 内で、最後にログ出力変わりに ローカル変数 _cwd1 の値を出力する
+	// ようにしているため、一見ちゃんと設定できているように見えるし
+	// ログ出力するために変数を使用しているため、コンパイルエラーにもならない
+	// (これがログ出力部分がなかったら、ローカル変数を使用していないという
+	//  コンパイルエラーとなるため、そこで気づける可能性がある。)
+	// -------------------------------------------------------------------------
+
+	return nil
+}

examples/basic/scope/scope_common_mistake2.go

@@ -0,0 +1,37 @@
+package scope
+
+import (
+	"os"
+
+	"github.com/devlights/gomy/output"
+)
+
+var (
+	_cwd2 string
+)
+
+func loadcwd2() error {
+	// ローカルスコープで _cwd2 が生成されるのを防ぐために
+	// err を var宣言 で先に宣言し、:= を使わないようにする
+	var err error
+
+	_cwd2, err = os.Getwd()
+	if err != nil {
+		return err
+	}
+
+	output.Stdoutl("[loadcwd]", _cwd2)
+
+	return nil
+}
+
+// CommonMistake2 -- CommonMistake1の間違い修正パターン (1)
+func CommonMistake2() error {
+	if err := loadcwd2(); err != nil {
+		return err
+	}
+
+	output.Stdoutl("[main]", _cwd2)
+
+	return nil
+}

examples/basic/scope/scope_common_mistake3.go

@@ -0,0 +1,40 @@
+package scope
+
+import (
+	"os"
+
+	"github.com/devlights/gomy/output"
+)
+
+type (
+	_pkginfo struct {
+		cwd string
+	}
+)
+
+var (
+	pkginfo _pkginfo
+)
+
+func loadcwd3() error {
+	cwd, err := os.Getwd()
+	if err != nil {
+		return err
+	}
+
+	pkginfo.cwd = cwd
+	output.Stdoutl("[loadcwd]", pkginfo.cwd)
+
+	return nil
+}
+
+// CommonMistake3 -- CommonMistake1の間違い修正パターン (2)
+func CommonMistake3() error {
+	if err := loadcwd3(); err != nil {
+		return err
+	}
+
+	output.Stdoutl("[main]", pkginfo.cwd)
+
+	return nil
+}