«

I often make use of this idiom to add something to an array in a hash of arrays:

@categories = {}

pagelist.each do |resource|
@categories[resource.tag] ||= []
@categories[resource.tag] << resource
end


.. is there a better way? Can the two @categories lines
be made into one?


Les

Leslie Viljoen wrote:
> @categories =3D {}
>
> pagelist.each do |resource|
> =A0 =A0 @categories[resource.tag] ||=3D []
> =A0 =A0 @categories[resource.tag] << resource
> end
>
>
> .. is there a better way?

@categories =3D Hash.new {Array.new}
pagelist.each do |resource|
@categories[resource.tag] << resource
end



=2D-=20
Jabber:
ICQ: 205544826

On Wed, 20 Feb 2008 07:11:31 +0900, "Leslie Viljoen" <> wrote:
> I often make use of this idiom to add something to an array in a hash of
> arrays:
>
> @categories = {}
>
> pagelist.each do |resource|
> @categories[resource.tag] ||= []
> @categories[resource.tag] << resource
> end
>
> .. is there a better way? Can the two @categories lines
> be made into one?

I sometimes do this sort of thing:

( @categories[resource.tag] ||= [] ) << resource

Although this might possibly be more readable:

category = ( @categories[resource.tag] ||= [] )
category << resource

Often it can make sense to factor the ||= portion into a
separate method:

def category_for(tag)
@categories[tag] ||= []
end

# ...

category_for(resource.tag) << resource

-mental

Sebastian Hungerecker wrote:
> Leslie Viljoen wrote:
>> @categories = {}
>>
>> pagelist.each do |resource|
>> @categories[resource.tag] ||= []
>> @categories[resource.tag] << resource
>> end
>>
>>
>> .. is there a better way?
>
> @categories = Hash.new {Array.new}
> pagelist.each do |resource|
> @categories[resource.tag] << resource
> end

That one still bites me from time to time...

irb(main):001:0> h = Hash.new {Array.new}
=> {}
irb(main):002:0> h[4]
=> []
irb(main):003:0> h[4] << 5
=> [5]
irb(main):004:0> h
=> {}


Try this...

irb(main):005:0> h = Hash.new {|h,k| h[k]=Array.new}
=> {}
irb(main):006:0> h[4] << 5
=> [5]
irb(main):007:0> h
=> {4=>[5]}

--
vjoel : Joel VanderWerf : path berkeley edu : 510 665 3407

On Feb 19, 4:11 pm, Leslie Viljoen <leslievilj...@gmail.com> wrote:
> I often make use of this idiom to add something to an array in a hash of arrays:
>
> @categories = {}
>
> pagelist.each do |resource|
> @categories[resource.tag] ||= []
> @categories[resource.tag] << resource
> end
>
> . is there a better way? Can the two @categories lines
> be made into one?
>
> Les

irb(main):001:0> h = Hash.new { |hash, key| hash[key] =[] }
=> {}
irb(main):002:0> h['foo'] << 33
=> [33]
irb(main):003:0> h['foo'] << 44
=> [33, 44]
irb(main):004:0> h
=> {"foo"=>[33, 44]}

On Wed, 20 Feb 2008 07:15:52 +0900, Sebastian Hungerecker <> wrote:
>> .. is there a better way?
>
> @categories = Hash.new {Array.new}
> pagelist.each do |resource|
> @categories[resource.tag] << resource
> end

Although it is specific to hashes, when this is applicable I do
think it can be a better approach than the more general approach
I outlined in my other email.

-mental

On Wed, Feb 20, 2008 at 07:11:31AM +0900, Leslie Viljoen wrote:
> I often make use of this idiom to add something to an array in a hash of arrays:
>
> @categories = {}
>
> pagelist.each do |resource|
> @categories[resource.tag] ||= []
> @categories[resource.tag] << resource
> end

@categories = {}

pagelist.each do |resource|
(@categories[resource.tag] ||= []) << resource
end

or

@categories = Hash.new {|h, k| h[k] = []}

pagelist.each do |resource|
@categories[resource.tag] << resource
end

--
Jos Backus
jos at catnook.com

[Note: parts of this message were removed to make it a legal post.]

On 2/19/08, Leslie Viljoen <> wrote:
>
> I often make use of this idiom to add something to an array in a hash of
> arrays:
>
> @categories = {}
>
> pagelist.each do |resource|
> @categories[resource.tag] ||= []
> @categories[resource.tag] << resource
> end
>
>
> .. is there a better way? Can the two @categories lines
> be made into one?


@categories = {}

pagelist.each do |resource|
(@catetegories[resource.tag] ||= []) << resource
end



Christopher

MenTaLguY wrote:
> Often it can make sense to factor the ||= portion into a
> separate method:
>
> def category_for(tag)
> @categories[tag] ||= []
> end
>
> # ...
>
> category_for(resource.tag) << resource

That's something I've used frequently. It's kind of a poor man's
dependency injection.

--
vjoel : Joel VanderWerf : path berkeley edu : 510 665 3407

If you don't mind being really cryptic about it:

@categories = resource.inject({}) {|a, v| (a[v.tag] ||= []) << v; a}

I think this works, but I'm not sure the "next guy" would ever figure
out what it does

On Feb 19, 2008, at 2:11 PM, Leslie Viljoen wrote:

> I often make use of this idiom to add something to an array in a
> hash of arrays:
>
> @categories = {}
>
> pagelist.each do |resource|
> @categories[resource.tag] ||= []
> @categories[resource.tag] << resource
> end
>
>
> .. is there a better way? Can the two @categories lines
> be made into one?
>
>
> Les

If you don't mind being really cryptic about it:

@categories = resource.inject({}) {|a, v| (a[v.tag] ||= []) << v; a}

I think this works, but I'm not sure the "next guy" would ever figure
out what it does