From: Benoit Daloze <eregontp@...>
Date: 2010-04-08T01:21:04+09:00
Subject: [ruby-core:29305] Re: Proposal: Module#thunk_method

--00163646b964b008920483a7f268
Content-Type: text/plain; charset=ISO-8859-1

Hi,

I think as Joshua, it's a very bad idea to do something like that.

Memoizing is something very quick to write in Ruby, and should *never* use
define_method.

Either use a constant when possible, or a regular method if you need to
evaluate it later.

I think these simple cases are really too less and should be avoided how
they are.

class Foo
 def value
   "some value" # Why would you use defined_method ?
  end
end

def value
  defined?(@a) ? @a : some_expensive_calculation
end

Any problem to this case ?

I saw the case over-complicated with the second rails link,
I think it can be improved by eval, that's the usual way to deal with the
"slow" define_method.

Regards,
B.D.

--00163646b964b008920483a7f268
Content-Type: text/html; charset=ISO-8859-1
Content-Transfer-Encoding: quoted-printable

Hi,<div><br></div><div>I think as Joshua, it&#39;s a very bad idea to do so=
mething like that.</div><div><br></div><meta charset=3D"utf-8"><div><span c=
lass=3D"Apple-style-span" style=3D"font-family: arial, sans-serif; font-siz=
e: 13px; border-collapse: collapse; ">Memoizing</span>=A0is something very =
quick to write in Ruby, and should *never* use define_method.</div>

<div><br></div><div>Either use a constant when possible, or a regular metho=
d if you need to evaluate it later.</div><div><br></div><div>I think these =
simple cases are really too less and should be avoided how they are.</div>

<div><br></div><div><span class=3D"Apple-style-span" style=3D"font-family: =
arial, sans-serif; font-size: 13px; border-collapse: collapse; ">class Foo<=
br>=A0def value</span></div><div><span class=3D"Apple-style-span" style=3D"=
font-family: arial, sans-serif; font-size: 13px; border-collapse: collapse;=
 ">=A0=A0 &quot;some value&quot; # Why would you use defined_method ?</span=
></div>

<div><span class=3D"Apple-style-span" style=3D"font-family: arial, sans-ser=
if; font-size: 13px; border-collapse: collapse; ">=A0=A0end<br>end<br><br>d=
ef value</span></div><div><span class=3D"Apple-style-span" style=3D"font-fa=
mily: arial, sans-serif; font-size: 13px; border-collapse: collapse; ">=A0=
=A0defined?(@a) ? @a : some_expensive_calculation<br>

end</span></div><div><span class=3D"Apple-style-span" style=3D"font-family:=
 arial, sans-serif; font-size: 13px; border-collapse: collapse; "><br></spa=
n></div><div><span class=3D"Apple-style-span" style=3D"font-family: arial, =
sans-serif; font-size: 13px; border-collapse: collapse; ">Any problem to th=
is case ?</span></div>

<div><br></div><div>I saw the case over-complicated with the second rails l=
ink,</div><div>I think it can be improved by eval, that&#39;s the usual way=
 to deal with the &quot;slow&quot; define_method.=A0</div><div><br></div>

<div>Regards,</div><div>B.D.</div>

--00163646b964b008920483a7f268--