Details
-
Improvement
-
Status: Closed
-
Major
-
Resolution: Fixed
-
None
-
None
Description
Certain CTORs accept two boolean flags
public SomeTransport( arg1, arg2, ..., bool useBufferedSockets = false, bool useFramedTransport = false)
The only valid combinations here are in fact (false,false), (true,false), (false,true) - the forth combination does not make sense because framed by design already acts as a buffer. Not to mention, that multiple boolean arguments are usually less coder-friendly.
Therefore, the parameterlist should be shortened to the more readable, maintainable and concise style like so (proposal):
public enum Buffering { None, BufferedTransport, FramedTransport } public SomeTransport( arg1, arg2, ..., Buffering buffering = Buffering.None)
Attachments
Issue Links
- links to