[ruby-dev:9070] Re: bang method returns string always

青山です。

On Thu, 3 Feb 2000 00:50:58 +0900,
[e-mail] (Yukihiro Matsumoto) wrote:

>   (1) tr!などは変化しないときnilを返すように戻す
>   (2) 変化の有無が知りたいときはno bang methodを使い、
>       original と比較する

(3) tr! などには(も)引数を増やして、nil (または、例外？)を返すようにも
    できるようにする。s.tr!("abc", "ABC", true) とか。

元の文字列との比較というのは、場合によっては結構重くなりませんか？
そうで無いのならかまいませんが、重くなる可能性があるのならば、比較無し
に変化の有無を知る方法も欲しいです。


とりあえず shellwords.rb は、こんな感じでしょうか。パッチの方が長くなっ
てしまうので、そのまま。ArgError --> ArgumentError の修正と、Array#+=
--> Array#push, String#+= --> String#concat による高速化、そして、引数が
nil, false だった場合に '' を返していたのを、型チェックに変更して、
String 以外の場合は例外としてみました。

# shellwords.rb
# original is shellwords.pl
#
# Usage:
#       require 'shellwords'
#       words = Shellwords.shellwords(line)
#
#	   or
#
#       require 'shellwords'
#       include Shellwords
#       words = shellwords(line)

module Shellwords
  def shellwords(line)
    unless line.kind_of?(String)
      raise ArgumentError, "Argument must be String class object."
    end
    line.sub!(/^\s+/, '')
    words = []
    while line != ''
      field = ''
      while true
	if line.sub!(/^"(([^"\\]|\\.)*)"/, '') && $& then #"
	  snippet = $1
	  snippet.gsub!(/\\(.)/, '\1')
	elsif line =~ /^"/ then #"
	  raise ArgumentError, "Unmatched double quote: #{line}"
        elsif line.sub!(/^'(([^'\\]|\\.)*)'/, '') && $& then #'
	  snippet = $1
	  snippet.gsub!(/\\(.)/, '\1')
	elsif line =~ /^'/ then #'
	  raise ArgumentError, "Unmatched single quote: #{line}"
	elsif line.sub!(/^\\(.)/, '') && $& then
	  snippet = $1
	elsif line.sub!(/^([^\s\\'"]+)/, '') && $& then #'
	  snippet = $1
	else
	  line.sub!(/^\s+/, '')
	  break
	end
	field.concat(snippet)
      end
      words.push(field)
    end
    words
  end
  module_function :shellwords
end


-- 
青山 和光 Wakou Aoyama <[e-mail]>