Question

I have a lot of code like this

var o = new SomeClass().DoSomething(someParam);

Is this a design flaw ?

This class is some kind of a builder class.It should contruct other object.

Answer 1

If you don’t need the instance after calling the method, you probably better with a static method on SomeClass and ends up with

var o = SomeClass.DoSomething(someParam);

and if for any reason, you absolutely need a new instance to execute DoSomething then you could create a static factory method to make the code more readable like :

var o = SomeClass.GetInstance().DoSomething(someParam)