Strange code example (teaser)

[James Lonero]

I saw this in a code sample on a JetBrains web page. Can you spot the problem(s)?

using System;
public class Class1 {
private Action _someAction;

public void Method() {
    var obj1 = new object();
    var obj2 = new object();

    \_someAction += () => {
        Console.WriteLine(obj1);
        Console.WriteLine(obj2);
    };

    // "Implicitly captured closure: obj2"
    \_someAction += () => {
        Console.WriteLine(obj1);
    };
}

}

This should be easy to figure out.

[Jon McKee]

Am I missing something? Besides the erroneous comment (obj1 is captured, not obj2) I see no issues :doh:

[Nish Nishant]

Not sure what that code's trying to do, but the delegate gets both functions added to its invocation list, and this is repeated each time the containing method is called.

Nish Nishant Consultant Software Architect Ganymede Software Solutions LLC www.ganymedesoftwaresolutions.com

[Jon McKee]

Yea. I guess I was assuming there was another error - not just pointless code. With the naming, I can't even be absolutely sure that isn't intended behavior :laugh:

[Lost User]

This is why you don't give matches or scissors to unsupervised children under 10 years old. If only software devs would consider similar wisdom for programmers with under 10 years experience.

Signature ready for installation. Please Reboot now.

[OriginalGriff]

James Lonero wrote:

Can you spot the problem(s)?

Yes. 1) Somebody actually wrote it. 2) See (1).

Bad command or file name. Bad, bad command! Sit! Stay! Staaaay... AntiTwitter: @DalekDave is now a follower!

[Pete OHanlon]

And what invokes the action?

This space for rent