Skip to content

Commit e76c6eb

Browse files
author
Tomas Korima
committed
functioning null-safe code
1 parent 5aba3d0 commit e76c6eb

1 file changed

Lines changed: 16 additions & 8 deletions

File tree

core/src/main/java/de/tesis/dynaware/grapheditor/core/selections/SelectionDragManager.java

Lines changed: 16 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -275,8 +275,10 @@ private void addPositionListeners(final Region master, final GModel model) {
275275
if (nodeSkin.isSelected() && !nodeSkin.getRoot().equals(master)) {
276276

277277
final Region slave1 = nodeSkin.getRoot();
278-
double offset = Optional.ofNullable(nodeLayoutXOffsets.get(node)).orElse(0d);
279-
slave1.setLayoutX((Double) n + offset);
278+
Double offset = nodeLayoutXOffsets.get(node);
279+
if(offset != null) {
280+
slave1.setLayoutX((Double) n + offset);
281+
}
280282
}
281283
}
282284

@@ -289,8 +291,10 @@ private void addPositionListeners(final Region master, final GModel model) {
289291
if (jointSkin.isSelected() && !jointSkin.getRoot().equals(master)) {
290292

291293
final Region slave2 = jointSkin.getRoot();
292-
double offset = Optional.ofNullable(jointLayoutXOffsets.get(joint)).orElse(0d);
293-
slave2.setLayoutX((Double) n + offset);
294+
Double offset = jointLayoutXOffsets.get(joint);
295+
if(offset != null) {
296+
slave2.setLayoutX((Double) n + offset);
297+
}
294298
}
295299
}
296300
}
@@ -305,8 +309,10 @@ private void addPositionListeners(final Region master, final GModel model) {
305309
if (nodeSkin.isSelected() && !nodeSkin.getRoot().equals(master)) {
306310

307311
final Region slave1 = nodeSkin.getRoot();
308-
double offset = Optional.ofNullable(nodeLayoutYOffsets.get(node)).orElse(0d);
309-
slave1.setLayoutY((Double) n + offset);
312+
Double offset = nodeLayoutYOffsets.get(node);
313+
if(offset != null) {
314+
slave1.setLayoutY((Double) n + offset);
315+
}
310316
}
311317
}
312318

@@ -319,8 +325,10 @@ private void addPositionListeners(final Region master, final GModel model) {
319325
if (jointSkin.isSelected() && !jointSkin.getRoot().equals(master)) {
320326

321327
final Region slave2 = jointSkin.getRoot();
322-
double offset = Optional.ofNullable(jointLayoutYOffsets.get(joint)).orElse(0d);
323-
slave2.setLayoutY((Double) n + jointLayoutYOffsets.get(joint));
328+
Double offset = jointLayoutYOffsets.get(joint);
329+
if(offset != null) {
330+
slave2.setLayoutY((Double) n + offset);
331+
}
324332
}
325333
}
326334
}

0 commit comments

Comments
 (0)