Cleanup

src/main/java/xyz/kwiecien/jfuck/OperationExtractor.java

@@ -18,13 +18,13 @@ public class OperationExtractor {
                 case '+':
                 case '-':
                     byte valueModification = (byte) (c == '+' ? 1 : -1);
-                    if (currentOperation instanceof ModifyStackValueOperation(byte value)) {
-                        currentOperation = new ModifyStackValueOperation((byte) (value + valueModification));
+                    if (currentOperation instanceof ModifyValueOperation(byte value)) {
+                        currentOperation = new ModifyValueOperation((byte) (value + valueModification));
                     } else {
                         if (currentOperation != null) {
                             operationLists.peek().add(currentOperation);
                         }
-                        currentOperation = new ModifyStackValueOperation(valueModification);
+                        currentOperation = new ModifyValueOperation(valueModification);
                     }
                     break;
                 case '<':

src/main/java/xyz/kwiecien/jfuck/operation/ModifyStackValueOperation.java → src/main/java/xyz/kwiecien/jfuck/operation/ModifyValueOperation.java

@@ -5,7 +5,7 @@ import java.lang.classfile.CodeBuilder;
 import static xyz.kwiecien.jfuck.operation.BytecodeConstants.DATA_VAR_INDEX;
 import static xyz.kwiecien.jfuck.operation.BytecodeConstants.PTR_VAR_INDEX;
 
-public record ModifyStackValueOperation(byte value) implements Operation {
+public record ModifyValueOperation(byte value) implements Operation {
     @Override
     public void appendBytecode(CodeBuilder cb) {
         cb.aload(DATA_VAR_INDEX).iload(PTR_VAR_INDEX).dup2() // dup2, because will be used to read value and then store incremented

src/test/java/xyz/kwiecien/jfuck/OperationExtractorTest.java

@@ -14,7 +14,7 @@ class OperationExtractorTest {
         var operations = new OperationExtractor().extract(code);
         assertEquals(List.of(
                 new Initialization(),
-                new ModifyStackValueOperation((byte) 5)
+                new ModifyValueOperation((byte) 5)
         ), operations);
     }
 
@@ -24,7 +24,7 @@ class OperationExtractorTest {
         var operations = new OperationExtractor().extract(code);
         assertEquals(List.of(
                 new Initialization(),
-                new ModifyStackValueOperation((byte) -5)
+                new ModifyValueOperation((byte) -5)
         ), operations);
     }
 
@@ -34,7 +34,7 @@ class OperationExtractorTest {
         var operations = new OperationExtractor().extract(code);
         assertEquals(List.of(
                 new Initialization(),
-                new ModifyStackValueOperation((byte) 2)
+                new ModifyValueOperation((byte) 2)
         ), operations);
     }
 
@@ -75,9 +75,9 @@ class OperationExtractorTest {
         assertEquals(List.of(
                 new Initialization(),
                 new ReadOperation(),
-                new ModifyStackValueOperation((byte) 1),
+                new ModifyValueOperation((byte) 1),
                 new ModifyPointerOperation(1),
-                new ModifyStackValueOperation((byte) -1),
+                new ModifyValueOperation((byte) -1),
                 new ModifyPointerOperation(-1),
                 new WriteOperation()
         ), operations);