Asp Forum - Inheriting from / Delegating to Integer - Currency class implementation

Pascal Ehlert

2/9/2008 12:35:00 PM

Howdy folks.

I'm currently working on a Currency class which I'd like to behave
exactly like Integer with some additional methods like #to_currency and
and overridden #to_s method etc.

My first approach was to inherit from Integer directly which failed,
because there is no constructor for this class. After some more
research I found the DelegateClass method which seemed to be exactly
what I need, however there was one major problem with it: Methods like
#+ and #- returned Integer, not Currency objects. So "(Currency.new(20)
+ Currency.new(30)).class" evaled to Integer and my custom methods have
been gone.

The latest approach was not to inherit at all but write the class from
scratch, carrying around a @value instance variable which stores an
Integer representation of the currency.
Unfortunately that idea didn't work to good either because I couldn't
add Currency objects to Fixnums anymore, "20 + Currency.new(30)" raised
a TypeError ("Currency can't be coerced into Fixnum"), although my
class implements a #to_i method.

I actually like best the second way, using DelegateClass.
Are there any ways to make the calculation methods return a Currency
object then?
Or do you at least have an idea how to solve the problem with the
latest approach?

Thank you

-- Pascal

7 Answers

Rick DeNatale

2/9/2008 1:54:00 PM

On 2/9/08, Pascal Ehlert <pascal.ehlert@odadata.eu> wrote:
> Howdy folks.
>
> I'm currently working on a Currency class which I'd like to behave
> exactly like Integer with some additional methods like #to_currency and
> and overridden #to_s method etc.
>
> My first approach was to inherit from Integer directly which failed,
> because there is no constructor for this class. After some more
> research I found the DelegateClass method which seemed to be exactly
> what I need, however there was one major problem with it: Methods like
> #+ and #- returned Integer, not Currency objects. So "(Currency.new(20)
> + Currency.new(30)).class" evaled to Integer and my custom methods have
> been gone.
>
> The latest approach was not to inherit at all but write the class from
> scratch, carrying around a @value instance variable which stores an
> Integer representation of the currency.
> Unfortunately that idea didn't work to good either because I couldn't
> add Currency objects to Fixnums anymore, "20 + Currency.new(30)" raised
> a TypeError ("Currency can't be coerced into Fixnum"), although my
> class implements a #to_i method.
>
> I actually like best the second way, using DelegateClass.
> Are there any ways to make the calculation methods return a Currency
> object then?
> Or do you at least have an idea how to solve the problem with the
> latest approach?
class Currency

attr_reader :base_value

def initialize(value)
@base_value = value
end

def +(other)
Currency.new(@base_value + other.to_currency.base_value)
end

def to_currency
self
end

def coerce(other)
[self, other.to_currency]
end

end

class Numeric
def to_currency
Currency.new(self)
end
end

Currency.new(10) + Currency.new(5) # => #<Currency:0x23ab4 @base_value=15>
10 + Currency.new(5) # => #<Currency:0x238e8 @base_value=15>

HTH

--
Rick DeNatale

My blog on Ruby
http://talklikeaduck.denh...

Pascal Ehlert

2/9/2008 2:25:00 PM

On 2/9/2008 Rick DeNatale wrote:
> def coerce(other)
> [self, other.to_currency]
> end

Whoa, that looks really interesting.
Thanks a lot, Rick!

However, is there another way to implement the Currency class using
DelegateClass?
It still feels wrong to me to write +, -, * and / on my own, although
they are exactly the same as in the Integer class.

-- Pascal

Robert Klemme

2/9/2008 5:27:00 PM

On 09.02.2008 15:25, Pascal Ehlert wrote:
> On 2/9/2008 Rick DeNatale wrote:
>> def coerce(other)
>> [self, other.to_currency]
>> end
>
> Whoa, that looks really interesting.
> Thanks a lot, Rick!
>
> However, is there another way to implement the Currency class using
> DelegateClass?

There might be but you will have to handle type differences anyway. You
will have to implement #coerce, #+ etc. in any case. I recommend to use
Rick's approach because it is much cleaner.

> It still feels wrong to me to write +, -, * and / on my own, although
> they are exactly the same as in the Integer class.

Actually, they are not the same. Adding currencies is significantly
more complex than adding ints. What happens if you add 5 EUR and 12
USD? All sorts of issues are lurking here. A currency value is (at
least) a decimal value and a currency identifier - at least this is the
most straightforward definition I can think of. You are doing yourself
a favor by doing it properly - it's not that difficult.

Kind regards

robert

Alex Shulgin

2/10/2008 9:27:00 AM

On Feb 9, 4:25 pm, Pascal Ehlert <pascal.ehl...@odadata.eu> wrote:
> On 2/9/2008 Rick DeNatale wrote:
>
> > def coerce(other)
> > [self, other.to_currency]
> > end
>
> Whoa, that looks really interesting.
> Thanks a lot, Rick!
>
> However, is there another way to implement the Currency class using
> DelegateClass?
> It still feels wrong to me to write +, -, * and / on my own, although
> they are exactly the same as in the Integer class.

Huh?.. And what is the result for $5 divided by $1? ;)

--
Cheers,
Alex

Rick DeNatale

2/10/2008 6:28:00 PM

On 2/10/08, Alex Shulgin <alex.shulgin@gmail.com> wrote:
> On Feb 9, 4:25 pm, Pascal Ehlert <pascal.ehl...@odadata.eu> wrote:
> > On 2/9/2008 Rick DeNatale wrote:
> >
> > > def coerce(other)
> > > [self, other.to_currency]
> > > end
> >
> > Whoa, that looks really interesting.
> > Thanks a lot, Rick!
> >
> > However, is there another way to implement the Currency class using
> > DelegateClass?
> > It still feels wrong to me to write +, -, * and / on my own, although
> > they are exactly the same as in the Integer class.
>
> Huh?.. And what is the result for $5 divided by $1? ;)

What, do you want me to do all of the OP's work for him for free?

--
Rick DeNatale

My blog on Ruby
http://talklikeaduck.denh...

Rick DeNatale

2/10/2008 6:33:00 PM

On 2/10/08, Rick DeNatale <rick.denatale@gmail.com> wrote:
> On 2/10/08, Alex Shulgin <alex.shulgin@gmail.com> wrote:
they are exactly the same as in the Integer class.
> >
> > Huh?.. And what is the result for $5 divided by $1? ;)
>
> What, do you want me to do all of the OP's work for him for free?

Actually, I rather blithely skipped over the real point in Alex's question.

Although my code shows how to do coercion via double dispatching in
Ruby, it ignores the domain question of what does it mean to add an
integer to a currency.

$5 + 1 what?

It's not clear that an integers and currency really SHOULD be
compatible in this way.

--
Rick DeNatale

My blog on Ruby
http://talklikeaduck.denh...

Stefan Rusterholz

2/10/2008 9:21:00 PM

Alex Shulgin wrote:
> On Feb 9, 4:25ï¿½pm, Pascal Ehlert <pascal.ehl...@odadata.eu> wrote:
>> DelegateClass?
>> It still feels wrong to me to write +, -, * and / on my own, although
>> they are exactly the same as in the Integer class.
>
> Huh?.. And what is the result for $5 divided by $1? ;)

Hrm, I don't know why my answer via google didn't show up. Maybe a
mistake on my part. So here again:
That would be a mathematically sound: 5.
If you need an example to see that this even makes sense, consider: You
have $5 in $1 pieces. How many $1 pieces do you have? Answer: $5/$1 = 5.

With $1*$5 it's a bit different as powers other than 0 and 1 of $ as I
can't find anything where that'd make sense :)

Regards
Stefan
--
Posted via http://www.ruby-....

comp.lang.ruby

Inheriting from / Delegating to Integer - Currency class implementation

Pascal Ehlert

Rick DeNatale

Pascal Ehlert

Robert Klemme

Alex Shulgin

Rick DeNatale

Rick DeNatale

Stefan Rusterholz

x Login to ForumsZone