🚈 🕡 🍶 कनिष्ठ प्रतिक्रिया डेवलपर्स के कोड-समीक्षा परीक्षण असाइनमेंट 🧓🏿 🥉 🙌🏻

यह क्या है

यह दूसरे परीक्षण कार्य के समाधान की एक कोड समीक्षा है। वीडियो सफल समाधान और त्रुटियों को दिखाता है, साथ ही उन्हें ठीक करने के लिए सुझाव भी देता है। यह नोट आम समस्याओं की पहचान करता है और "समय टिकट" के साथ लिंक प्रदान करता है।

एक प्रतिक्रिया को चुना

वीडियो

आम समस्याएं

खराब रीडमी;
एस्लिंट चेतावनी, अतिरिक्त कंसोल हैं। (redux-logger गिनती नहीं करता है);
वेब आइकन आगे नहीं बढ़ा (कार्य की अनुपस्थित पठन);
वेब आइकन घटक में आगे बढ़ाया जाता है (और यह रिड्यूसर, या एक्शन में बेहतर होगा);
यदि कोई त्रुटि के साथ अनुरोध वापस आया तो पासवर्ड साफ़ नहीं किया गया है;
यदि फ़ील्ड खाली है (या फ़ील्ड में से एक) तो लॉगिन फ़ॉर्म में सबमिट बटन उपलब्ध है;
लॉगिन फ़ॉर्म में बटन सबमिट करें एंटर को दबाने का समर्थन नहीं करता है;
घटकों / कंटेनरों में कोई विभाजन नहीं (उन पर लागू नहीं होता है जो अन्य दृष्टिकोणों द्वारा साझा किए जाते हैं);
सर्वर के अनुरोधों के लिए URL पूरी तरह से प्रेषित है (स्ट्रिंग का एक स्थिर के रूप में कोई डिजाइन नहीं है)
त्रुटि / अधिसूचना "गलत उपयोगकर्ता नाम / पासवर्ड" - साफ़ नहीं किया गया है;
त्रुटि "गलत उपयोगकर्ता नाम / पासवर्ड" - सर्वर से एक निरंतर द्वारा आउटपुट है;
त्रुटि पाठ कोड में हार्डकोड है। सर्वर से निरंतर द्वारा शब्दकोश तक पहुंच नहीं है;
"पुराने कोड" को हटाया नहीं गया है, अर्थात, कोड जो कहीं भी उपयोग नहीं किया गया है;
वादों में कैच ब्लॉक नहीं होता है, सर्वर जवाब ok नहीं होने पर कोई त्रुटि नहीं होती है;
घटकों को नोड_मॉड्यूल्स में होस्ट किया जाता है;
प्रोप प्रकार गायब हैं या अपर्याप्त रूप से वर्णित हैं।
एक फ़ाइल में ढेर पर कार्रवाई और रिड्यूसर (या अन्य सभी रिड्यूसर में एक कार्रवाई में)। "मॉड्यूल" में कोई विभाजन नहीं है, अर्थात्, प्रत्येक इकाई - अपने स्वयं के कार्यों और इसके पुनर्विकास;

सभी टाइमस्टैम्पड समाधान

केवल त्रुटियों को यहां इंगित किया गया है, और अच्छे अंक नोट नहीं किए गए हैं , जिनमें से एक महान कई हैं, इसलिए मैं उन लोगों के लिए एक पंक्ति में सब कुछ देखने की सलाह देता हूं जो प्रतिक्रिया में विकास में खुद को "जून" मानते हैं।

6m00s - आर्थर डोनकोवत्सेव कमिट

7m40s - फ़ंक्शन नाम में टाइपो

8m07s - अतुल्यकालिक अनुरोधों को कार्रवाई में नहीं डाला गया

9 एम 30 एस - पावेल पिमकिन प्रतिबद्ध

10m07s - एक फ़ाइल में सभी क्रियाएं। मॉड्यूल में कोई विभाजन नहीं।

10m25s - आइकन (डेटा गणना) को हटाने के घटक में बनाया गया है। एक reducer या कार्रवाई में बेहतर है।

11m42s सर्गेई ZackFox प्रतिबद्ध

12m28s - "शांत" शिलालेख। इसे "न्यूट्रली" करना बेहतर है ताकि समान कार्य तब नियोक्ता को तुरंत भेजे जा सकें।

13m05s - यह दर्शाता है कि "डाउनलोड" पूरा हो गया है। अर्थात्, तीन क्रियाओं के बजाय: REQUEST / SUCCESS / STOP, आप दो से मिल सकते हैं: REQUEST / SUCCESS।

16m16s - दिमित्री पेट्रोव प्रतिबद्ध

18m16s - var का उपयोग करना

18m34s - URL पते का हिस्सा स्थिर नहीं है

21m15s - Yefim रोटी प्रतिबद्ध

21m17s - खराब प्रतिबद्ध संदेश

22m15s एक ही कार्रवाई के नाम हैं।

24m16s - कत्सुरा व्लादिस्लाव कमिट

25m17s - (त्रुटि नहीं) - एक रिड्यूसर में तैयार किया गया डेटा

27m38s - e.target , बेहतर e.currentTarget का उपयोग e.currentTarget

28m20s - == , लेकिन यह आवश्यक होगा ===

२ एम ३३ एस - कंपोनेंटविल यूमाउंट का उपयोग करना

29m00s (त्रुटि नहीं) - "सर्वर सत्यापन से पहले" के बारे में तर्क देना।