Skip to content

Commit c3d27b5

Browse files
committed
Moved value switch.
1 parent d596b3f commit c3d27b5

File tree

11 files changed

+16
-64
lines changed

11 files changed

+16
-64
lines changed

ReClass.NET/Forms/ScannerForm.cs

Lines changed: 16 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -734,6 +734,14 @@ private IScanComparer CreateComparer(ScanSettings settings)
734734
if (!long.TryParse(dualValueBox.Value1, numberStyle, null, out var value1)) throw new InvalidInputException(dualValueBox.Value1);
735735
if (!long.TryParse(dualValueBox.Value2, numberStyle, null, out var value2) && checkBothInputFields) throw new InvalidInputException(dualValueBox.Value2);
736736

737+
if (compareType == ScanCompareType.Between || compareType == ScanCompareType.BetweenOrEqual)
738+
{
739+
if (value1 > value2)
740+
{
741+
Utils.Swap(ref value1, ref value2);
742+
}
743+
}
744+
737745
switch (settings.ValueType)
738746
{
739747
case ScanValueType.Byte:
@@ -769,6 +777,14 @@ int CalculateSignificantDigits(string input, NumberFormatInfo numberFormat)
769777
var nf2 = NumberFormat.GuessNumberFormat(dualValueBox.Value2);
770778
if (!double.TryParse(dualValueBox.Value2, NumberStyles.Float, nf2, out var value2) && checkBothInputFields) throw new InvalidInputException(dualValueBox.Value2);
771779

780+
if (compareType == ScanCompareType.Between || compareType == ScanCompareType.BetweenOrEqual)
781+
{
782+
if (value1 > value2)
783+
{
784+
Utils.Swap(ref value1, ref value2);
785+
}
786+
}
787+
772788
var significantDigits = Math.Max(
773789
CalculateSignificantDigits(dualValueBox.Value1, nf1),
774790
CalculateSignificantDigits(dualValueBox.Value2, nf2)

ReClass.NET/MemoryScanner/Comparer/ByteMemoryComparer.cs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,6 @@ public ByteMemoryComparer(ScanCompareType compareType, byte value1, byte value2)
1515
{
1616
CompareType = compareType;
1717

18-
if (compareType == ScanCompareType.Between || compareType == ScanCompareType.BetweenOrEqual)
19-
{
20-
if (value1 > value2)
21-
{
22-
Utils.Swap(ref value1, ref value2);
23-
}
24-
}
25-
2618
Value1 = value1;
2719
Value2 = value2;
2820
}

ReClass.NET/MemoryScanner/Comparer/DoubleMemoryComparer.cs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,6 @@ public DoubleMemoryComparer(ScanCompareType compareType, ScanRoundMode roundType
2121
{
2222
CompareType = compareType;
2323

24-
if (compareType == ScanCompareType.Between || compareType == ScanCompareType.BetweenOrEqual)
25-
{
26-
if (value1 > value2)
27-
{
28-
Utils.Swap(ref value1, ref value2);
29-
}
30-
}
31-
3224
RoundType = roundType;
3325
this.significantDigits = Math.Max(significantDigits, 1);
3426
Value1 = Math.Round(value1, this.significantDigits, MidpointRounding.AwayFromZero);

ReClass.NET/MemoryScanner/Comparer/FloatMemoryComparer.cs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -21,14 +21,6 @@ public FloatMemoryComparer(ScanCompareType compareType, ScanRoundMode roundType,
2121
{
2222
CompareType = compareType;
2323

24-
if (compareType == ScanCompareType.Between || compareType == ScanCompareType.BetweenOrEqual)
25-
{
26-
if (value1 > value2)
27-
{
28-
Utils.Swap(ref value1, ref value2);
29-
}
30-
}
31-
3224
RoundType = roundType;
3325
this.significantDigits = Math.Max(significantDigits, 1);
3426
Value1 = (float)Math.Round(value1, this.significantDigits, MidpointRounding.AwayFromZero);

ReClass.NET/MemoryScanner/Comparer/IntegerMemoryComparer.cs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,6 @@ public IntegerMemoryComparer(ScanCompareType compareType, int value1, int value2
1515
{
1616
CompareType = compareType;
1717

18-
if (compareType == ScanCompareType.Between || compareType == ScanCompareType.BetweenOrEqual)
19-
{
20-
if (value1 > value2)
21-
{
22-
Utils.Swap(ref value1, ref value2);
23-
}
24-
}
25-
2618
Value1 = value1;
2719
Value2 = value2;
2820
}

ReClass.NET/MemoryScanner/Comparer/LongMemoryComparer.cs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,6 @@ public LongMemoryComparer(ScanCompareType compareType, long value1, long value2)
1515
{
1616
CompareType = compareType;
1717

18-
if (compareType == ScanCompareType.Between || compareType == ScanCompareType.BetweenOrEqual)
19-
{
20-
if (value1 > value2)
21-
{
22-
Utils.Swap(ref value1, ref value2);
23-
}
24-
}
25-
2618
Value1 = value1;
2719
Value2 = value2;
2820
}

ReClass.NET/MemoryScanner/Comparer/ShortMemoryComparer.cs

Lines changed: 0 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -15,14 +15,6 @@ public ShortMemoryComparer(ScanCompareType compareType, short value1, short valu
1515
{
1616
CompareType = compareType;
1717

18-
if (compareType == ScanCompareType.Between || compareType == ScanCompareType.BetweenOrEqual)
19-
{
20-
if (value1 > value2)
21-
{
22-
Utils.Swap(ref value1, ref value2);
23-
}
24-
}
25-
2618
Value1 = value1;
2719
Value2 = value2;
2820
}

ReClass.NET_Tests/MemoryScanner/Comparer/ByteMemoryComparerTest.cs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,6 @@ public void TestConstructor(ScanCompareType compareType, byte value1, byte value
2323
Check.That(sut.ValueSize).IsEqualTo(sizeof(byte));
2424
Check.That(sut.Value1).IsOneOf(value1, value2);
2525
Check.That(sut.Value2).IsOneOf(value1, value2);
26-
if (compareType == ScanCompareType.Between || compareType == ScanCompareType.BetweenOrEqual)
27-
{
28-
Check.That(sut.Value1 <= sut.Value2).IsTrue();
29-
}
3026
}
3127

3228
public static TheoryData<ScanCompareType, byte, byte, byte[], bool, ScanResult> GetTestCompareBasicData() => new TheoryData<ScanCompareType, byte, byte, byte[], bool, ScanResult>

ReClass.NET_Tests/MemoryScanner/Comparer/IntegerMemoryComparerTest.cs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,6 @@ public void TestConstructor(ScanCompareType compareType, int value1, int value2)
2323
Check.That(sut.ValueSize).IsEqualTo(sizeof(int));
2424
Check.That(sut.Value1).IsOneOf(value1, value2);
2525
Check.That(sut.Value2).IsOneOf(value1, value2);
26-
if (compareType == ScanCompareType.Between || compareType == ScanCompareType.BetweenOrEqual)
27-
{
28-
Check.That(sut.Value1 <= sut.Value2).IsTrue();
29-
}
3026
}
3127

3228
public static TheoryData<ScanCompareType, int, int, byte[], bool, ScanResult> GetTestCompareBasicData() => new TheoryData<ScanCompareType, int, int, byte[], bool, ScanResult>

ReClass.NET_Tests/MemoryScanner/Comparer/LongMemoryComparerTest.cs

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -23,10 +23,6 @@ public void TestConstructor(ScanCompareType compareType, long value1, long value
2323
Check.That(sut.ValueSize).IsEqualTo(sizeof(long));
2424
Check.That(sut.Value1).IsOneOf(value1, value2);
2525
Check.That(sut.Value2).IsOneOf(value1, value2);
26-
if (compareType == ScanCompareType.Between || compareType == ScanCompareType.BetweenOrEqual)
27-
{
28-
Check.That(sut.Value1 <= sut.Value2).IsTrue();
29-
}
3026
}
3127

3228
public static TheoryData<ScanCompareType, long, long, byte[], bool, ScanResult> GetTestCompareBasicData() => new TheoryData<ScanCompareType, long, long, byte[], bool, ScanResult>

0 commit comments

Comments
 (0)