2009-05-19 9 views
5

आज सुबह कुछ आम तौर पर भयानक जावास्क्रिप्ट कोड पर काम के दौरान, मैं भर में आया था (कई स्थानों में) के बाद:न्याय की योग्यता - क्या कोई गर्मी गर्मियों में करता है?

// make moveAmount negative 
moveAmount = moveAmount - (moveAmount * 2); 

मैं सीधे पुरुष कि यह लिखा से भर में मौजूद हैं; वह यहां सात साल तक एक डेवलपर रहा है। दूसरी तरफ, मैंने अभी शुरू किया है, मैं बहुत जूनियर हूं और जैक को जानने का दावा नहीं करता हूं।

फिर भी मुझे कई सालों के विकासशील सॉफ़्टवेयर के बाद सरल तर्क के लड़के के असर के बारे में सोचने में मुझे आश्चर्य हुआ है।

मेरा प्रश्न यह है कि यहां अन्य लोगों को यह देखने के लिए क्या डेवलपर की समग्र क्षमता ले सकती है (3 सप्ताह पहले), या यह बिल्कुल भी प्रतिबिंबित करता है?

कोई भी दृढ़ संकल्प को इंगित करेगा?

मैं, खुद, लगातार डर में काम करता हूं कि ऐसे निर्णय मेरे बने होते हैं।

+0

errm moveAmount = 0-moveAmount; मुझे आशा है कि चाल प्रत्येक मामले में एक सकारात्मक संख्या है। –

+0

शून्य की आवश्यकता नहीं है – mysomic

+0

रक्षात्मक कोडिंग; पी। इसके अलावा मैंने पढ़ा 'क्या कोई भी दृढ़ संकल्प को इंगित करेगा?' गलत तरीके से। इसलिए स्पष्ट टिप्पणी। –

उत्तर

30

सरल समाधान: उसे कोड की रेखा दिखाएं और पूछें कि उसने ऐसा क्यों किया। यह आप दोनों को कुछ सीखने का अवसर है। हो सकता है कि किसी ब्राउज़र या किसी अन्य समस्या में एक बग था (राउंडिंग दिमाग में आती है) ताकि कोड इसके बिना तोड़ सके। या उसने गलती की। किसी भी तरह से पूछना, इसे साफ़ कर देगा।

और जब आप सही हैं कि अन्य लोग आपको आपके द्वारा देखे गए कोड से न्याय करेंगे, यह केवल चीज नहीं है जिसे वे ध्यान में रखते हैं (और यदि ऐसा है, तो आप गलत पर काम कर रहे हैं जगह - छोड़ें जबकि आप अभी भी सचेत हैं)। वे यह भी देखेंगे कि आप विनम्र, उत्सुक, सहायक कब हैं।

ये चीजें आपके द्वारा लिखे गए कोड से अधिक गिनती हैं: कोड को गुस्सा सहकर्मियों की तुलना में अधिक आसानी से तय किया जा सकता है।

+1

सहमत हुए। मैं सिर्फ विनम्रता से कहूंगा "अरे, मैं कोड की इस पंक्ति में आया था और सोच रहा था कि आपने ऐसा क्यों किया"। –

+0

भले ही वह ब्लश करता है और फिर यह अगले प्रतिबद्धता में तय हो जाता है, सूक्ष्म उल्लेख इसे ठीक कर देता है, और वह duhs –

+1

के लिए पुरानी कोड जांच के माध्यम से हो सकता है एक और अंगूठे। आप कभी नहीं जानते कि क्या ऐसा करने के लिए कुछ आर्केन-लेकिन-वैध कारण हो सकता है। – fenomas

3

वैसे यह काम करता है ताकि आप एपी **** की तरह दिख सकें यदि आपने उसे बताया कि आप उस कोड को नापसंद करते हैं और शायद सही तरीके से :) तो फिर वह 7 साल पहले उस कोड को पहले जावास्क्रिप्ट के रूप में कर सकता था लिखा है, लेकिन किसी भी कोड के सिर्फ एक लाइन से गंभीरता से किसी का न्याय नहीं करते हैं।

+0

मैं निश्चित रूप से – mysomic

+1

को इंगित नहीं करता जैसा कि हारून कहता है, ऐसा कुछ कारण हो सकता है जो अब मान्य नहीं है और इस कोड को इस तरह क्यों लिखा गया था। शायद कुछ दूर-दूर की पटकथा भाषा में यह कोड अच्छा था। जब तक आपसे पूछा नहीं जाता कि आप कभी नहीं जानते। –

+0

मैं इसे इंगित करता हूं। आप अपने सहकर्मियों को सीधे उनके साथ रहने के लिए देनदार हैं और आप दोनों को बेहतर बनाने के लिए मिलकर काम करने की कोशिश करते हैं। – mquander

1

यह एक उदाहरण खोजने और डेवलपर के बारे में सामान्यीकृत करने के लिए मोहक है, लेकिन जब तक कि इस तरह के बकवास का ट्रैक रिकॉर्ड न हो, मैं इसे एक अलग घटना मानता हूं। सीधे शब्दों में उससे पूछो क्यों वह नहीं लिखा था:

moveAmount = moveAmount * (-1) 
+3

.... या moveAmount = -moveAmount – mysomic

+0

// वास्तव में डेव (या जो भी) ढूंढ रहा है? moveAmount = moveAmount * (moveAmount * 2); उतना ही कष्टप्रद हो सकता है। –

1

इस व्यवहार के मामले में माउंटेमाउंट = -moveAmount से अलग है जहां 2 * moveAmount एक अतिप्रवाह कारण हो सकता है अगर moveAmount काफी बड़ा (जैसे 2^32-1 था उदाहरण के लिए)।

मुझे नहीं पता कि वह उसका इरादा था या नहीं।

1

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

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

... और इसके लिए इसे ओवरटार्ड या तनाव या जो कुछ भी होने से संबंधित है ... नीचे की रेखा यह है कि कोड गुणवत्ता महत्वपूर्ण है। अगर उसने इस तरह एक लाइनर लिखा है, तो और क्या लिखा जा सकता है जो इतनी आसानी से खड़ा नहीं होता?

4

आपका उदाहरण गरीब कोड हो सकता है, लेकिन मैं भी बदतर देखा है (सब के बाद, वहाँ हालात जहां यह काम करेगा हैं, कुछ कोड भी है कि यह नहीं कह सकते)। मुझे लगता है कि आप वास्तव में पूछ रहे सवाल यह है: क्या कोड की कोई भी रेखा इतनी भयानक है कि इसका मतलब है कि डेवलपर को उस लाइन के आधार पर अक्षम माना जा सकता है?

मैं नहीं कहता हूं। मेरे दो बच्चे हैं जो बच्चों के रूप में भयानक स्लीपर थे। मैं फ्लू के साथ काम करने आया हूँ। मैं मौत-मार्च मोड में काम कर रहा हूं जहां मैं एक पंक्ति में बीस घंटे काम कर रहा हूं। ऐसी परिस्थितियों में कोई भी कोड की एक भयानक रेखा लिख ​​सकता है। (यही कारण है कि ऐसी परिस्थितियों से बचा जाना चाहिए है।)

दी, मैं आशा करता हूं मैं बाद में भयानक कोड स्थान होता है, और इसे ठीक।

1

के बाद से वह यह 3 सप्ताह पहले लिखा था, मैं कहूंगा कि वह इस बात के लिए आलोचना के लिए खुला होना चाहिए। यह एक अपरिपक्वता दिखाता है जो व्यावसायिक रूप से काम करने के 7 साल बाद वास्तव में अक्षम है।

कम से कम यह किया है तो वह है |:

moveAmount -= moveAmount * 2; 

वह कम flack प्राप्त होंगे। यह कम से कम दिखाता है कि वह अन्य ऑपरेटरों से अवगत है और चीजों को और अधिक पठनीय बनाने के लिए कुछ प्रयास कर रहा है।

मुझे नहीं लगता है कि आप हमेशा एक पंक्ति के साथ लोगों का न्याय कर सकते हैं, लेकिन कोड आप किसी व्यक्ति की योग्यता के बारे में इतना बता सकता है कि एक पंक्ति एक व्यक्ति के बारे में बहुत बता सकते हैं।

+3

आपका संस्करण स्पष्ट रूप से कम पठनीय है। –

+0

जबकि मैं नहीं कह सकता कि ओपी में कोडर का उपयोग क्यों नहीं किया गया - =, मैं कह सकता हूं कि मैं हाल ही में ऑपरेटर के साथ-साथ वृद्धि और कमी से दूर क्यों गया हूं: काम पर मैं वर्तमान में 3 का उपयोग कर रहा हूं भाषाएं - 2 मैं पूरी तरह से नया हूं (और तीसरा मैंने केवल थोड़ी सी उपयोग की है) - और चौथे स्थान पर ब्रश कर रहा हूं। मैं घर परियोजनाओं के लिए कुछ अन्य लोगों का उपयोग करता हूं। कुछ ++ और -, कुछ + =, आदि, दोनों को अनुमति देते हैं, और कोई भी न तो अनुमति देता है। मैंने छोड़ दिया है - कम से कम अभी तक - सीधे रखने की कोशिश कर रहा है कौन सी भाषा ऑपरेटरों की अनुमति देता है और बस इसके साथ चिपक जाता है: x = x + 1 इस प्रकार मैं आसानी से - =, + = आदि का उपयोग नहीं कर सकता। – PTBNL

2

निर्णय के डर के बारे में आराम करो। यह सच है, हम सभी को किसी बिंदु पर किसी के द्वारा तय किया जा रहा है, भले ही यह कुछ पॉइंट बालों वाले बॉस या नरभक्षी छोटे अपस्टार्ट हो। मुझे लगता है कि इस तरह के मुठभेड़ से कुछ उपयोगी कैसे प्राप्त करें, इस बारे में सोचने के लिए मुख्य बात है।

जाहिर है, वहाँ यह की खातिर चीजों के बारे में griping लोग होने जा रहे हैं, लेकिन जहां कुछ सीखा जा करने के लिए नहीं है तो आप भी कई ऐसी ही स्थितियों मिल जाएगा।

आपके सहकर्मी के साथ संवाद शुरू करने के लिए उपर्युक्त सुझाव उत्कृष्ट है; यह सीधे ऐसे मुठभेड़ में खिला सकता है, जिसमें से आप या दोनों आप कुछ महत्वपूर्ण सीख सकते हैं।

1

मुझे लगता है कि यह निगल पर निर्भर करेगा। यदि अलग किया गया है, तो आपके द्वारा दिखाए गए कोड की रेखा कुछ थकाऊ प्रोग्रामर से अपेक्षा की जाती है, भले ही एक बहुत ही सक्षम व्यक्ति। यह कुछ उदाहरण है जो बहुत चालाक और थके हुए लोग इतनी चतुर चीजें नहीं करते हैं। यदि, दूसरी तरफ उस तरह का कोड अक्सर प्रकट होता है तो शायद आप किसी के साथ मुड़ते दिमाग से काम कर रहे हैं।

चीज़ें है कि एक बुरा प्रोग्रामर के स्पष्ट संकेत हैं गलत व्यवहारों से संबंधित हैं। अच्छे प्रोग्रामर बड़ी विधियां नहीं करेंगे। यहां तक ​​कि थक गए, वे अपने प्रकार को कम से कम युग्मित रखने की कोशिश करेंगे। वे एक्सेसर्स या गुणों के माध्यम से उन्हें उजागर करने के बजाय सार्वजनिक क्षेत्रों को उजागर करने से बचेंगे। वे कॉपी-पेस्ट करने के बजाए बार-बार कोड पुन: उपयोग के रूप में विरासत और एकत्रीकरण की कोशिश करेंगे ... सूची लंबी है। वे निगलने के प्रकार हैं जो मुझे लगभग तुरंत महसूस करते हैं कि यह गर्मियों में है। :-)

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