Рассмотрим такой метод:

@Override
public String toString()
{
    final StringBuilder sb = new StringBuilder();
    for (final Room room : map)
    {
        sb.append(room.toString());
        sb.append(System.getProperty("line.separator")); // THIS IS IMPORTANT
    }
    return sb.toString();
}

System.getProperty("line.separator") можно вызывать много раз.

Должен ли я кэшировать это значение сpublic final static String lineSeperator = System.getProperty("line.separator") а позже использовать только?lineSeperator

Или жеSystem.getProperty("line.separator") так быстро, как с использованием статического поля?