Go: 제안: cmd/vet: 수의사는 time.Time 유형(또는 이를 포함하는 유형)이 지도 키로 사용될 때 경고해야 합니다.

에 만든 2017년 12월 04일 · 3코멘트 · 출처: golang/go

이동 버전 go1.9.2 linux/amd64

예

package main

import "time"

type T struct {
    time.Time
}

var a map[T]string

func main() {
}

$ go vet a.go
$ [nothing reported]

$ go vet a.go
$ time.Time is not a good map key type.

time.Time과 마찬가지로 맵 키 유형으로 사용할 수 없는 다른 유형이 있을 수 있습니다.

보고된 바 없음

FrozenDueToAge Proposal

출처

go101

가장 유용한 댓글

나는 이것이 수의사 검사의 기준에 부합하지 않는다고 생각합니다.

시간을 사용하는 것은 완벽하게 유효합니다. 시간을 지도 키로 사용하는 것이 좋습니다. 유효한 코드에 대해서는 너무 시끄럽다고 생각합니다.

그러나 누군가 GitHub에서 몇 퍼센트의 시간을 확인하기 위해 분석을 할 수 있습니다. 시간 키 맵이 유효합니다.

bradfitz 에 2017년 12월 04일

👍2

나는 이것이 수의사 검사의 기준에 부합하지 않는다고 생각합니다.

시간을 사용하는 것은 완벽하게 유효합니다. 시간을 지도 키로 사용하는 것이 좋습니다. 유효한 코드에 대해서는 너무 시끄럽다고 생각합니다.

그러나 누군가 GitHub에서 몇 퍼센트의 시간을 확인하기 위해 분석을 할 수 있습니다. 시간 키 맵이 유효합니다.

bradfitz 에 2017년 12월 04일

👍2

관심이 있는 경우 이러한 유형의 문제를 감지하는 린터를 작성했습니다. https://github.com/m3db/build-tools/blob/master/linters/badtime/README.md

지금 gometalinter와 통합하려고 합니다.

richardartoul 에 2017년 12월 04일

👍1

이제 수의사가 테스트의 일부가 되었기 때문에 새로운 검사에 대한 기준은 더욱 높아졌습니다. 때때로 시간을 갖는 것이 _is_ 유효하기 때문에. 시간을 지도 키로 사용하므로 수의사에서 이를 거부하는 것은 옳지 않습니다.

megacheck 또는 다른 린터 중 하나에 이것을 추가하는 것에 대해

rsc 에 2017년 12월 04일

이 페이지가 도움이 되었나요?

0 / 5 - 0 등급