http://www.perlmonks.org?node_id=894080


in reply to Re^5: parsing XML fragments (xml log files) with... a regex
in thread parsing XML fragments (xml log files) with XML::Parser

Why are you so defensive when I reply to your reply? I point out things that you didn't point out and that your response actually points against and you feel the need to reply with "Duh! I knew that" and to call me defensive.

But you also didn't comment on the main point of my reply: that you seem to have missed the main point of my original posting. Doing pre-/post-processing is mostly counter to the whole point of what I was providing a demonstration of. Restricting yourself to solutions that don't touch the code is perfectly opposed to the purpose of code that was offered up as "here is code you can easily change". It isn't just "one way to fix it", it is closer to a misuse of the example.

So? All I did was identify the features others might have to add to suit their needs.

Yes, and that in itself is actually going against the point of what I wrote. I wasn't providing a subroutine for people to just use. I was providing an example of code that somebody could similarly create to meet their own needs or that somebody might choose to customize to meet their needs.

If the prospective consumer of this example code is unable to determine on their own what the code handles, then they should not be trying to use the code.

You replied exactly like one would reply to "Here is a black box that claims to parse XML but only partially". Treating it like a black box is counter to the whole purpose and hints that I failed at conveying the point to you and thus likely to others and so some clarifications were in order.

Pointing out how the code can be easily modified was not being defensive; it was trying to get back to / reinforce the original point.

- tye        

  • Comment on Re^6: parsing XML fragments (xml log files) with... a regex

Replies are listed 'Best First'.
Re^7: parsing XML fragments (xml log files) with... a regex
by ikegami (Patriarch) on Mar 18, 2011 at 23:00 UTC

    Yes, and that in itself is actually going against the point of what I wrote

    You gave code that "somebody might choose to customize to meet their needs." I really don't care if telling them what they may need to customise goes against some point you were trying to make.

    But you also didn't comment on the main point of my reply

    That sometimes, the simplest approach is to role our your own? I have no comment to make about that.

    Treating it like a black box is counter to the whole purpose and hints that I failed at conveying the point to you and thus likely to others and so some clarifications were in order.

    Granted.

    (Upd: I believe the post-processor mention started in the belief that you might have had more that wasn't shown. As for the pre-processor, that's probably the easiest way to handle the character encoding. )

    If the prospective consumer of this example code is unable to determine on their own what the code handles, then they should not be trying to use the code.

    Perhaps. I'll have to think about that.

Re^7: parsing XML fragments (xml log files) with... a regex
by ikegami (Patriarch) on Mar 18, 2011 at 22:57 UTC

    Yes, and that in itself is actually going against the point of what I wrote

    You gave code that "somebody might choose to customize to meet their needs." I really don't care if telling them what they may need to customise goes against some point you were trying to make.

    But you also didn't comment on the main point of my reply

    That sometimes, the simplest approach is to role our your own? I have no comment to make about that.

    Treating it like a black box is counter to the whole purpose and hints that I failed at conveying the point to you and thus likely to others and so some clarifications were in order.

    Granted.

    If the prospective consumer of this example code is unable to determine on their own what the code handles, then they should not be trying to use the code.

    Perhaps. I'll have to think about that.