Write csv entries in correct order

Author: ckittl

CHANGELOG.md

@@ -23,6 +23,7 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 - Fix JavaDoc creation
     - Create JavaDoc with java 17 instead of java 8
     - Let JavDoc pass, if there are warnings **ATTENTION:** Should be removed, when JavaDoc is fixed! (cf. Issue [#494](https://github.com/ie3-institute/PowerSystemDataModel/issues/494))
+- `BufferedCsvWriter` writes columns in the order, that the headline elements are defined [#434](https://github.com/ie3-institute/PowerSystemDataModel/issues/393)
 
 ### Changed
 - BREAKING: PvInput Model parameter name height changed to elevationAngle [#393](https://github.com/ie3-institute/PowerSystemDataModel/issues/393) :warning:

src/main/java/edu/ie3/datamodel/io/csv/BufferedCsvWriter.java

@@ -27,10 +27,11 @@ public class BufferedCsvWriter extends BufferedWriter {
       "Direct appending is prohibited. Use write instead.";
 
   /**
-   * Build a new CsvBufferedWriter
+   * Build a new CsvBufferedWriter. The order of headline elements given in this constructor defines
+   * the order of columns in file
    *
    * @param filePath String representation of the full path to the target file
-   * @param headLineElements Elements of the csv head line
+   * @param headLineElements Elements of the csv headline
    * @param csvSep csv separator char
    * @param append true to append to an existing file, false to overwrite an existing file (if any),
    *     if no file exists, a new one will be created in both cases
@@ -47,7 +48,8 @@ public BufferedCsvWriter(
   /**
    * Build a new CsvBufferedWriter. This is a "convenience" Constructor. The absolute file path is
    * assembled by concatenation of {@code baseFolder} and {@code fileDefinition}'s file path
-   * information.
+   * information. The order of headline elements in {@code fileDefinition} defines the order of
+   * columns in file
    *
    * @param baseFolder Base folder, from where the file hierarchy should start
    * @param fileDefinition The foreseen shape of the file
@@ -73,15 +75,16 @@ public BufferedCsvWriter(String baseFolder, CsvFileDefinition fileDefinition, bo
    */
   public synchronized void write(Map<String, String> entityFieldData)
       throws IOException, SinkException {
-    /* Check against eligible head line elements */
+    /* Check against eligible headline elements */
     if (entityFieldData.size() != headLineElements.length
         || !entityFieldData.keySet().containsAll(Arrays.asList(headLineElements)))
       throw new SinkException(
           "The provided data does not meet the pre-defined head line elements '"
               + String.join(",", headLineElements)
               + "'.");
 
-    String[] entries = entityFieldData.values().toArray(new String[0]);
+    String[] entries =
+        Arrays.stream(headLineElements).map(entityFieldData::get).toArray(String[]::new);
     writeOneLine(entries);
   }
 

src/test/groovy/edu/ie3/datamodel/io/csv/BufferedCsvWriterTest.groovy

@@ -5,6 +5,7 @@
  */
 package edu.ie3.datamodel.io.csv
 
+import edu.ie3.datamodel.exceptions.FileException
 import edu.ie3.datamodel.exceptions.SinkException
 import edu.ie3.util.io.FileIOUtils
 import org.apache.commons.io.FilenameUtils
@@ -78,4 +79,33 @@ class BufferedCsvWriterTest extends Specification {
 		def e = thrown(SinkException)
 		e.message == "The provided data does not meet the pre-defined head line elements 'a,b,c'."
 	}
+
+	def "The buffered csv writer writes out content in the order specified by the headline elements"() {
+		given:
+		def targetFile = FilenameUtils.concat(tmpDirectory.toString(), "order_test.csv")
+		def writer = new BufferedCsvWriter(targetFile, ["c", "b", "a"] as String[], ",", false)
+		writer.writeFileHeader()
+		def content = [
+				"c": "z",
+				"a": "x",
+				"b": "y"
+		]
+
+		when:
+		writer.write(content)
+		writer.close()
+		/* Read in the content */
+		def writtenContent = ""
+		def headline = ""
+		try(BufferedReader reader = new BufferedReader(new FileReader(targetFile))) {
+			headline = reader.readLine()
+			writtenContent = reader.readLine()
+		} catch (Exception e) {
+			throw new FileException("Unable to read content of test file '"+targetFile+"'.", e)
+		}
+
+		then:
+		headline == "c,b,a"
+		writtenContent == "z,y,x"
+	}
 }