[ fields[ 1..4 ], fields[ 4..5 ], 0, 0 ].flatten
didn't work when I tried it, but I was probably doing something wrong. I'd like sth.execute() to take an array instead of requiring me to splat it, which would allow for sth.execute( [ ... ] )
vs. the annoying ary = [ ... ]; sth.execute( *ary )
. Ideally, it would do something like:
def execute( *args )
which would allow for sth.execute( fields[ 1..4 ], fields[ 4..5 ], 0, 0 )
, but I'm sure there was a good reason to not do that. I haven't cared enough to find out what that reason was.
And, yes, do..end is more idiomatic, but I'm not comfortable with it yet. I'm sure I'll be making the shift soon enough. :-)
My criteria for good software:
- Does it work?
- Can someone else come in, make a change, and be reasonably certain no bugs were introduced?