メソッド内で、クラス new するのは良くない？

Question

自作の下記コード(抜粋)中のメソッド
get_text_from_page_source_xml
が呼びだされる度に、
REXML::Document.new(xml).new
してしまうので、
その都度、インスタンスが作成されるのは、良くないのでは？

と思いつつも...

メッソド get_text_from_page_source_xml 内なので、
ローカル変数同様、.new された変数もローカル変数と同様に
メッソドの実行が終われば、その領域も破棄される

ので、問題ない？

という風に理解していますが、正しいでしょうか？

xml の element を抽出するには、
REXML::Document.new(xml)
xml ソースを引数にして .new しないとできないし...

のように、

自分の理解の整理がいまひとつできないので、ご教示いただけますでしょうか？

■ 自作コード(抜粋)

require 'rexml/document'

def get_text_from_page_source_xml(**resource_ids)

  xml = @cbelement.page_source
  doc = REXML::Document.new(xml)

  resource_ids.each do |rid,v|
    arid  = "com.aspiro.tidal:id/" + rid.to_s # arid: actual resource id
    xpath = "//*[@resource-id='#{arid}']" # need ' at before and end #{id}

    begin
      text = doc.elements[xpath].attributes["text"]
      resource_ids[rid] = text
    rescue => e
      resource_ids[rid] = nil
    end
  end

  return resource_ids
end

id0 = :artistName
id1 = :mediaItemTitle
texts = get_text_from_page_source_xml(id0 => :nil, id1 => :nil)

Comments

[rhiroe]

メソッドの中でクラス.newがダメ！となれば、
文字列、配列、ハッシュ、日付、何を生成するのも
全てダメということになってしまいますね...。

[roswell]

ご教示いただきたいところは

>呼びだされる度に、
>REXML::Document.new(xml).new
>してしまうので、
>その都度、インスタンスが作成されるのは

なのですが、そこは問題ないでしょうか？

Nokogiri を用いれば
doc = Nokogiri::XML(xml)
.new しなくて書けます。

標準の REXML::Document だと、
doc = REXML::Document.new(xml)
で、毎度 .new されるので気になっています。

Answer 1

気にせず new して下さい。
doc = Nokogiri::XML(xml)　でnewが無いようにみえますが、Nokogiri内部でnewしてくれているだけです。

Mugheart　さんの言うとおり、
method内でnewし無い方がよいとなると、何もできないです

Comments

[roswell]

ご教示ありがとうございます。

> method内でnewし無い方がよいとなると、何もできないです
は、理解はしているのですが...

使いまわしでよさそうなので、1回 new して、そのインスタンスを用いて
と思っていましたが、引数に毎度 (xml) を渡さなければならないので、その
都度 new されるのが気になっていました。

REXML::Document.new のマニュアル
https://docs.ruby-lang.org/ja/latest/method/REXML=3a=3aDocument/s/new.html
みると、Singleton method になっているので、
問題なさそうだとの認識に至りました。

[winterboum]

＞　みると、Singleton method になっているので、問題なさそうだとの認識に至りました。
なんだ、かなりの使い手ではないですか

[roswell]

> なんだ、かなりの使い手ではないですか

頻繁にcallされる method で、毎度.new されるとその度に、
インスタンスが作られる。
渡す xml の内容も変化するので、
doc = REXML::Document.new(xml)
を呼び出す必要があるし...
と頭が整理できなかったので、質問に至りました。

アドバイスありがとうございます。