Better generation

core/src/mindustry/ai/BaseRegistry.java

@@ -72,7 +72,11 @@ public class BaseRegistry{
 
                 part.tier = schem.tiles.sumf(s -> s.block.buildCost / s.block.buildCostMultiplier);
 
-                (part.core != null ? cores : parts).add(part);
+                if(part.core != null){
+                    cores.add(part);
+                }else if(part.requiredItem == null){
+                    parts.add(part);
+                }
 
                 if(drills > 0){
                     Tmp.v1.scl(1f / drills).scl(1f / tilesize);

core/src/mindustry/maps/generators/BaseGenerator.java

@@ -1,5 +1,6 @@
 package mindustry.maps.generators;
 
+import arc.func.*;
 import arc.math.*;
 import arc.math.geom.*;
 import arc.struct.*;
@@ -23,13 +24,17 @@ public class BaseGenerator{
     private static final Schematic tmpSchem2 = new Schematic(new Array<>(), new StringMap(), 0, 0);
     private static final Vec2 axis = new Vec2(), rotator = new Vec2();
 
+    private final static int range = 200;
+
     private Tiles tiles;
     private Team team;
     private ObjectMap<Item, OreBlock> ores = new ObjectMap<>();
+    private Array<Tile> cores;
 
     public void generate(Tiles tiles, Array<Tile> cores, Tile spawn, Team team, Sector sector){
         this.tiles = tiles;
         this.team = team;
+        this.cores = cores;
 
         for(Block block : content.blocks()){
             if(block instanceof OreBlock && block.asFloor().itemDrop != null){
@@ -41,8 +46,9 @@ public class BaseGenerator{
         Array<Block> wallsLarge = content.blocks().select(b -> b instanceof Wall && b.size == 2);
 
         float bracket = 0.1f;
-        int range = 200;
         int wallAngle = 70; //180 for full coverage
+        double resourceChance = 0.4;
+        double nonResourceChance = 0.0007;
         BasePart coreschem = bases.cores.getFrac(bracket);
 
         Block wall = wallsSmall.getFrac(bracket), wallLarge = wallsLarge.getFrac(bracket);
@@ -60,29 +66,27 @@ public class BaseGenerator{
             }
         }
 
-        //first pass: random schematics
+        //first pass: random item specific schematics
-        for(Tile core : cores){
+        pass(tile -> {
-            core.circle(range, (x, y) -> {
+            if(tile.overlay().itemDrop != null && Mathf.chance(resourceChance)){
-                Tile tile = tiles.getn(x, y);
-                if(tile.overlay().itemDrop != null){
                 Array<BasePart> parts = bases.forItem(tile.overlay().itemDrop);
                 if(!parts.isEmpty()){
-                        tryPlace(parts.random(), x, y);
+                    tryPlace(parts.random(), tile.x, tile.y);
                 }
+            }else if(tile.overlay().itemDrop == null && Mathf.chance(nonResourceChance)){
+                tryPlace(bases.parts.random(), tile.x, tile.y);
             }
         });
-        }
 
         if(wallAngle > 0){
-            //second pass: small walls
+
-            for(Tile core : cores){
+            //small walls
-                core.circle(range, (x, y) -> {
+            pass(tile -> {
-                    Tile tile = tiles.getn(x, y);
                 if(tile.block().alwaysReplace){
                     boolean any = false;
 
                     for(Point2 p : Geometry.d8){
-                            if(Angles.angleDist(Angles.angle(p.x, p.y), spawn.angleTo(core)) > wallAngle){
+                        if(Angles.angleDist(Angles.angle(p.x, p.y), spawn.angleTo(tile.x, tile.y)) > wallAngle){
                             continue;
                         }
 
@@ -98,15 +102,13 @@ public class BaseGenerator{
                     }
                 }
             });
-            }
 
-            //third pass: large walls
+            //large walls
-            for(Tile core : cores){
+            pass(curr -> {
-                core.circle(range, (x, y) -> {
                 int walls = 0;
                 for(int cx = 0; cx < 2; cx++){
                     for(int cy = 0; cy < 2; cy++){
-                            Tile tile = tiles.get(x + cx, y + cy);
+                        Tile tile = tiles.get(curr.x + cx, curr.y + cy);
                         if(tile == null || tile.block().size != 1 || (tile.block() != wall && !tile.block().alwaysReplace)) return;
 
                         if(tile.block() == wall){
@@ -116,15 +118,21 @@ public class BaseGenerator{
                 }
 
                 if(walls >= 3){
-                        tiles.getn(x, y).setBlock(wallLarge, team);
+                    curr.setBlock(wallLarge, team);
                 }
             });
         }
     }
+
+    void pass(Cons<Tile> cons){
+        Tile core = cores.first();
+        //for(Tile core : cores){
+        core.circle(range, (x, y) -> cons.get(tiles.getn(x, y)));
+        //}
     }
 
     boolean tryPlace(BasePart part, int x, int y){
-        int rotation = 0;
+        int rotation = Mathf.range(2);
         axis.set((int)(part.schematic.width / 2f), (int)(part.schematic.height / 2f));
         Schematic result = rotate(part.schematic, rotation);
 
@@ -151,6 +159,9 @@ public class BaseGenerator{
                 if(tile.block instanceof Drill){
                     tile.block.iterateTaken(tile.x + cx, tile.y + cy, (ex, ey) -> {
                         tiles.getn(ex, ey).setOverlay(ores.get(part.requiredItem));
+
+                        //random ores nearby to make it look more natural
+                        tiles.getc(ex + Mathf.range(1), ey + Mathf.range(1)).setOverlay(ores.get(part.requiredItem));
                     });
                 }
             }

core/src/mindustry/world/Tiles.java

@@ -1,6 +1,7 @@
 package mindustry.world;
 
 import arc.func.*;
+import arc.math.*;
 import arc.math.geom.*;
 import arc.util.ArcAnnotate.*;
 
@@ -54,6 +55,13 @@ public class Tiles implements Iterable<Tile>{
         return array[y*width + x];
     }
 
+    /** @return a tile at coordinates, clamped. */
+    public @NonNull Tile getc(int x, int y){
+        x = Mathf.clamp(x, 0, width - 1);
+        y = Mathf.clamp(y, 0, height - 1);
+        return array[y*width + x];
+    }
+
     /** @return a tile at an iteration index [0, width * height] */
     public @NonNull Tile geti(int idx){
         return array[idx];