apply feedback

Author: Maximilian Stiede

chunky/src/java/se/llbit/chunky/renderer/scene/sky/CelestialBodyType.java

@@ -24,7 +24,7 @@
 import se.llbit.chunky.renderer.Refreshable;
 import se.llbit.chunky.renderer.SceneIOProvider;
 import se.llbit.chunky.renderer.scene.Scene;
-import se.llbit.chunky.resources.Texture;
+import se.llbit.chunky.renderer.scene.sky.celestialbodies.CelestialBodyType;
 import se.llbit.json.JsonObject;
 import se.llbit.math.QuickMath;
 import se.llbit.math.Ray;
@@ -116,7 +116,7 @@ public class Sun implements JsonSerializable {
 
   private final Refreshable scene;
 
-  private CelestialBodyType type = new CelestialBodyType.Sun();
+  private CelestialBodyType type = CelestialBodyType.DEFAULT;
 
   /**
    * Sun radius

chunky/src/java/se/llbit/chunky/renderer/scene/sky/Sun.java

@@ -0,0 +1,55 @@
+package se.llbit.chunky.renderer.scene.sky.celestialbodies;
+
+import se.llbit.chunky.renderer.SceneIOProvider;
+import se.llbit.chunky.resources.Texture;
+import se.llbit.json.JsonObject;
+import se.llbit.log.Log;
+import se.llbit.util.Registerable;
+
+import java.util.HashMap;
+import java.util.Map;
+import java.util.function.Supplier;
+
+public abstract class CelestialBodyType implements Registerable {
+  private static final String CONFIG_TYPE_KEY = "celestialBodyType";
+
+  public final static CelestialBodyType DEFAULT = new Sun();
+
+  public static final Map<String, Supplier<CelestialBodyType>> TYPES = new HashMap<>(3);
+  static {
+    TYPES.put(Sun.ID, Sun::new);
+    TYPES.put(Moon.ID, Moon::new);
+    TYPES.put(Custom.ID, Custom::new);
+  }
+
+  public static CelestialBodyType newFromJson(JsonObject obj) {
+    String typeStr = obj.get(CONFIG_TYPE_KEY).asString(Sun.ID);
+    Supplier<CelestialBodyType> create = TYPES.get(typeStr);
+    if(create == null) {
+      Log.warnf("Unknown celestial body type \"%s\"", typeStr);
+      return DEFAULT;
+    }
+    CelestialBodyType type = create.get();
+    type.importFromJson(obj);
+    return type;
+  }
+
+  protected void importFromJson(JsonObject obj) {}
+
+  public void appendToConfig(JsonObject obj) {
+    obj.add(CONFIG_TYPE_KEY, getId());
+  }
+
+  /**
+   * will be called when a scene is loaded to load associated custom textures
+   */
+  public void loadCustomTextures(SceneIOProvider ioContext) {
+  }
+
+  public abstract Texture getTexture();
+
+  @Override
+  public String getDescription() {
+    return getName();
+  }
+}

chunky/src/java/se/llbit/chunky/renderer/scene/sky/celestialbodies/CelestialBodyType.java

@@ -0,0 +1,74 @@
+package se.llbit.chunky.renderer.scene.sky.celestialbodies;
+
+import se.llbit.chunky.renderer.SceneIOProvider;
+import se.llbit.chunky.resources.Texture;
+import se.llbit.json.JsonObject;
+import se.llbit.log.Log;
+import se.llbit.resources.ImageLoader;
+
+import java.io.File;
+import java.io.IOException;
+
+public class Custom extends CelestialBodyType {
+  public static final String ID = "CUSTOM";
+
+  @Override
+  public String getId() {
+    return ID;
+  }
+
+  @Override
+  public String getName() {
+    return "Custom";
+  }
+
+  @Override
+  public String getDescription() {
+    return "Custom celestial body texture";
+  }
+
+  private final Texture texture = new Texture();
+  private String fileName;
+
+  public Custom() {
+    texture.setTexture(Sun.texture);
+  }
+
+  @Override
+  protected void importFromJson(JsonObject obj) {
+    fileName = obj.get("customTextureFile").asString(null);
+  }
+
+  @Override
+  public void appendToConfig(JsonObject obj) {
+    super.appendToConfig(obj);
+    obj.add("customTextureFile", fileName);
+  }
+
+  public void loadCustomTextures(SceneIOProvider ioContext) {
+    if (fileName != null) {
+      try {
+        setFile(ioContext.resolveLinkedFile(fileName));
+      } catch (IOException ex) {
+        Log.error("Failed to find custom skymap file: " + fileName);
+      }
+    }
+  }
+
+  @Override
+  public Texture getTexture() {
+    return texture;
+  }
+
+  public String getFileName() {
+    return fileName;
+  }
+
+  public void setFile(File file) {
+    try {
+      texture.setTexture(ImageLoader.read(file));
+    } catch (IOException ex) {
+      Log.error("Failed to load custom skymap: " + file, ex);
+    }
+  }
+}