package com.khorn.terraincontrol.configuration.settingType;

import com.khorn.terraincontrol.exception.InvalidConfigException;
import com.khorn.terraincontrol.util.helpers.StringHelper;

import java.util.Arrays;
import java.util.List;

/**
 * Reads and writes a list of strings. Strings are read using
 * {@link StringHelper#readCommaSeperatedString(String)}, and written with a
 * ", " between each string.
 *
 */
class StringListSetting extends Setting<List<String>>
{
    private String[] defaultValue;

    StringListSetting(String name, String... defaultValue)
    {
        super(name);
        this.defaultValue = defaultValue;
    }

    @Override
    public List<String> getDefaultValue()
    {
        return Arrays.asList(defaultValue);
    }

    @Override
    public List<String> read(String string) throws InvalidConfigException
    {
        return Arrays.asList(StringHelper.readCommaSeperatedString(string));
    }

    @Override
    public String write(List<String> value)
    {
        return StringHelper.join(value, ", ");
    }

}

package com.khorn.terraincontrol.configuration.settingType;

import com.khorn.terraincontrol.exception.InvalidConfigException;
import com.khorn.terraincontrol.util.helpers.StringHelper;

import java.util.Arrays;
import java.util.List;

/**
 * Reads and writes a list of strings. Strings are read using
 * {@link StringHelper#readCommaSeperatedString(String)}, and written with a
 * ", " between each string.
 *
 */
class StringListSetting extends Setting<List<String>>
{
    private String[] defaultValue;

    StringListSetting(String name, String... defaultValue)
    {
        super(name);
        this.defaultValue = defaultValue;
    }

    @Override
    public List<String> getDefaultValue()
    {
        return Arrays.asList(defaultValue);
    }

    @Override
    public List<String> read(String string) throws InvalidConfigException
    {
        String[] split = StringHelper.readCommaSeperatedString(string);
        for (int i = 0; i < split.length; i++)
        {
            // Trimming the values allows "Value1, Value2" instead of
            // "Value1,Value2"
            split[i] = split[i];
        }
        return Arrays.asList(split);
    }

    @Override
    public String write(List<String> value)
    {
        return StringHelper.join(value, ", ");
    }

}

--- 
+++ 
@@ -31,14 +31,7 @@
     @Override
     public List<String> read(String string) throws InvalidConfigException
     {
-        String[] split = StringHelper.readCommaSeperatedString(string);
-        for (int i = 0; i < split.length; i++)
-        {
-            // Trimming the values allows "Value1, Value2" instead of
-            // "Value1,Value2"
-            split[i] = split[i];
-        }
-        return Arrays.asList(split);
+        return Arrays.asList(StringHelper.readCommaSeperatedString(string));
     }
 
     @Override
Remove some unneeded code, as StringHelper.readCommaSeperatedString already trims values
