added logging

Author: i-make-robots

pom.xml

@@ -58,23 +58,32 @@
             <artifactId>json</artifactId>
             <version>20201115</version>
         </dependency>
+        <!-- unit testing -->
         <dependency>
             <groupId>org.junit.jupiter</groupId>
             <artifactId>junit-jupiter-api</artifactId>
             <version>5.0.0-M5</version>
             <scope>test</scope>
         </dependency>
-        <!-- for logging -->
+        <!-- unit testing -->
         <dependency>
-            <groupId>ch.qos.logback</groupId>
-            <artifactId>logback-classic</artifactId>
-            <version>1.2.10</version>
+            <groupId>org.assertj</groupId>
+            <artifactId>assertj-swing-junit</artifactId>
+            <version>3.9.2</version>
+            <scope>test</scope>
         </dependency>
         <!-- for logging -->
+        <!-- https://mvnrepository.com/artifact/org.slf4j/slf4j-api -->
         <dependency>
             <groupId>org.slf4j</groupId>
             <artifactId>slf4j-api</artifactId>
-            <version>1.7.32</version>
+            <version>1.7.35</version>
+        </dependency>
+        <!-- for logging -->
+        <dependency>
+            <groupId>ch.qos.logback</groupId>
+            <artifactId>logback-classic</artifactId>
+            <version>1.2.11</version>
         </dependency>
     </dependencies>
 

src/main/java/com/marginallyClever/nodeGraphSwing/Donatello.java

@@ -17,12 +17,16 @@
 import java.util.ArrayList;
 import java.util.List;
 
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
+
 /**
  * {@link Donatello} is a Graphic User Interface to edit a {@link NodeGraph}.
  * @author Dan Royer
  * @since 2022-02-01
  */
 public class Donatello extends JPanel {
+    private static final Logger logger = LoggerFactory.getLogger(Donatello.class);
     /**
      * Used by save and load actions
       */
@@ -522,7 +526,7 @@ public static void setSystemLookAndFeel() {
         try {
             UIManager.setLookAndFeel(UIManager.getSystemLookAndFeelClassName());
         } catch (Exception e) {
-            System.out.println("System look and feel could not be set.");
+            logger.debug("System look and feel could not be set.");
         }
     }
 

src/main/java/com/marginallyClever/nodeGraphSwing/PropertiesHelper.java

@@ -2,6 +2,8 @@
 
 import com.marginallyClever.nodeGraphCore.DAO4JSONFactory;
 import com.marginallyClever.nodeGraphCore.NodeFactory;
+import org.slf4j.Logger;
+import org.slf4j.LoggerFactory;
 
 import java.util.*;
 import java.util.List;
@@ -12,32 +14,36 @@
  * @since 2022-02-01
  */
 public class PropertiesHelper {
+    private static final Logger logger = LoggerFactory.getLogger(PropertiesHelper.class);
+
     public static void showProperties() {
-        System.out.println("------------------------------------------------");
+        logger.debug("------------------------------------------------");
         Properties p = System.getProperties();
         List<String> names = new ArrayList<>(p.stringPropertyNames());
         Collections.sort(names);
         for (String name : names) {
-            System.out.println( name +" = "+ p.get(name));
+            logger.debug("{} = {}", name, p.get(name));
         }
-        System.out.println("------------------------------------------------");
+        logger.debug("------------------------------------------------");
     }
 
     public static void listAllNodes() {
-        String add = "Nodes: ";
+        String sum = "";
+        String add = "";
         for (String n : NodeFactory.getNames()) {
-            System.out.print(add + n);
+            sum += add + n;
             add = ", ";
         }
-        System.out.println(".");
+        logger.debug("Nodes: {}.",sum);
     }
 
     public static void listAllDAO() {
-        String add="DAOs: ";
+        String sum = "";
+        String add = "";
         for(String n : DAO4JSONFactory.getNames()) {
-            System.out.print(add + n);
+            sum += add + n;
             add = ", ";
         }
-        System.out.println(".");
+        logger.debug("DAOs: {}.",sum);
     }
 }

src/main/java/module-info.java

@@ -8,6 +8,7 @@
 module com.marginallyClever.nodeGraphCore {
     requires java.desktop;
     requires org.json;
+    requires org.slf4j;
 
     uses com.marginallyClever.nodeGraphCore.NodeRegistry;
     provides com.marginallyClever.nodeGraphCore.NodeRegistry with

src/main/resources/logback.xml

@@ -0,0 +1,40 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<!DOCTYPE configuration>
+<configuration>
+
+    <if condition='property("DEV").equals("true")'>
+        <then>
+            <appender name="STDOUT" class="ch.qos.logback.core.ConsoleAppender">
+                <encoder>
+                    <pattern>%d %-5level %logger{35} - %msg %n</pattern>
+                </encoder>
+            </appender>
+
+            <logger name="com.marginallyClever.makelangelo.nodeGraphSwing" level="debug" />
+            <root>
+                <appender-ref ref="STDOUT" />
+            </root>
+        </then>
+    </if>
+
+    <appender name="FILE" class="ch.qos.logback.core.rolling.RollingFileAppender">
+        <file>${user.home}/.makelangelo/makelangelo.log</file>
+        <rollingPolicy class="ch.qos.logback.core.rolling.FixedWindowRollingPolicy">
+            <fileNamePattern>${user.home}/.makelangelo/makelangelo.%i.log</fileNamePattern>
+            <minIndex>1</minIndex>
+            <maxIndex>1</maxIndex>
+        </rollingPolicy>
+
+        <triggeringPolicy class="com.marginallyClever.convenience.log.RollOncePerSessionTriggeringPolicy"/>
+        <encoder>
+            <pattern>%d{HH:mm:ss.SSS} [%thread] %-5level %logger - %msg%n</pattern>
+        </encoder>
+    </appender>
+
+    <logger name="com.marginallyClever" level="debug" />
+
+    <root level="info">
+        <appender-ref ref="FILE" />
+    </root>
+
+</configuration>