PSOTWIFLI

[Marc Clifton]

Jim_Snyder wrote:

I would like the bottom form if the number of parentheses matched.

Wow, people's eyes are good. Fixed, but still inline. But I like your version.

Latest Article - Class-less Coding - Minimalist C# and Why F# and Function Programming Has Some Advantages Learning to code with python is like learning to swim with those little arm floaties. It gives you undeserved confidence and will eventually drown you. - DangerBunny Artificial intelligence is the only remedy for natural stupidity. - CDP1802

[Marc Clifton]

GuyThiebaut wrote:

insert a field in the middle of the others and the thing will go kaboom

Good point. :)

Latest Article - Class-less Coding - Minimalist C# and Why F# and Function Programming Has Some Advantages Learning to code with python is like learning to swim with those little arm floaties. It gives you undeserved confidence and will eventually drown you. - DangerBunny Artificial intelligence is the only remedy for natural stupidity. - CDP1802

[Jim_Snyder]

For some Markdown Formatting reason, I cannot get ``Processing.Fire`` to display in the block.

[Richard Deeming]

As always, the only valid answer is: it depends. :) Does it make sense for the properties to be writeable? Particularly in an EventArgs class, where you probably don't want one rogue handler to change the data that other handlers see. If you have lots of parameters, and you don't always need to pass all of them, an object initializer might be cleaner. But you could do something similar with optional parameters, or provide constructor overloads for each valid set of parameters. And if you're in that situation, it might be time to refactor the class anyway, because it's probably breaking the SRP.

---

"These people looked deep within my soul and assigned me a number based on the order in which I joined." - Homer

[Jim_Snyder]

The *nerve* of some folks!

[Marc Clifton]

Richard Deeming wrote:

Does it make sense for the properties to be writeable?

5! Yes, that's the hidden "gotcha". When using a constructor, the class can be initialized like this:

public class ProcessEventArgs : EventArgs
{
public IMembrane FromMembrane { get; protected set; }
public IReceptor FromReceptor { get; protected set; }
public IMembrane ToMembrane { get; protected set; }
public IReceptor ToReceptor { get; protected set; }
public ISemanticType SemanticType { get; protected set; }

public ProcessEventArgs(IMembrane fromMembrane, IReceptor fromReceptor, IMembrane toMembrane, IReceptor toReceptor, ISemanticType st)
{
FromMembrane = fromMembrane;
FromReceptor = fromReceptor;
ToMembrane = toMembrane;
ToReceptor = toReceptor;
SemanticType = st;
}
}

Note the protected setters. :)

Latest Article - Class-less Coding - Minimalist C# and Why F# and Function Programming Has Some Advantages Learning to code with python is like learning to swim with those little arm floaties. It gives you undeserved confidence and will eventually drown you. - DangerBunny Artificial intelligence is the only remedy for natural stupidity. - CDP1802

[Marc Clifton]

Jim_Snyder wrote:

For some Markdown Formatting reason, I cannot get Processing.Fire to display in the block.

Bizarre. Works for me! :-D [edit] Oh, in you're example. Yeah, I was wondering about that. [/edit]

Latest Article - Class-less Coding - Minimalist C# and Why F# and Function Programming Has Some Advantages Learning to code with python is like learning to swim with those little arm floaties. It gives you undeserved confidence and will eventually drown you. - DangerBunny Artificial intelligence is the only remedy for natural stupidity. - CDP1802

[Pete OHanlon]

The answer, of course, is that it all depends. The second example has the potential to introduce errors that the first doesn't if you're looking at validation to make sure that the values in ProcessEventArgs aren't null (validating them in the constructor of course). More importantly, the second example implies that you have opened up the scope of the properties to be read/write as opposed to read only. It really depends on what the use case is for ProcessEventArgs.

This space for rent

[Richard Deeming]

Or better yet, like this:

public class ProcessEventArgs : EventArgs
{
public IMembrane FromMembrane { get; }
public IReceptor FromReceptor { get; }
public IMembrane ToMembrane { get; }
public IReceptor ToReceptor { get; }
public ISemanticType SemanticType { get; }

public ProcessEventArgs(IMembrane fromMembrane, IReceptor fromReceptor, IMembrane toMembrane, IReceptor toReceptor, ISemanticType semanticType)
{
FromMembrane = fromMembrane;
FromReceptor = fromReceptor;
ToMembrane = toMembrane;
ToReceptor = toReceptor;
SemanticType = semanticType;
}
}

(Assuming you're using the C# 6 compiler or later. Otherwise, private set; would have a similar effect.) Also, named parameters can make the constructor look more like the object initializer:

Processing.Fire(this, new ProcessEventArgs(
fromMembrane: fromMembrane,
fromReceptor: fromReceptor,
toMembrane: membrane,
toReceptor: target,
semanticType: obj
));

---

"These people looked deep within my soul and assigned me a number based on the order in which I joined." - Homer

[Johnny J]

I'm old school, so the second one any day! :thumbsup:

Anything that is unrelated to elephants is irrelephant
Anonymous
-----
The problem with quotes on the internet is that you can never tell if they're genuine
Winston Churchill, 1944
-----
I'd just like a chance to prove that money can't make me happy.
Me, all the time

[Tim Carmichael]

Arriving late to the party (comments indicate multiple edits have been made). For my older eyes, I prefer the first; I can see what the arguments are on each line without having to scan a list looking for something.

[Marc Clifton]

Richard Deeming wrote:

Also, named parameters can make the constructor look more like the object initializer:

Ah, I've never used that syntax in C#. And yes, omitting the setter or marking it private too. :cool:

Latest Article - Class-less Coding - Minimalist C# and Why F# and Function Programming Has Some Advantages Learning to code with python is like learning to swim with those little arm floaties. It gives you undeserved confidence and will eventually drown you. - DangerBunny Artificial intelligence is the only remedy for natural stupidity. - CDP1802

[Marc Clifton]

Pete O'Hanlon wrote:

More importantly, the second example implies that you have opened up the scope of the properties to be read/write as opposed to read only

5! Yup.

Latest Article - Class-less Coding - Minimalist C# and Why F# and Function Programming Has Some Advantages Learning to code with python is like learning to swim with those little arm floaties. It gives you undeserved confidence and will eventually drown you. - DangerBunny Artificial intelligence is the only remedy for natural stupidity. - CDP1802

[Marc Clifton]

Tim Carmichael wrote:

I prefer the first; I can see what the arguments are on each line without having to scan a list looking for something.

Me too, but as others have pointed out, it allows for the properties to be write-able.

Latest Article - Class-less Coding - Minimalist C# and Why F# and Function Programming Has Some Advantages Learning to code with python is like learning to swim with those little arm floaties. It gives you undeserved confidence and will eventually drown you. - DangerBunny Artificial intelligence is the only remedy for natural stupidity. - CDP1802

[Mladen Jankovic]

Parameters that are required for establishing invariant should be part of constructor. For others, do as you're pleased. I prefer #2, but I also like read-only field/properties, but that subject is already covered by others in this thread.

[Tim Carmichael]

Clarification... I prefer the properties to be listed, one per line, as in option 1 versus having them all listed on a single line.

[Jeremy Falcon]

For me, it depends. If it's just a few params I tend to go for the latter since a couple params isn't hard to remember. If you end up with a constructor or method that takes a crap ton of them then I use the former always so there's no guesswork as to what the params are. For me, code is like art. You make it look pretty and readable on a case-by-case basis. If it makes sense to do something then you do it, but it doesn't always mean you do the same thing every time for the rest of your life. It's on a case-by-case basis.

Jeremy Falcon

[Jeremy Falcon]

Johnny J. wrote:

I'm old school, so the second one any day!

Hey there's this new fancy language called COBOL you should check out. :rolleyes:

Jeremy Falcon

[dandy72]

The former, even if only because the list of args provided can keep growing and it'll remain readable. Whereas in the latter form, you've probably already reached the limit of what you can see without scrolling horizontally. If that's the dumbest reason imaginable, then I'll still insist it gives me the ability to merely glance at the code to figure out what it's doing.

[dandy72]

Jeremy Falcon wrote:

For me, code is like art. You make it look pretty and readable on a case-by-case basis. If it makes sense to do something then you do it, but it doesn't always mean you do the same thing every time for the rest of your life.

Whenever I make my code pretty I invariably have to revisit it one more time to make it functional.