groovy

Map or Property Sort

Details

  • Type: New Feature New Feature
  • Status: Closed Closed
  • Priority: Minor Minor
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 1.7.2, 1.8-beta-1
  • Component/s: groovy-jdk
  • Labels:
    None
  • Environment:
    All
  • Number of attachments :
    0

Description

It would be handy to have a sort method on a Map or Property that would sort by the name/key since this is something that is commonly done. A possible implementation for a Property class might look like this:

Properties.metaClass.sort = {
new TreeMap(delegate)
}

Example usage would be:

System.properties.sort()

Issue Links

Activity

Hide
blackdrag blackdrag added a comment -

you mean

System.properties.sort()

vs.

new TreeMap(System.properties)

that's all you suggest, right? In general there is a problem with sorting a map and that is... is it sorting by key, or by value, with what Comperator? Depending on the usage each case make in itself make sense. But your suggestions does not reflect any of this.

Show
blackdrag blackdrag added a comment - you mean
System.properties.sort()
vs.
new TreeMap(System.properties)
that's all you suggest, right? In general there is a problem with sorting a map and that is... is it sorting by key, or by value, with what Comperator? Depending on the usage each case make in itself make sense. But your suggestions does not reflect any of this.
Hide
Christopher Judd added a comment -

Yes. I would rather type System.properties.sort() to sort. An overloaded sort method that takes a Comperator would be a good idea as well. But at least for Properties a default of ordering by the name in alphabetical order make sense.

Show
Christopher Judd added a comment - Yes. I would rather type System.properties.sort() to sort. An overloaded sort method that takes a Comperator would be a good idea as well. But at least for Properties a default of ordering by the name in alphabetical order make sense.
Hide
Paul King added a comment -

Just noticed that there is already support for this:

System.properties.sort{ it.key }
Show
Paul King added a comment - Just noticed that there is already support for this:
System.properties.sort{ it.key }
Hide
Paul King added a comment -

Added sort() and sort(Comparator) variants.

Show
Paul King added a comment - Added sort() and sort(Comparator) variants.

People

Vote (0)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: