कोड गंध एक क्लासिक हैं।  यह गंध करता है क्योंकि ऐसे कई उदाहरण हैं जहां इसे संपादित या सुधार किया जा सकता है।  इनमें से अधिकतर गंध कुछ गलत होने का संकेत मात्र हैं। वे अनिवार्य रूप से निश्चित नहीं हैं ... (हालांकि आपको इसे देखना चाहिए।)   पिछला कोड बदबू आ रही है   भाग I   भाग द्वितीय   भाग III   भाग IV   भाग V   भाग VI   भाग VII   भाग VIII   भाग IX   भाग X   भाग XI   भाग XII   भाग XIII   भाग XIV   भाग XV   भाग XVI   भाग XVII   भाग XVIII   भाग XIX   भाग XX   भाग XXI  चलो जारी रखते है...  कोड गंध 106 - उत्पादन निर्भर कोड    उत्पादन परिवेश के लिए IFs जाँच न जोड़ें।  TL; DR: उत्पादन से संबंधित शर्तों को जोड़ने से बचें  समस्या  तेजी से विफल सिद्धांत उल्लंघन  टेस्टेबिलिटी की कमी  समाधान  यदि पूरी तरह से आवश्यक हो, तो वातावरण को मॉडल करें और उन   का परीक्षण करें। सभी  संदर्भ  कभी-कभी, हमें विकास और उत्पादन में अलग-अलग व्यवहार बनाने की आवश्यकता होती है।  उदाहरण के लिए पासवर्ड की ताकत।  इस मामले में, हमें पर्यावरण को ताकत की रणनीति के साथ कॉन्फ़िगर करने और रणनीति का परीक्षण करने की आवश्यकता है, न कि पर्यावरण को ही।  नमूना कोड  गलत   def send_welcome_email(email_address, environment): if ENVIRONMENT_NAME == "production": print(f"Sending welcome email to {email_address} from Bob Builder <bob@builder.com>") else: print("Emails are sent only on production") send_welcome_email("john@doe.com", "development") # Emails are sent only on production send_welcome_email("john@doe.com", "production") # Sending welcome email to john@doe.com from Bob Builder <bob@builder.com>  सही   class ProductionEnvironment: FROM_EMAIL = "Bob Builder <bob@builder.com>" class DevelopmentEnvironment: FROM_EMAIL = "Bob Builder Development <bob@builder.com>" # We can unit test environments # and even implement different sending mechanisms def send_welcome_email(email_address, environment): print(f"Sending welcome email to {email_address} from {environment.FROM_EMAIL}") # We can delegate into a fake sender (and possible logger) # and unit test it send_welcome_email("john@doe.com", DevelopmentEnvironment()) # Sending welcome email to john@doe.com from Bob Builder Development <bob@builder.com> send_welcome_email("john@doe.com", ProductionEnvironment()) # Sending welcome email to john@doe.com from Bob Builder <bob@builder.com>  खोज    मैनुअल [एक्स]  यह एक डिजाइन गंध है।  हमें खाली विकास/उत्पादन विन्यास बनाने और उन्हें अनुकूलन योग्य बहुरूपी वस्तुओं के साथ सौंपने की आवश्यकता है।  टैग  युग्मन  निष्कर्ष  अनुपयुक्त सशर्त जोड़ने से बचें।  व्यावसायिक नियम सौंपते हुए कॉन्फ़िगरेशन बनाएं।  अमूर्त, प्रोटोकॉल और इंटरफेस का प्रयोग करें, कठिन पदानुक्रमों से बचें।  संबंधों   कोड गंध 56 - प्रीप्रोसेसर  और जानकारी   कष्टप्रद ifs से कैसे बचें  क्रेडिट    . पर   द्वारा फोटो Unsplash बर्मिंघम संग्रहालय ट्रस्ट  यह ट्वीट @   . से प्रेरित था Jan Giacomelli   ट्विटर  जटिलता तकनीकी अपरिपक्वता का प्रतीक है। उपयोग की सादगी एक अच्छी तरह से डिजाइन उत्पाद का वास्तविक संकेत है चाहे वह एटीएम हो या पैट्रियट मिसाइल।   डेनियल टी. लिंग   सॉफ्टवेयर इंजीनियरिंग महान उद्धरण  कोड गंध 107 - चर पुन: उपयोग    चर का पुन: उपयोग करने से दायरे और सीमाओं का पालन करना कठिन हो जाता है  TL; DR: अलग-अलग उद्देश्यों के लिए एक ही चर को न पढ़ें और न लिखें  समस्या  पठनीयता  छिपी हुई समस्याएं  समाधान  चर का पुन: उपयोग न करें  स्कोप को अलग करने की  विधि निकालें  संदर्भ  स्क्रिप्ट की प्रोग्रामिंग करते समय वेरिएबल्स का पुन: उपयोग करना आम बात है।  इससे भ्रम होता है और डिबगिंग कठिन हो जाती है।  हमें जितना हो सके इसका दायरा कम करना चाहिए।  नमूना कोड  गलत   // print line total double total = item.getPrice() * item.getQuantity(); System.out.println("Line total: " + total ); // print amount total total = order.getTotal() - order.getDiscount(); System.out.println( "Amount due: " + total ); // variable is reused  सही   function printLineTotal() { double total = item.getPrice() * item.getQuantity(); System.out.println("Line total: " + total ); } function printAmountTotal() { double total = order.getTotal() - order.getDiscount(); System.out.println( "Amount due: " + total ); }  खोज    स्वचालित []  चर परिभाषा और उपयोग खोजने के लिए लिंटर पार्स ट्री का उपयोग कर सकते हैं।  टैग  पठनीयता  निष्कर्ष  परिवर्तनीय नामों का पुन: उपयोग करने से बचें। अधिक विशिष्ट और भिन्न नामों का प्रयोग करें।  संबंधों   कोड गंध 03 - कार्य बहुत लंबे हैं  और जानकारी   002 रिफैक्टरिंग - निकालने की विधि  क्रेडिट    . पर   द्वारा फोटो Unsplash सिगमंड  सामान्यता से पहले सादगी, पुन: उपयोग से पहले उपयोग करें।   केवलिन हेनी   सॉफ्टवेयर इंजीनियरिंग महान उद्धरण  कोड गंध 108 - फ्लोट अभिकथन    दो फ्लोट नंबरों को एक ही मान लेना एक बहुत ही कठिन समस्या है  TL; DR: फ़्लोट्स की तुलना न करें  समस्या  गलत परीक्षा परिणाम  नाजुक परीक्षण  तेजी से विफल सिद्धांत उल्लंघन  समाधान  जब तक आपको वास्तविक प्रदर्शन संबंधी चिंताएं न हों तब तक फ़्लोट से बचें  मनमाने ढंग से सटीक संख्याओं का प्रयोग करें  यदि आपको फ्लोट की तुलना सहिष्णुता के साथ तुलना करने की आवश्यकता है।  संदर्भ  फ्लोट नंबरों की तुलना करना एक पुरानी कंप्यूटर विज्ञान समस्या है।  सामान्य समाधान थ्रेशोल्ड तुलनाओं का उपयोग करना है।  हम अनुशंसा करते हैं कि फ़्लोट से बिल्कुल भी बचें और अनंत सटीक संख्याओं का उपयोग करने का प्रयास करें।  नमूना कोड  गलत   Assert.assertEquals(0.0012f, 0.0012f); // Deprecated Assert.assertTrue(0.0012f == 0.0012f); // Not JUnit - Smell  सही   Assert.assertEquals(0.0012f, 0.0014f, 0.0002); // true Assert.assertEquals(0.0012f, 0.0014f, 0.0001); // false // last parameter is the delta threshold Assert.assertEquals(12 / 10000, 12 / 10000); // true Assert.assertEquals(12 / 10000, 14 / 10000); // false  खोज    स्वचालित []  फ्लोट्स की जांच से बचने के लिए हम अपने परीक्षण ढांचे पर एक चेक con   जोड़ सकते हैं। assertEquals()  टैग  टेस्ट गंध  निष्कर्ष  हमें हमेशा फ्लोट्स की तुलना करने से बचना चाहिए।  संबंधों   कोड गंध 71 - दशमलव के रूप में प्रच्छन्न जादू तैरता है  और जानकारी   तेजी से विफल  क्रेडिट    . पर   द्वारा फोटो Unsplash मीका बॉमिस्टर  भगवान ने प्राकृतिक संख्याएं बनाईं; बाकी सब आदमी का काम है।   लियोपोल्ड क्रोनकर   सॉफ्टवेयर इंजीनियरिंग महान उद्धरण  कोड गंध 109 - स्वचालित गुण    यदि आप 4 कोड गंधों को मिलाते हैं तो क्या होता है?  टीएल; डीआर: गेटर्स से बचें, सेटर्स से बचें, मेटाप्रोग्रामिंग से बचें। व्यवहार के बारे में सोचो।  समस्या  सूचना छिपाने का उल्लंघन   अस्थिरता    उल्लंघन तेजी से विफल सिद्धांत  गुण सेट करते समय डुप्लिकेट कोड  समाधान   स्वचालित सेटर्स और गेटर्स निकालें  संदर्भ  सेटर्स और गेटर्स एक खराब उद्योग प्रथा है।  कई आईडीई इस कोड गंध का पक्ष लेते हैं।  कुछ भाषाएं एनीमिक मॉडल और डीटीओ बनाने के लिए स्पष्ट समर्थन प्रदान करती हैं।  नमूना कोड  गलत   class Person { public string name { get; set; } }  सही   class Person { private string name public Person(string personName) { name = personName; //imutable //no getters, no setters } //... more protocol, probably accessing private variable name }  खोज    स्वचालित []  यह एक भाषा विशेषता है।  हमें अपरिपक्व भाषाओं से बचना चाहिए या उनकी सबसे खराब प्रथाओं को मना करना चाहिए।  टैग  कैप्सूलीकरण  निष्कर्ष  हमें अपनी संपत्तियों को उजागर करने से पहले सावधानी से सोचने की जरूरत है।  पहला कदम गुणों के बारे में सोचना बंद करना और पूरी तरह से व्यवहार पर ध्यान केंद्रित करना है।  संबंधों   कोड गंध 28 - सेटर्स   कोड गंध 68 - गेटर्स   कोड गंध 70 - एनीमिक मॉडल जेनरेटर   कोड गंध 40 - डीटीओ   कोड गंध 01 - एनीमिक मॉडल  और जानकारी   W3 स्कूल   आलस्य I - मेटाप्रोग्रामिंग   आलस्य II - कोड विजार्ड्स   रिफैक्टरिंग 001 - सेटर्स निकालें   उत्परिवर्ती की दुष्ट शक्ति   तेजी से विफल  क्रेडिट    . पर   द्वारा फोटो Unsplash कोनी  एक तंग समय सीमा के तहत काम करने और फिर भी सफाई के लिए समय निकालने से ज्यादा कठिन कुछ नहीं है।   केंट बेकी   सॉफ्टवेयर इंजीनियरिंग महान उद्धरण  कोड गंध 110 - डिफ़ॉल्ट के साथ स्विच करता है    डिफ़ॉल्ट का अर्थ है 'वह सब कुछ जो हम अभी तक नहीं जानते'। हम भविष्य की भविष्यवाणी नहीं कर सकते।  TL; DR: अपने मामलों में एक डिफ़ॉल्ट खंड न जोड़ें। इसे अपवाद के लिए बदलें। स्पष्टवादी बनें।  समस्या  युग्मन   तेजी से विफल सिद्धांत उल्लंघन  खुला बंद सिद्धांत उल्लंघन  समाधान  अगर और मामलों को बहुरूपता से बदलें  डिफ़ॉल्ट कोड को अपवाद में बदलें  संदर्भ  मामलों का उपयोग करते समय, हम आमतौर पर एक डिफ़ॉल्ट मामला जोड़ते हैं ताकि यह विफल न हो।  बिना सबूत के निर्णय लेने से असफल होना हमेशा बेहतर होता है।  चूंकि   भी एक गंध हैं, इसलिए हम उनसे बच सकते हैं। केस और स्विच  नमूना कोड  गलत   switch (value) { case value1: // if value1 matches the following will be executed.. doSomething(); break; case value2: // if value2 matches the following will be executed.. doSomethingElse(); break; default: // if value does not presently match the above values // or future values // the following will be executed doSomethingSpecial(); break; }  सही   switch (value) { case value1: // if value1 matches the following will be executed.. doSomething(); break; case value2: // if value2 matches the following will be executed.. doSomethingElse(); break; case value3: case value4: // We currently know these options exist doSomethingSpecial(); break; default: // if value does not match the above values we need to take a decision throw new Exception('Unexpected case ' + value + ' we need to consider it'); break; }  खोज    अर्ध स्वचालित [x]  जब तक कोई अपवाद न हो, हम अपने लिंटर्स को डिफ़ॉल्ट उपयोगों पर हमें चेतावनी देने के लिए कह सकते हैं।  टैग  तेजी से विफल  निष्कर्ष  मजबूत कोड लिखने का मतलब यह नहीं है कि हमें सबूत के बिना निर्णय लेने की जरूरत है।  संबंधों   कोड गंध 36 - स्विच/केस/elseif/else/if कथन  और जानकारी   तेजी से विफल  क्रेडिट    . पर   द्वारा फोटो Unsplash जोशुआ वोरोनीकी  किसी सुविधा को जोड़ने की लागत केवल उसे कोड करने में लगने वाला समय नहीं है। लागत में भविष्य के विस्तार में बाधा भी शामिल है। चाल उन विशेषताओं को चुनना है जो एक दूसरे से नहीं लड़ती हैं।   जॉन कार्मैक   सॉफ्टवेयर इंजीनियरिंग महान उद्धरण  और अभी के लिए बस इतना ही…  अगला लेख 5 और कोड गंधों की व्याख्या करेगा!

Ball

Twitter

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

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

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

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

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

About Author

टिप्पणियाँ

लेबल

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

Related Stories

क्लाउड सॉनेट 3.5 सिस्टम प्रॉम्प्ट लीक: एक फोरेंसिक विश्लेषण

फ्लोकी का वल्लाह भारत के श्रीलंका दौरे के लिए सहयोगी प्रायोजक के रूप में शामिल हुआ

फ़ोरम से फ़ीड तक: सोशल मीडिया एल्गोरिदम डिजिटल इंटरैक्शन को कैसे आकार देते हैं

क्लाउड सॉनेट 3.5 सिस्टम प्रॉम्प्ट लीक: एक फोरेंसिक विश्लेषण

फ्लोकी का वल्लाह भारत के श्रीलंका दौरे के लिए सहयोगी प्रायोजक के रूप में शामिल हुआ

फ़ोरम से फ़ीड तक: सोशल मीडिया एल्गोरिदम डिजिटल इंटरैक्शन को कैसे आकार देते हैं

Light-Mode

Classic

Newspaper

Dark-Mode

Neon Noir

Minty

HN StartUps