測驗代碼中有很多示例隱藏了 testify 等測驗庫中的包名稱。
例如：
import (
"testing"
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
)
func TestSomething(t *testing.T) {
assert := assert.New(t)
require := require.New(t)
var a string = "Hello"
var b string = "Hello"
assert.Equal(a, b, "The two words should be the same.")
}
用區域變數覆寫包命名空間的值不應該是一個明確的決定并且很少發生嗎？
import (
"testing"
"github.com/stretchr/testify/assert"
"github.com/stretchr/testify/require"
)
func TestSomething(t *testing.T) {
assertions := assert.New(t)
req := require.New(t)
var a string = "Hello"
var b string = "Hello"
assert.Equal(a, b, "The two words should be the same.")
}
我希望正確的行為是為包添加別名，就像pkgassert我覺得不習慣的那樣，或者確保變數沒有設定為與包相同的名稱：assertions := assert.New(t).
其他例子：
- 好的：
buf := bytes.Buffer{} - 壞的：
bytes := bytes.Buffer{}
注意：我不關注變數命名模式，因為 Go 傾向于推薦單個字母，或者在您可能使用緩沖區的狹窄范圍內使用短變數。由于函式長度，假設更長的名稱是可以接受和需要的。
如果這是測驗包中所需的行為，我想知道為什么？它似乎違反了 Go 在“無魔法”和可讀性方面的一個關鍵原則。
如果由此產生不良副作用，我也想知道。我的假設是這意味著在匯入后，由于變數陰影，對同一范圍內的包的任何呼叫都將不起作用
編輯
Go中的Scope Shadowing是一本不錯的讀物，并在使用方面給出了一些很好的觀點。我從這篇文章和 Stack 評論者那里得到的主要收獲：
它可以增強可讀性。（我認為這可能是因為 Go 更喜歡對包進行簡潔的命名，在 Go 中使用可變長度的包名稱。）
Parallel Test Issues was something I recall reading a while back. I didn't understand the recommendation to "hide" the loop variable by putting in:
for _, tc := range tests { tc := tc t.Run(tc.name, func(t *testing.T) { t.Parallel() // Here you test tc.value against a test function. // Let's use t.Log as our test function :-) t.Log(tc.value) }) }}
With a better understanding of shadowing, I'm seeing this "hiding" was talking about shadowing the caller scope variable of tc by assigning inside the loop, thereby shadowing.
This seems like a good use case, though I still feel it is being "clever" rather if not commented, and violates the idiomatic Go goals of being clear and readable with no "magic".
uj5u.com熱心網友回復：
哇，這是可怕的代碼。雖然它可能不是編譯器強制規則，但如果我看到這個：
import "github.com/stretchr/testify/assert"
然后在該檔案中，就我而言，任何拼寫assert的識別符號都是該包中的頂級匯入，句點。任何打破這種模式的人，以“聰明”之類的名義，都是自找麻煩。人們的肌肉記憶被訓練來識別頂級進口，所以請不要做這樣的事情。如果您想避免想到新名稱，只需洗掉一些字母：
asrt := assert.New(t)
或添加一個字母：
nAssert := assert.New(t)
或別名匯入：
import tAssert "github.com/stretchr/testify/assert"
請不要遵循該代碼中給出的糟糕示例。
uj5u.com熱心網友回復：
在查看了所有反饋和寫作之后，我將在這里總結調查結果。
如果您在看到來自 Goland 之類的工具的 Go linting 警報后正在調查此行為，希望這個答案可以為您節省相同的時間，并明確表示這是可以的、慣用的，并且僅在某些情況下在初始化后重用實際的包呼叫才會有任何問題有經驗。
使用區域變數隱藏包名稱
這是一個見仁見智的問題，沒有官方宣告表明這被認為是非慣用代碼。
除非為了清晰和可讀性而有意這樣做，否則意見總體上傾向于避免陰影，但這又是意見，而不是要求。
一旦被覆寫，包將在該范圍內不可用，因為包名稱已被區域變數遮蔽。
我觀察到這種行為在 Go 中似乎比在某些語言中更可能發生，因為 Go 提示的包名稱很短，不是多詞，并且易于輸入。
assert := assert.New(t)當這種情況發生時，這似乎是不可避免的。與往常一樣，在此類問題中，一致性是關鍵。如果團隊決定從不隱藏包名稱，則
assertions := assert.New(t)可以使用 ，或者如前所述，包可以使用別名，例如import tassert "github.com/stretchr/testify/assert". 這是一種意見標準，無論哪種方式都是合法的并且被認為是慣用的。在測驗的情況下，這通常是在初始變數集不再使用后進一步使用包而完成的。下面使用
isis 包的示例。包作者對此發表評論（這也是固執己見，但與is := is.New(t).
package package_test
import (
"testing"
"package"
iz "github.com/matryer/is"
)
func TestFuncName(t *testing.T) {
is := iz.New(t)
got := FuncName()
want := ""
is.Equal(got,want) // AssertMessage
}
陰影行為可以解決 Goroutine 變數問題
陰影是一種已知的行為，并在Effective Go 中有明確記錄。
錯誤在于在 Go for 回圈中，回圈變數在每次迭代中都被重用，因此 req 變數在所有 goroutine 之間共享。那不是我們想要的。
寫
req := req起來可能很奇怪，但在 Go 中這樣做是合法和慣用的。你會得到一個同名變數的新版本，故意在本地隱藏回圈變數，但每個 goroutine 都是唯一的。
這可以通過將變數作為引數傳遞到嵌入式 func 中來解決，而無需隱藏，從而簡化代碼。根據指南，兩者都是合法的并且被認為是慣用的。
測驗陰影
與此 goroutine 行為相關，但在測驗的背景關系中，并行測驗在 goroutine 中運行，因此也可以從這種模式中受益。
The tc := tc statement is required in the parallel test example because the closure runs in a goroutine. More generally, the tc := tc idiom used in the context of closures where the closure can be executed after the start of the next loop iteration - credit @gopher in comments
The gist on Be Careful With Table Driven Tests mentions this behavior as well in the How To Solve This.
Detecting
The check go vet can check for shadowing, but is not considered part of the stable package, and was removed here related to GitHub Issue 29260 and GitHub Issue 34053.
The change seems to have been prompted for compatibility purposes due to the requirement of unsafeptr flag. Additional comments seem to point towards also requiring better heuristics before allowing in go vet without being experimental.
- Go Vet Shadow
- Instructions on running via
go vet: Package Variables With Instructions:
go install golang.org/x/tools/go/analysis/passes/shadow/cmd/shadow
go vet -vettool=$(which shadow)
For those wanting to lint this, golangci-lint does have an option to configure shadowing checks in the configuration (search for check-shadowing as no bookmark to that text block`.
Further Reading
- Scope and Shadowing in Go
- Scopes and Closures
轉載請註明出處，本文鏈接：https://www.uj5u.com/net/338495.html
標籤：go
下一篇：模板無法正確決議嵌入FS