in reply to Perl OO - Class Data

In general, you can't use objects as keys to a hash.

Given your current usage pattern, I would suggest reversing everything and, instead of $references->{$objref} = "value", try: 

  $references->{$objref->{_team}}{$objref->{_name}} = $objref;

[download]
That's assuming that no two objects can have the same name. If that's not the case, perhaps you really want arrays: 
  push @{$references->{$objref->{_team}}}, $objref;

[download]
Both examples here split everything up by team so that you can just skip over an entire hash or array rather than skipping over individual team members.

(This is in addition to japhy's advice to use the same variable name to refer to the same object ;-})

Replies are listed 'Best First'.
Re^2: Perl OO - Class Data
by dsheroh (Monsignor) on Jun 26, 2006 at 16:50 UTC
    In general, you can't use objects as keys to a hash.

    Just to expand on that a little...

    If you attempt to use an object as a hash key, Perl will convert the reference to a text string and use that as the actual hash key. While this is quite handy for generating unique IDs, it also means that the keys of the hash are no longer usable as objects. (This applies to all references, actually, not just objects.)

    So using keys %ds to retrieve your User objects can't work. However, if you construct %ds with $ds{$objref} = $objref (the posted code doesn't put any data into %ds, so I don't know whether that's how you're doing it or not), then you should be able to retrieve all your objects using values %ds in your foreach.

    But it also looks like your test sub, in addition to only looking for Users on the opposing team, is also going against every one of them rather than a random one. If you want randomness, then you'll probably want to do something like 

    my @objlist = values %ds;
my $randobj = $objlist[rand @objlist];

    [download]
[reply]
[d/l]
[select]
      That cleared up a lot of confusion for me. Thanks!!!
[reply]

In Section Seekers of Perl Wisdom