v0.3.0c: centralized general sign placement logic, fixed a bug where traps would sometimes appear over signs.

src/com/shatteredpixel/shatteredpixeldungeon/levels/CavesLevel.java

@@ -144,13 +144,7 @@ public class CavesLevel extends RegularLevel {
 			}
 		}
 		
-		while (true) {
-			int pos = roomEntrance.random();
-			if (pos != entrance) {
-				map[pos] = Terrain.SIGN;
-				break;
-			}
-		}
+		placeSign();
 		
 		if (Dungeon.bossLevel( Dungeon.depth + 1 )) {
 			return;

src/com/shatteredpixel/shatteredpixeldungeon/levels/CityLevel.java

@@ -77,13 +77,7 @@ public class CityLevel extends RegularLevel {
 			}
 		}
 		
-		while (true) {
-			int pos = roomEntrance.random();
-			if (pos != entrance) {
-				map[pos] = Terrain.SIGN;
-				break;
-			}
-		}
+		placeSign();
 	}
 	
 	@Override

src/com/shatteredpixel/shatteredpixeldungeon/levels/HallsLevel.java

@@ -94,13 +94,7 @@ public class HallsLevel extends RegularLevel {
 			}
 		}
 		
-		while (true) {
-			int pos = roomEntrance.random();
-			if (pos != entrance) {
-				map[pos] = Terrain.SIGN;
-				break;
-			}
-		}
+		placeSign();
 	}
 	
 	@Override

src/com/shatteredpixel/shatteredpixeldungeon/levels/LastShopLevel.java

@@ -147,13 +147,7 @@ public class LastShopLevel extends RegularLevel {
 		}
 		
 		if (Imp.Quest.isCompleted()) {
-			while (true) {
-				int pos = roomEntrance.random();
-				if (pos != entrance) {
-					map[pos] = Terrain.SIGN;
-					break;
-				}
-			}
+			placeSign();
 		}
 	}
 	

src/com/shatteredpixel/shatteredpixeldungeon/levels/PrisonBossLevel.java

@@ -256,13 +256,7 @@ public class PrisonBossLevel extends RegularLevel {
 			}
 		}
 		
-		while (true) {
-			int pos = roomEntrance.random();
-			if (pos != entrance) {
-				map[pos] = Terrain.SIGN;
-				break;
-			}
-		}
+		placeSign();
 		
 		Point door = roomExit.entrance();
 		arenaDoor = door.x + door.y * WIDTH;

src/com/shatteredpixel/shatteredpixeldungeon/levels/PrisonLevel.java

@@ -119,13 +119,7 @@ public class PrisonLevel extends RegularLevel {
 			}
 		}
 		
-		while (true) {
-			int pos = roomEntrance.random();
-			if (pos != entrance) {
-				map[pos] = Terrain.SIGN;
-				break;
-			}
-		}
+		placeSign();
 	}
 	
 	@Override

src/com/shatteredpixel/shatteredpixeldungeon/levels/RegularLevel.java

@@ -168,6 +168,16 @@ public abstract class RegularLevel extends Level {
 		
 		return true;
 	}
+
+	protected void placeSign(){
+		while (true) {
+			int pos = roomEntrance.random();
+			if (pos != entrance || traps.get(pos) == null) {
+				map[pos] = Terrain.SIGN;
+				break;
+			}
+		}
+	}
 	
 	protected boolean initRooms() {
 

src/com/shatteredpixel/shatteredpixeldungeon/levels/SewerBossLevel.java

@@ -196,13 +196,7 @@ public class SewerBossLevel extends RegularLevel {
 			}
 		}
 		
-		while (true) {
-			int pos = roomEntrance.random();
-			if (pos != entrance) {
-				map[pos] = Terrain.SIGN;
-				break;
-			}
-		}
+		placeSign();
 	}
 	
 	@Override

src/com/shatteredpixel/shatteredpixeldungeon/levels/SewerLevel.java

@@ -93,13 +93,7 @@ public class SewerLevel extends RegularLevel {
 			}
 		}
 		
-		while (true) {
-			int pos = roomEntrance.random();
-			if (pos != entrance) {
-				map[pos] = Terrain.SIGN;
-				break;
-			}
-		}
+		placeSign();
 	}
 	
 	@Override