[ruby-list:6335] Re: ruby 流に添削して下さい

さくです．

In article <[e-mail]>,
    [e-mail] writes:
nkon> ruby 流の書き方に添削して欲しいのですが、構わないでしょうか。
nkon> データメンバの初期化、配列の使い方、イテレータの使い方
nkon> 文字列の処理方法など、がとくに良くわからなかったので、
nkon> 思考錯誤で書いています。

/正規表現/とイテレータ {}，あとは後置する修飾子(「式 if 式」とか)を使
うとRubyっぽくなります．

キモのparse_lineを書き換えてみますね．

to_sもイテレータを使って書き換えたほうが綺麗になるはずなのですが，「何
回ループしたか」をiでカウントしなきゃならないことには変りないのでやめ
ました．

  # 一行解釈
  def parse_line(line)
    # ま，「改行を取る」という意味では chomp のほうがよいので．
    line.chomp!
    # 正規表現は文字列処理の要(本当は . は [0-9A-Fa-f] を使うべき)
    line =~ /^:(..)((..)(..))(..)(.*)(..)$/
    # 正規表現中の () の中身は $1,... に入っている．
    length, address, high, low, type, data, checksum =
      $1.hex, $2.hex/2, $3.hex, $4.hex, $5, $6, $7.hex
    sum = length + high + low
    # イテレータ scan を使ってループを書換えます．
    data.scan(/(..)(..)/) { |a, b|
      sum += a.hex + b.hex
      @memory[address] = (b+a).hex
      address += 1;
      # やはり @memory[address++] = (b+a).hex と書きたい人はいるの？:)
    }
    # 修飾子を使うと raise というキーワードが目立つ．
    raise "Ihx8m Checksum ERROR" if (~(sum-1) & 0x00ff) != checksum
  end  # end of parse_line

例外を投げるのか，偽を返すのかも悩みどころですが．
@memory は parse_line で初期化するより initialize で初期化したほうがい
いのでは？
-- 
[Mr.] OZAWA Sakuro
<mailto:[e-mail]>
<http://www.shiojiri.ne.jp/%7Ecrouton/>