Rubyのスクリプト

質問

つぎのようなスクリプトを組みました。これはこれで、動いておりますが、お願いです。
（１）Rubyらしい書き方にするには、どこを改めればよいですか？
（２）ref_file=open("myReference.txt") の部分は、
外部のレファレンスを用いて名前を翻訳しているところです。
この部分を、関数呼び出しでやってしまいたい。どのように組みなおせば良いですか？
宜しくお願いいたします。

---------------------
おおまかな説明：ディレクトリ「myData」の中にあるlogファイルをひとつずつオープン。
まず、「Target」という単語を探し、その直後にある単語を掴んで、外部のレファレンスファイル「myReference.txt」からrealNameを持ってくる。
その後、処理－－－－－を実行する。

myPath='C:\myData\'
print Dir.getwd,"\n"
Dir.chdir(myPath)

Dir.foreach(myPath) { |myFileName|
if /.log/i =~myFileName
file_source_body=open(myFileName, "r")

while text=file_source_body.gets do
if(/Target (\w+\/)/=~text) then
myTargetName=$1

ref_file=open("myReference.txt") # （２）のアイテムの部分
while ref_info=ref_file.gets do
(/#{myTargetName}\s(\S+)/i=~ref_info) && (realName=$1)
end
ref_file.close
end
end

-------
-------
-------
}

回答No.5

koko_u_ · ベストアンサー

>誠に申し訳ない。ご指導ください。
うーん、困ったな。
面倒だから「答え」を書いちゃえ。 # 削除されたらゴメンね。

------
require 'csv'
class Reference
　# CSV 形式の ref_file を読み込んで、インスタンス変数 @ref_hash に格納
　def initialize(ref_file)
　　@ref_hash = {}
　　CSV.open(ref_file) do |row|
　　　@ref_hash[row[0]] = row[1]
　　end
　end

　# key の辞書引きの結果を返す。辞書になければ nil を返す
　def [](key)
　　@ref_hash[key]
　end
end
-----------------
irb でこんな感じで動くはず
irb(main):000:0> ref = Reference.new("Reference.txt")
irb(main):001:0> puts ref['aaaa']
=> "AAAA"
-----------------

クラスっちゅうのは「何か」の抽象化なので、今回は「辞書ファイル」を抽象化して Reference クラスを作成しました。

initialize メソッドは Reference クラスのインスタンスを作成する際の初期化ルーチンで、Reference.new() した時に呼ばれます。今回は "Reference.txt" のようなファイルを引数として、辞書としての自分自身を作成しています。その実体はただのハッシュ。

Ruby では [] 自体もメソッドとして定義できて、その方法は上記の通り、そしてメソッドの呼出し方が通常のメソッドのように foo.bar(baz) ではなくて foo[baz] となります。foo.[](baz) でも呼べるけど見難いだけ。

お礼

koko_u_さん、ありがとうございます。
教えていただいた、内容で、１点エラーが出ました。
10:in `open': wrong number of arguments (1 for 2) (ArgumentError)
CSV.openをハイライトさせて、F1でヘルプを呼び出し、
モード（この場合は、read only　ということで 'r')を追加して
うまく行きました。

以下をRDEのエディタウィンドウに記述して、実行させました。
---------------------------
require 'csv'
class Reference
def initialize(ref_file)
@ref_hash = {}
CSV.open(ref_file,'r') do |row|
@ref_hash[row[0]] = row[1]
end
end

def [](key)
@ref_hash[key]
end
end

ref = Reference.new("myReference.txt")
puts ref['aaaa']
-----------------

CSV.openのところは、下記でもやってみて、OKであることを確認しました。
CSV.foreach(ref_file){|row|
@ref_hash[row[0]] = row[1]
}
-----------------

foo[baz] の, fooとか、barは、ヘルプの中に、よく使われている
言葉なのですね。これも、理解しました。

interactive rubyでの動きも、確認しました。でも、このモードで
丹念に、すべてのステートメントを入力するのは、結構大変ですね。
しかし、このモードは、うまく使えるかもしれないと、思っております。

尚、クラスのことに関しては、難しいです。サブルーチン或いは、関数の代わりにはなっていることは分かりますが、理解するには、時間が必要です。
ありがとうございました。感謝します。

回答No.6

koko_u_

>モード（この場合は、read only　ということで 'r')を追加して
>うまく行きました。
スマン。適当に回答してるのがバレちゃった。

>CSV.openのところは、下記でもやってみて、OKであることを確認しました。
>CSV.foreach(ref_file){|row|
>@ref_hash[row[0]] = row[1]
>}
複数行に渡るコードブロックは do ... end で書く方が Ruby っぽいか？

>interactive rubyでの動きも、確認しました。でも、このモードで
>丹念に、すべてのステートメントを入力するのは、結構大変ですね。
普通に無理です。適当な .rb ファイルを書いて、
irb(main):000:0>load 'foo.rb'
でどうぞ。

私は irb はあんまり使ってなくて、xyzzy + ruby-interaction-mode で試しに動かしてみて、問題なければそれをソースにコピペしています。

回答No.4

notnot

IO.foreachと同じにするなら、

io=open(myFileName,"r")
begin
io.each{|text|
text.chomp!
print text,"\n"
}
ensure
io.close
end

また、
text.chomp!
print text,"\n"
よりは
puts text.chomp
のほうがRubyらしいと思います。

あと私も単純なケースでは、and や or を多用します。

参考文献ですが、１冊読んだのなら、あとはリファレンスで十分だと思います。http://www.ruby-lang.org/ja/man/

お礼

notnotさん、ありがとうございます。
よく理解できました。
今後とも、宜しくお願いいたします。

回答No.3

koko_u_

>@ref_hash = {}
>この表現で、ハッシュメモリ（３つ）に、入ってくれるものでしょうか？
ここは Hash.new してるだけ。

Reference.txt が CSV形式なら手抜きで

CSV.open(ref_file) do |row|
　@ref_hash[row[0]] = row[1]
end

とかして下さい。

>（２）このクラスのメッソドとしての　def [](key)
>メッソドの名称を仮に、nameCnvertとすると、
こっちは完了していて、Reference のメソッド [] を委譲によって定義しているだけです。

ref = Reference.new('Reference.txt')
で Reference オブジェクトを作成すれば

ref['ほげほげ'] で辞書引きの結果が得られます。

補足

koko_u_さん、ありがとうございます。
結論から言って、難しい。まだ、理解できていません。
とにかく、動かそうとしています。下記のように、cut&tryしました。

-------------myReference.txtの内容-----------
aaaa,AAAA
bbbb,BBBB
cccc,CCCC

-------------Ruby script trial---------------
myPath='D:\MyFolder\Study_Programming\hash'
print Dir.getwd,"\n"
Dir.chdir(myPath)

class Reference
ref_fil="myReference.txt"
def initialize(ref_file)
@ref_hash = {}
end

def [](key)
CSV.open(ref_file) do |row|
@ref_hash[row[0]] = row[1]
end
end
end

ref = Reference.new('Reference.txt')
print ref['aaaa'],"\n"
------------------------------------------

実行結果です
------------------------------------------
D:/MyFolder/Study_Programming/hash
D:/MyFolder/Study_Programming/hash/rb25E.tmp:12:in `[]': uninitialized constant Reference::CSV (NameError)
from D:/MyFolder/Study_Programming/hash/rb25E.tmp:19
Completed(1)
------------------------------------------

誠に申し訳ない。ご指導ください。

回答No.2

notnot

ファイルから1行ずつ読んで処理するなら、

file_source_body=open(myFileName, "r")
while text=file_source_body.gets do

じゃなくて、

IO.foreach(myFileName) do |text|

ですかね。明示的にcloseしないといけない書き方は避けられるなら避けるほうがいいです。closeするなら、begin ～ ensure ～ end を使う。

(/#{myTargetName}\s(\S+)/i=~ref_info) && (realName=$1)

は、その上と同じパターンで if のほうがいいと思うけど、if を使わないなら、優先度の低い and を使って、

/#{myTargetName}\s(\S+)/i=~ref_info and realName=$1

お礼

notnotさん、ありがとうございます。
下記の２つのスクリプトは、同じであることを、理解しました。
小生の持っている本（たのしいRuby　第２版、Rubyではじめる気軽なプログラミング）
には、IO.foreach(myFileName){|text|・・・・は書かれていないと思います。
他に、本を買うとしたら、どの本が良いでしょうか？

------------------------
myFileName="test.csv"
IO.foreach(myFileName){|text|
text.chomp!
print text,"\n"
}
-------------------------
io=open(myFileName,"r")
io.each{|text|
text.chomp!
print text,"\n"
}
-------------------------

（２）(/#{myTargetName}\s(\S+)/i=~ref_info) && (realName=$1)
に関しては、Perlで、覚えたやり方で、Rubyでも通用することを理解し、
やはり、これからも使ってしまうと思います。
&& (andでも構いませんが)で、いくつもつなぐ事で、見た目は良くないですが、
複雑な条件が、１行で記述できて、結構、スクリプトがすっきりしてしまいます。

回答No.1

koko_u_

>Rubyらしい書き方にするには、どこを改めればよいですか？
さぁ？ open を差し替えたいなら、IO.foreach(path) { |line| ... } を使うくらいしか。

>関数呼び出しでやってしまいたい。どのように組みなおせば良いですか？
単純なところでは

class Reference
　def initialize(ref_file)
　　@ref_hash = {}
　　...
　end

　def [](key)
　　@ref_hash[key]
　end
end

みたいなのをデッチ上げて、最初に myReference.txt をまるごと読み込む。

補足

名前を変換するのを、ハッシュメモリを使ってやってしまうというのは、
「なるほど！！！」、感心しました。今まで、ハッシュと言うものは聞いた
ことがありましたが、こういうふうに使えるとは、知りませんでした。

本を買ってきて、Classを読んでおります。
今一、理解しづらいところです。

class Reference
　def initialize(ref_file)
　　@ref_hash = {}
　　...
　end

　def [](key)
　　@ref_hash[key]
　end
end

－－－－－－－
（１）仮に、myReference.txt が次のようなファイルであるとして、

aaaa,AAAA
bbbb,BBBB
cccc,CCCC

－－－－－－－－－
@ref_hash = {}
この表現で、ハッシュメモリ（３つ）に、入ってくれるものでしょうか？
（確認しようとして、やってはいるのですが、如何せん、呼び出し方がまだ、分からず・・・

（２）このクラスのメッソドとしての　def [](key)
メッソドの名称を仮に、nameCnvertとすると、
def nameCnvert(key)
@ref_hash[key]
end

これで、良いでしょうか？

（３）このメッソドの呼び出し方は?
nameCnvertは、メッソドであるゆえ、オブジェクトが必要・・・しかし、
何を持ってくればよいのか?

大変申し訳ない。宜しくお願いいたします。