cleaner way to write this?

[John Salerno]

Paul Rubin wrote:

> I like
> 
>     if dlg.ShowModal() == wx.ID_OK:
>          db_name = dlg.GetValue()
>     else:
>          db_name = None
>     dlg.Destroy()
>     return db_name
> 
> better than
> 
>     db_name = None
>     if dlg.ShowModal() == wx.ID_OK:
>          db_name = dlg.GetValue()
>     dlg.Destroy()
>     return db_name
>

Thanks guys! I didn't think about setting db_name to None in that way!