From: shugo@... (MAEDA Shugo)
Date: 1997-09-11T21:33:54+09:00
Subject: [ruby-list:4280] Re: [Q] tk ext...

前田です。

In message "[ruby-list:4272] [Q] tk ext..."
Kikutani Makoto <kikutani@jdc.ericsson.se> wrote:

|そろそろ、rubyで実用プログラムを書きたいので、添削をお願いしようと
|思います。

最近ruby-listでrubyのプログラムがよく流れるようになって
いい感じですね:-)

|# initial window size
|$width = '500'
|$height = '100'
|$vwidth = 30
|$pr_time_step = 100

これらはインスタンス変数にしてコンストラクタで指定できる
ようにした方がよいような気がします。

class WViewer

  DEFAULT_WIDTH = '500'
  DEFAULT_HEIGHT = '100'
  DEFAULT_VWIDTH = 30
  DEFAULT_PR_TIME_STEP = 100

  def initialize(signame,
		 width = DEFAULT_WIDTH, heigth = DEFAULT_HEIGHT,
		 vwidth = DEFAULT_VWIDTH, pr_time_step = DEFAULT_PR_TIME_STEP)
    @width = width
    @height = height
    @vwidth = vwidth
    @pr_time_step = pr_time_step
    ...

のような感じでいかがでしょう？

-- 
前田 修吾