Beefy Boxes and Bandwidth Generously Provided by pair Networks
Syntactic Confectionery Delight
 
PerlMonks  

Re^19: eval to replace die?

by BrowserUk (Pope)
on Oct 09, 2010 at 19:51 UTC ( #864413=note: print w/replies, xml ) Need Help??


in reply to Re^18: eval to replace die?
in thread eval to replace die?

You see what happens, right?

Yes. You've made a change to the program, so you run it through your test suite. ( You do have one don't you?)

You discover the problem and you correct it. End of.

But let me assure you it's tame against the shit I've seen in production over the years.

Then fix your process.


Examine what is said, not who speaks -- Silence betokens consent -- Love the truth but pardon error.
"Science is about questioning the status quo. Questioning authority".
In the absence of evidence, opinion is indistinguishable from prejudice.

Replies are listed 'Best First'.
Re^20: eval to replace die?
by mithaldu (Monk) on Oct 09, 2010 at 20:08 UTC
    With an exception that is a structure there is no need to correct any problem. The test suite still passes because there is no problem.

    Which was the whole point of this entire debate: It is much easier to run into trouble by using string-only errors.
      It is much easier to run into trouble

      Coding a feature change is not "running into trouble". Doing a half-assed job of it is just sloppy. Allowing that sloppiness to get into production is bad process. You cannot correct bad process by over-engineering.

      Your derived contrived example code made no sense.

      There was no reason to parse the error message. You never did anything with the results of the parsing.

      Which was the whole point of this entire debate:

      You think? Read again.

        You cannot correct bad process by over-engineering.
        You are right, i cannot. I can merely lessen the chance of others actions based on bad process being a danger and i will gladly do so.
        There was no reason to parse the error message. You never did anything with the results of the parsing.
        There was a reason, since i did something with the result, which i stated in the second paragraph. The value of the change (positive/negative) determined the direction of the change and based on that a css style which is used to colour the message. This not contrived, but slightly anonymized code i've encountered, written by others.
        You think? Read again.
        I would like to ask you to look at the original starting post of this node tree and ponder the intent of the message the user is asking the perlmonks about. I am of the opinion that it correlates very closely with what i said.

Log In?
Username:
Password:

What's my password?
Create A New User
Node Status?
node history
Node Type: note [id://864413]
help
Chatterbox?
[thezip]: Thanks guys for your comments! :-)
[1nickt]: Quick survey: has anyone used or even heard of autobox?
[Corion]: thezip: I would expect some (fancier?) numeric formatting to fail from these test failures
[1nickt]: ( Sometimes when idle I browse remote corners of the code repo at $work ... usually this yields knowledge of projects to decline and coworkers to avoid ... )
[LanX]: sure
[Corion]: 1nickt: Finding autobox in production would give me pause, yes
[LanX]: efficient survey
[MidLifeXis]: And under MINGW64_NT-6.1 MYHOST 2.6.0(0.304/5/3) 2016-09-09 09:46 x86_64 Msys there seem to be issues with escapes in external build tool calls.
[Corion]: I mean, it's a technical feat it achieves, but... why? ;)
[MidLifeXis]: And it also has the 0.14 version of the tarball in its manifest.

How do I use this? | Other CB clients
Other Users?
Others scrutinizing the Monastery: (12)
As of 2017-03-23 17:22 GMT
Sections?
Information?
Find Nodes?
Leftovers?
    Voting Booth?
    Should Pluto Get Its Planethood Back?



    Results (291 votes). Check out past polls.