Jshint: टर्नरी लाइन ब्रेकिंग

को निर्मित 5 जुल॰ 2015 · 19टिप्पणियाँ · स्रोत: jshint/jshint

var a = (tern)
    ? b
    : c;

यह कोड चेतावनी देता है: Bad line breaking before '?'.

जशिंट प्रवाह नियंत्रण है:

पहले पकड़ो "?" टोकन और फिर इसे अभिव्यक्ति समारोह में पास करें

  infix("?", function(left, that) {
    increaseComplexityCount();
    that.left = left;
    that.right = expression(10);
    advance(":");
    that["else"] = expression(10);
    return that;
  }, 30);

फंक्शन एक्सप्रेशन में यह तर्क इसके प्रारंभ के पास होता है

    var isDangerous =
      state.option.asi &&
      state.tokens.prev.line !== startLine(state.tokens.curr) &&
      _.contains(["]", ")"], state.tokens.prev.id) &&
      _.contains(["[", "("], state.tokens.curr.id);

    if (isDangerous)
      warning("W014", state.tokens.curr, state.tokens.curr.id);

ऐसा प्रतीत होता है कि JSHint कोड गलत तरीके से ")" और "?" के बीच लाइन ब्रेक को पकड़ रहा है। अपने एएसआई तर्क में। इस मामले में एएसआई को आवेदन नहीं करना चाहिए। एक टर्नरी को कई लाइनों में तोड़ने से कोड को पढ़ना आसान हो जाता है। उदाहरण:

var a = (tern1 > 0)
        ? b
        : (tern2 > x)
            ? c
            : d,
    e = 1;

Needs Discussion P1

स्रोत

prettydiff

👍4

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

@prettydiff
यह गुजर जाता है

/* jshint laxbreak: true */
var a = (tern)
    ? b
    : c;

तो, चेतावनियां न पाने के लिए बस उस विकल्प को चालू करें।

laxbreak को निकट भविष्य में हटा दिया जाएगा। इसे हटाने के लिए उठाया गया ये है मुद्दा - #1867

lukeapage 5 जुल॰ 2015

👍4

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

आप अभी भी JSHint का उपयोग क्यों कर रहे हैं, और न केवल JSLint जो कि बहुत बेहतर है?

kaizer200 5 जुल॰ 2015

👎5 👍1

@ kaizer200 JSHint कहीं अधिक बार उपयोग किया जाने वाला टूल है। मेरा लक्ष्य दोनों की डिफ़ॉल्ट सेटिंग्स के अनुरूप जितना हो सके उतना करीब से प्राप्त करना है। इस तरह मैं कम से कम संभव समायोजन के साथ किसी भी उपकरण के अनुरूप किसी भी उपयोगकर्ता की इच्छा का सटीक रूप से प्रतिनिधित्व कर सकता हूं।

prettydiff 5 जुल॰ 2015

मुझे लगता है कि यह वांछित है, क्योंकि यह परीक्षण है:

code = [ 'var a = b ', '? c : d;' ];
TestRun(test)
  .addError(2, "Bad line breaking before '?'.")
  .test(code, { es3: true });

( tests/unit/options.js , लाइन 1583)

nicolo-ribaudo 5 जुल॰ 2015

तथ्य यह है कि एक परीक्षण है इसका मतलब यह नहीं है कि यह वांछित है। यह अधिक jscs का क्षेत्र है

caitp 5 जुल॰ 2015

@caitp क्या आप इसे ठीक कर रहे हैं?

nicolo-ribaudo 5 जुल॰ 2015

यह लैक्सब्रेक द्वारा अक्षम नहीं है? मुझे पता है कि यह पदावनत है, लेकिन इसे v3 . तक नहीं हटाया जाएगा

lukeapage 5 जुल॰ 2015

v3 के लिए कोई वास्तविक रोडमैप नहीं है, इससे छुटकारा पाना सुरक्षित है --- यह किसी के निर्माण को नहीं तोड़ेगा

caitp 5 जुल॰ 2015

@caitp उसी टोकन के लिए, JSHint को डिफ़ॉल्ट रूप से &&, ||, ... से पहले लाइन ब्रेक की भी अनुमति देनी चाहिए।

nicolo-ribaudo 5 जुल॰ 2015

एक रोडमैप है, मैं और @jugglinmike ने इस पर चर्चा की है। यह सब कुछ v3 मील के पत्थर पर है।

यह लोगों के निर्माण को नहीं तोड़ेगा, लेकिन यह उन चेतावनियों को रोक देगा जिन पर लोग भरोसा करते हैं, जब कोई चेतावनी नहीं है और बहुत कम दस्तावेज है कि विकल्प हटा दिया गया है (और लोगों को चेंजलिस्ट को देखने और नोटिस करने के लिए कोई बड़ा संस्करण टक्कर नहीं है कि उन्हें जेएसएससी का उपयोग करना चाहिए )

lukeapage 5 जुल॰ 2015

कोई भी वास्तविक रूप से इस पर भरोसा नहीं कर रहा है, और मील का पत्थर रोडमैप नहीं है। V3 मील का पत्थर सदियों से है, कोई समय सीमा नहीं है और कोई दबाव नहीं है। सभी रिलीज़ के लिए एक परिवर्तन लॉग है, न कि केवल प्रमुख रिलीज़ के लिए, और अन्य फर्जी चेतावनियाँ पहले ही हटा दी गई हैं। इसे अनिश्चित काल तक रखने के लिए कोई बाध्यकारी तर्क नहीं है, यह जेएससी क्षेत्र है

caitp 5 जुल॰ 2015

v3 मील का पत्थर सदियों से रहा है

हमने हाल ही में इसे मंजूरी दी है और इसे एक छोटी, प्राप्त करने योग्य सूची में बनाया है।

इसके लिए कुछ पीआर इंतजार कर रहे हैं।

सभी रिलीज़ के लिए एक परिवर्तन लॉग है, न कि केवल प्रमुख रिलीज़ के लिए, और अन्य फर्जी चेतावनियाँ पहले ही हटा दी गई हैं। इसे अनिश्चित काल तक रखने के लिए कोई बाध्यकारी तर्क नहीं है, यह जेएससी क्षेत्र है

मैं मानता हूँ, बस कब। मैं सिर्फ _both_ क्लासिंग से असहमत हूं, बहिष्कृत नियमों के बारे में चेतावनी नहीं देता क्योंकि यह एक ब्रेकिंग परिवर्तन है, इसलिए v3 तक विलय नहीं करना और साथ ही, बहिष्कृत नियमों को हटा देना। अगर जशिंट अति सावधान रहना चाहता है, तो अति सावधान रहें। यदि वह नियमों को हटाना चाहता है, तो एक प्रमुख संस्करण को टक्कर दें या लोगों को चेतावनी दें।

यदि आप कुछ समय पहले jshint को सेट करते हैं तो यह याद रखना बहुत आसान है कि विकल्प बहिष्कृत हो गए हैं - क्या हम चाहते हैं कि लोगों को जो अनुभव मिलता है वह उनके लिए एक बग उठाना है और फिर इसे बहिष्कृत बताया जाए? या इसे स्पष्ट करें?

lukeapage 5 जुल॰ 2015

कोई भी वास्तविक रूप से इस पर भरोसा नहीं कर रहा है

मैं किसी प्रकार की शैली की जाँच के लिए jshint पर भरोसा करता था जब तक कि मुझे jscs के बारे में पता नहीं चला और उस पर स्विच नहीं किया। मुझे यकीन है कि बहुत से लोग अभी भी हैं।

lukeapage 5 जुल॰ 2015

कोई भी वास्तविक रूप से इस पर भरोसा नहीं कर रहा है

GitHub पर खोजा जा रहा है... https://github.com/search?l=json&q=%22laxbreak%22%3A+false&ref=searchresults&type=Code&utf8=%E2%9C%93

nicolo-ribaudo 5 जुल॰ 2015

@prettydiff
यह गुजर जाता है

/* jshint laxbreak: true */
var a = (tern)
    ? b
    : c;

तो, चेतावनियां न पाने के लिए बस उस विकल्प को चालू करें।

lukeapage 5 जुल॰ 2015

👍4

इसे #1867 के डुप्लीकेट के रूप में बंद करने पर कोई आपत्ति है?

lukeapage 5 जुल॰ 2015

मेरा वोट इसे एक नया बग बनाना और इसे ठीक से ठीक करना है। यहां कोई प्रमुख संस्करण टक्कर की आवश्यकता नहीं है

caitp 5 जुल॰ 2015

@lukeapage अगर v3.0 में लैक्सब्रेक हटा दिया जाता है तो क्या यह कोड नमूना v3.0 में चेतावनियां उत्पन्न करेगा और v3.0 रिलीज पर अनुमानित समयरेखा क्या है?

prettydiff 6 जुल॰ 2015

यदि v3.0 में शिथिलता को हटा दिया जाता है तो क्या यह कोड नमूना v3.0 में चेतावनियां उत्पन्न करेगा?

योजना लैक्सब्रेक विकल्प को हटाने की है और अब लैक्सब्रेक के मामलों में चेतावनी नहीं देने की है।
विकल्प का उपयोग एक चेतावनी उत्पन्न कर सकता है कि विकल्प बहिष्कृत है और इसे हटाया जा सकता है।

v3.0 रिलीज पर अनुमानित समयरेखा क्या है

मुझे यकीन नहीं है, समीक्षा/अंतिम रूप देने के लिए पीआर की एक कतार है और इसमें बहुत सारी बग ठीक करें। मुझे संदेह है कि यह अगले महीने में होगा।

कैट एक अधिक दीर्घकालिक योगदानकर्ता है और अन्य योगदानकर्ता/मालिक उसके साथ सहमत होने और अगली रिलीज में चेतावनियों को हटाने का निर्णय ले सकते हैं, जिस बिंदु पर आप चेतावनियां देखना बंद कर देंगे और फिर v3 में आपको चेतावनी मिलनी शुरू हो जाएगी कि लैक्सब्रेक विकल्प बहिष्कृत है (क्या आपको अभी इसका उपयोग शुरू करने का निर्णय लेना चाहिए)।

lukeapage 6 जुल॰ 2015

@lukeapage इसके साथ अब आप इस बग को बंद कर सकते हैं (मेरे दृष्टिकोण से)।

prettydiff 6 जुल॰ 2015

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

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

Jshint: टर्नरी लाइन ब्रेकिंग

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

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

संबंधित मुद्दों