groovy

@ToString optionally exclude fields with null values

Details

  • Type: Improvement Improvement
  • Status: Closed Closed
  • Priority: Trivial Trivial
  • Resolution: Fixed
  • Affects Version/s: None
  • Fix Version/s: 2.0-beta-3
  • Component/s: None
  • Labels:
    None
  • Number of attachments :
    0

Description

Particularly on objects with a high number of fields, having the ability to exclude fields which are null from the output of the generated toString would be a nice to have.

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Paul King added a comment -

There is a bit of a hack which might work for you now as a workaround depending on your circumstances (shown for Immutable):

import groovy.transform.Immutable
@Immutable class Name {
  String title, firstname, surname
  String toString() {
    _toString().replaceAll(/(, )?[a-z]*:null(, )?/, '')
  }
}

println new Name(firstname: 'Cher')
Show
Paul King added a comment - There is a bit of a hack which might work for you now as a workaround depending on your circumstances (shown for Immutable): import groovy.transform.Immutable @Immutable class Name { String title, firstname, surname String toString() { _toString().replaceAll(/(, )?[a-z]*: null (, )?/, '') } } println new Name(firstname: 'Cher')
Hide
Permalink
Andre Steingress added a comment -

Is someone working on this issue? Otherwise I'd be willing to contribute

Show
Andre Steingress added a comment - Is someone working on this issue? Otherwise I'd be willing to contribute
Hide
Permalink
Andre Steingress added a comment -

issued a pull request: https://github.com/groovy/groovy-core/pull/26

Show
Andre Steingress added a comment - issued a pull request: https://github.com/groovy/groovy-core/pull/26
Hide
Permalink
Paul King added a comment -

Wondering whether "ignoreNulls" is good enough instead of "ignoreNullValues"?

Show
Paul King added a comment - Wondering whether "ignoreNulls" is good enough instead of "ignoreNullValues"?
Hide
Permalink
Paul King added a comment -

Thanks Andre, applied your patch with a few tweaks (I noticed that some other earlier changes that were supposed to be there for 2.0 hadn't made it - think I had them on a different branch that wasn't committed at some point - anyway fixed now - but the earlier change was the main cause of the tweaks)

Show
Paul King added a comment - Thanks Andre, applied your patch with a few tweaks (I noticed that some other earlier changes that were supposed to be there for 2.0 hadn't made it - think I had them on a different branch that wasn't committed at some point - anyway fixed now - but the earlier change was the main cause of the tweaks)

People

  • Assignee:
    Paul King
    Reporter:
    Michael Bishop
Vote (0)
Watch (1)

Dates

  • Created:
    Updated:
    Resolved: