Is this OK?

[wossname]

I'd like your opinions on this line of code...

'type Boundary is a custom class
'frmViewEditBoundary is a form that returns a Boundary instance
 
Dim temp As Boundary = (New frmViewEditBoundary).ShowDialog()

Is this acceptable coding practice? Or is there a cleaner way to get the same effect.

The code works fine but I'm not sure if its in line with oop practice. I liked it because it saves having to dim the form on a separate line of code. The form is not needed afterwards.

What are your thoughts please?

[crptcblade]

As long as you don't need to refer to the same instance more than once, I see no problem with it. Doesn't get much cleaner.

[wossname]

Good. I've never seen this technique used elsewhere though, which makes me wonder if there is a reason!

[LITHIA]

ive always liked the idea you can declare the variable and assign it in the same line.
some people think it makes VB.NET feel more like C++ by doing this, but i just see it as a more efficient way of writing code.
i dont see anything wrong with it!