|
Thanks for your reply. I am starting to understand the code, and it seems to be too intelligent a design for a lay person like me.
|
|
|
|
|
If you have to ask, then the answer is probably YES.
|
|
|
|
|
Looks like the natural progression of a first time C# project.
It was only in wine that he laid down no limit for himself, but he did not allow himself to be confused by it.
― Confucian Analects: Rules of Confucius about his food
|
|
|
|
|
No. Spaghetti tastes good.
Nothing succeeds like a budgie without teeth.
|
|
|
|
|
I don't know what this is, but it's not spaghetti code.
Spaghetti code is when you break the sequential flow of your code by jumping to somewhere else, without the possibility to come back.
One would think it is not used anymore these days, but that is not really true. Every time you write a return in the middle of a method you actually break the sequential flow, which is technically speaking spaghetti code. Granted this has not the disastrous consequences as the old goto, but still...
|
|
|
|
|
|
People go to C# to escape scary old C++ and this is where they end up.
|
|
|
|
|
Without a good view of the whole dish along with remarks and dev notes, I'd call it ziti. Might be baked or half-baked. Depending on the entire code base, it might even be baked raviolis.
I can't call it spaghetti just on a declaration snippet, but I'm not a purist when I'm looking at existing code. In practical terms, it really depends on why you are there also. If you have an issue, trace it through. If it's just theoretical or just jumps out at you, it may fall under the 'if it ain't broke' theory, again depending on what your specific task is and allotted time.
Now, What's for dinner?
|
|
|
|
|
MikeCO10 wrote: What's for dinner?
Fish.
Software Zen: delete this;
|
|
|
|
|
Not spaghetti code, but it is an antipattern.
Remove it, and if nothing breaks, leave it gone.
Otherwise, if you figure out what it's for, add comments. As it sits now it's expensive trash.
|
|
|
|
|
oh my. These look like prime candidates for deletion. if the rest of the code depends on it, you might be dealing with a prime application for abortion. 
|
|
|
|
|
short: yes
Long: abstraction for the sake of patterns is a waste when it's not needed. I knew someone who abstracted everything and basically made a soup of interfaces for a very simple implementation, the abstraction was not used in a beneficial way, and caused way more headaches just trying to troubleshoot.
the KISS ideology is good, but I like the "Keep it as simple as possible"; if you know multiple things will inherit or implement code, build for that. if not, keep the code as direct and simple as possible, you can always refactor later.
|
|
|
|
|
Seasons 1 to 5 of The Expanse binge-watched, books 1 through 9 re-read.
I'm ready for Season 6.
I think ... pity it's one-per-week though.
Anyone else ready?
"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!
|
|
|
|
|
Quote: Anyone else ready?
Yup! Ready and waiting! 
Get me coffee and no one gets hurt!
|
|
|
|
|
Read somewhere recently that this would be the last series they make, so it looks as if we won't see all of the books covered which is rather a shame. This has been rather an enjoyable outing.
|
|
|
|
|
Bingewatched my first series, called "Katla", and two more episodes to go.
SSssshh..
Bastard Programmer from Hell 
"If you just follow the bacon Eddy, wherever it leads you, then you won't have to think about politics." -- Some Bell.
|
|
|
|
|
The Icelandic one? Watched that a few weeks ago. 
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
Yup. Very enjoyable 
Bastard Programmer from Hell
"If you just follow the bacon Eddy, wherever it leads you, then you won't have to think about politics." -- Some Bell.
|
|
|
|
|
Nope.
>64
If you can keep your head while those about you are losing theirs, perhaps you don't understand the situation.
|
|
|
|
|
You Are Not Prepared - YouTube[^]
Bastard Programmer from Hell
"If you just follow the bacon Eddy, wherever it leads you, then you won't have to think about politics." -- Some Bell.
|
|
|
|
|
I'm too cheap for Amazon Prime. I'll wait for all the series to come out, get prime and binge for a 2 month period, then cancel prime. No need to feed my consumerism by getting extra fast shipping. Plus book 9 is a Christmas present for my wife, so I won't be able to read it till she is done with the book on Dec 26th around midnight
Hogan
|
|
|
|
|
I'm in!
Like you I've binged seasons 1-5 and reread all 9 books. Though one episode a week is going to test my mettle. I'm sure I'll binge the all of season 6 again once the last episode has dropped.
|
|
|
|
|
I posted on this a while back and never got a resolution.
Using VS2019. In SOME of my code editor windows there appears these horizontal dotted lines. They appear to be indenting indicators of some kind.
See this pic
You can see the lines highligted in red on the left. Yet they DON'T appear on the right code window. Some windows they show up, others they don't.
These really bug me. I've tried all manner of things in Settings. Anyone know how to make these go away?
If it's not broken, fix it until it is.
Everything makes sense in someone's mind.
Ya can't fix stupid.
|
|
|
|
|
That looks like "view whitespace". The file on the left has a mixture of spaces and tabs. The file on the right is only using tabs.
Ctrl+R, Ctrl+W is the default keyboard shortcut to toggle it; otherwise, the option is within the "Edit" menu.
"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer
|
|
|
|
|
Richard Deeming wrote: Ctrl+R, Ctrl+W is the default keyboard shortcut to toggle it; otherwise, the option is within the "Edit" menu.
This toggles on/off some little right arrows. Those are the whitespace indicators. There's nothing on the Edit menu that I can see that affects this.
But even if this was a setting of some kind, why is it on in some files and not in others?? It should be on/off across all files.
If it's not broken, fix it until it is.
Everything makes sense in someone's mind.
Ya can't fix stupid.
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
