From e950575d4e11d27acbb6682b248de8f477a9ba50 Mon Sep 17 00:00:00 2001 From: olim Date: Tue, 23 Apr 2024 17:09:31 +0100 Subject: create mixin to render to the sign input screen and basic converter for k,m,b --- .../skyblocker/mixins/SignEditScreenMixin.java | 32 ++++++++++ .../hysky/skyblocker/skyblock/SignCalculator.java | 74 ++++++++++++++++++++++ 2 files changed, 106 insertions(+) create mode 100644 src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java create mode 100644 src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java (limited to 'src/main/java') diff --git a/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java b/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java new file mode 100644 index 00000000..a88f97ef --- /dev/null +++ b/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java @@ -0,0 +1,32 @@ +package de.hysky.skyblocker.mixin; + + +import de.hysky.skyblocker.skyblock.SignCalculator; +import net.minecraft.block.entity.SignBlockEntity; +import net.minecraft.client.gui.DrawContext; +import net.minecraft.client.gui.screen.ingame.AbstractSignEditScreen; +import org.spongepowered.asm.mixin.Final; +import org.spongepowered.asm.mixin.Mixin; +import org.spongepowered.asm.mixin.Shadow; +import org.spongepowered.asm.mixin.injection.At; +import org.spongepowered.asm.mixin.injection.Inject; +import org.spongepowered.asm.mixin.injection.callback.CallbackInfo; + +import java.util.Objects; + +@Mixin(AbstractSignEditScreen.class) +public abstract class SignEditScreenMixin{ + @Shadow + @Final + private String[] messages; + + @Inject(method = "render", at = @At("HEAD")) + private void skyblocker$render(DrawContext context, int mouseX, int mouseY, float delta, CallbackInfo ci) { + //if the sign is being used to enter number send it to the sign calculator + if (Objects.equals(messages[1], "^^^^^^^^^^^^^^^")) { + SignCalculator.renderSign(context, messages); + } + + } + +} diff --git a/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java b/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java new file mode 100644 index 00000000..ab0d417c --- /dev/null +++ b/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java @@ -0,0 +1,74 @@ +package de.hysky.skyblocker.skyblock; + +import net.minecraft.client.MinecraftClient; +import net.minecraft.client.gui.DrawContext; +import net.minecraft.text.Text; +import net.minecraft.util.Formatting; +import net.minecraft.util.Util; +import net.minecraft.util.math.Vec3i; + +import java.awt.*; +import java.awt.image.LookupTable; +import java.text.DecimalFormat; +import java.util.HashMap; +import java.util.regex.Matcher; +import java.util.regex.Pattern; + +public class SignCalculator { + + private static final MinecraftClient CLIENT = MinecraftClient.getInstance(); + + private static final Pattern NUMBER_PATTERN = Pattern.compile("(\\d+\\.?\\d*)([kmb]?)"); + private static final DecimalFormat FORMATTER = new DecimalFormat("#,###.##"); + + private static final HashMap magnitudeValues = Util.make(new HashMap<>(), map -> { + map.put("k", 1000); + map.put("m", 1000000); + map.put("b", 1000000000); + }); + + private static String input; + private static Double output; + + public static void renderSign(DrawContext context, String[] messages){ + input = messages[0]; + + calculateValue(); + + render(context); + } + + public static void calculateValue() { + Matcher numberMatcher = NUMBER_PATTERN.matcher(input.toLowerCase()); + if (!numberMatcher.matches()) { + output = null; + return; + } + double number = Double.parseDouble(numberMatcher.group(1)); // + String magnitude = numberMatcher.group(2); + + if (!magnitude.isEmpty()) { + if (!magnitudeValues.containsKey(magnitude)) {//its invalid if its another letter + output = null; + return; + } + number *= magnitudeValues.get(magnitude); + } + + output = number; + + } + + + private static void render(DrawContext context) { + Text text; + if (output == null) { + text = Text.literal("test").formatted(Formatting.RED); + } + else { + text = Text.literal(input +" = " + FORMATTER.format(output)).formatted(Formatting.GREEN); + } + + context.drawCenteredTextWithShadow(CLIENT.textRenderer, text,context.getScaledWindowWidth() /2 , 55,0xFFFFFFFF); + } +} -- cgit From e618d9e1e57b17c55b7b589e8a7e3f2ca2e95428 Mon Sep 17 00:00:00 2001 From: olim Date: Wed, 24 Apr 2024 13:12:46 +0100 Subject: calculations work should probaly move to utils function and need to add config and proper text --- .../skyblocker/mixins/SignEditScreenMixin.java | 8 + .../hysky/skyblocker/skyblock/SignCalculator.java | 223 +++++++++++++++++++-- 2 files changed, 214 insertions(+), 17 deletions(-) (limited to 'src/main/java') diff --git a/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java b/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java index a88f97ef..b160b3ac 100644 --- a/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java +++ b/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java @@ -26,6 +26,14 @@ public abstract class SignEditScreenMixin{ if (Objects.equals(messages[1], "^^^^^^^^^^^^^^^")) { SignCalculator.renderSign(context, messages); } + } + + @Inject(method = "finishEditing", at = @At("HEAD")) + private void skyblocker$finishEditing(CallbackInfo ci) { + //if the sign is being used to enter number get number from calculator for if maths has been done + if (Objects.equals(messages[1], "^^^^^^^^^^^^^^^")) { + messages[0] = SignCalculator.getNewValue(); + } } diff --git a/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java b/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java index ab0d417c..66b32028 100644 --- a/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java +++ b/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java @@ -1,16 +1,14 @@ package de.hysky.skyblocker.skyblock; +import de.hysky.skyblocker.skyblock.dungeon.puzzle.waterboard.Switch; import net.minecraft.client.MinecraftClient; import net.minecraft.client.gui.DrawContext; import net.minecraft.text.Text; import net.minecraft.util.Formatting; import net.minecraft.util.Util; -import net.minecraft.util.math.Vec3i; -import java.awt.*; -import java.awt.image.LookupTable; import java.text.DecimalFormat; -import java.util.HashMap; +import java.util.*; import java.util.regex.Matcher; import java.util.regex.Pattern; @@ -18,48 +16,239 @@ public class SignCalculator { private static final MinecraftClient CLIENT = MinecraftClient.getInstance(); - private static final Pattern NUMBER_PATTERN = Pattern.compile("(\\d+\\.?\\d*)([kmb]?)"); - private static final DecimalFormat FORMATTER = new DecimalFormat("#,###.##"); - + public enum TokenType { + NUMBER, OPERATOR, L_PARENTHESIS, R_PARENTHESIS + } + public static class Token { + public TokenType type; + String value; + int tokenLength; + } + private static final Pattern NUMBER_PATTERN = Pattern.compile("(\\d+\\.?\\d*)([kmbs]?)"); private static final HashMap magnitudeValues = Util.make(new HashMap<>(), map -> { + map.put("s", 64); map.put("k", 1000); map.put("m", 1000000); map.put("b", 1000000000); }); + + + + private static final DecimalFormat FORMATTER = new DecimalFormat("#,###.##"); + + private static String lastInput; private static String input; private static Double output; public static void renderSign(DrawContext context, String[] messages){ input = messages[0]; - calculateValue(); + + //only update output if new input + if (!input.equals(lastInput)) { // + try { + output = evaluate(shunt(lex(input))); + } catch (Exception e){ + output = null; //todo log + } + } + render(context); + + lastInput = input; + } + + public static String getNewValue() { + if (output == null) { + return ""; + } + return Double.toString(output); + } + + private static List lex(String input) { + List tokens = new ArrayList<>(); + input = input.replace(" ", "").toLowerCase().replace("x","*"); + int i = 0; + while (i < input.length()) { + Token token = new Token(); + switch (input.charAt(i)) { + case '+','-','*','/' -> { + token.type = TokenType.OPERATOR; + token.value = String.valueOf(input.charAt(i)); + token.tokenLength = 1; + } + + case '(' -> { + token.type = TokenType.L_PARENTHESIS; + token.value = String.valueOf(input.charAt(i)); + token.tokenLength = 1; + //add implicit multiplication when there is a number before brackets + if (!tokens.isEmpty() ) { + TokenType lastType = tokens.get(tokens.size()-1).type; + if (lastType == TokenType.R_PARENTHESIS || lastType == TokenType.NUMBER) { + Token mutliplyToken = new Token(); + mutliplyToken.type = TokenType.OPERATOR; + mutliplyToken.value = "*"; + tokens.add(mutliplyToken); + } + } + } + + case ')' -> { + token.type = TokenType.R_PARENTHESIS; + token.value = String.valueOf(input.charAt(i)); + token.tokenLength = 1; + } + + default -> { + token.type = TokenType.NUMBER; + Matcher numberMatcher = NUMBER_PATTERN.matcher(input.substring(i)); + if (!numberMatcher.find()) {//invalid value to lex + throw new UnsupportedOperationException(); + } + int end = numberMatcher.end(); + token.value = input.substring(i,i + end); + token.tokenLength = end; + } + } + tokens.add(token); + + i += token.tokenLength; + } + + return tokens; } - public static void calculateValue() { - Matcher numberMatcher = NUMBER_PATTERN.matcher(input.toLowerCase()); + private static List shunt(List tokens) { + // This is an implementation of the shunting yard algorithm + // Converts equation to use reverse polish notation + + Deque operatorStack = new ArrayDeque<>(); + List outputQueue = new ArrayList<>(); + + for (Token shuntingToken : tokens) + switch (shuntingToken.type) { + case NUMBER -> { + outputQueue.add(shuntingToken); + } + case OPERATOR -> { + int precedence = getPrecedence(shuntingToken.value); + while (!operatorStack.isEmpty()) { + Token leftToken = operatorStack.peek(); + if (leftToken.type == TokenType.L_PARENTHESIS) { + break; + } + assert (leftToken.type == TokenType.OPERATOR); //todo why is this here + int leftPrecedence = getPrecedence(leftToken.value); + if (leftPrecedence >= precedence) { + outputQueue.add(operatorStack.pop()); + continue; + } + break; + } + operatorStack.push(shuntingToken); + } + case L_PARENTHESIS -> { + operatorStack.push(shuntingToken); + } + case R_PARENTHESIS -> { + while (true) { + if (operatorStack.isEmpty()) { + throw new UnsupportedOperationException("Unbalanced left parenthesis"); + } + Token leftToken = operatorStack.pop(); + if (leftToken.type == TokenType.L_PARENTHESIS) { + break; + } + outputQueue.add(leftToken); + } + } + } + //empty the operator stack + while (!operatorStack.isEmpty()) { + Token leftToken = operatorStack.pop(); + if (leftToken.type == TokenType.L_PARENTHESIS) { + throw new UnsupportedOperationException("Unbalanced left parenthesis"); + } + outputQueue.add(leftToken); + } + + return outputQueue.stream().toList(); + } + private static int getPrecedence(String operator) { + switch (operator) { + case "+","-" -> { + return 0; + } + case "*","/" -> { + return 1; + } + default -> { + throw new UnsupportedOperationException(); + } + } + } + + /** + * + * @param tokens list of Tokens in reverse polish notation + * @return answer to equation + */ + private static double evaluate(List tokens) { + Deque values = new ArrayDeque<>(); + for (Token token : tokens) { + switch (token.type) { + case NUMBER -> { + values.push(calculateValue(token.value)); + } + case OPERATOR -> { + double right = values.pop(); + double left = values.pop(); + switch (token.value) { + case "+" -> { + values.push(left + right); + } + case "-" -> { + values.push(left - right); + } + case "/" -> { + values.push(left / right); + } + case "*" -> { + values.push(left * right); + } + } + } + case L_PARENTHESIS, R_PARENTHESIS -> { + throw new UnsupportedOperationException("equation is not in RPN"); + } + } + } + return values.pop(); + } + + private static double calculateValue(String value) { + Matcher numberMatcher = NUMBER_PATTERN.matcher(value.toLowerCase()); if (!numberMatcher.matches()) { - output = null; - return; + throw new UnsupportedOperationException(); } - double number = Double.parseDouble(numberMatcher.group(1)); // + double number = Double.parseDouble(numberMatcher.group(1)); String magnitude = numberMatcher.group(2); if (!magnitude.isEmpty()) { if (!magnitudeValues.containsKey(magnitude)) {//its invalid if its another letter - output = null; - return; + throw new UnsupportedOperationException(); } number *= magnitudeValues.get(magnitude); } - output = number; - + return number; } + private static void render(DrawContext context) { Text text; if (output == null) { -- cgit From aa97d9a5bb3e9274ac04b47c5d711c20e7b30087 Mon Sep 17 00:00:00 2001 From: olim Date: Wed, 24 Apr 2024 16:26:50 +0100 Subject: fix amount outputs to be full numbers --- .../java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java | 7 ++++++- src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java | 9 +++++++-- 2 files changed, 13 insertions(+), 3 deletions(-) (limited to 'src/main/java') diff --git a/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java b/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java index b160b3ac..3c1667f0 100644 --- a/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java +++ b/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java @@ -32,7 +32,12 @@ public abstract class SignEditScreenMixin{ private void skyblocker$finishEditing(CallbackInfo ci) { //if the sign is being used to enter number get number from calculator for if maths has been done if (Objects.equals(messages[1], "^^^^^^^^^^^^^^^")) { - messages[0] = SignCalculator.getNewValue(); + boolean isPrice = messages[2].contains("price"); + String value = SignCalculator.getNewValue(isPrice); + if (value.length() >= 15) { + value = value.substring(0,15); + } + messages[0] = value; } } diff --git a/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java b/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java index 66b32028..e71442a5 100644 --- a/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java +++ b/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java @@ -60,11 +60,16 @@ public class SignCalculator { lastInput = input; } - public static String getNewValue() { + public static String getNewValue(Boolean isPrice) { if (output == null) { return ""; } - return Double.toString(output); + //price can except decimals and exponents + if (isPrice) { + return output.toString(); + } + //amounts want an integer number so round + return Long.toString(Math.round(output)); } private static List lex(String input) { -- cgit From 6ae6896b147cd9e012f52e15a2d5ac0beb898c44 Mon Sep 17 00:00:00 2001 From: olim Date: Wed, 24 Apr 2024 17:14:09 +0100 Subject: move the calculator to a util function --- .../hysky/skyblocker/skyblock/SignCalculator.java | 207 +------------------- .../java/de/hysky/skyblocker/utils/Calculator.java | 213 +++++++++++++++++++++ 2 files changed, 216 insertions(+), 204 deletions(-) create mode 100644 src/main/java/de/hysky/skyblocker/utils/Calculator.java (limited to 'src/main/java') diff --git a/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java b/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java index e71442a5..cf621ca7 100644 --- a/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java +++ b/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java @@ -1,6 +1,7 @@ package de.hysky.skyblocker.skyblock; import de.hysky.skyblocker.skyblock.dungeon.puzzle.waterboard.Switch; +import de.hysky.skyblocker.utils.Calculator; import net.minecraft.client.MinecraftClient; import net.minecraft.client.gui.DrawContext; import net.minecraft.text.Text; @@ -16,25 +17,6 @@ public class SignCalculator { private static final MinecraftClient CLIENT = MinecraftClient.getInstance(); - public enum TokenType { - NUMBER, OPERATOR, L_PARENTHESIS, R_PARENTHESIS - } - public static class Token { - public TokenType type; - String value; - int tokenLength; - } - private static final Pattern NUMBER_PATTERN = Pattern.compile("(\\d+\\.?\\d*)([kmbs]?)"); - private static final HashMap magnitudeValues = Util.make(new HashMap<>(), map -> { - map.put("s", 64); - map.put("k", 1000); - map.put("m", 1000000); - map.put("b", 1000000000); - }); - - - - private static final DecimalFormat FORMATTER = new DecimalFormat("#,###.##"); private static String lastInput; @@ -48,13 +30,12 @@ public class SignCalculator { //only update output if new input if (!input.equals(lastInput)) { // try { - output = evaluate(shunt(lex(input))); + output = Calculator.calculate(input); } catch (Exception e){ output = null; //todo log } } - render(context); lastInput = input; @@ -72,192 +53,10 @@ public class SignCalculator { return Long.toString(Math.round(output)); } - private static List lex(String input) { - List tokens = new ArrayList<>(); - input = input.replace(" ", "").toLowerCase().replace("x","*"); - int i = 0; - while (i < input.length()) { - Token token = new Token(); - switch (input.charAt(i)) { - case '+','-','*','/' -> { - token.type = TokenType.OPERATOR; - token.value = String.valueOf(input.charAt(i)); - token.tokenLength = 1; - } - - case '(' -> { - token.type = TokenType.L_PARENTHESIS; - token.value = String.valueOf(input.charAt(i)); - token.tokenLength = 1; - //add implicit multiplication when there is a number before brackets - if (!tokens.isEmpty() ) { - TokenType lastType = tokens.get(tokens.size()-1).type; - if (lastType == TokenType.R_PARENTHESIS || lastType == TokenType.NUMBER) { - Token mutliplyToken = new Token(); - mutliplyToken.type = TokenType.OPERATOR; - mutliplyToken.value = "*"; - tokens.add(mutliplyToken); - } - } - } - - case ')' -> { - token.type = TokenType.R_PARENTHESIS; - token.value = String.valueOf(input.charAt(i)); - token.tokenLength = 1; - } - - default -> { - token.type = TokenType.NUMBER; - Matcher numberMatcher = NUMBER_PATTERN.matcher(input.substring(i)); - if (!numberMatcher.find()) {//invalid value to lex - throw new UnsupportedOperationException(); - } - int end = numberMatcher.end(); - token.value = input.substring(i,i + end); - token.tokenLength = end; - } - } - tokens.add(token); - - i += token.tokenLength; - } - - return tokens; - } - - private static List shunt(List tokens) { - // This is an implementation of the shunting yard algorithm - // Converts equation to use reverse polish notation - - Deque operatorStack = new ArrayDeque<>(); - List outputQueue = new ArrayList<>(); - - for (Token shuntingToken : tokens) - switch (shuntingToken.type) { - case NUMBER -> { - outputQueue.add(shuntingToken); - } - case OPERATOR -> { - int precedence = getPrecedence(shuntingToken.value); - while (!operatorStack.isEmpty()) { - Token leftToken = operatorStack.peek(); - if (leftToken.type == TokenType.L_PARENTHESIS) { - break; - } - assert (leftToken.type == TokenType.OPERATOR); //todo why is this here - int leftPrecedence = getPrecedence(leftToken.value); - if (leftPrecedence >= precedence) { - outputQueue.add(operatorStack.pop()); - continue; - } - break; - } - operatorStack.push(shuntingToken); - } - case L_PARENTHESIS -> { - operatorStack.push(shuntingToken); - } - case R_PARENTHESIS -> { - while (true) { - if (operatorStack.isEmpty()) { - throw new UnsupportedOperationException("Unbalanced left parenthesis"); - } - Token leftToken = operatorStack.pop(); - if (leftToken.type == TokenType.L_PARENTHESIS) { - break; - } - outputQueue.add(leftToken); - } - } - } - //empty the operator stack - while (!operatorStack.isEmpty()) { - Token leftToken = operatorStack.pop(); - if (leftToken.type == TokenType.L_PARENTHESIS) { - throw new UnsupportedOperationException("Unbalanced left parenthesis"); - } - outputQueue.add(leftToken); - } - - return outputQueue.stream().toList(); - } - private static int getPrecedence(String operator) { - switch (operator) { - case "+","-" -> { - return 0; - } - case "*","/" -> { - return 1; - } - default -> { - throw new UnsupportedOperationException(); - } - } - } - - /** - * - * @param tokens list of Tokens in reverse polish notation - * @return answer to equation - */ - private static double evaluate(List tokens) { - Deque values = new ArrayDeque<>(); - for (Token token : tokens) { - switch (token.type) { - case NUMBER -> { - values.push(calculateValue(token.value)); - } - case OPERATOR -> { - double right = values.pop(); - double left = values.pop(); - switch (token.value) { - case "+" -> { - values.push(left + right); - } - case "-" -> { - values.push(left - right); - } - case "/" -> { - values.push(left / right); - } - case "*" -> { - values.push(left * right); - } - } - } - case L_PARENTHESIS, R_PARENTHESIS -> { - throw new UnsupportedOperationException("equation is not in RPN"); - } - } - } - return values.pop(); - } - - private static double calculateValue(String value) { - Matcher numberMatcher = NUMBER_PATTERN.matcher(value.toLowerCase()); - if (!numberMatcher.matches()) { - throw new UnsupportedOperationException(); - } - double number = Double.parseDouble(numberMatcher.group(1)); - String magnitude = numberMatcher.group(2); - - if (!magnitude.isEmpty()) { - if (!magnitudeValues.containsKey(magnitude)) {//its invalid if its another letter - throw new UnsupportedOperationException(); - } - number *= magnitudeValues.get(magnitude); - } - - return number; - } - - - private static void render(DrawContext context) { Text text; if (output == null) { - text = Text.literal("test").formatted(Formatting.RED); + text = Text.literal("Invalid Equation").formatted(Formatting.RED); } else { text = Text.literal(input +" = " + FORMATTER.format(output)).formatted(Formatting.GREEN); diff --git a/src/main/java/de/hysky/skyblocker/utils/Calculator.java b/src/main/java/de/hysky/skyblocker/utils/Calculator.java new file mode 100644 index 00000000..61cdf169 --- /dev/null +++ b/src/main/java/de/hysky/skyblocker/utils/Calculator.java @@ -0,0 +1,213 @@ +package de.hysky.skyblocker.utils; + +import de.hysky.skyblocker.skyblock.SignCalculator; +import net.minecraft.util.Util; + +import java.util.*; +import java.util.regex.Matcher; +import java.util.regex.Pattern; + +public class Calculator { + + public enum TokenType { + NUMBER, OPERATOR, L_PARENTHESIS, R_PARENTHESIS + } + public static class Token { + public TokenType type; + String value; + int tokenLength; + } + private static final Pattern NUMBER_PATTERN = Pattern.compile("(\\d+\\.?\\d*)([kmbs]?)"); + private static final HashMap magnitudeValues = Util.make(new HashMap<>(), map -> { + map.put("s", 64); + map.put("k", 1000); + map.put("m", 1000000); + map.put("b", 1000000000); + }); + + + private static List lex(String input) { + List tokens = new ArrayList<>(); + input = input.replace(" ", "").toLowerCase().replace("x","*"); + int i = 0; + while (i < input.length()) { + Token token = new Token(); + switch (input.charAt(i)) { + case '+','-','*','/' -> { + token.type = TokenType.OPERATOR; + token.value = String.valueOf(input.charAt(i)); + token.tokenLength = 1; + } + + case '(' -> { + token.type = TokenType.L_PARENTHESIS; + token.value = String.valueOf(input.charAt(i)); + token.tokenLength = 1; + //add implicit multiplication when there is a number before brackets + if (!tokens.isEmpty() ) { + TokenType lastType = tokens.get(tokens.size()-1).type; + if (lastType == TokenType.R_PARENTHESIS || lastType == TokenType.NUMBER) { + Token mutliplyToken = new Token(); + mutliplyToken.type = TokenType.OPERATOR; + mutliplyToken.value = "*"; + tokens.add(mutliplyToken); + } + } + } + + case ')' -> { + token.type = TokenType.R_PARENTHESIS; + token.value = String.valueOf(input.charAt(i)); + token.tokenLength = 1; + } + + default -> { + token.type = TokenType.NUMBER; + Matcher numberMatcher = NUMBER_PATTERN.matcher(input.substring(i)); + if (!numberMatcher.find()) {//invalid value to lex + throw new UnsupportedOperationException(); + } + int end = numberMatcher.end(); + token.value = input.substring(i,i + end); + token.tokenLength = end; + } + } + tokens.add(token); + + i += token.tokenLength; + } + + return tokens; + } + + private static List shunt(List tokens) { + // This is an implementation of the shunting yard algorithm + // Converts equation to use reverse polish notation + + Deque operatorStack = new ArrayDeque<>(); + List outputQueue = new ArrayList<>(); + + for (Token shuntingToken : tokens) + switch (shuntingToken.type) { + case NUMBER -> { + outputQueue.add(shuntingToken); + } + case OPERATOR -> { + int precedence = getPrecedence(shuntingToken.value); + while (!operatorStack.isEmpty()) { + Token leftToken = operatorStack.peek(); + if (leftToken.type == TokenType.L_PARENTHESIS) { + break; + } + assert (leftToken.type == TokenType.OPERATOR); //todo why is this here + int leftPrecedence = getPrecedence(leftToken.value); + if (leftPrecedence >= precedence) { + outputQueue.add(operatorStack.pop()); + continue; + } + break; + } + operatorStack.push(shuntingToken); + } + case L_PARENTHESIS -> { + operatorStack.push(shuntingToken); + } + case R_PARENTHESIS -> { + while (true) { + if (operatorStack.isEmpty()) { + throw new UnsupportedOperationException("Unbalanced left parenthesis"); + } + Token leftToken = operatorStack.pop(); + if (leftToken.type == TokenType.L_PARENTHESIS) { + break; + } + outputQueue.add(leftToken); + } + } + } + //empty the operator stack + while (!operatorStack.isEmpty()) { + Token leftToken = operatorStack.pop(); + if (leftToken.type == TokenType.L_PARENTHESIS) { + throw new UnsupportedOperationException("Unbalanced left parenthesis"); + } + outputQueue.add(leftToken); + } + + return outputQueue.stream().toList(); + } + private static int getPrecedence(String operator) { + switch (operator) { + case "+","-" -> { + return 0; + } + case "*","/" -> { + return 1; + } + default -> { + throw new UnsupportedOperationException(); + } + } + } + + /** + * + * @param tokens list of Tokens in reverse polish notation + * @return answer to equation + */ + private static double evaluate(List tokens) { + Deque values = new ArrayDeque<>(); + for (Token token : tokens) { + switch (token.type) { + case NUMBER -> { + values.push(calculateValue(token.value)); + } + case OPERATOR -> { + double right = values.pop(); + double left = values.pop(); + switch (token.value) { + case "+" -> { + values.push(left + right); + } + case "-" -> { + values.push(left - right); + } + case "/" -> { + values.push(left / right); + } + case "*" -> { + values.push(left * right); + } + } + } + case L_PARENTHESIS, R_PARENTHESIS -> { + throw new UnsupportedOperationException("equation is not in RPN"); + } + } + } + return values.pop(); + } + + private static double calculateValue(String value) { + Matcher numberMatcher = NUMBER_PATTERN.matcher(value.toLowerCase()); + if (!numberMatcher.matches()) { + throw new UnsupportedOperationException(); + } + double number = Double.parseDouble(numberMatcher.group(1)); + String magnitude = numberMatcher.group(2); + + if (!magnitude.isEmpty()) { + if (!magnitudeValues.containsKey(magnitude)) {//its invalid if its another letter + throw new UnsupportedOperationException(); + } + number *= magnitudeValues.get(magnitude); + } + + return number; + } + + public static double calculate(String equation) { + return evaluate(shunt(lex(equation))); + } + +} -- cgit From 05c44d712d1a4f950733fae875cb97792e4fef1e Mon Sep 17 00:00:00 2001 From: olim Date: Wed, 24 Apr 2024 17:33:53 +0100 Subject: add messages to exceptions --- .../hysky/skyblocker/skyblock/SignCalculator.java | 2 +- .../java/de/hysky/skyblocker/utils/Calculator.java | 22 ++++++++++++++-------- 2 files changed, 15 insertions(+), 9 deletions(-) (limited to 'src/main/java') diff --git a/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java b/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java index cf621ca7..cbff07c9 100644 --- a/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java +++ b/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java @@ -32,7 +32,7 @@ public class SignCalculator { try { output = Calculator.calculate(input); } catch (Exception e){ - output = null; //todo log + output = null; } } diff --git a/src/main/java/de/hysky/skyblocker/utils/Calculator.java b/src/main/java/de/hysky/skyblocker/utils/Calculator.java index 61cdf169..fb2b8b11 100644 --- a/src/main/java/de/hysky/skyblocker/utils/Calculator.java +++ b/src/main/java/de/hysky/skyblocker/utils/Calculator.java @@ -65,7 +65,7 @@ public class Calculator { token.type = TokenType.NUMBER; Matcher numberMatcher = NUMBER_PATTERN.matcher(input.substring(i)); if (!numberMatcher.find()) {//invalid value to lex - throw new UnsupportedOperationException(); + throw new UnsupportedOperationException("invalid character"); } int end = numberMatcher.end(); token.value = input.substring(i,i + end); @@ -80,10 +80,13 @@ public class Calculator { return tokens; } - private static List shunt(List tokens) { - // This is an implementation of the shunting yard algorithm - // Converts equation to use reverse polish notation + /** + * This is an implementation of the shunting yard algorithm to convert the equation to reverse polish notation + * @param tokens equation in infix notation order + * @return equation in RPN order + */ + private static List shunt(List tokens) { Deque operatorStack = new ArrayDeque<>(); List outputQueue = new ArrayList<>(); @@ -99,7 +102,7 @@ public class Calculator { if (leftToken.type == TokenType.L_PARENTHESIS) { break; } - assert (leftToken.type == TokenType.OPERATOR); //todo why is this here + assert (leftToken.type == TokenType.OPERATOR); int leftPrecedence = getPrecedence(leftToken.value); if (leftPrecedence >= precedence) { outputQueue.add(operatorStack.pop()); @@ -145,7 +148,7 @@ public class Calculator { return 1; } default -> { - throw new UnsupportedOperationException(); + throw new UnsupportedOperationException("invalid operator"); } } } @@ -173,6 +176,9 @@ public class Calculator { values.push(left - right); } case "/" -> { + if (right == 0) { + throw new UnsupportedOperationException("Can not divide by 0"); + } values.push(left / right); } case "*" -> { @@ -191,14 +197,14 @@ public class Calculator { private static double calculateValue(String value) { Matcher numberMatcher = NUMBER_PATTERN.matcher(value.toLowerCase()); if (!numberMatcher.matches()) { - throw new UnsupportedOperationException(); + throw new UnsupportedOperationException("invalid number"); } double number = Double.parseDouble(numberMatcher.group(1)); String magnitude = numberMatcher.group(2); if (!magnitude.isEmpty()) { if (!magnitudeValues.containsKey(magnitude)) {//its invalid if its another letter - throw new UnsupportedOperationException(); + throw new UnsupportedOperationException("invalid magnitude"); } number *= magnitudeValues.get(magnitude); } -- cgit From 0619e195661f4ed285786ea679a27e1f28f9dd6e Mon Sep 17 00:00:00 2001 From: olim Date: Wed, 24 Apr 2024 17:41:35 +0100 Subject: add config option --- src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java | 7 +++++-- src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java | 2 +- 2 files changed, 6 insertions(+), 3 deletions(-) (limited to 'src/main/java') diff --git a/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java b/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java index 3c1667f0..867edb7d 100644 --- a/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java +++ b/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java @@ -1,8 +1,11 @@ package de.hysky.skyblocker.mixin; +import de.hysky.skyblocker.config.SkyblockerConfigManager; import de.hysky.skyblocker.skyblock.SignCalculator; +import de.hysky.skyblocker.utils.Utils; import net.minecraft.block.entity.SignBlockEntity; +import net.minecraft.client.MinecraftClient; import net.minecraft.client.gui.DrawContext; import net.minecraft.client.gui.screen.ingame.AbstractSignEditScreen; import org.spongepowered.asm.mixin.Final; @@ -23,7 +26,7 @@ public abstract class SignEditScreenMixin{ @Inject(method = "render", at = @At("HEAD")) private void skyblocker$render(DrawContext context, int mouseX, int mouseY, float delta, CallbackInfo ci) { //if the sign is being used to enter number send it to the sign calculator - if (Objects.equals(messages[1], "^^^^^^^^^^^^^^^")) { + if (Utils.isOnSkyblock() && SkyblockerConfigManager.get().general.enableSignCalculator && Objects.equals(messages[1], "^^^^^^^^^^^^^^^")) { SignCalculator.renderSign(context, messages); } } @@ -31,7 +34,7 @@ public abstract class SignEditScreenMixin{ @Inject(method = "finishEditing", at = @At("HEAD")) private void skyblocker$finishEditing(CallbackInfo ci) { //if the sign is being used to enter number get number from calculator for if maths has been done - if (Objects.equals(messages[1], "^^^^^^^^^^^^^^^")) { + if (Utils.isOnSkyblock() && SkyblockerConfigManager.get().general.enableSignCalculator && Objects.equals(messages[1], "^^^^^^^^^^^^^^^")) { boolean isPrice = messages[2].contains("price"); String value = SignCalculator.getNewValue(isPrice); if (value.length() >= 15) { diff --git a/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java b/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java index cbff07c9..be393b49 100644 --- a/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java +++ b/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java @@ -56,7 +56,7 @@ public class SignCalculator { private static void render(DrawContext context) { Text text; if (output == null) { - text = Text.literal("Invalid Equation").formatted(Formatting.RED); + text = Text.translatable("text.autoconfig.skyblocker.option.general.enableSignCalculator.invalidEquation").formatted(Formatting.RED); } else { text = Text.literal(input +" = " + FORMATTER.format(output)).formatted(Formatting.GREEN); -- cgit From 79ded04ec6834ad28f23f9db0f205689deda467d Mon Sep 17 00:00:00 2001 From: olim Date: Thu, 25 Apr 2024 11:45:23 +0100 Subject: be more lenient with brackets --- src/main/java/de/hysky/skyblocker/utils/Calculator.java | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) (limited to 'src/main/java') diff --git a/src/main/java/de/hysky/skyblocker/utils/Calculator.java b/src/main/java/de/hysky/skyblocker/utils/Calculator.java index fb2b8b11..c8dcb6d8 100644 --- a/src/main/java/de/hysky/skyblocker/utils/Calculator.java +++ b/src/main/java/de/hysky/skyblocker/utils/Calculator.java @@ -132,7 +132,8 @@ public class Calculator { while (!operatorStack.isEmpty()) { Token leftToken = operatorStack.pop(); if (leftToken.type == TokenType.L_PARENTHESIS) { - throw new UnsupportedOperationException("Unbalanced left parenthesis"); + //technically unbalanced left parenthesis error but just assume they are close after the equation and ignore them from here + continue; } outputQueue.add(leftToken); } -- cgit From feee94e2c2ce01262f3eb47d7b356ca9821bdb79 Mon Sep 17 00:00:00 2001 From: olim Date: Fri, 26 Apr 2024 18:01:09 +0100 Subject: add command add command to be able to use the calculator and fix crash with emty equations --- .../java/de/hysky/skyblocker/SkyblockerMod.java | 2 + .../skyblocker/mixins/SignEditScreenMixin.java | 4 +- .../hysky/skyblocker/skyblock/SignCalculator.java | 67 ---------------------- .../skyblock/calculators/CalculatorCommand.java | 57 ++++++++++++++++++ .../skyblock/calculators/SignCalculator.java | 62 ++++++++++++++++++++ .../java/de/hysky/skyblocker/utils/Calculator.java | 14 ++--- 6 files changed, 129 insertions(+), 77 deletions(-) delete mode 100644 src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java create mode 100644 src/main/java/de/hysky/skyblocker/skyblock/calculators/CalculatorCommand.java create mode 100644 src/main/java/de/hysky/skyblocker/skyblock/calculators/SignCalculator.java (limited to 'src/main/java') diff --git a/src/main/java/de/hysky/skyblocker/SkyblockerMod.java b/src/main/java/de/hysky/skyblocker/SkyblockerMod.java index 3336cefb..9840c02f 100644 --- a/src/main/java/de/hysky/skyblocker/SkyblockerMod.java +++ b/src/main/java/de/hysky/skyblocker/SkyblockerMod.java @@ -7,6 +7,7 @@ import de.hysky.skyblocker.config.ImageRepoLoader; import de.hysky.skyblocker.config.SkyblockerConfigManager; import de.hysky.skyblocker.debug.Debug; import de.hysky.skyblocker.skyblock.*; +import de.hysky.skyblocker.skyblock.calculators.CalculatorCommand; import de.hysky.skyblocker.skyblock.chat.ChatRuleAnnouncementScreen; import de.hysky.skyblocker.skyblock.chat.ChatRulesHandler; import de.hysky.skyblocker.skyblock.crimson.kuudra.Kuudra; @@ -132,6 +133,7 @@ public class SkyblockerMod implements ClientModInitializer { Shortcuts.init(); ChatRulesHandler.init(); ChatRuleAnnouncementScreen.init(); + CalculatorCommand.init(); DiscordRPCManager.init(); LividColor.init(); FishingHelper.init(); diff --git a/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java b/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java index 867edb7d..88c54878 100644 --- a/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java +++ b/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java @@ -2,10 +2,8 @@ package de.hysky.skyblocker.mixin; import de.hysky.skyblocker.config.SkyblockerConfigManager; -import de.hysky.skyblocker.skyblock.SignCalculator; +import de.hysky.skyblocker.skyblock.calculators.SignCalculator; import de.hysky.skyblocker.utils.Utils; -import net.minecraft.block.entity.SignBlockEntity; -import net.minecraft.client.MinecraftClient; import net.minecraft.client.gui.DrawContext; import net.minecraft.client.gui.screen.ingame.AbstractSignEditScreen; import org.spongepowered.asm.mixin.Final; diff --git a/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java b/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java deleted file mode 100644 index be393b49..00000000 --- a/src/main/java/de/hysky/skyblocker/skyblock/SignCalculator.java +++ /dev/null @@ -1,67 +0,0 @@ -package de.hysky.skyblocker.skyblock; - -import de.hysky.skyblocker.skyblock.dungeon.puzzle.waterboard.Switch; -import de.hysky.skyblocker.utils.Calculator; -import net.minecraft.client.MinecraftClient; -import net.minecraft.client.gui.DrawContext; -import net.minecraft.text.Text; -import net.minecraft.util.Formatting; -import net.minecraft.util.Util; - -import java.text.DecimalFormat; -import java.util.*; -import java.util.regex.Matcher; -import java.util.regex.Pattern; - -public class SignCalculator { - - private static final MinecraftClient CLIENT = MinecraftClient.getInstance(); - - private static final DecimalFormat FORMATTER = new DecimalFormat("#,###.##"); - - private static String lastInput; - private static String input; - private static Double output; - - public static void renderSign(DrawContext context, String[] messages){ - input = messages[0]; - - - //only update output if new input - if (!input.equals(lastInput)) { // - try { - output = Calculator.calculate(input); - } catch (Exception e){ - output = null; - } - } - - render(context); - - lastInput = input; - } - - public static String getNewValue(Boolean isPrice) { - if (output == null) { - return ""; - } - //price can except decimals and exponents - if (isPrice) { - return output.toString(); - } - //amounts want an integer number so round - return Long.toString(Math.round(output)); - } - - private static void render(DrawContext context) { - Text text; - if (output == null) { - text = Text.translatable("text.autoconfig.skyblocker.option.general.enableSignCalculator.invalidEquation").formatted(Formatting.RED); - } - else { - text = Text.literal(input +" = " + FORMATTER.format(output)).formatted(Formatting.GREEN); - } - - context.drawCenteredTextWithShadow(CLIENT.textRenderer, text,context.getScaledWindowWidth() /2 , 55,0xFFFFFFFF); - } -} diff --git a/src/main/java/de/hysky/skyblocker/skyblock/calculators/CalculatorCommand.java b/src/main/java/de/hysky/skyblocker/skyblock/calculators/CalculatorCommand.java new file mode 100644 index 00000000..f9b8b4cf --- /dev/null +++ b/src/main/java/de/hysky/skyblocker/skyblock/calculators/CalculatorCommand.java @@ -0,0 +1,57 @@ +package de.hysky.skyblocker.skyblock.calculators; + +import com.mojang.brigadier.Command; +import com.mojang.brigadier.CommandDispatcher; +import com.mojang.brigadier.arguments.StringArgumentType; +import de.hysky.skyblocker.SkyblockerMod; +import de.hysky.skyblocker.utils.Calculator; +import de.hysky.skyblocker.utils.Constants; +import net.fabricmc.fabric.api.client.command.v2.ClientCommandRegistrationCallback; +import net.fabricmc.fabric.api.client.command.v2.FabricClientCommandSource; +import net.minecraft.client.MinecraftClient; +import net.minecraft.command.CommandRegistryAccess; +import net.minecraft.text.MutableText; +import net.minecraft.text.Text; +import net.minecraft.util.Formatting; + +import java.text.DecimalFormat; + +import static com.mojang.brigadier.arguments.StringArgumentType.getString; +import static net.fabricmc.fabric.api.client.command.v2.ClientCommandManager.argument; +import static net.fabricmc.fabric.api.client.command.v2.ClientCommandManager.literal; + +public class CalculatorCommand { + private static final MinecraftClient CLIENT = MinecraftClient.getInstance(); + + private static final DecimalFormat FORMATTER = new DecimalFormat("#,###.##"); + + public static void init() { + ClientCommandRegistrationCallback.EVENT.register(CalculatorCommand::calculate); + } + + private static void calculate(CommandDispatcher dispatcher, CommandRegistryAccess registryAccess) { + dispatcher.register(literal(SkyblockerMod.NAMESPACE) + .then(literal("calculate") + .then(argument("equation", StringArgumentType.greedyString()) + .executes(context -> doCalculation(getString(context, "equation")) ) + ) + ) + ); + } + + private static int doCalculation(String calculation) { + MutableText text = Constants.PREFIX.get(); + try { + text.append(Text.literal(FORMATTER.format(Calculator.calculate(calculation))).formatted(Formatting.GREEN)); + } catch (UnsupportedOperationException e) { + text.append(Text.literal("text.autoconfig.skyblocker.option.general.enableSignCalculator.invalidEquation").formatted(Formatting.RED)); + } + + if (CLIENT == null || CLIENT.player == null) { + return 0; + } + + CLIENT.player.sendMessage(text, false); + return Command.SINGLE_SUCCESS; + } +} diff --git a/src/main/java/de/hysky/skyblocker/skyblock/calculators/SignCalculator.java b/src/main/java/de/hysky/skyblocker/skyblock/calculators/SignCalculator.java new file mode 100644 index 00000000..7dbeabd9 --- /dev/null +++ b/src/main/java/de/hysky/skyblocker/skyblock/calculators/SignCalculator.java @@ -0,0 +1,62 @@ +package de.hysky.skyblocker.skyblock.calculators; + +import de.hysky.skyblocker.utils.Calculator; +import net.minecraft.client.MinecraftClient; +import net.minecraft.client.gui.DrawContext; +import net.minecraft.text.Text; +import net.minecraft.util.Formatting; + +import java.text.DecimalFormat; + +public class SignCalculator { + + private static final MinecraftClient CLIENT = MinecraftClient.getInstance(); + + private static final DecimalFormat FORMATTER = new DecimalFormat("#,###.##"); + + private static String lastInput; + private static String input; + private static Double output; + + public static void renderSign(DrawContext context, String[] messages){ + input = messages[0]; + + + //only update output if new input + if (!input.equals(lastInput)) { // + try { + output = Calculator.calculate(input); + } catch (Exception e){ + output = null; + } + } + + render(context); + + lastInput = input; + } + + public static String getNewValue(Boolean isPrice) { + if (output == null) { + return ""; + } + //price can except decimals and exponents + if (isPrice) { + return output.toString(); + } + //amounts want an integer number so round + return Long.toString(Math.round(output)); + } + + private static void render(DrawContext context) { + Text text; + if (output == null) { + text = Text.translatable("text.autoconfig.skyblocker.option.general.enableSignCalculator.invalidEquation").formatted(Formatting.RED); + } + else { + text = Text.literal(input +" = " + FORMATTER.format(output)).formatted(Formatting.GREEN); + } + + context.drawCenteredTextWithShadow(CLIENT.textRenderer, text,context.getScaledWindowWidth() /2 , 55,0xFFFFFFFF); + } +} diff --git a/src/main/java/de/hysky/skyblocker/utils/Calculator.java b/src/main/java/de/hysky/skyblocker/utils/Calculator.java index c8dcb6d8..19155708 100644 --- a/src/main/java/de/hysky/skyblocker/utils/Calculator.java +++ b/src/main/java/de/hysky/skyblocker/utils/Calculator.java @@ -1,6 +1,5 @@ package de.hysky.skyblocker.utils; -import de.hysky.skyblocker.skyblock.SignCalculator; import net.minecraft.util.Util; import java.util.*; @@ -148,9 +147,7 @@ public class Calculator { case "*","/" -> { return 1; } - default -> { - throw new UnsupportedOperationException("invalid operator"); - } + default -> throw new UnsupportedOperationException("Invalid operator"); } } @@ -188,24 +185,27 @@ public class Calculator { } } case L_PARENTHESIS, R_PARENTHESIS -> { - throw new UnsupportedOperationException("equation is not in RPN"); + throw new UnsupportedOperationException("Equation is not in RPN"); } } } + if (values.isEmpty()) { + throw new UnsupportedOperationException("Equation is empty"); + } return values.pop(); } private static double calculateValue(String value) { Matcher numberMatcher = NUMBER_PATTERN.matcher(value.toLowerCase()); if (!numberMatcher.matches()) { - throw new UnsupportedOperationException("invalid number"); + throw new UnsupportedOperationException("Invalid number"); } double number = Double.parseDouble(numberMatcher.group(1)); String magnitude = numberMatcher.group(2); if (!magnitude.isEmpty()) { if (!magnitudeValues.containsKey(magnitude)) {//its invalid if its another letter - throw new UnsupportedOperationException("invalid magnitude"); + throw new UnsupportedOperationException("Invalid magnitude"); } number *= magnitudeValues.get(magnitude); } -- cgit From 08140d6b262a9b694cbc9d14189249d10c10eef6 Mon Sep 17 00:00:00 2001 From: olim Date: Thu, 2 May 2024 19:38:55 +0100 Subject: clean code and fix to 1.20.6 --- .../skyblocker/mixins/SignEditScreenMixin.java | 10 ++++---- .../skyblock/calculators/CalculatorCommand.java | 2 +- .../skyblock/calculators/SignCalculator.java | 15 +++++------- .../java/de/hysky/skyblocker/utils/Calculator.java | 27 +++++++++++----------- 4 files changed, 25 insertions(+), 29 deletions(-) (limited to 'src/main/java') diff --git a/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java b/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java index 88c54878..d1800e9b 100644 --- a/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java +++ b/src/main/java/de/hysky/skyblocker/mixins/SignEditScreenMixin.java @@ -1,4 +1,4 @@ -package de.hysky.skyblocker.mixin; +package de.hysky.skyblocker.mixins; import de.hysky.skyblocker.config.SkyblockerConfigManager; @@ -16,7 +16,7 @@ import org.spongepowered.asm.mixin.injection.callback.CallbackInfo; import java.util.Objects; @Mixin(AbstractSignEditScreen.class) -public abstract class SignEditScreenMixin{ +public abstract class SignEditScreenMixin { @Shadow @Final private String[] messages; @@ -32,15 +32,13 @@ public abstract class SignEditScreenMixin{ @Inject(method = "finishEditing", at = @At("HEAD")) private void skyblocker$finishEditing(CallbackInfo ci) { //if the sign is being used to enter number get number from calculator for if maths has been done - if (Utils.isOnSkyblock() && SkyblockerConfigManager.get().general.enableSignCalculator && Objects.equals(messages[1], "^^^^^^^^^^^^^^^")) { + if (Utils.isOnSkyblock() && SkyblockerConfigManager.get().general.enableSignCalculator && Objects.equals(messages[1], "^^^^^^^^^^^^^^^")) { boolean isPrice = messages[2].contains("price"); String value = SignCalculator.getNewValue(isPrice); if (value.length() >= 15) { - value = value.substring(0,15); + value = value.substring(0, 15); } messages[0] = value; } - } - } diff --git a/src/main/java/de/hysky/skyblocker/skyblock/calculators/CalculatorCommand.java b/src/main/java/de/hysky/skyblocker/skyblock/calculators/CalculatorCommand.java index f9b8b4cf..d78e2c0d 100644 --- a/src/main/java/de/hysky/skyblocker/skyblock/calculators/CalculatorCommand.java +++ b/src/main/java/de/hysky/skyblocker/skyblock/calculators/CalculatorCommand.java @@ -33,7 +33,7 @@ public class CalculatorCommand { dispatcher.register(literal(SkyblockerMod.NAMESPACE) .then(literal("calculate") .then(argument("equation", StringArgumentType.greedyString()) - .executes(context -> doCalculation(getString(context, "equation")) ) + .executes(context -> doCalculation(getString(context, "equation"))) ) ) ); diff --git a/src/main/java/de/hysky/skyblocker/skyblock/calculators/SignCalculator.java b/src/main/java/de/hysky/skyblocker/skyblock/calculators/SignCalculator.java index 7dbeabd9..82d86ba7 100644 --- a/src/main/java/de/hysky/skyblocker/skyblock/calculators/SignCalculator.java +++ b/src/main/java/de/hysky/skyblocker/skyblock/calculators/SignCalculator.java @@ -12,21 +12,19 @@ public class SignCalculator { private static final MinecraftClient CLIENT = MinecraftClient.getInstance(); - private static final DecimalFormat FORMATTER = new DecimalFormat("#,###.##"); + private static final DecimalFormat FORMATTER = new DecimalFormat("#,###.##"); private static String lastInput; private static String input; private static Double output; - public static void renderSign(DrawContext context, String[] messages){ + public static void renderSign(DrawContext context, String[] messages) { input = messages[0]; - - //only update output if new input if (!input.equals(lastInput)) { // try { output = Calculator.calculate(input); - } catch (Exception e){ + } catch (Exception e) { output = null; } } @@ -52,11 +50,10 @@ public class SignCalculator { Text text; if (output == null) { text = Text.translatable("text.autoconfig.skyblocker.option.general.enableSignCalculator.invalidEquation").formatted(Formatting.RED); - } - else { - text = Text.literal(input +" = " + FORMATTER.format(output)).formatted(Formatting.GREEN); + } else { + text = Text.literal(input + " = " + FORMATTER.format(output)).formatted(Formatting.GREEN); } - context.drawCenteredTextWithShadow(CLIENT.textRenderer, text,context.getScaledWindowWidth() /2 , 55,0xFFFFFFFF); + context.drawCenteredTextWithShadow(CLIENT.textRenderer, text, context.getScaledWindowWidth() / 2, 55, 0xFFFFFFFF); } } diff --git a/src/main/java/de/hysky/skyblocker/utils/Calculator.java b/src/main/java/de/hysky/skyblocker/utils/Calculator.java index 19155708..bb6ed768 100644 --- a/src/main/java/de/hysky/skyblocker/utils/Calculator.java +++ b/src/main/java/de/hysky/skyblocker/utils/Calculator.java @@ -11,11 +11,13 @@ public class Calculator { public enum TokenType { NUMBER, OPERATOR, L_PARENTHESIS, R_PARENTHESIS } + public static class Token { public TokenType type; String value; int tokenLength; } + private static final Pattern NUMBER_PATTERN = Pattern.compile("(\\d+\\.?\\d*)([kmbs]?)"); private static final HashMap magnitudeValues = Util.make(new HashMap<>(), map -> { map.put("s", 64); @@ -24,15 +26,14 @@ public class Calculator { map.put("b", 1000000000); }); - private static List lex(String input) { List tokens = new ArrayList<>(); - input = input.replace(" ", "").toLowerCase().replace("x","*"); + input = input.replace(" ", "").toLowerCase().replace("x", "*"); int i = 0; while (i < input.length()) { Token token = new Token(); switch (input.charAt(i)) { - case '+','-','*','/' -> { + case '+', '-', '*', '/' -> { token.type = TokenType.OPERATOR; token.value = String.valueOf(input.charAt(i)); token.tokenLength = 1; @@ -43,8 +44,8 @@ public class Calculator { token.value = String.valueOf(input.charAt(i)); token.tokenLength = 1; //add implicit multiplication when t