Refactors null object pattern to java-11
This commit is contained in:
parent
a5038c4329
commit
8e060ad0ad
@ -37,12 +37,16 @@ public class App {
|
||||
* @param args command line args
|
||||
*/
|
||||
public static void main(String[] args) {
|
||||
|
||||
Node root =
|
||||
new NodeImpl("1", new NodeImpl("11", new NodeImpl("111", NullNode.getInstance(),
|
||||
NullNode.getInstance()), NullNode.getInstance()), new NodeImpl("12",
|
||||
NullNode.getInstance(), new NodeImpl("122", NullNode.getInstance(),
|
||||
NullNode.getInstance())));
|
||||
Node root = new NodeImpl("1",
|
||||
new NodeImpl("11",
|
||||
new NodeImpl("111", NullNode.getInstance(), NullNode.getInstance()),
|
||||
NullNode.getInstance()
|
||||
),
|
||||
new NodeImpl("12",
|
||||
NullNode.getInstance(),
|
||||
new NodeImpl("122", NullNode.getInstance(), NullNode.getInstance())
|
||||
)
|
||||
);
|
||||
|
||||
root.walk();
|
||||
}
|
||||
|
@ -26,15 +26,11 @@ package com.iluwatar.nullobject;
|
||||
import org.junit.jupiter.api.Test;
|
||||
|
||||
/**
|
||||
*
|
||||
* Application test
|
||||
*
|
||||
*/
|
||||
public class AppTest {
|
||||
|
||||
@Test
|
||||
public void test() {
|
||||
String[] args = {};
|
||||
App.main(args);
|
||||
App.main(new String[]{});
|
||||
}
|
||||
}
|
||||
|
@ -42,14 +42,14 @@ public class NullNodeTest {
|
||||
*/
|
||||
@Test
|
||||
public void testGetInstance() {
|
||||
final NullNode instance = NullNode.getInstance();
|
||||
final var instance = NullNode.getInstance();
|
||||
assertNotNull(instance);
|
||||
assertSame(instance, NullNode.getInstance());
|
||||
}
|
||||
|
||||
@Test
|
||||
public void testFields() {
|
||||
final NullNode node = NullNode.getInstance();
|
||||
final var node = NullNode.getInstance();
|
||||
assertEquals(0, node.getTreeSize());
|
||||
assertNull(node.getName());
|
||||
assertNull(node.getLeft());
|
||||
|
@ -23,22 +23,21 @@
|
||||
|
||||
package com.iluwatar.nullobject;
|
||||
|
||||
import ch.qos.logback.classic.Logger;
|
||||
import ch.qos.logback.classic.spi.ILoggingEvent;
|
||||
import ch.qos.logback.core.AppenderBase;
|
||||
import org.junit.jupiter.api.AfterEach;
|
||||
import org.junit.jupiter.api.BeforeEach;
|
||||
import org.junit.jupiter.api.Test;
|
||||
import org.slf4j.LoggerFactory;
|
||||
|
||||
import java.util.LinkedList;
|
||||
import java.util.List;
|
||||
|
||||
import static org.junit.jupiter.api.Assertions.assertEquals;
|
||||
import static org.junit.jupiter.api.Assertions.assertNotNull;
|
||||
import static org.junit.jupiter.api.Assertions.assertSame;
|
||||
import static org.junit.jupiter.api.Assertions.assertTrue;
|
||||
|
||||
import ch.qos.logback.classic.Logger;
|
||||
import ch.qos.logback.classic.spi.ILoggingEvent;
|
||||
import ch.qos.logback.core.AppenderBase;
|
||||
import java.util.LinkedList;
|
||||
import java.util.List;
|
||||
import org.junit.jupiter.api.AfterEach;
|
||||
import org.junit.jupiter.api.BeforeEach;
|
||||
import org.junit.jupiter.api.Test;
|
||||
import org.slf4j.LoggerFactory;
|
||||
|
||||
/**
|
||||
* Date: 12/26/15 - 11:44 PM
|
||||
*
|
||||
@ -75,12 +74,12 @@ public class TreeTest {
|
||||
private static final Node TREE_ROOT;
|
||||
|
||||
static {
|
||||
final NodeImpl level1B = new NodeImpl("level1_b", NullNode.getInstance(), NullNode.getInstance());
|
||||
final NodeImpl level2B = new NodeImpl("level2_b", NullNode.getInstance(), NullNode.getInstance());
|
||||
final NodeImpl level3A = new NodeImpl("level3_a", NullNode.getInstance(), NullNode.getInstance());
|
||||
final NodeImpl level3B = new NodeImpl("level3_b", NullNode.getInstance(), NullNode.getInstance());
|
||||
final NodeImpl level2A = new NodeImpl("level2_a", level3A, level3B);
|
||||
final NodeImpl level1A = new NodeImpl("level1_a", level2A, level2B);
|
||||
final var level1B = new NodeImpl("level1_b", NullNode.getInstance(), NullNode.getInstance());
|
||||
final var level2B = new NodeImpl("level2_b", NullNode.getInstance(), NullNode.getInstance());
|
||||
final var level3A = new NodeImpl("level3_a", NullNode.getInstance(), NullNode.getInstance());
|
||||
final var level3B = new NodeImpl("level3_b", NullNode.getInstance(), NullNode.getInstance());
|
||||
final var level2A = new NodeImpl("level2_a", level3A, level3B);
|
||||
final var level1A = new NodeImpl("level1_a", level2A, level2B);
|
||||
TREE_ROOT = new NodeImpl("root", level1A, level1B);
|
||||
}
|
||||
|
||||
@ -112,17 +111,17 @@ public class TreeTest {
|
||||
|
||||
@Test
|
||||
public void testGetLeft() {
|
||||
final Node level1 = TREE_ROOT.getLeft();
|
||||
final var level1 = TREE_ROOT.getLeft();
|
||||
assertNotNull(level1);
|
||||
assertEquals("level1_a", level1.getName());
|
||||
assertEquals(5, level1.getTreeSize());
|
||||
|
||||
final Node level2 = level1.getLeft();
|
||||
final var level2 = level1.getLeft();
|
||||
assertNotNull(level2);
|
||||
assertEquals("level2_a", level2.getName());
|
||||
assertEquals(3, level2.getTreeSize());
|
||||
|
||||
final Node level3 = level2.getLeft();
|
||||
final var level3 = level2.getLeft();
|
||||
assertNotNull(level3);
|
||||
assertEquals("level3_a", level3.getName());
|
||||
assertEquals(1, level3.getTreeSize());
|
||||
@ -132,7 +131,7 @@ public class TreeTest {
|
||||
|
||||
@Test
|
||||
public void testGetRight() {
|
||||
final Node level1 = TREE_ROOT.getRight();
|
||||
final var level1 = TREE_ROOT.getRight();
|
||||
assertNotNull(level1);
|
||||
assertEquals("level1_b", level1.getName());
|
||||
assertEquals(1, level1.getTreeSize());
|
||||
@ -140,7 +139,7 @@ public class TreeTest {
|
||||
assertSame(NullNode.getInstance(), level1.getLeft());
|
||||
}
|
||||
|
||||
private class InMemoryAppender extends AppenderBase<ILoggingEvent> {
|
||||
private static class InMemoryAppender extends AppenderBase<ILoggingEvent> {
|
||||
private final List<ILoggingEvent> log = new LinkedList<>();
|
||||
|
||||
public InMemoryAppender() {
|
||||
@ -154,7 +153,7 @@ public class TreeTest {
|
||||
}
|
||||
|
||||
public boolean logContains(String message) {
|
||||
return log.stream().anyMatch(event -> event.getMessage().equals(message));
|
||||
return log.stream().map(ILoggingEvent::getMessage).anyMatch(message::equals);
|
||||
}
|
||||
|
||||
public int getLogSize() {
|
||||
|
Loading…
x
Reference in New Issue
Block a user