[ruby-list:42875] Re: system の実行タイミングのトラブル

From: "Hideo Konami" <[e-mail]>
Subject: [ruby-list:42874] Re: system の実行タイミングのトラブル
Date: Sat, 14 Oct 2006 21:09:55 +0900

>   def initialize(psfile)
>      @ps = psfile
>      File.open(@ps)
>      @ps.puts EPS_HEADER
>   end

@psがファイル名だとすると、 String#putsがないと怒られます。

def initialize(psfile)
  @ps = File.open(psfile)
  @ps.puts EPS_HEADER
end

こう書きたかったのでしょうか？

> というふうには書けそうもないので，クローズ忘れという
> どじが発生してしまうわけです。うまいやり方はないものでしょ
> うか。

あります。
まずtestabilityを考えると Postscript.new は IO を取るといいでしょう。
IO を取ることでテスト時に StringIO を使うことができます。

また、initializeはあくまで初期化が目的なので、実際の出力はしないほうがいいと思います。
なので print_header メソッドに分けました。
書き出す時に必ず print_header を呼び出すように Postscript.for_io を
Postscript.new の代わりに使います。

EPS_HEADER = "header\n"
def honyarara() "honyarara\n" end

class Postscript 
  def self.open(psfile)
    File.open(psfile, "w") do |io|
      ps = for_io io
      yield(ps)
    end
  end

  def self.for_io(io)
    ps = new io
    ps.print_header
    ps
  end

  def initialize(io)
    @io = io
  end

  def print_header
    @io.puts EPS_HEADER
  end

  def set_color(hogehoge)
    honyarara
    @io.puts "setrgbcolor #{hogehoge}"
  end

end

require 'stringio'
io = StringIO.new
ps = Postscript.new(io)
ps.set_color('red')
io.string                       # => "setrgbcolor red\n"


file = "/tmp/a"
Postscript.open(file) do |ps|
  ps.set_color('red')
end

--
rubikitch
http://www.rubyist.net/~rubikitch/