Tslint: ऑब्जेक्ट स्प्रेड को प्राथमिकता नहीं दी जानी चाहिए जब ऑब्जेक्ट.असाइन का पहला तर्क एक क्लास इंस्टेंस हो

को निर्मित 25 मई 2017 · 3टिप्पणियाँ · स्रोत: palantir/tslint

बग रिपोर्ट

__TSLint संस्करण__: 5.3

टाइपस्क्रिप्ट कोड पंक्तिबद्ध किया जा रहा है

value = Object.assign(new Model(), value)

prefer-object-spread सक्षम के साथ

वास्तविक व्यवहार

लिंट की विफलता, ऑब्जेक्ट स्प्रेड का उपयोग करने के लिए ऑटो-फिक्स

अपेक्षित् व्यवहार

कुछ नहीं। इस नियम को उन मामलों को अनदेखा करना चाहिए जहां Object.assign का पहला तर्क एक वर्ग उदाहरण है।

अगर --project की आपूर्ति की जाती है, तो हम जांच सकते हैं कि पहले तर्क का प्रकार वर्ग-जैसा है या नहीं
यदि कोई परियोजना प्रदान नहीं की जाती है और हम केवल वाक्यात्मक जाँच कर रहे हैं, तो सबसे अच्छा हम new ... जाँच कर सकते हैं और अभिव्यक्ति को अनदेखा कर सकते हैं

जैसा कि https://github.com/palantir/tslint/issues/2618#issuecomment -303879255 में बताया गया है

Fixed Bug

स्रोत

adidahiya

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

यदि पहला तर्क एक वस्तु शाब्दिक नहीं है, तो नियम केवल एक त्रुटि उत्पन्न करता है यदि परिणाम किसी चीज़ को सौंपा गया हो।
ऑब्जेक्ट का रिटर्न वैल्यू असाइन करना। असाइन करना केवल तभी समझ में आता है जब पहला तर्क CallExpression, NewExpression या इसी तरह का हो। बाकी सब कुछ, उदाहरण के लिए एक पहचानकर्ता या PropertyAccessExpression अभी भी एक त्रुटि होनी चाहिए।
और इसके लिए टाइप चेकर की आवश्यकता नहीं है।

ajafff 25 मई 2017

👍2

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

Object.assign अपने पहले तर्क पर दुष्प्रभाव हैं। (इसीलिए टाइपस्क्रिप्ट { ...x, ...y } से Object.assign({}, x, y) ट्रांसपाइल करता है।) नियम तभी सक्रिय होना चाहिए जब पहला तर्क एक ताजा वस्तु शाब्दिक हो।

andy-hanson 25 मई 2017

नियम केवल तभी सक्रिय होना चाहिए जब पहला तर्क एक ताजा वस्तु शाब्दिक हो।

ठीक है, तकनीकी रूप से, तुम सही हो। लेकिन मुझे अक्सर ऐसी गलतियाँ दिखाई देती हैं जहाँ प्रोग्रामर पहले तर्क पर होने वाले दुष्प्रभावों के बारे में सावधान नहीं होते हैं और वे वैसे भी एक गैर-शाब्दिक मूल्य प्रदान करते हैं, जब वे _actually_ वस्तु के प्रसार के समान ही काम करते हैं। इसलिए मैं अब भी चाहूंगा कि यह नियम उसे पकड़ ले। शायद यह ऑप्ट-इन हो सकता है ...

adidahiya 25 मई 2017