রিশ্যার কৌতূহল: "পরামিতি কেবল পূর্বশর্ত চেক (গুলি) এর জন্য ব্যবহৃত হয়।"

Question 1

কেন রিশার্পার এই কোডটির জন্য আমাকে বিচার করছেন?

    private Control GetCorrespondingInputControl(SupportedType supportedType, object settingValue)
    {
        this.ValidateCorrespondingValueType(supportedType, settingValue);

        switch(supportedType)
        {
            case SupportedType.String:
                return new TextBox { Text = (string)settingValue };
            case SupportedType.DateTime:
                return new MonthPicker { Value = (DateTime)settingValue, ShowUpDown = true };
            default:
                throw new ArgumentOutOfRangeException(string.Format("The supported type value, {0} has no corresponding user control defined.", supportedType));
        }
    }

    private void ValidateCorrespondingValueType(SupportedType supportedType, object settingValue)
    {
        Type type;

        switch(supportedType)
        {
            case SupportedType.String:
                type = typeof(string);
                break;
            case SupportedType.DateTime:
                type = typeof(DateTime);
                break;
            default:
                throw new ArgumentOutOfRangeException(string.Format("The supported type value, {0} has no corresponding Type defined.", supportedType));
        }
        string exceptionMessage = string.Format("The specified setting value is not assignable to the supported type, [{0}].", supportedType);
        if(settingValue.GetType() != type)
        {
            throw new InvalidOperationException(exceptionMessage);
        }
    }

দ্বিতীয় পদ্ধতি ভ্যালিডেটপ্রেসিংভালিউটিটাইপের "সেটিংওয়ালু" প্যারামিটারটি রিশার্পার দ্বারা নিম্নলিখিত বার্তার সাথে ধূসর করা হয়েছে: "পরামিতি 'সেটিংওয়ালু' কেবল পূর্বশর্ত চেক (গুলি) এর জন্য ব্যবহৃত হয়।"

Question 2

এটি বিচার করছে না, এটি সাহায্য করার চেষ্টা করছে :)

যদি রিশার্পার দেখেন যে প্যারামিটারটি কেবলমাত্র ব্যতিক্রম ছুঁড়ে ফেলার জন্য চেক হিসাবে ব্যবহৃত হয়, তবে এটি এটি ধূসর করে দেয় যা ইঙ্গিত করে যে আপনি এটি "বাস্তব" কাজের জন্য আসলে ব্যবহার করছেন না। এটি সম্ভবত একটি ভুল - আপনি যে প্যারামিটারটি ব্যবহার করছেন না কেন তা পাস করবেন? এটি সাধারণত ইঙ্গিত করে যে আপনি এটি প্রাক-শর্তে ব্যবহার করেছেন, তবে তারপরে কোডের অন্য কোনও জায়গায় এটি ব্যবহার করতে ভুলে গেছেন (বা আর প্রয়োজন নেই)।

যেহেতু পদ্ধতিটি একটি ValidateCorrespondingValueTypeদৃser়বিজ্ঞানের পদ্ধতি (এটি যা যা করা হয় সবগুলিই এটির বৈধতা রয়েছে), আপনি পুনঃশিকারের টীকা বিশিষ্টতাগুলি বিশেষত বৈশিষ্ট্যটি ব্যবহার করে একটি বার্তা পদ্ধতি হিসাবে চিহ্নিত করে বার্তাটি দমন করতে পারেন [AssertionMethod]:

[AssertionMethod]
private void ValidateCorrespondingValueType(SupportedType supportedType, object settingValue)
{
  // …
}

Question 3

মজার বিষয় হল, আপনি nameofযদি সি # 6 তে নতুন কার্যকারিতাটি ব্যবহার করেন তবে রিশার্পার ব্যাকআপ করেছে :

static void CheckForNullParameters(IExecutor executor, ILogger logger)
{
    if (executor == null)
    {
        throw new ArgumentNullException(nameof(executor));
    }

    if (logger == null)
    {
        throw new ArgumentNullException(nameof(logger));
    }
}

Question 4

নিম্নলিখিতটি সমস্যার সমাধান করে (রিসার্পার 2016.1.1, ভিএস2015 এ), তবে আমি নিশ্চিত নই যে এটি 'ডান' সমস্যাটি সমাধান করে। যাই হোক না কেন, এটি এই বিষয় সম্পর্কিত রিসার্পার মেকানিক্সে অস্পষ্টতা দেখায়:

এটি সতর্কবার্তা দেয়:

    private void CheckForNull(object obj)
    {
        if (ReferenceEquals(obj, null))
        {
            throw new Exception();
        }
    }

তবে এটি করে না:

    private void CheckForNull(object obj)
    {
        if (!ReferenceEquals(obj, null))
        {
            return;
        }
        throw new Exception();
    }

এটি আকর্ষণীয় যে সমতুল্য কোড (বিপর্যয়টি রিসার্পার দ্বারা করা হয়েছিল: ডি) বিভিন্ন ফলাফল দেয়। দেখে মনে হচ্ছে যে প্যাটার্নের মিলটি কেবল দ্বিতীয় সংস্করণটি গ্রহণ করে না।

Question 5

এই সমস্যার আমার পছন্দের সমাধান করা resharper মনে প্যারামিটার হয় ব্যবহার করা হয়েছে। এই যেমন একটি বৈশিষ্ট্য ব্যবহার উপর একটি সুবিধা আছে UsedImplicitlyকারন যদি কখনও আপনি কি স্টপ যে পরামিতি ব্যবহার করে, resharper আপনি আবার সাবধান শুরু হবে। আপনি যদি কোনও বৈশিষ্ট্য ব্যবহার করেন তবে পুনঃভাগটি ভবিষ্যতের প্রকৃত সতর্কতাগুলিও ধরবে না।

প্যারামিটারটি ব্যবহার করা হয়েছে বলে মনে করে পুনরায় ভাগ করার সহজ উপায় হ'ল throwকোনও পদ্ধতির পরিবর্তে । এর পরিবর্তে ...

if(myPreconditionParam == wrong)
    throw new Exception(...);

...তুমি লেখ:

if(myPreconditionParam == wrong)
    new Exception(...).ThrowPreconditionViolation();

এটি ভবিষ্যতের প্রোগ্রামারদের জন্য দুর্দান্তভাবে স্ব-ডকুমেন্টিং, এবং পুনরায় ভাগ করা ঝকঝকে ত্যাগ করে।

থ্রোপ্রেকন্ডিশনভায়োলশনের বাস্তবায়ন নগণ্য:

public static class WorkAroundResharperBugs 
{
    //NOT [Pure] so resharper shuts up; the aim of this method is to make resharper 
    //shut up about "Parameter 'Foobaar' is used only for precondition checks" 
    //optionally: [DebuggerHidden]
    public static void ThrowPreconditionViolation(this Exception e)
    {
        throw e;
    }
}

ব্যতিক্রম সম্পর্কিত একটি এক্সটেনশন পদ্ধতি হ'ল নাম স্থান দূষণ, তবে এটি মোটামুটি রয়েছে।

Question 6

অন্যরা ইতিমধ্যে প্রশ্নের উত্তর দিয়েছেন, কিন্তু সতর্কতাটি বন্ধ করার নিম্নলিখিত উপায়গুলির কথা কেউ উল্লেখ করেনি।

কেবলমাত্র সেই পদ্ধতির জন্য এটিকে বন্ধ করতে পদ্ধতির স্বাক্ষরের উপরে এটি যুক্ত করুন:

    // ReSharper disable once ParameterOnlyUsedForPreconditionCheck.Local

এটি পুরো ফাইলটির জন্য বন্ধ করার জন্য শ্রেণীর ঘোষণার উপরে এটি যুক্ত করুন:

     // ReSharper disable ParameterOnlyUsedForPreconditionCheck.Local