Kasus Peninjauan Kode 1

Saya bekerja untuk perusahaan yang tidak memiliki praktik tinjauan kode. Untuk pengembangan diri dan memperluas wawasan saya, saya ingin menerima sedikit kritik yang membangun.

Sekarang saya mengusulkan untuk mengurai kasus berulang dengan banyak cabang.

Tugas

Pengguna bermaksud untuk menyeret file dengan mouse dari satu folder folder ke yang lain. Anda perlu menulis metode dispatcher yang memeriksa esensi acara dan kemungkinan memprosesnya, jika perlu, mengklarifikasi detailnya, lalu memanggil metode yang diinginkan atau menampilkan pesan tentang batasan.

Jika pengguna seret dan lepaskan dari satu folder ke folder dan folder lain ada di bagian yang berbeda, maka periksa copyability. Jika Anda dapat menyalin, maka salin. Kalau tidak, berikan pesan yang tidak bisa Anda salin. Itu tidak dapat disalin karena alasan: tidak ada izin tertulis; tidak cukup ruang kosong; sistem file tidak mendukung karakter dalam nama; nama file di folder tujuan akan memiliki jalur yang terlalu panjang; sudah ada file dengan nama yang sama di folder (buka dialog untuk menimpa file, jika pengguna setuju, lalu timpa).

Jika folder tujuan berada di partisi yang sama dengan file, maka pindahkan file tersebut. Tidak dapat bergerak: tidak ada izin tulis; path tujuan lengkap akan terlalu panjang, sudah ada file dengan nama yang sama di folder (buka dialog); file adalah sistem dan tidak dapat dihapus; sudah ada file dengan nama itu (buka dialog untuk menimpa file, jika pengguna setuju, lalu timpa).

Jika pengguna memindahkan file ke jendela lain, tetapi memiliki jalur yang sama, kemudian buat salinan file tersebut (Tambahkan "salin #" ke nama, di mana # adalah angka positif terkecil yang membuat file tersebut unik). Tidak dapat membuat salinan: tidak ada izin menulis; jalan penuh terlalu panjang; ruang kosong tidak cukup.

Jika pengguna ditransfer dengan tombol kanan, maka panggil dialog untuk memilih tindakan (salin / pindahkan / buat pintasan / buat salinan).

Jika pengguna merilis file di jendela yang sama (file rusak) dengan tombol kiri, maka jangan lakukan apa pun. Dan jika benar, maka sarankan untuk membuat salinan atau pintasan. Jika file tidak jatuh di jendela folder, maka jangan lakukan apa pun.

Seiring waktu, kondisi baru dapat muncul, tindakan baru, kondisi yang sudah dijelaskan untuk mengubah tindakan dapat berubah.

Solusi Diskusi

Saya menawarkan solusi kontroversial saya di Jawa, di mana saya mencapai tingkat tertinggi kedua bersarang jika:

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); } }