কোড রিভিউ বেস্ট প্র্যাক্টিস

class CodeReviewComment:
    def __init__(self, line_number, category, text):
        self.line_number = line_number
        self.category = category  # "substance" অথবা "style"
        self.text = text

class PullRequestReview:
    def __init__(self):
        self.comments = []

    def add_comment(self, comment):
        self.comments.append(comment)

def review_quality_report(comments):
    total = len(comments)
    style_count = sum(1 for c in comments if c.category == "style")
    substance_count = total - style_count
    style_pct = (style_count / total) * 100 if total else 0
    flagged = style_pct > 80
    print(f"  মোট কমেন্ট: {total} | substance: {substance_count} | style: {style_count} ({style_pct:.1f}%)")
    if flagged:
        print("  ⚠ এই রিভিউ অতিরিক্তভাবে style-কেন্দ্রিক (৮০%+) -- এই ধরনের চেক automated linter দিয়ে")
        print("    করানো যেত, রিভিউয়ারের সময় substance-এ ব্যয় হওয়া উচিত ছিল।")
    else:
        print("  ✓ রিভিউটি যথেষ্ট substance-কেন্দ্রিক।")
    return {"total": total, "style_pct": style_pct, "flagged": flagged}

# উদাহরণ A -- substance-heavy রিভিউ
review_a = PullRequestReview()
for c in [
    CodeReviewComment(42, "substance", "এই ফাংশনটি নেগেটিভ quantity হ্যান্ডেল করছে না -- edge case মিস হচ্ছে।"),
    CodeReviewComment(58, "substance", "এই লজিক OrderService-এর দায়িত্ব, এখানে থাকা উচিত না (SRP লঙ্ঘন)।"),
    CodeReviewComment(70, "substance", "এই পাথের জন্য কোনো ইউনিট টেস্ট নেই।"),
    CodeReviewComment(12, "substance", "ম্যাজিক নাম্বার 86400 ব্যবহার হয়েছে, নামকরণ করা কনস্ট্যান্ট ভালো হতো।"),
    CodeReviewComment(5, "style", "এখানে একটা এক্সট্রা ব্ল্যাংক লাইন আছে।"),
]:
    review_a.add_comment(c)

print("রিভিউ A:")
report_a = review_quality_report(review_a.comments)

print()
print("রিভিউ B:")
review_b = PullRequestReview()
for c in [
    CodeReviewComment(3, "style", "ভেরিয়েবল নাম camelCase না snake_case হওয়া উচিত।"),
    CodeReviewComment(9, "style", "ইনডেন্টেশন ২ স্পেস, ৪ স্পেস হওয়া উচিত।"),
    CodeReviewComment(15, "style", "কোটেশন মার্ক ডাবল না সিঙ্গেল হওয়া উচিত।"),
    CodeReviewComment(20, "style", "লাইনের শেষে ট্রেইলিং স্পেস আছে।"),
    CodeReviewComment(1, "style", "ইম্পোর্ট অর্ডার ঠিক নেই।"),
    CodeReviewComment(30, "substance", "এখানে নাল-চেক মিসিং, crash হতে পারে।"),
]:
    review_b.add_comment(c)

report_b = review_quality_report(review_b.comments)