Click here to Skip to main content
15,400,029 members

Welcome to the Lounge

   

For discussing anything related to a software developer's life but is not for programming questions. Got a programming question?

The Lounge is rated Safe For Work. If you're about to post something inappropriate for a shared office environment, then don't post it. No ads, no abuse, and no programming questions. Trolling, (political, climate, religious or whatever) will result in your account being removed.

 
AnswerRe: code sexiness question Pin
Richard Deeming29-Nov-21 22:01
mveRichard Deeming29-Nov-21 22:01 
GeneralRe: code sexiness question Pin
Super Lloyd29-Nov-21 22:22
MemberSuper Lloyd29-Nov-21 22:22 
GeneralRe: code sexiness question Pin
Richard Deeming29-Nov-21 22:34
mveRichard Deeming29-Nov-21 22:34 
GeneralRe: code sexiness question Pin
Super Lloyd29-Nov-21 22:39
MemberSuper Lloyd29-Nov-21 22:39 
GeneralRe: code sexiness question Pin
Daniel Pfeffer29-Nov-21 22:26
professionalDaniel Pfeffer29-Nov-21 22:26 
GeneralRe: code sexiness question Pin
Richard Deeming29-Nov-21 22:35
mveRichard Deeming29-Nov-21 22:35 
GeneralRe: code sexiness question Pin
snorkie30-Nov-21 3:08
professionalsnorkie30-Nov-21 3:08 
AnswerRe: code sexiness question Pin
OriginalGriff29-Nov-21 22:38
mveOriginalGriff29-Nov-21 22:38 
I'd use the first version - that way if SelectedObject is changed (by another thread for example) the non-null value is preserved and the app doesn't crash.

It's the way I handle event raising - my standard template code is:
C#
/// <summary>
/// Event to indicate Description
/// </summary>
public event EventHandler Name;
/// <summary>
/// Called to signal to subscribers that Description
/// </summary>
/// <param name="e"></param>
protected virtual void OnName(EventArgs e)
    {
    EventHandler eh = Name;
    if (eh != null)
        {
        eh(this, e);
        }
    }
That way, in the (unlikely) event that the last handler is removed from the c=hain, the app doesn't crash and does something sensible.
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!

GeneralRe: code sexiness question Pin
Richard Deeming29-Nov-21 22:54
mveRichard Deeming29-Nov-21 22:54 
GeneralRe: code sexiness question Pin
OriginalGriff29-Nov-21 23:22
mveOriginalGriff29-Nov-21 23:22 
AnswerRe: code sexiness question Pin
11917640 Member 30-Nov-21 0:19
Member11917640 Member 30-Nov-21 0:19 
GeneralRe: code sexiness question Pin
Andre_Prellwitz2-Dec-21 5:07
MemberAndre_Prellwitz2-Dec-21 5:07 
AnswerRe: code sexiness question Pin
Gerry Schmitz30-Nov-21 10:55
mveGerry Schmitz30-Nov-21 10:55 
GeneralRe: code sexiness question Pin
Andre_Prellwitz2-Dec-21 12:12
MemberAndre_Prellwitz2-Dec-21 12:12 
AnswerRe: code sexiness question Pin
Gary R. Wheeler30-Nov-21 12:02
MemberGary R. Wheeler30-Nov-21 12:02 
AnswerRe: code sexiness question Pin
Fueled By Decaff1-Dec-21 3:02
MemberFueled By Decaff1-Dec-21 3:02 
GeneralRe: code sexiness question Pin
Daniele Rota Nodari1-Dec-21 3:18
MemberDaniele Rota Nodari1-Dec-21 3:18 
GeneralRe: code sexiness question Pin
englebart2-Dec-21 12:51
professionalenglebart2-Dec-21 12:51 
GeneralRx DevOps Pin
BillWoodruff29-Nov-21 18:46
mveBillWoodruff29-Nov-21 18:46 
GeneralThis song is stuck in my head and it keeps seeming more relevant Pin
honey the codewitch29-Nov-21 13:51
mvahoney the codewitch29-Nov-21 13:51 
GeneralRe: This song is stuck in my head and it keeps seeming more relevant Pin
Super Lloyd29-Nov-21 16:05
MemberSuper Lloyd29-Nov-21 16:05 
GeneralRe: This song is stuck in my head and it keeps seeming more relevant Pin
honey the codewitch29-Nov-21 16:36
mvahoney the codewitch29-Nov-21 16:36 
GeneralRe: This song is stuck in my head and it keeps seeming more relevant Pin
Super Lloyd29-Nov-21 18:30
MemberSuper Lloyd29-Nov-21 18:30 
AdminThere's no survey because... Pin
Chris Maunder29-Nov-21 11:52
cofounderChris Maunder29-Nov-21 11:52 
GeneralRe: There's no survey because... Pin
PIEBALDconsult29-Nov-21 12:04
professionalPIEBALDconsult29-Nov-21 12:04 

General General    News News    Suggestion Suggestion    Question Question    Bug Bug    Answer Answer    Joke Joke    Praise Praise    Rant Rant    Admin Admin   

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.


Straw Poll

Do you expect users to read the instructions when using your apps?
  Results   400 votes