In article <[EMAIL PROTECTED]>, [EMAIL PROTECTED] (Rob Richardson) writes: >--- Peter Scott <[EMAIL PROTECTED]> wrote: >> In article <[EMAIL PROTECTED]>, >> [EMAIL PROTECTED] (Rob Richardson) writes: >> >sub GetTrainsByTime >> >{ >> > my $self = shift; >> > my @result; >> > my @temp; >> > my $aTrain; >> > my $aTrainName; >> > >> > # First we build a temporary array of references to trains >> > foreach $aTrainName (keys $self) >> ^ % missing >> > { >> > push @temp, $self{aTrainName}; >> ^ $ missing >> >> > } >> > >> > @result = sort {$a->GetCrewCall() cmp $b->GetCrewCall()} @temp; >> > return @result; >> >} >> >> (1) If you wanted to put the keys of a hash into an array, just do it >> all at once: >> >> @temp = keys %$self > >I am not putting the keys of the hash into the temporary array. I'm >putting the values of the hash into the array.
Whoops, okay, then do @temp = values %$self >> (2) If you want to sort the keys of a hash, there's no need to put >> them into an array. sort takes a list as input: >> >> return sort { $a->GetCrewCall cmp $b->GetCrewCall } keys %self >> > >Since I am not using the keys of the hash, the above line won't work >for me. return sort { $a->GetCrewCall cmp $b->GetCrewCall } values %self -- Peter Scott http://www.perldebugged.com -- To unsubscribe, e-mail: [EMAIL PROTECTED] For additional commands, e-mail: [EMAIL PROTECTED]