try catch - "Variable declared but never used" ?

[What-A-Tool]

In the following code, "catch" is underlined in blue, and the error "the
variable fx is declared but never used" is displayed when I mouse hover over
it.

Am I doing something wrong, or do I just ignore it?

private void btnCalc_Click(o bject sender, System.EventArg s e)

{

try

{

int intNum = Int32.Parse(txt NumEntry.Text);

long lngNum = Int64.Parse(txt NumEntry.Text);

}

catch (System.FormatE xception fx) //Non integer input

{

MessageBox.Show ("Invalid entry!", "Entry Error!",

MessageBoxButto ns.OK, MessageBoxIcon. Exclamation);

}

finally

{

//unexpected error

}

}

Thanks - Sean

[Robbe Morris [C# MVP]]

If you aren't going to use it, then don't reference it

catch (System.FormatE xception)

--
Robbe Morris - 2004-2006 Microsoft MVP C#
http://www.eggheadcafe.com/forums/merit.asp

"What-A-Tool" <Di************ *****@DieDieDie .Com> wrote in message
news:6AtMf.5336 9$Ug4.7762@duke read12...

In the following code, "catch" is underlined in blue, and the error "the
variable fx is declared but never used" is displayed when I mouse hover
over it.

Am I doing something wrong, or do I just ignore it?

private void btnCalc_Click(o bject sender, System.EventArg s e)

{

try

{

int intNum = Int32.Parse(txt NumEntry.Text);

long lngNum = Int64.Parse(txt NumEntry.Text);

}

catch (System.FormatE xception fx) //Non integer input

{

MessageBox.Show ("Invalid entry!", "Entry Error!",

MessageBoxButto ns.OK, MessageBoxIcon. Exclamation);

}

finally

{

//unexpected error

}

}

Thanks - Sean

[Boblemar]

What-A-Tool a écrit :

In the following code, "catch" is underlined in blue, and the error "the
variable fx is declared but never used" is displayed when I mouse hover over
it.

Am I doing something wrong, or do I just ignore it?

private void btnCalc_Click(o bject sender, System.EventArg s e)

{

try

{

int intNum = Int32.Parse(txt NumEntry.Text);

long lngNum = Int64.Parse(txt NumEntry.Text);

}

catch (System.FormatE xception fx) //Non integer input

{

MessageBox.Show ("Invalid entry!", "Entry Error!",

MessageBoxButto ns.OK, MessageBoxIcon. Exclamation);

}

finally

{

//unexpected error

}

}

Thanks - Sean

You can write the following line into your catch block :
fx.ToString();
there won't be any warning anymore and you will have the possibility to
look at the error at debug time if you want

Bob

[What-A-Tool]

"Boblemar" <ar************ *@logicim.com> wrote in message
news:uM******** ******@TK2MSFTN GP09.phx.gbl...

You can write the following line into your catch block :
fx.ToString();
there won't be any warning anymore and you will have the possibility to
look at the error at debug time if you want

Bob

Simple solution :-)

Just seemed strange, cause I coded it the way I saw in several different
examples, and nothing was ever mentioned about errors being pointed.

Thanks - Sean

[Sandeep]

This is just a warning. You can change your error level at the compler
option.

"What-A-Tool" <Se**@IHateSpam .Com> wrote in message
news:QdBMf.5338 5$Ug4.43518@duk eread12...

"Boblemar" <ar************ *@logicim.com> wrote in message
news:uM******** ******@TK2MSFTN GP09.phx.gbl...

You can write the following line into your catch block :
fx.ToString();
there won't be any warning anymore and you will have the possibility to
look at the error at debug time if you want

Bob

Simple solution :-)

Just seemed strange, cause I coded it the way I saw in several different
examples, and nothing was ever mentioned about errors being pointed.

Thanks - Sean

[Jon Skeet [C# MVP]]

Sandeep <Sa*****@mailin ator.com> wrote:

This is just a warning. You can change your error level at the compler
option.

True, but it's far better to clean up the warning in the way suggested
by Robbe. Turning off warnings may well cause grief when you come up
against a problem which would have normally raised a warning.

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too

[Bruce Wood]

This is a personal bugbear of mine. It's a very useful warning in the
case of local variables, but it's very annoying in the case of "catch".
In the case of "catch," if you eliminate the variable name, then as
Boblemar points out you can't see the exception at debug time: you can
put a breakpoint in the catch body, but you can't see what exception
was caught.

I usually use some stupid trick like Boblemar's in order to fool the
compiler into shutting up, but I wish that variables declared in
"catch" clauses were an exception to this message.

[Jon Skeet [C# MVP]]

Bruce Wood <br*******@cana da.com> wrote:

This is a personal bugbear of mine. It's a very useful warning in the
case of local variables, but it's very annoying in the case of "catch".
In the case of "catch," if you eliminate the variable name, then as
Boblemar points out you can't see the exception at debug time: you can
put a breakpoint in the catch body, but you can't see what exception
was caught.

I usually use some stupid trick like Boblemar's in order to fool the
compiler into shutting up, but I wish that variables declared in
"catch" clauses were an exception to this message.

One alternative to actually calling something at runtime would be to
call a method which was conditionally removed - i.e.

DevNullUtil.DoN othingWith(e)

where DoNothingWith had a conditional attribute which would never be
defined.

Not something I've ever done myself, but just a thought...

--
Jon Skeet - <sk***@pobox.co m>
http://www.pobox.com/~skeet Blog: http://www.msmvps.com/jon.skeet
If replying to the group, please do not mail me too