कोड गंध एक क्लासिक हैं।  यह गंध करता है क्योंकि ऐसे कई उदाहरण हैं जहां इसे संपादित या सुधार किया जा सकता है।  इनमें से अधिकतर गंध कुछ गलत होने का संकेत मात्र हैं। वे अनिवार्य रूप से निश्चित नहीं हैं ... (हालांकि आपको इसे देखना चाहिए।)   पिछला कोड बदबू आ रही है   भाग I   भाग द्वितीय   भाग III   भाग IV   भाग V   भाग VI   भाग VII   भाग VIII   भाग IX   भाग X   भाग XI   भाग XII   भाग XIII   भाग XIV   भाग XV   भाग XVI   भाग XVII   भाग XVIII   भाग XIX   भाग XX  चलो जारी रखते है...  कोड गंध 101 - बूलियन के खिलाफ तुलना    बूलियन से तुलना करते समय, हम जादुई कास्टिंग करते हैं और अप्रत्याशित परिणाम प्राप्त करते हैं।  TL; DR: सत्य से तुलना न करें। या तो आप सच्चे हैं, या झूठे हैं या आपको तुलना नहीं करनी चाहिए  समस्या  हिडन कास्टिंग  कम से कम आश्चर्य सिद्धांत का उल्लंघन।    सिद्धांत उल्लंघन तेजी से विफल  समाधान  बूलियन का प्रयोग करें  बूलियन को   के साथ न मिलाएं बूलियन कास्टेबल ऑब्जेक्ट्स  संदर्भ  कई भाषाओं ने बूलियन क्रॉसिंग डोमेन को मान दिया है।  नमूना कोड  गलत   #!/bin/bash if [ false ]; then echo "True" else echo "False" fi # this evaluates to true since # "false" is a non-empty string if [ false ] = true; then echo "True" else echo "False" fi # this also evaluates to true  सही   #!/bin/bash if false ; then echo "True" else echo "False" fi # this evaluates to false  खोज    स्वचालित [x]  लिंटर स्पष्ट तुलना और चेतावनियों की जांच कर सकते हैं।  टैग  कास्टिंग्स  निष्कर्ष  कई गैर-बूलियन को बूलियन के रूप में उपयोग करना एक सामान्य उद्योग प्रथा है। बूलियन का उपयोग करते समय हमें बहुत सख्त होना चाहिए।  संबंधों   कोड गंध 69 - बिग बैंग (जावास्क्रिप्ट हास्यास्पद कास्टिंग)  और जानकारी   तेजी से विफल  क्रेडिट    . पर   द्वारा फोटो Unsplash माइकल हेल्ड  यदि यह काम नहीं करता है, तो इससे कोई फर्क नहीं पड़ता कि यह कितनी तेजी से काम नहीं करता है।  -  मिच रावेरा  कोड गंध 102 - तीर कोड    नेस्टेड IFs और Elses को पढ़ना और परीक्षण करना बहुत कठिन है  TL; DR: नेस्टेड IFs से बचें। और भी बेहतर: सभी IFs से बचें  समस्या  पठनीयता  समाधान  निकालने की विधि  बूलियन स्थितियों को मिलाएं  आकस्मिक IFs निकालें  संदर्भ  प्रक्रियात्मक कोड में, जटिल नेस्टेड आईएफएस देखना बहुत आम है। यह समाधान ऑब्जेक्ट-ओरिएंटेड प्रोग्रामिंग की तुलना में स्क्रिप्टिंग से अधिक संबंधित है।  नमूना कोड  गलत   if (actualIndex < totalItems) { if (product[actualIndex].Name.Contains("arrow")) { do { if (product[actualIndex].price == null) { // handle no price } else { if (!(product[actualIndex].priceIsCurrent())) { // add price } else { if (!hasDiscount) { // handle discount } else { // etc } } } actualIndex++; } while (actualIndex < totalCounf && totalPrice < wallet.money); } else actualIndex++; } return actualIndex; }  सही   foreach (products as currentProduct) addPriceIfDefined(currentProduct) addPriceIfDefined() { //Several extracts }  खोज    स्वचालित [x]  चूंकि कई लिंटर पेड़ों को पार्स कर सकते हैं, हम घोंसले के स्तर के लिए संकलन-समय की जांच कर सकते हैं।  टैग  पठनीयता  जटिलता  निष्कर्ष    बाद, हमें कोड क्लीनर को छोड़ देना चाहिए जितना हमने पाया। अंकल बॉब की सलाह के  इस समस्या को रिफैक्टर करना आसान है।  संबंधों   कोड गंध 78 - कॉलबैक नरक   कोड गंध 03 - कार्य बहुत लंबे हैं   कोड गंध 36 - स्विच/केस/elseif/else/if कथन  और जानकारी   सी2 विकी   फ़्लैटनिंग एरो कोड   रिफैक्टरिंग.कॉम  सॉफ्टवेयर इंजीनियरिंग का उद्देश्य जटिलता को नियंत्रित करना है, न कि उसे बनाना।   - पामेला ज़वेस  कोड गंध 103 - डबल एनकैप्सुलेशन    अपने स्वयं के एक्सेसर विधियों को कॉल करना एक अच्छा इनकैप्सुलेशन विचार प्रतीत हो सकता है। लेकिन यह नहीं है।  टीएल; डीआर: निजी इस्तेमाल के लिए भी सेटर्स और गेटर्स का उपयोग न करें  समस्या  सेटर्स  टिककर खेल  निजी विशेषताओं को उजागर करना  समाधान   बसने वालों को हटा दें  गेटर्स हटाएं  अपने गुणों की रक्षा करें  संदर्भ  90 के दशक में डबल एनकैप्सुलेशन का उपयोग करना एक मानक प्रक्रिया थी।  हम निजी इस्तेमाल के लिए भी कार्यान्वयन विवरण छिपाना चाहते थे।  यह एक और गंध छिपा रहा था जब बहुत सारे कार्य डेटा संरचना और आकस्मिक कार्यान्वयन पर निर्भर करते हैं।  उदाहरण के लिए, हम किसी वस्तु के आंतरिक प्रतिनिधित्व को बदल सकते हैं और उसके बाहरी प्रोटोकॉल पर भरोसा कर सकते हैं।  लागत/लाभ इसके लायक नहीं है।  नमूना कोड  गलत   contract MessageContract { string message = "Let's trade"; function getMessage() public constant returns(string) { return message; } function setMessage(string newMessage) public { message = newMessage; } function sendMessage() public constant { this.send(this.getMessage()); // We can access property but make a self call instead } }  सही   contract MessageContract { string message = "Let's trade"; function sendMessage() public constant { this.send(message); } }  खोज    अर्ध-स्वचालित [x]  हम गेटर्स और सेटर्स का अनुमान लगा सकते हैं और जांच सकते हैं कि क्या उन्हें एक ही वस्तु से बुलाया गया है।  टैग  कैप्सूलीकरण  निष्कर्ष  आकस्मिक कार्यान्वयन की रक्षा के लिए डबल एनकैप्सुलेशन एक ट्रेंडी विचार था, लेकिन यह संरक्षित से अधिक उजागर हुआ।  संबंधों   कोड गंध 37 - संरक्षित गुण   कोड गंध 28 - सेटर्स   कोड गंध 68 - गेटर्स  और जानकारी   स्टैक एक्सचेंज   reddit   इन्फोवर्ल्ड  क्रेडिट    . पर   द्वारा फोटो Unsplash रे हेनेसी  उस अवधारणा को समाहित करें जो भिन्न होती है।  -  एरिच गामा  कोड गंध 104 - सही पर जोर दें    बूलियन के खिलाफ जोर देने से त्रुटि ट्रैकिंग अधिक कठिन हो जाती है।  टीएल; डीआर: जब तक आप बूलियन की जांच नहीं कर रहे हैं, तब तक सत्य का दावा न करें  समस्या  तेजी से विफल सिद्धांत  समाधान  जांचें कि क्या बूलियन स्थिति को बेहतर ढंग से फिर से लिखा जा सकता है  पक्ष मुखरएक्वाल्स  संदर्भ  बूलियन का दावा करते समय हमारे परीक्षण इंजन हमारी बहुत मदद नहीं कर सकते हैं।  वे बस हमें बताते हैं कि कुछ विफल रहा।  त्रुटि ट्रैकिंग अधिक कठिन हो जाती है।  नमूना कोड  गलत   <? final class RangeUnitTest extends TestCase { function testValidOffset() { $range = new Range(1, 1); $offset = $range->offset(); $this->assertTrue(10 == $offset); // No functional essential description :( // Accidental description provided by tests is very bad } } // When failing Unit framework will show us // // 1 Test, 1 failed // Failing asserting true matches expected false :( // () <-- no business description :( // // <Click to see difference> - Two booleans // (and a diff comparator will show us two booleans)  सही   <? final class RangeUnitTest extends TestCase { function testValidOffset() { $range = new Range(1, 1); $offset = $range->offset(); $this->assertEquals(10, $offset, 'All pages must have 10 as offset'); // Expected value should always be first argument // We add a functional essential description // to complement accidental description provided by tests } } // When failing Unit framework will show us // // 1 Test, 1 failed // Failing asserting 0 matches expected 10 // All pages must have 10 as offset <-- business description // // <Click to see difference> // (and a diff comparator will help us and it will be a great help // for complex objects like objects or jsons)  खोज    अर्ध-स्वचालित [x]  कुछ लिंटर हमें चेतावनी देते हैं कि क्या हम इस स्थिति को सेट करने के बाद बूलियन के खिलाफ जाँच कर रहे हैं।  हमें इसे और अधिक विशिष्ट जांच में बदलने की जरूरत है।  टैग  टेस्ट गंध  निष्कर्ष  अपने बुलियन दावे को फिर से लिखने का प्रयास करें और आप विफलताओं को बहुत तेज़ी से ठीक कर देंगे।  संबंधों   कोड गंध 101 - बूलियन के खिलाफ तुलना   कोड गंध 07 - बूलियन चर  और जानकारी   तेजी से विफल  क्रेडिट    . पर   द्वारा फोटो Unsplash जोएल डी वीरेन्ड  मैंने अंत में सीखा है कि 'ऊपर की ओर संगत' का क्या अर्थ है। इसका मतलब है कि हमें अपनी सभी पुरानी गलतियों को रखना होगा।  -  डेनी वैन टैसेलो  कोड गंध 105 - कॉमेडियन तरीके    पेशेवर और सार्थक नामों का प्रयोग करें  TL; DR: अनौपचारिक या आक्रामक न हों  समस्या  पठनीयता  अव्यवसायिक कार्य  समाधान  अच्छे और पेशेवर नाम चुनें।  संदर्भ  हमारे पेशे का एक रचनात्मक पक्ष है।  कभी-कभी हम ऊब जाते हैं और मजाकिया बनने की कोशिश करते हैं।  नमूना कोड  गलत   function erradicateAndMurderAllCustomers(); // unprofessional and offensive  सही   function deleteAllCustomers(); // more declarative and professional  खोज    अर्ध-स्वचालित [x]  हमारे पास निषिद्ध शब्दों की एक सूची हो सकती है।  हम उन्हें कोड समीक्षाओं में भी देख सकते हैं।  नाम प्रासंगिक हैं, इसलिए स्वचालित लिंटर के लिए यह एक मुश्किल काम होगा।  नामकरण परंपराएं सामान्य होनी चाहिए और इसमें सांस्कृतिक शब्दजाल शामिल नहीं होना चाहिए।  टैग  नामकरण  निष्कर्ष  अपने कोड में चीजों को नाम देने के तरीके में पेशेवर बनें।  एक वेरिएबल को एक मूर्खतापूर्ण नाम देकर कॉमेडियन बनने की कोशिश न करें।  आपको प्रोडक्शन कोड लिखना चाहिए ताकि भविष्य के सॉफ्टवेयर डेवलपर्स (यहां तक कि आप भी) आसानी से समझ सकें।  संबंधों   कोड गंध 38 - सार नाम  और जानकारी   नाम में क्या रखा है  क्रेडिट    . पर   द्वारा फोटो Unsplash स्टीवर्ट मुनरो  यह 'उपयोगकर्ता बेवकूफ हैं, और कार्यक्षमता से भ्रमित हैं' जीनोम की मानसिकता एक बीमारी है। अगर आपको लगता है कि आपके उपयोगकर्ता बेवकूफ हैं, तो केवल बेवकूफ ही इसका इस्तेमाल करेंगे।  -  लिनुस टॉर्वाल्ड्स   सॉफ्टवेयर इंजीनियरिंग महान उद्धरण  और अभी के लिए बस इतना ही…  अगला लेख 5 और कोड गंधों की व्याख्या करेगा!

Keep

Winner of Noonies 2020 for the award Hacker Noon Contributor of the Year - REFACTORING.

2020 - Hacker Noon Contributor of the Year in REFACTORING

2022 - HackerNoon Contributor of the Year - Clean Code

2022 - HackerNoon Contributor of the Year - Debugging

2022 - HackerNoon Contributor of the Year - Github

2022 - HackerNoon Contributor of the Year - Machine Learning

2022 - HackerNoon Contributor of the Year - Quality Assurance

2022 - HackerNoon Contributor of the Year - Refactoring

2022 - HackerNoon Contributor of the Year - Software Architecture

2022 - the Greatest Storyteller

Buy My Book

Share your ideas with me!

Read My Stories

Buy Me a Coffee

Software Engineer and Book Author

यह ऑडियो कहानी की मूल भाषा में निर्मित है!

बहुत लंबा; पढ़ने के लिए

अपने कोड के बदबूदार हिस्सों को कैसे खोजें [भाग XXI]

अपने कोड के बदबूदार हिस्सों को कैसे खोजें [भाग XXI]

About Author

टिप्पणियाँ

लेबल

इस लेख में चित्रित किया गया था

Related Stories

उपयोगकर्ता-केंद्रित क्रिप्टो उत्पाद बनाना: ग्राहक प्रतिक्रिया का महत्व

10 Reasons Why Publishing on HackerNoon Will Skyrocket Your Reach and Impact

अपने वर्कफ़्लो को 10 गुना बेहतर कैसे बनाएं: 17 ज़रूरी ऐप्स

डिजिटल खानाबदोशों सुनो: थाईलैंड के नए डीटीवी वीज़ा के बारे में आपको क्या जानना चाहिए

उपयोगकर्ता-केंद्रित क्रिप्टो उत्पाद बनाना: ग्राहक प्रतिक्रिया का महत्व

10 Reasons Why Publishing on HackerNoon Will Skyrocket Your Reach and Impact

अपने वर्कफ़्लो को 10 गुना बेहतर कैसे बनाएं: 17 ज़रूरी ऐप्स

डिजिटल खानाबदोशों सुनो: थाईलैंड के नए डीटीवी वीज़ा के बारे में आपको क्या जानना चाहिए

Light-Mode

Classic

Newspaper

Dark-Mode

Neon Noir

Minty

HN StartUps