Go: प्रस्ताव: cmd/vet: पशु चिकित्सक को समय आने पर चेतावनी देनी चाहिए। समय प्रकार (या प्रकार इसे एम्बेड करें) का उपयोग मानचित्र कुंजियों के रूप में किया जाता है।

को निर्मित 4 दिस॰ 2017 · 3टिप्पणियाँ · स्रोत: golang/go

आप गो के किस संस्करण का उपयोग कर रहे हैं ( `go version` )?

गो संस्करण 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.

समय की तरह। समय, कुछ अन्य प्रकार भी हो सकते हैं, जो मानचित्र कुंजी प्रकारों के रूप में उपयोग करने में सक्षम नहीं हैं।

इसके बजाय आपने क्या देखा?

कुछ भी रिपोर्ट नहीं किया गया

FrozenDueToAge Proposal

स्रोत

go101

सबसे उपयोगी टिप्पणी

मुझे नहीं लगता कि यह एक पशु चिकित्सक जांच के लिए बार से मिलता है।

समय का उपयोग करना पूरी तरह से मान्य है। मानचित्र कुंजी के रूप में समय, यदि आप जानते हैं कि आप क्या कर रहे हैं। मुझे लगता है कि यह वैध कोड के लिए बहुत शोर होगा।

लेकिन कोई व्यक्ति GitHub पर कुछ विश्लेषण कर सकता है यह देखने के लिए कि कितने प्रतिशत समय है। समय-कुंजी मानचित्र मान्य हैं।

bradfitz 4 दिस॰ 2017

👍2

सभी 3 टिप्पणियाँ

मुझे नहीं लगता कि यह एक पशु चिकित्सक जांच के लिए बार से मिलता है।

bradfitz 4 दिस॰ 2017

👍2

मैंने एक लिंटर लिखा है जो इस प्रकार की समस्या का पता लगाता है यदि आप रुचि रखते हैं: https://github.com/m3db/build-tools/blob/master/linters/badtime/README.md

मैं इसे अभी gometalinter के साथ एकीकृत करने का प्रयास कर रहा हूं।

richardartoul 4 दिस॰ 2017

👍1

अब वह पशु चिकित्सक गो टेस्ट का हिस्सा है, नए चेक के लिए बार और भी अधिक है। क्योंकि यह _is_ कभी-कभी समय के लिए मान्य होता है। मानचित्र कुंजी के रूप में समय, पशु चिकित्सक में इसे अस्वीकार करना ठीक नहीं है।

आप इसे मेगाचेक या किसी अन्य लिंटर में जोड़ने के बारे में @dominikh से बात कर सकते हैं।

rsc 4 दिस॰ 2017

क्या यह पृष्ठ उपयोगी था?

0 / 5 - 0 रेटिंग्स

आप गो के किस संस्करण का उपयोग कर रहे हैं ( go version )?