Using Lombok logger

Author: Bernardo-MG

pom.xml

@@ -207,6 +207,15 @@
          <version>${dice.version}</version>
       </dependency>
       <!-- ============================================== -->
+      <!-- ================== LOMBOK ==================== -->
+      <!-- ============================================== -->
+      <dependency>
+         <!-- Lombok -->
+         <groupId>org.projectlombok</groupId>
+         <artifactId>lombok</artifactId>
+         <scope>provided</scope>
+      </dependency>
+      <!-- ============================================== -->
       <!-- ================== LOGGERS =================== -->
       <!-- ============================================== -->
       <dependency>

src/main/java/com/bernardomg/tabletop/dice/cli/Main.java

@@ -16,11 +16,9 @@
 
 package com.bernardomg.tabletop.dice.cli;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import com.bernardomg.tabletop.dice.cli.menu.DiceMenu;
 
+import lombok.extern.slf4j.Slf4j;
 import picocli.CommandLine;
 
 /**
@@ -29,19 +27,15 @@
  * @author Bernardo Martínez Garrido
  *
  */
+@Slf4j
 public class Main {
 
-    /**
-     * Logger.
-     */
-    private static final Logger LOGGER = LoggerFactory.getLogger(Main.class);
-
     public static void main(final String[] args) {
         final Integer exitCode;
 
         exitCode = new CommandLine(new DiceMenu()).execute(args);
 
-        LOGGER.debug("Exited with code {}", exitCode);
+        log.debug("Exited with code {}", exitCode);
 
         System.exit(exitCode);
     }

src/main/java/com/bernardomg/tabletop/dice/cli/command/DiceGathererCommand.java

@@ -20,9 +20,6 @@
 import java.util.stream.Collectors;
 import java.util.stream.StreamSupport;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import com.bernardomg.tabletop.dice.Dice;
 import com.bernardomg.tabletop.dice.cli.version.ManifestVersionProvider;
 import com.bernardomg.tabletop.dice.interpreter.DiceGatherer;
@@ -31,6 +28,7 @@
 import com.bernardomg.tabletop.dice.parser.DiceParser;
 import com.google.common.collect.Iterables;
 
+import lombok.extern.slf4j.Slf4j;
 import picocli.CommandLine.Command;
 import picocli.CommandLine.Model.CommandSpec;
 import picocli.CommandLine.Parameters;
@@ -44,24 +42,20 @@
  */
 @Command(name = "gather", description = "Gathers dice from an expression", mixinStandardHelpOptions = true,
         versionProvider = ManifestVersionProvider.class)
+@Slf4j
 public final class DiceGathererCommand implements Runnable {
 
-    /**
-     * Logger.
-     */
-    private static final Logger LOGGER = LoggerFactory.getLogger(DiceGathererCommand.class);
-
     /**
      * Expression to roll.
      */
     @Parameters(index = "0", description = "The expression to roll", paramLabel = "EXP")
-    private String              expression;
+    private String      expression;
 
     /**
      * Command specification. Used to get the line output.
      */
     @Spec
-    private CommandSpec         spec;
+    private CommandSpec spec;
 
     /**
      * Default constructor.
@@ -78,14 +72,14 @@ public final void run() {
         final String                          diceText;
         final PrintWriter                     writer;
 
-        LOGGER.debug("Running expression {}", expression);
+        log.debug("Running expression {}", expression);
 
         parser = new DefaultDiceParser();
         gatherer = new DiceGatherer();
 
         diceSets = parser.parse(expression, gatherer);
 
-        LOGGER.debug("Gathered dice sets {}", diceSets);
+        log.debug("Gathered dice sets {}", diceSets);
 
         writer = spec.commandLine()
             .getOut();

src/main/java/com/bernardomg/tabletop/dice/cli/command/DiceRollCommand.java

@@ -20,9 +20,6 @@
 import java.util.stream.Collectors;
 import java.util.stream.StreamSupport;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
 import com.bernardomg.tabletop.dice.cli.version.ManifestVersionProvider;
 import com.bernardomg.tabletop.dice.history.RollHistory;
 import com.bernardomg.tabletop.dice.history.RollResult;
@@ -31,6 +28,7 @@
 import com.bernardomg.tabletop.dice.parser.DefaultDiceParser;
 import com.bernardomg.tabletop.dice.parser.DiceParser;
 
+import lombok.extern.slf4j.Slf4j;
 import picocli.CommandLine.Command;
 import picocli.CommandLine.Model.CommandSpec;
 import picocli.CommandLine.Option;
@@ -45,36 +43,32 @@
  */
 @Command(name = "roll", description = "Rolls an expression", mixinStandardHelpOptions = true,
         versionProvider = ManifestVersionProvider.class)
+@Slf4j
 public final class DiceRollCommand implements Runnable {
 
-    /**
-     * Logger.
-     */
-    private static final Logger LOGGER = LoggerFactory.getLogger(DiceRollCommand.class);
-
     /**
      * Expression to roll.
      */
     @Parameters(index = "0", description = "The expression to roll", paramLabel = "EXP")
-    private String              expression;
+    private String      expression;
 
     /**
      * Roll history flag.
      */
     @Option(names = "-history", description = "Prints the roll history", defaultValue = "false")
-    private Boolean             history;
+    private Boolean     history;
 
     /**
      * Full history flag.
      */
     @Option(names = "-fullHistory", description = "Prints a detailed roll history", defaultValue = "false")
-    private Boolean             historyDetailed;
+    private Boolean     historyDetailed;
 
     /**
      * Command specification. Used to get the line output.
      */
     @Spec
-    private CommandSpec         spec;
+    private CommandSpec spec;
 
     /**
      * Default constructor.
@@ -110,7 +104,7 @@ public final void run() {
         final PrintWriter                  writer;
         String                             valuesText;
 
-        LOGGER.debug("Running expression {}", expression);
+        log.debug("Running expression {}", expression);
 
         parser = new DefaultDiceParser();
         roller = new DiceRoller();
@@ -119,8 +113,8 @@ public final void run() {
 
         totalRoll = rolls.getTotalRoll();
 
-        LOGGER.debug("Total roll {}", totalRoll);
-        LOGGER.debug("History: {}", rolls.toString());
+        log.debug("Total roll {}", totalRoll);
+        log.debug("History: {}", rolls.toString());
 
         writer = spec.commandLine()
             .getOut();
@@ -131,14 +125,14 @@ public final void run() {
         writer.printf("Total roll: %d%n", totalRoll);
 
         if (history) {
-            LOGGER.debug("Printing roll history");
+            log.debug("Printing roll history");
 
             writer.println("------------");
             writer.printf("Roll history: %s%n", rolls.toString());
         }
 
         if (historyDetailed) {
-            LOGGER.debug("Printing detailed roll history");
+            log.debug("Printing detailed roll history");
 
             writer.println("------------");
             writer.println("Detailed roll history");

src/main/java/com/bernardomg/tabletop/dice/cli/version/ManifestVersionProvider.java

@@ -22,9 +22,7 @@
 import java.util.jar.Attributes;
 import java.util.jar.Manifest;
 
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
+import lombok.extern.slf4j.Slf4j;
 import picocli.CommandLine;
 import picocli.CommandLine.IVersionProvider;
 
@@ -34,13 +32,9 @@
  * @author Bernardo Martínez Garrido
  *
  */
+@Slf4j
 public final class ManifestVersionProvider implements IVersionProvider {
 
-    /**
-     * Logger.
-     */
-    private static final Logger LOGGER  = LoggerFactory.getLogger(ManifestVersionProvider.class);
-
     /**
      * Project title. Used to identify the correct manifest.
      */
@@ -74,7 +68,7 @@ public final String[] getVersion() throws Exception {
             try {
                 manifest = new Manifest(url.openStream());
             } catch (final IOException ex) {
-                LOGGER.error("Unable to read from {}", url);
+                log.error("Unable to read from {}", url);
                 // TODO: Use detailed error
                 throw new RuntimeException();
             }