|
In my defense, I had very little context to work with. This could be some automated system that's working on transactions coming in 24/7.
Besides, it's an easy fix. Why risk it?
|
|
|
|
|
Overvalidation requires overtesting which itself causes more churn. That's the brutal irony of the situation.
The other day I spent almost an entire day poring over a routine because the test for it was broken - but it wasn't even the test itself, but bad metrics in a font file i downloaded.
I also can't count the number of times I've had to remove parameter validation as routines get nested inside the internals instead of being part of that API's surface area as things evolve, or to extend them with optional (read: nullable) parameters.
But worse is a situation like you describe. It's not an edge case - and the reason for that is you don't know all the situations in which the machine can be reset, necessarily - or do you? What about an EMP? Ergo, it's a failure condition that's not being handled gracefully. Therefore, it's a design flaw, and one that would have been non-obvious to catch in the design phase.
You got bit, only I don't think you did anything wrong, because erring on the side of overvalidating is better than the alternative, but honestly, I would have done your time validation at another level, like halt if the clock isn't greater than the last time you read it, and I would have skipped the transaction amount accumulation checking altogether barring a scenario you can think of where it's necessary.
Can you pull it out?
Can you adjust the clock code so it checks that the clock's good on start, instead of checking it where you are? That way all you theoretically have to do to restart it on a machine reset it set the clock.
I'm flying blind here, so I may be talking out my backside, but all of this is just stuff that occurred to me.
To err is human. Fortune favors the monsters.
|
|
|
|
|
Sounds like your data model needs to track the PLC and the case as 2 separate entities.
I think the new PLC should start as a new PLC-30 (0 initial start) placed in CASE-10. PLC-04 previously in CASE-10 is retired/scrapped.
Just keep an eye on the collectors as they will find new ways to trick the cash control reports. Like entering negative currency on PLC-04 to balance what goes in their pocket!
|
|
|
|
|
I usually go for minimal validation for exactly the reason you describe.
I ask myself, do I really need this value for the process to work?
If so, it's required.
And also, should a value have a maximum length for it to be saved in the database?
If so, I add a maximum length.
The latter is rarely a problem since I give my field lengths a value that's definitely high enough (usually something like four times the length I'd expect).
And often I make a unique validation, like on a unique code or key.
Other than that, anything goes (unless explicitly specified).
If the data is wrong I can always tell my customer it's their data.
Of course I can write an extra validation, but they should also validate their process so it won't happen again.
|
|
|
|
|
kmoorevs wrote: Meter reading must be greater than or equal to the previous reading.
I hope that the machine itself submits the reading. If it is read by an operative who then posts it, you have greater problems.
I still submit my electricity meter reading 'manually' and have once got the digits wrong which gave an excessively high reading, followed by a much lower correct reading. Now my electric meter has an odometer style display that is in the 900000s and only has six digits; when it reaches 999999 it will roll back to 000000 - I am already worrying if my supplier will be able to cope with that.
The flip side is that 'modern' design principles (XP - Extreme Programming) say you should only cater for the 'known' and not pre-guess the 'unknown'. It is called YAGNI "You Ain't Going to Need It" / "You Aren't Gonna Need It".
|
|
|
|
|
I think that's up there with rolling back the odometer on a car. Create a new "machine account" (with maybe a link to the old one).
"Before entering on an understanding, I have meditated for a long time, and have foreseen what might happen. It is not genius which reveals to me suddenly, secretly, what I have to say or to do in a circumstance unexpected by other people; it is reflection, it is meditation." - Napoleon I
|
|
|
|
|
I am looking for contracting companies hiring programmers/developers.
Any recommendations?
ed
|
|
|
|
|
Most of my connections are from personal network (been on the same gig since 2003).
Are you looking for work?
Charlie Gilley
“They who can give up essential liberty to obtain a little temporary safety deserve neither liberty nor safety.” BF, 1759
Has never been more appropriate.
|
|
|
|
|
Yes. Just quit my current job as the boss is a real a** h***
Can send you a resume, if you like.
Thank you for your reply.
ed
|
|
|
|
|
Quote: Yes. Just quit my current job as the boss is a real a** h***
Quite risky for such a statement as long you have your employer in your profile? 
|
|
|
|
|
flexjobs.com. [^] I know people that have found great gigs from this site. Costs $ subscription per year, but effective.
you may also want to get a premium LinkedIn account. I have one, and it has been super helpful for me.
|
|
|
|
|
Thank you. I will look into it.
|
|
|
|
|
|
|
My idea of marketing myself as a contractor: "I may be a whore, but I ain't cheap."
Software Zen: delete this;
|
|
|
|
|
Yes. Me too. That is an avenue I am pursuing, currently.
|
|
|
|
|
I appreciate being put on "Hot Threads.
However, I don't find the caption "
"Why don't you get a haircut and get a real job?" to be a laughing matter.
|
|
|
|
|
That's a line from a song by George Thorogood and the Destroyers.
Bond
Keep all things as simple as possible, but no simpler. -said someone, somewhere
|
|
|
|
|
Funny Note...
You probably have a decent haircut by 1970s standards...
And you are LOOKING for a job...
So, the comment kinda nailed it. It simply brought up a memory, and it made me look.
I thought NOTHING less of you upon reading the comment, because the comment is the kind of sh!t I say!
Good Luck!
|
|
|
|
|
Long-haired, freaky-looking people need not apply.
|
|
|
|
|
I replaced my 4 year old iPhone yesterday. Unlike the happy dancing people on TV, I do not get excited about a new device (PC, Phone, ...) The whole process of getting the new device set up to work just like the one it replaced is a series of slow, frustrating steps. Maybe showing once again that I'm old. 
|
|
|
|
|
Craig Robbins wrote: that I'm old
It's the mileage. I'm 33, I'm in computers since the late 90s, I lost the excitement for new equipment* in the 2010s, replaced by the frustration of setting it up.
*Well, not lost lost, but it's definitely milder and temperated by the effort required.
GCS/GE d--(d) s-/+ a C+++ U+++ P-- L+@ E-- W+++ N+ o+ K- w+++ O? M-- V? PS+ PE Y+ PGP t+ 5? X R+++ tv-- b+(+++) DI+++ D++ G e++ h--- r+++ y+++* Weapons extension: ma- k++ F+2 X
|
|
|
|
|
A new PC was great back in the day, when it meant your startup time got shorter by a few seconds, you could play the newest games, and you got a new OS that definitely looked better than the previous one (32 colors? 64 colors!? TRUE COLORS!?).
Don't forget the bigger monitor with a better resolution that actually made a difference!
And the switch from floppy to CD to CD-RW and DVD and Blu-Ray, or finally a USB!
Back then a new PC meant you got something definitely better (well, save for Vista) 
A new PC now means it has a new OS that looks and feels pretty much like the old one, you probably won't notice a huge difference in performance, but it'll cost you a day getting all your tools re-installed 
A new resolution just means your code doesn't end at 3/4 of your screen, but halfway.
|
|
|
|
|
Sander Rossel wrote: A new resolution just means your code doesn't end at 3/4 of your screen, but halfway.
Crying in 150% magnification with 1920x1080 on a 22" screen.
GCS/GE d--(d) s-/+ a C+++ U+++ P-- L+@ E-- W+++ N+ o+ K- w+++ O? M-- V? PS+ PE Y+ PGP t+ 5? X R+++ tv-- b+(+++) DI+++ D++ G e++ h--- r+++ y+++* Weapons extension: ma- k++ F+2 X
|
|
|
|
|
Haven't bought a new desktop machine this century, I think. Do they still have that "new computer" smell?? ![Rose | [Rose]](https://www.codeproject.com/script/Forums/Images/rose.gif)
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
