कोड समीक्षा मामला 1

मैंने एक कंपनी के लिए काम किया जिसमें कोड समीक्षा अभ्यास का अभाव है। आत्म-सुधार और अपने क्षितिज को व्यापक बनाने के लिए, मैं थोड़ी रचनात्मक आलोचना प्राप्त करना चाहूंगा।

अब मैं शाखाओं के एक बहुतायत के साथ आवर्ती मामले को पार्स करने का प्रस्ताव करता हूं।

कार्य

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

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

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

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

यदि उपयोगकर्ता दाएं बटन के साथ स्थानांतरित होता है, तो कार्रवाई (कॉपी / मूव / शॉर्टकट बनाने / कॉपी बनाने) का चयन करने के लिए संवाद को कॉल करें।

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

समय के साथ, नई स्थितियां दिखाई दे सकती हैं, नई क्रियाएं, बदलती कार्रवाइयों के लिए पहले से वर्णित स्थितियां बदल सकती हैं।

चर्चा समाधान

मैं अपना विवादास्पद समाधान जावा में प्रस्तुत करता हूं, जिसमें मैं घोंसले के दूसरे उच्चतम स्तर पर पहुंच गया हूं:

public static void dispatchFileDropping( FileDragNDropEvent event ) { //------------------------------------------------------ //   // (     ). boolean A = isTargetPlaceIsDirectoryWindow(event); boolean B = isTargetDirEqualsSourceDir(event); boolean C = isTargetVolumeEqualsSourceVolume(event); boolean D = isMouseRightButtonUsed(event); boolean E = isSystemFileDroped(event); boolean F = isTargetVolumeHasFreeSpace(event); boolean G = isTargetDirWritable(event); boolean H = isSourceDirCleanable(event); boolean I = isFileNameOkForTarget(event); boolean J = isNewFileFullPathOkForTargetLimit(event); boolean K = isTargetDirHasSameNamedFile(event); boolean L = isTargetDirSameNamedFileIsWritable(event); Actions userChoise = (A & D) ? askUserForAction(event) : null; if (userChoise == Actions.CANCEL) return; boolean M = (userChoise == Actions.COPY); boolean N = (userChoise == Actions.CLONE); boolean O = (userChoise == Actions.MOVE); boolean P = (userChoise == Actions.LINK); //------------------------------------------------------ //      . boolean copyCase = (M & !K) | (A & !B & !C & !D & !K); boolean copyRewriteCase = (M & K) | (A & !B & !C & !D & K); boolean cloneCase = N | (A & B & !D); boolean moveCase = (O & !K) | (A & !B & C & !D & !K); boolean moveRewriteCase = (O & K) | (A & !B & C & !D & K); boolean createLinkCase = P; //------------------------------------------------------ //      //  . if (copyRewriteCase | moveRewriteCase) { if (askUserWantToRewrite() == Answers.NO) return; } //------------------------------------------------------ //     . boolean isPossibleToCopy = F & G & I & J; boolean isPossibleToCopyRewrite = isPossibleToCopy & L; boolean isPossibleToClone = isPossibleToCopy; boolean isPossibleToMove = isPossibleToCopy & !E & H; boolean isPossibleToMoveRewrite = isPossibleToMove & L; boolean isPossibleToLink = isPossibleToCopy & !K; //------------------------------------------------------ //   ,  , //    . String errorMessage = ""; if (copyCase & !isPossibleToCopy) { errorMessage = "  ."; } else if (copyRewriteCase & !isPossibleToCopyRewrite) { errorMessage = "  ."; } else if (cloneCase & !isPossibleToClone) { errorMessage = "  ."; } else if (moveCase & !isPossibleToMove) { errorMessage = "  ."; } else if (moveRewriteCase & !isPossibleToMoveRewrite) { errorMessage = "    ."; } else if (createLinkCase & !isPossibleToLink) { errorMessage = "  ."; } String reasons = " : \n"; if (!F) { reasons += "--     \n"; } if (!G) { reasons += "--        \n"; } if (!I) { reasons += "--      \n"; } if (!J) { reasons += "--       \n"; } if (moveCase | moveRewriteCase) { if (E) { reasons += "--     \n"; } if (!H) { reasons += "--       \n"; } } else if (copyRewriteCase | moveRewriteCase) { if (!L) { reasons += "--      \n"; } } else if (createLinkCase) { if (K) { reasons += "--       \n"; } } if (errorMessage.isEmpty()) { if (copyCase) copy(event); if (copyRewriteCase) copyRewrite(event); if (cloneCase) clone(event); if (moveCase) move(event); if (moveRewriteCase) moveRewrite(event); if (createLinkCase) createLink(event); } else { showMessage(errorMessage + reasons); } }