Merge pull request #31 from ScoreUnder/tweak-remove-key-restriction

Remove restriction from left/right click keys

src/itdelatrisu/opsu/Options.java

@@ -719,7 +719,7 @@ public class Options {
 	 */
 	public static int getGameKeyLeft() {
 		if (keyLeft == Keyboard.KEY_NONE)
-			keyLeft = Input.KEY_Z;
+			setGameKeyLeft(Input.KEY_Z);
 		return keyLeft;
 	}
 
@@ -729,21 +729,39 @@ public class Options {
 	 */
 	public static int getGameKeyRight() {
 		if (keyRight == Keyboard.KEY_NONE)
-			keyRight = Input.KEY_X;
+			setGameKeyRight(Input.KEY_X);
 		return keyRight;
 	}
 
 	/**
 	 * Sets the left game key.
+	 * This will not be set to the same key as the right game key, nor to reserved keys like ESCAPE.
 	 * @param key the keyboard key
+	 * @return {@code true} if the key was set, {@code false} if it was rejected
 	 */
-	public static void setGameKeyLeft(int key) { keyLeft = key; }
+	public static boolean setGameKeyLeft(int key) {
+		if (key == keyRight && key != Keyboard.KEY_NONE)
+			return false;
+		if (!isValidClickKey(key))
+			return false;
+		keyLeft = key;
+		return true;
+	}
 
 	/**
 	 * Sets the right game key.
+	 * This will not be set to the same key as the left game key, nor to reserved keys like ESCAPE.
 	 * @param key the keyboard key
+	 * @return {@code true} if the key was set, {@code false} if it was rejected
 	 */
-	public static void setGameKeyRight(int key) { keyRight = key; }
+	public static boolean setGameKeyRight(int key) {
+		if (key == keyLeft && key != Keyboard.KEY_NONE)
+			return false;
+		if (!isValidClickKey(key))
+			return false;
+		keyRight = key;
+		return true;
+	}
 
 	/**
 	 * Returns the beatmap directory.
@@ -831,6 +849,10 @@ public class Options {
 		return osu;
 	}
 
+	public static boolean isValidClickKey(int keyCode) {
+		return keyCode != Keyboard.KEY_ESCAPE;
+	}
+
 	/**
 	 * Reads user options from the options file, if it exists.
 	 */
@@ -941,20 +963,10 @@ public class Options {
 						GameOption.DISABLE_SOUNDS.setValue(Boolean.parseBoolean(value));
 						break;
 					case "keyOsuLeft":
-						if ((value.length() == 1 && Character.isLetterOrDigit(value.charAt(0))) ||
+						setGameKeyLeft(Keyboard.getKeyIndex(value));
-							(value.length() == 7 && value.startsWith("NUMPAD"))) {
-							i = Keyboard.getKeyIndex(value);
-							if (keyRight != i)
-								keyLeft = i;
-						}
 						break;
 					case "keyOsuRight":
-						if ((value.length() == 1 && Character.isLetterOrDigit(value.charAt(0))) ||
+						setGameKeyRight(Keyboard.getKeyIndex(value));
-							(value.length() == 7 && value.startsWith("NUMPAD"))) {
-							i = Keyboard.getKeyIndex(value);
-							if (keyLeft != i)
-								keyRight = i;
-						}
 						break;
 					case "MouseDisableWheel":
 						GameOption.DISABLE_MOUSE_WHEEL.setValue(Boolean.parseBoolean(value));

src/itdelatrisu/opsu/states/OptionsMenu.java

@@ -243,9 +243,17 @@ public class OptionsMenu extends BasicGameState {
 			g.setColor(Utils.COLOR_BLACK_ALPHA);
 			g.fillRect(0, 0, width, height);
 			g.setColor(Color.white);
+
+			String prompt;
+			if (keyEntryLeft) {
+				prompt = "Please press the new left-click key";
+			} else {
+				prompt = "Please press the new right-click key";
+			}
+
 			Utils.FONT_LARGE.drawString(
-					(width / 2) - (Utils.FONT_LARGE.getWidth("Please enter a letter or digit.") / 2),
+					(width / 2) - (Utils.FONT_LARGE.getWidth(prompt) / 2),
-					(height / 2) - Utils.FONT_LARGE.getLineHeight(), "Please enter a letter or digit."
+					(height / 2) - Utils.FONT_LARGE.getLineHeight(), prompt
 			);
 		}
 
@@ -339,12 +347,10 @@ public class OptionsMenu extends BasicGameState {
 	public void keyPressed(int key, char c) {
 		// key entry state
 		if (keyEntryLeft || keyEntryRight) {
-			if (Character.isLetterOrDigit(c)) {
+			if (keyEntryLeft)
-				if (keyEntryLeft && Options.getGameKeyRight() != key)
 				Options.setGameKeyLeft(key);
-				else if (keyEntryRight && Options.getGameKeyLeft() != key)
+			else
 				Options.setGameKeyRight(key);
-			}
 			keyEntryLeft = keyEntryRight = false;
 			return;
 		}