क्या यह वस्तु-जीवनकाल-विस्तार-समापन C # संकलक बग है?


136

जब मैं C # संकलक (4.0 यदि वह मायने रखता है) की ओर से कुछ अत्यंत जिज्ञासु कोड-जीन में भाग लेने पर (जीवन को) वैधता से बंद करने की संभावना के बारे में एक प्रश्न का उत्तर दे रहा था ।

सबसे छोटा रिप्रो जो मैं पा सकता हूं वह निम्नलिखित है:

  1. एक लैम्ब्डा बनाएं जो एक स्थानीय को कैप्चर करता है जिसमें युक्त प्रकार का एक स्थिर तरीका है।
  2. किसी ऑब्जेक्ट के इंस्टेंस फ़ील्ड में जनरेट किया गया प्रतिनिधि-संदर्भ असाइन करें ।

परिणाम: कंपाइलर एक क्लोजर-ऑब्जेक्ट बनाता है जो उस ऑब्जेक्ट को संदर्भित करता है जो लैम्बडा का निर्माण करता है, जब इसका कोई कारण नहीं होता है - प्रतिनिधि का 'इनर' टार्गेट एक स्टैटिक तरीका है, और लैम्ब्डा-क्रिएटिंग-ऑब्जेक्ट का इंस्टेंस सदस्यों को जरूरत नहीं है जब प्रतिनिधि को निष्पादित किया जाता है, तो उसे (और नहीं) छुआ जाता है। प्रभावी रूप से, संकलक कार्य कर रहा है जैसे प्रोग्रामर ने thisबिना कारण के कब्जा कर लिया है।

class Foo
{
    private Action _field;

    public void InstanceMethod()
    {
        var capturedVariable = Math.Pow(42, 1);

        _field = () => StaticMethod(capturedVariable);
    }

    private static void StaticMethod(double arg) { }
}

रिलीज़ बिल्ड से उत्पन्न कोड ('सरल' C # से विघटित) इस तरह दिखता है:

public void InstanceMethod()
{

    <>c__DisplayClass1 CS$<>8__locals2 = new <>c__DisplayClass1();

    CS$<>8__locals2.<>4__this = this; // What's this doing here?

    CS$<>8__locals2.capturedVariable = Math.Pow(42.0, 1.0);
    this._field = new Action(CS$<>8__locals2.<InstanceMethod>b__0);
}

[CompilerGenerated]
private sealed class <>c__DisplayClass1
{
    // Fields
    public Foo <>4__this; // Never read, only written to.
    public double capturedVariable;

    // Methods
    public void <InstanceMethod>b__0()
    {
        Foo.StaticMethod(this.capturedVariable);
    }
}

इस बात का निरीक्षण करें कि <>4__thisक्लोजर ऑब्जेक्ट का फील्ड ऑब्जेक्ट रेफरेंस से पॉपुलेटेड है, लेकिन कभी भी (कोई कारण नहीं है) से पढ़ा जाता है।

तो यहां क्या हो रहा है? क्या भाषा-विनिर्देश इसके लिए अनुमति देते हैं? क्या यह संकलक बग / विषमता है या वस्तु को संदर्भित करने के लिए बंद करने के लिए एक अच्छा कारण है (कि मैं स्पष्ट रूप से गायब हूं)? यह मुझे चिंतित करता है क्योंकि यह करीबी-खुश प्रोग्रामर (मेरे जैसे) के लिए एक नुस्खा जैसा लगता है कि अनजाने में स्मृति-लीक (यदि प्रतिनिधि को एक इवेंट-हैंडलर के रूप में इस्तेमाल किया गया था) कल्पना करें।


19
दिलचस्प। मेरे लिए एक बग की तरह लग रहा है। ध्यान दें कि यदि आप एक इंस्टेंस फ़ील्ड पर असाइन नहीं करते हैं (जैसे यदि आप मान लौटाते हैं), तो यह कैप्चर नहीं करता हैthis
जॉन स्कीट

15
मैं वीएस 11 डेवलपर प्रीव्यू के साथ इसे रीप्रो नहीं कर सकता। VS2010SP1 में repro कर सकते हैं। यह तय है :)
लेप्पी डे

2
यह VS2008SP1 में भी होता है। VS2010SP1 के लिए, यह 3.5 और 4.0 दोनों के लिए होता है।
लेप्पी

5
हम्म, बग इस पर लागू करने के लिए एक बहुत बड़ा शब्द है। संकलक सिर्फ थोड़ा अक्षम कोड उत्पन्न करता है। निश्चित रूप से रिसाव नहीं, यह कचरा एक समस्या के बिना एकत्र करता है। यह संभवत: तब तय हुआ जब उन्होंने एसिंक्स कार्यान्वयन पर काम किया।
हंस पसंत

7
@ हंस, यह एक समस्या के बिना कचरा इकट्ठा नहीं करेगा यदि प्रतिनिधि वस्तु के जीवनकाल में जीवित रहेगा, और ऐसा होने से रोकने के लिए कुछ भी नहीं है।
सॉफ्टमैक्स

जवाबों:


24

यकीन है कि एक बग की तरह लग रहा है। इसे मेरे संज्ञान में लाने के लिए धन्यवाद। मैं इस पर गौर करूंगा। यह संभव है कि यह पहले से ही पाया गया है और तय हो गया है।


7

यह एक बग या अनावश्यक प्रतीत होता है:

मैं आपको आईएल लैंग में छूट देता हूं:

.method public hidebysig 
    instance void InstanceMethod () cil managed 
{
    // Method begins at RVA 0x2074
    // Code size 63 (0x3f)
    .maxstack 4
    .locals init (
        [0] class ConsoleApplication1.Program/Foo/'<>c__DisplayClass1'   'CS$<>8__locals2'
    )

    IL_0000: newobj instance void ConsoleApplication1.Program/Foo/'<>c__DisplayClass1'::.ctor()
    IL_0005: stloc.0
    IL_0006: ldloc.0
    IL_0007: ldarg.0
    IL_0008: stfld class ConsoleApplication1.Program/Foo ConsoleApplication1.Program/Foo/'<>c__DisplayClass1'::'<>4__this' //Make ref to this
    IL_000d: nop
    IL_000e: ldloc.0
    IL_000f: ldc.r8 42
    IL_0018: ldc.r8 1
    IL_0021: call float64 [mscorlib]System.Math::Pow(float64, float64)
    IL_0026: stfld float64 ConsoleApplication1.Program/Foo/'<>c__DisplayClass1'::capturedVariable
    IL_002b: ldarg.0
    IL_002c: ldloc.0
    IL_002d: ldftn instance void ConsoleApplication1.Program/Foo/'<>c__DisplayClass1'::'<InstanceMethod>b__0'()
    IL_0033: newobj instance void [mscorlib]System.Action::.ctor(object, native int)
    IL_0038: stfld class [mscorlib]System.Action ConsoleApplication1.Program/Foo::_field
    IL_003d: nop
    IL_003e: ret
} // end of method Foo::InstanceMethod

उदाहरण 2:

class Program
{
    static void Main(string[] args)
    {
    }


    class Foo
    {
        private Action _field;

        public void InstanceMethod()
        {
            var capturedVariable = Math.Pow(42, 1);

            _field = () => Foo2.StaticMethod(capturedVariable);  //Foo2

        }

        private static void StaticMethod(double arg) { }
    }

    class Foo2
    {

        internal static void StaticMethod(double arg) { }
    }


}

in cl: (नोट !! अब यह संदर्भ समाप्त हो गया है!)

public hidebysig 
        instance void InstanceMethod () cil managed 
    {
        // Method begins at RVA 0x2074
        // Code size 56 (0x38)
        .maxstack 4
        .locals init (
            [0] class ConsoleApplication1.Program/Foo/'<>c__DisplayClass1' 'CS$<>8__locals2'
        )

        IL_0000: newobj instance void ConsoleApplication1.Program/Foo/'<>c__DisplayClass1'::.ctor()
        IL_0005: stloc.0
        IL_0006: nop //No this pointer
        IL_0007: ldloc.0
        IL_0008: ldc.r8 42
        IL_0011: ldc.r8 1
        IL_001a: call float64 [mscorlib]System.Math::Pow(float64, float64)
        IL_001f: stfld float64 ConsoleApplication1.Program/Foo/'<>c__DisplayClass1'::capturedVariable
        IL_0024: ldarg.0 //No This ref
        IL_0025: ldloc.0
        IL_0026: ldftn instance void ConsoleApplication1.Program/Foo/'<>c__DisplayClass1'::'<InstanceMethod>b__0'()
        IL_002c: newobj instance void [mscorlib]System.Action::.ctor(object, native int)
        IL_0031: stfld class [mscorlib]System.Action ConsoleApplication1.Program/Foo::_field
        IL_0036: nop
        IL_0037: ret
    }

छूट 3:

class Program
{
    static void Main(string[] args)
    {
    }

    static void Test(double arg)
    {

    }

    class Foo
    {
        private Action _field;

        public void InstanceMethod()
        {
            var capturedVariable = Math.Pow(42, 1);

            _field = () => Test(capturedVariable);  

        }

        private static void StaticMethod(double arg) { }
    }


}

IL में: (यह सूचक वापस आ गया है)

IL_0006: ldloc.0
IL_0007: ldarg.0
IL_0008: stfld class ConsoleApplication1.Program/Foo ConsoleApplication1.Program/Foo/'<>c__DisplayClass1'::'<>4__this' //Back again.

और सभी तीन मामलों में विधि- b__0 () - समान दिखें:

instance void '<InstanceMethod>b__0' () cil managed 
    {
        // Method begins at RVA 0x2066
        // Code size 13 (0xd)
        .maxstack 8

        IL_0000: ldarg.0
        IL_0001: ldfld float64 ConsoleApplication1.Program/Foo/'<>c__DisplayClass1'::capturedVariable
                   IL_0006: call void ConsoleApplication1.Program/Foo::StaticMethod(float64) //Your example
                    IL_0006: call void ConsoleApplication1.Program/Foo2::StaticMethod(float64)//Example 2
        IL_0006: call void ConsoleApplication1.Program::Test(float64) //Example 3
        IL_000b: nop
        IL_000c: ret
    }

और सभी 3 मामलों में एक स्थिर विधि का संदर्भ है, इसलिए यह इसे और अधिक विषम बनाता है। तो इस लिट्ल एनालिसिस के बाद, मैं इसके बग को / अच्छे के लिए नहीं कहूंगा। !


मुझे लगता है कि इसका मतलब यह है कि यह नेड वर्ग द्वारा उत्पन्न लंबोदर अभिव्यक्ति के अंदर एक मूल वर्ग से स्थिर तरीकों का उपयोग करने के लिए एक बुरा विचार है? मुझे आश्चर्य है कि अगर Foo.InstanceMethodइसे स्थिर बनाया जाता है, तो क्या यह संदर्भ को भी हटा देगा? मैं जानने के लिए आभारी रहूँगा।
Ivaylo Slavov

1
@ इव्लो: अगर Foo.InstanceMethodस्टैटिक भी होते, तो देखने में कोई उदाहरण नहीं होता, और इसलिए किसी भी तरह के thisक्लोजर पर कब्जा करने का कोई तरीका नहीं था ।
ऐनी

1
@ इवालो स्लाव यदि उदाहरण विधि स्थिर थी, तो क्षेत्र को स्थिर होना पड़ता है, मैंने कोशिश की - और 'यह सूचक' नहीं होगा।
निकोलस

@ निक्लास, धन्यवाद। अंत में मुझे लगता है कि लंबोदा बनाने के लिए स्थैतिक तरीके इस अनावश्यक सूचक की कमी की गारंटी देंगे।
Ivaylo Slavov

Slavov @Ivaylo, तो लगता .. :)
निकलस
हमारी साइट का प्रयोग करके, आप स्वीकार करते हैं कि आपने हमारी Cookie Policy और निजता नीति को पढ़ और समझा लिया है।
Licensed under cc by-sa 3.0 with attribution required.