কোড পর্যালোচনা করার সময়, আমি সাধারণত সমস্যাগুলি কীভাবে সমাধান করবেন সে সম্পর্কে সুনির্দিষ্ট সুপারিশ করার চেষ্টা করি। তবে পর্যালোচনার জন্য যে কেউ সীমিত সময় ব্যয় করতে পারে তার কারণে এটি সর্বদা ভাল কাজ করে না। এই ক্ষেত্রেগুলিতে আমি এটি আরও কার্যকর মনে করি যদি বিকাশকারী নিজেই কোনও সমাধান নিয়ে আসে।
আজ আমি কিছু কোড পর্যালোচনা করে দেখতে পেলাম যে কোনও শ্রেণি স্পষ্টতই ভালভাবে ডিজাইন করা হয়নি not এটিতে বেশ কয়েকটি alচ্ছিক বৈশিষ্ট্য রয়েছে যা কেবলমাত্র নির্দিষ্ট কিছু বস্তুর জন্য নির্ধারিত ছিল এবং অন্যদের জন্য ফাঁকা রেখেছিল। এটি সমাধানের স্ট্যান্ডার্ড উপায় হ'ল শ্রেণি ভাগ করে নেওয়া এবং উত্তরাধিকার ব্যবহার করা। তবে এই সুনির্দিষ্ট ক্ষেত্রে এই সমাধানটি জিনিসগুলিকে অত্যধিক জটিল বলে মনে হয়েছে। আমি নিজেই এই সফ্টওয়্যারটির বিকাশের সাথে জড়িত ছিলাম না এবং সমস্ত মডিউলগুলির সাথে পরিচিত নই। অতএব আমি নির্দিষ্ট সিদ্ধান্ত নিতে যথেষ্ট বুদ্ধিমান বোধ করিনি।
আর একটি সাধারণ ঘটনা যা আমি অনেকবার অভিজ্ঞতা পেয়েছি তা হ'ল আমি একটি স্পষ্টত অর্থহীন বা এমনকি বিভ্রান্তিমূলক কার্য, শ্রেণি বা পরিবর্তনশীল নামটি পাই তবে আমি নিজে একটি ভাল নাম নিয়ে আসতে সক্ষম নই।
সুতরাং সাধারণত, একজন পর্যালোচক হিসাবে, "এই কোডটি ত্রুটিযুক্ত কারণ ..., এটি অন্যভাবে করুন" বা আপনার কি কোনও নির্দিষ্ট সমাধান নিয়ে আসতে হবে?