improving my writing - 2005

[Kaypee]

below is what I have as part of registry class
Public Function DoesKeyExist() As Boolean

Dim oReg As RegistryKey

Dim bExist As Boolean = False

Try

oReg = m_MSRegRoot.Ope nSubKey(m_sKeyP ath, False)

If Not oReg Is Nothing Then

bExist = True

End If

Catch ex As Exception

'

Finally

If Not oReg Is Nothing Then

oReg.Close()

oReg = Nothing

End If

End Try

Return bExist

End Function

2005 IDE warns 'Variable oReg is used before it has been assigned a value.
A null reference exception could result at runtime'

My question is - how am I supposed to do this?

[Cor Ligthert [MVP]]

Terry,

dim oReg as RegistryKey = Nothing

I strongly disagree with you. The only thing what you do with this is
creating extra code and disable the checking on wrong used code.

Cor

[Terry Olsen]

When you declare the variable, assign it an initial value...such as:

dim oReg as RegistryKey = Nothing

*** Sent via Developersdex http://www.developersdex.com ***

[Kaypee]

:-0

OMG. I use that in most other places.

Why did my brain not associate that with the error message recieved?
Thanks Terry
"Terry Olsen" <to******@hotma il.com> wrote in message
news:%2******** **********@TK2M SFTNGP12.phx.gb l...

When you declare the variable, assign it an initial value...such as:

dim oReg as RegistryKey = Nothing

*** Sent via Developersdex http://www.developersdex.com ***

[Cor Ligthert [MVP]]

"Kaypee"

Public Function DoesKeyExist() As Boolean
Dim oReg As RegistryKey
Dim bExist As Boolean = False
Try
oReg = m_MSRegRoot.Ope nSubKey(m_sKeyP ath, False)
If Not oReg Is Nothing Then
bExist = True
End If
Catch ex As Exception
'
Finally
If Not oReg Is Nothing Then
oReg.Close()
oReg = Nothing
End If
End Try
Return bExist
End Function

As the warning tells you have a look at the use of oReg, in most places you
have a lot of code around it, as far as I can see it now, does that code
nothing.

Cor

[Cor Ligthert [MVP]]

Terry,

dim oReg as RegistryKey = Nothing

I strongly disagree with you. The only thing what you do with this is
creating extra code and disable the checking on wrong used code.

Cor

[Dennis]

Cor, I agree that there should be no need to assign a variable to nothing.
Is there a way to turn off these types of warning's in VB 2005? They would
be quite annoying. I haven't gotten VB2005 yet but do plan to do so when it
comes out and am trying to learn as much about it as I can from this group.
--
Dennis in Houston
"Cor Ligthert [MVP]" wrote:

"Kaypee"

Public Function DoesKeyExist() As Boolean
Dim oReg As RegistryKey
Dim bExist As Boolean = False
Try
oReg = m_MSRegRoot.Ope nSubKey(m_sKeyP ath, False)
If Not oReg Is Nothing Then
bExist = True
End If
Catch ex As Exception
'
Finally
If Not oReg Is Nothing Then
oReg.Close()
oReg = Nothing
End If
End Try
Return bExist
End Function

As the warning tells you have a look at the use of oReg, in most places you
have a lot of code around it, as far as I can see it now, does that code
nothing.

Cor

[Larry Lard]

I am confused...

Kaypee wrote:

below is what I have as part of registry class
Public Function DoesKeyExist() As Boolean
Dim oReg As RegistryKey
Dim bExist As Boolean = False
Try
oReg = m_MSRegRoot.Ope nSubKey(m_sKeyP ath, False)
If Not oReg Is Nothing Then
bExist = True
End If
Catch ex As Exception
'
Finally
If Not oReg Is Nothing Then
oReg.Close()
oReg = Nothing
End If
End Try
Return bExist
End Function

2005 IDE warns 'Variable oReg is used before it has been assigned a value.
A null reference exception could result at runtime'
Terry Olsen wrote: When you declare the variable, assign it an initial value...such as:

dim oReg as RegistryKey = Nothing

Terry,

Perhaps you could explain to me why the IDE is complaining? As far as I
can see the variable is declared and then immediately assigned a value
before ever being accessed. So what's the problem?

--
Larry Lard
Replies to group please

[Cor Ligthert [MVP]]

Dennis,

Because I have placed something about this somewhere else, did I get a
message from Herfried. (We agree about this before you misunderstand it).

What you ask is in the properties from myproject in the solution explorer.

There you see when you open that a tabpage with tabs in front

There you can tell at 'compile' what is a 'warning' 'none' or an 'error'.

My opinion is that this is something that should be set at end of
programming to see if there are some things not completly done. And when the
warning is correct set of again to none, to prevent that more important
warnings are overseen.

I hope this helps,

Cor

[Cor Ligthert [MVP]]

Dennis

My message can be read wrong. The text is mine not from Herfried.

Cor