Ignore:
Timestamp:
2017-02-12T16:32:18+01:00 (3 years ago)
Author:
Don-vip
Message:

refactor handling of null values - use Java 8 Optional where possible

Location:
trunk/src/org/openstreetmap/josm/data/preferences
Files:
3 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/data/preferences/AbstractProperty.java

    r11496 r11553  
    3838            if (this == obj)
    3939                return true;
    40             if (obj == null)
    41                 return false;
    42             if (getClass() != obj.getClass())
     40            if (obj == null || getClass() != obj.getClass())
    4341                return false;
    4442            @SuppressWarnings("unchecked")
     
    285283        if (this == obj)
    286284            return true;
    287         if (obj == null)
    288             return false;
    289         if (getClass() != obj.getClass())
     285        if (obj == null || getClass() != obj.getClass())
    290286            return false;
    291287        AbstractProperty<?> other = (AbstractProperty<?>) obj;
  • trunk/src/org/openstreetmap/josm/data/preferences/PreferencesReader.java

    r11453 r11553  
    1616import java.util.List;
    1717import java.util.Map;
     18import java.util.Optional;
    1819import java.util.SortedMap;
    1920import java.util.TreeMap;
     
    173174                        setting = new StringSetting(null);
    174175                    } else {
    175                         String value = parser.getAttributeValue(null, "value");
    176                         if (value == null) {
    177                             throw new XMLStreamException(tr("value expected"), parser.getLocation());
    178                         }
    179                         setting = new StringSetting(value);
     176                        setting = new StringSetting(Optional.ofNullable(parser.getAttributeValue(null, "value"))
     177                                .orElseThrow(() -> new XMLStreamException(tr("value expected"), parser.getLocation())));
    180178                    }
    181179                    if (defaults) {
  • trunk/src/org/openstreetmap/josm/data/preferences/PreferencesWriter.java

    r10824 r11553  
    66import java.util.List;
    77import java.util.Map;
     8import java.util.Optional;
    89import java.util.stream.Stream;
    910
     
    6869    private void addTime(Setting<?> setting) {
    6970        if (defaults) {
    70             Long time = setting.getTime();
    71             if (time == null)
    72                 throw new IllegalStateException();
    73             out.write("' time='" + time);
     71            out.write("' time='" + Optional.ofNullable(setting.getTime()).orElseThrow(IllegalStateException::new));
    7472        }
    7573    }
Note: See TracChangeset for help on using the changeset viewer.