use stringJoiner for better readability

- delete size because it was never used

Author: ddamke

src/main/java/de/doubleslash/keeptime/view/ProjectReport.java

@@ -19,6 +19,7 @@
 import static de.doubleslash.keeptime.view.ReportController.NOTE_DELIMETER;
 
 import java.lang.invoke.MethodHandles;
+import java.util.StringJoiner;
 
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
@@ -28,35 +29,23 @@ public class ProjectReport {
    /** The slf4j-logger for this class. */
    private static final Logger LOG = LoggerFactory.getLogger(MethodHandles.lookup().lookupClass());
 
-   private int numberOfNotEmptyNotes;
+   private final StringJoiner sb;
 
-   private final int size;
-
-   private final StringBuilder sb;
-
-   public ProjectReport(final int size) {
-      this.size = size;
-      this.sb = new StringBuilder(2 * 1024);
+   public ProjectReport() {
+      this.sb = new StringJoiner(NOTE_DELIMETER);
    }
 
    public void appendToWorkNotes(final String currentWorkNote) {
+
       if (!currentWorkNote.isEmpty()) {
-         this.numberOfNotEmptyNotes++;
-         if (this.numberOfNotEmptyNotes >1) {
-            this.sb.append(NOTE_DELIMETER);
-         }
-         this.sb.append(currentWorkNote.trim());
+         this.sb.add(currentWorkNote.trim());
       } else {
          LOG.debug("Skipping empty note.");
       }
    }
 
-   public int getNumberOfNotEmptyNotes() {
-      return this.numberOfNotEmptyNotes;
-   }
-
    public String getNotes() {
-      return this.sb.toString();
+      return sb.toString();
    }
 
 }

src/main/java/de/doubleslash/keeptime/view/ReportController.java

@@ -424,7 +424,7 @@ private Button createCopyProjectButton(final List<Work> projectWork) {
 
       final EventHandler<ActionEvent> eventListener = actionEvent -> {
          LOG.debug("Copy to Clipboard clicked.");
-         final ProjectReport pr = new ProjectReport(projectWork.size());
+         final ProjectReport pr = new ProjectReport();
          for (int j = 0; j < projectWork.size(); j++) {
             final Work work = projectWork.get(j);
             final String currentWorkNote = work.getNotes();

src/test/java/de/doubleslash/keeptime/view/ProjectReportTest.java

@@ -37,7 +37,7 @@ public class ProjectReportTest {
 
    @BeforeEach
    void setUp() {
-      this.uut = new ProjectReport(3);
+      this.uut = new ProjectReport();
    }
 
    @Test
@@ -60,7 +60,7 @@ public void testAppendToWorkNotes_EmptyNoteInTheMiddle() {
 
    @Test
    public void testAppendToWorkNotes_NoEmptyNote() {
-      this.uut = new ProjectReport(3);
+      this.uut = new ProjectReport();
       this.uut.appendToWorkNotes("note 1");
       this.uut.appendToWorkNotes("note 2");
       this.uut.appendToWorkNotes("note 3");
@@ -70,7 +70,7 @@ public void testAppendToWorkNotes_NoEmptyNote() {
 
    @Test
    public void testAppendToWorkNotes_EmptyNotesAtTheEnd() {
-      this.uut = new ProjectReport(4);
+      this.uut = new ProjectReport();
       this.uut.appendToWorkNotes("note 1");
       this.uut.appendToWorkNotes("note 2");
       this.uut.appendToWorkNotes(EMPTY_NOTE);