From: Charles Hixson <charleshixsn@...>
Date: 2013-04-19T10:36:21+09:00
Subject: Re: Match against multiple patterns problem

This is a multi-part message in MIME format.
--------------020103060900090401070801
Content-Type: text/plain; charset=ISO-8859-1; format=flowed
Content-Transfer-Encoding: 7bit

On 04/18/2013 11:17 AM, Wayne Brisette wrote:
> Wouldn't using the case statement be a better option than all the 
> elsif statements? Seems like this is the perfect situation to use case.
>
> Wayne
>
The case expression might be a bit cleaner, but that would appear to 
coerce using the $~ pattern match value, which I don't like at all.  So 
I think I'll stick to cleaning up the typo, and changing len(s) to 
s.length.  Then I'll merge the initial if onto the rest, so everything 
is the same form.

I'm glad I hadn't misremembered how the assignment should be written, 
but the error messages sure indicated (to me) that I had.  Now that the 
problem is explained I've cleared it up a lot.  (E.g., those != nil 
things turned out to be superfluous.)  The current version is:
         if    not s.is_a?(String)    or    s.length < 1    then    
return    ["1",""]
         if    (m = /^[^\W_]+/.match(s) )    then    return    ["a", m[0]]
         if    (m = /^\s+/.match(s) )        then    return    ["s", m[0]]
         if    (m = /^[[:cntrl:]]+/)         then    return    ["c", m[0]]
         if    (m = /^[[:punct:]_]+/)        then    return    ["p", m[0]]
         if    (m = /^[^\w\s[:cntrl:][:punct:]]+/) != nil
                                             then    return    ["r", m[0]]
which looks a lot better.

-- 
Charles Hixson


--------------020103060900090401070801
Content-Type: text/html; charset=ISO-8859-1
Content-Transfer-Encoding: 7bit

<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    On 04/18/2013 11:17 AM, Wayne Brisette wrote:
    <blockquote
      cite="mid:1366309020.87459.YahooMailNeo@web181701.mail.ne1.yahoo.com"
      type="cite">
      <meta http-equiv="Context-Type" content="text/html;
        charset=iso-8859-1">
      <div>
        <div>Wouldn't using the case statement be a better option than
          all the elsif statements? Seems like this is the perfect
          situation to use case.&nbsp;</div>
        <div><br>
        </div>
        <div>Wayne</div>
        <div><br>
        </div>
      </div>
    </blockquote>
    The case expression might be a bit cleaner, but that would appear to
    coerce using the $~ pattern match value, which I don't like at all.&nbsp;
    So I think I'll stick to cleaning up the typo, and changing len(s)
    to s.length.&nbsp; Then I'll merge the initial if onto the rest, so
    everything is the same form.<br>
    <br>
    I'm glad I hadn't misremembered how the assignment should be
    written, but the error messages sure indicated (to me) that I had.&nbsp;
    Now that the problem is explained I've cleared it up a lot.&nbsp; (E.g.,
    those != nil things turned out to be superfluous.)&nbsp; The current
    version is:<br>
    <tt>&nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; if&nbsp;&nbsp;&nbsp; not s.is_a?(String)&nbsp;&nbsp;&nbsp; or&nbsp;&nbsp;&nbsp; s.length &lt; 1&nbsp;&nbsp;&nbsp;
      then&nbsp;&nbsp;&nbsp; return&nbsp;&nbsp;&nbsp; ["1",""]<br>
      &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; if&nbsp;&nbsp;&nbsp; (m = /^[^\W_]+/.match(s) )&nbsp;&nbsp;&nbsp; then&nbsp;&nbsp;&nbsp; return&nbsp;&nbsp;&nbsp;
      ["a", m[0]]<br>
      &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; if&nbsp;&nbsp;&nbsp; (m = /^\s+/.match(s) )&nbsp; &nbsp; &nbsp; &nbsp; then&nbsp;&nbsp;&nbsp; return&nbsp;&nbsp;&nbsp;
      ["s", m[0]]<br>
      &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; if&nbsp;&nbsp;&nbsp; (m = /^[[:cntrl:]]+/)&nbsp;&nbsp; &nbsp; &nbsp; &nbsp; then&nbsp;&nbsp;&nbsp; return&nbsp;&nbsp;&nbsp;
      ["c", m[0]]<br>
      &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; if&nbsp;&nbsp;&nbsp; (m = /^[[:punct:]_]+/)&nbsp; &nbsp; &nbsp; &nbsp; then&nbsp;&nbsp;&nbsp; return&nbsp;&nbsp;&nbsp;
      ["p", m[0]]<br>
      &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; if&nbsp;&nbsp;&nbsp; (m = /^[^\w\s[:cntrl:][:punct:]]+/) != nil<br>
      &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; &nbsp;&nbsp;&nbsp; then&nbsp;&nbsp;&nbsp; return&nbsp;&nbsp;&nbsp;
      ["r", m[0]]</tt><br>
    which looks a lot better.<br>
    <br>
    <pre class="moz-signature" cols="72">-- 
Charles Hixson</pre>
  </body>
</html>

--------------020103060900090401070801--