Changeset 11128 in josm for trunk


Ignore:
Timestamp:
2016-10-14T21:29:40+02:00 (3 years ago)
Author:
simon04
Message:

see #13798 - Refactoring: better variable names

No functional change.

File:
1 edited

Legend:

Unmodified
Added
Removed
  • trunk/src/org/openstreetmap/josm/gui/dialogs/validator/ValidatorTreePanel.java

    r11127 r11128  
    1414import java.util.List;
    1515import java.util.Map;
    16 import java.util.Map.Entry;
    1716import java.util.Set;
    1817import java.util.function.Predicate;
     
    178177            filterToUse = filterToUse.and(e -> e.getPrimitives().stream().anyMatch(filter::contains));
    179178        }
    180         Map<Severity, Map<String, Map<String, List<TestError>>>> errorTreeDeep
     179        Map<Severity, Map<String, Map<String, List<TestError>>>> errorsBySeverityMessageDescription
    181180            = errors.stream().filter(filterToUse).collect(
    182181                    Collectors.groupingBy(TestError::getSeverity, () -> new EnumMap<>(Severity.class),
     
    185184                                    ))));
    186185
    187         List<TreePath> expandedPaths = new ArrayList<>();
    188         errorTreeDeep.forEach((s, severityErrorsDeep) -> {
     186        final List<TreePath> expandedPaths = new ArrayList<>();
     187        errorsBySeverityMessageDescription.forEach((severity, errorsByMessageDescription) -> {
    189188            // Severity node
    190             DefaultMutableTreeNode severityNode = new GroupTreeNode(s);
     189            final DefaultMutableTreeNode severityNode = new GroupTreeNode(severity);
    191190            rootNode.add(severityNode);
    192191
    193             if (oldSelectedRows.contains(s)) {
     192            if (oldSelectedRows.contains(severity)) {
    194193                expandedPaths.add(new TreePath(new Object[] {rootNode, severityNode}));
    195194            }
    196195
    197             Map<String, List<TestError>> severityErrors = severityErrorsDeep.get("");
    198             if (severityErrors != null) {
    199                 for (Entry<String, List<TestError>> msgErrors : severityErrors.entrySet()) {
    200                     // Message node
    201                     List<TestError> errs = msgErrors.getValue();
    202                     String msg = tr("{0} ({1})", msgErrors.getKey(), errs.size());
    203                     DefaultMutableTreeNode messageNode = new DefaultMutableTreeNode(msg);
     196            final Map<String, List<TestError>> errorsWithEmptyMessageByDescription = errorsByMessageDescription.get("");
     197            if (errorsWithEmptyMessageByDescription != null) {
     198                errorsWithEmptyMessageByDescription.forEach((description, errors) -> {
     199                    final String msg = tr("{0} ({1})", description, errors.size());
     200                    final DefaultMutableTreeNode messageNode = new DefaultMutableTreeNode(msg);
    204201                    severityNode.add(messageNode);
    205202
    206                     if (oldSelectedRows.contains(msgErrors.getKey())) {
     203                    if (oldSelectedRows.contains(description)) {
    207204                        expandedPaths.add(new TreePath(new Object[] {rootNode, severityNode, messageNode}));
    208205                    }
    209206
    210                     errs.stream().map(DefaultMutableTreeNode::new).forEach(messageNode::add);
    211                 }
    212             }
    213 
    214             severityErrorsDeep.forEach((description, errorlist) -> {
    215                 if (description.isEmpty()) {
     207                    errors.stream().map(DefaultMutableTreeNode::new).forEach(messageNode::add);
     208                });
     209            }
     210
     211            errorsByMessageDescription.forEach((message, errorsByDescription) -> {
     212                if (message.isEmpty()) {
    216213                    return;
    217214                }
    218215                // Group node
    219                 DefaultMutableTreeNode groupNode;
    220                 if (errorlist.size() > 1) {
    221                     groupNode = new GroupTreeNode(description);
     216                final DefaultMutableTreeNode groupNode;
     217                if (errorsByDescription.size() > 1) {
     218                    groupNode = new GroupTreeNode(message);
    222219                    severityNode.add(groupNode);
    223                     if (oldSelectedRows.contains(description)) {
     220                    if (oldSelectedRows.contains(message)) {
    224221                        expandedPaths.add(new TreePath(new Object[] {rootNode, severityNode, groupNode}));
    225222                    }
     
    228225                }
    229226
    230                 errorlist.forEach((message, errs) -> {
     227                errorsByDescription.forEach((description, errors) -> {
    231228                    // Message node
    232                     String msg;
     229                    final String msg;
    233230                    if (groupNode != null) {
    234                         msg = tr("{0} ({1})", message, errs.size());
     231                        msg = tr("{0} ({1})", description, errors.size());
    235232                    } else {
    236                         msg = tr("{0} - {1} ({2})", message, description, errs.size());
    237                     }
    238                     DefaultMutableTreeNode messageNode = new DefaultMutableTreeNode(msg);
     233                        msg = tr("{0} - {1} ({2})", message, description, errors.size());
     234                    }
     235                    final DefaultMutableTreeNode messageNode = new DefaultMutableTreeNode(msg);
    239236                    if (groupNode != null) {
    240237                        groupNode.add(messageNode);
     
    243240                    }
    244241
    245                     if (oldSelectedRows.contains(message)) {
     242                    if (oldSelectedRows.contains(description)) {
    246243                        if (groupNode != null) {
    247244                            expandedPaths.add(new TreePath(new Object[] {rootNode, severityNode, groupNode, messageNode}));
     
    251248                    }
    252249
    253                     errs.stream().map(DefaultMutableTreeNode::new).forEach(messageNode::add);
     250                    errors.stream().map(DefaultMutableTreeNode::new).forEach(messageNode::add);
    254251                });
    255252            });
Note: See TracChangeset for help on using the changeset viewer.