fenix/Simple-Calculator
1
0
Fork 0
mirror of https://github.com/SimpleMobileTools/Simple-Calculator.git synced 2025-03-11 00:40:08 +01:00
Code Issues Packages Projects Releases Wiki Activity

fixing some negative number related glitches

Browse Source
This commit is contained in:
tibbi 2020-11-05 22:28:47 +01:00
parent 2abd8ae27e
commit 2ec342d4db
1 changed files with 36 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

48
app/src/main/kotlin/com/simplemobiletools/calculator/helpers/CalculatorImpl.kt
View File

@ -124,7 +124,9 @@ class CalculatorImpl(calculator: Calculator, val context: Context) {
}
private fun calculateResult(update: Boolean = true) {
if (update) updateFormula()
if (update) {
updateFormula()
}
val operation = OperationFactory.forId(lastOperation!!, baseValue, secondValue)
if (operation != null) {
@ -140,12 +142,26 @@ class CalculatorImpl(calculator: Calculator, val context: Context) {
}
fun handleOperation(operation: String) {
if (inputDisplayedFormula.isEmpty()) {
inputDisplayedFormula = "0"
}
if (operation != ROOT) {
if (inputDisplayedFormula.last() == '+' || inputDisplayedFormula.last() == '-' || inputDisplayedFormula.last() == '*' || inputDisplayedFormula.last() == '/' || inputDisplayedFormula.last() == '^' || inputDisplayedFormula.last() == '%') {
if (inputDisplayedFormula.last() == '+' ||
inputDisplayedFormula.last() == '-' ||
inputDisplayedFormula.last() == '*' ||
inputDisplayedFormula.last() == '/' ||
inputDisplayedFormula.last() == '^' ||
inputDisplayedFormula.last() == '%') {
inputDisplayedFormula = inputDisplayedFormula.dropLast(1)
inputDisplayedFormula += getSign(operation)
} else {
if (!inputDisplayedFormula.contains('+') && !inputDisplayedFormula.contains('-') && !inputDisplayedFormula.contains('*') && !inputDisplayedFormula.contains('/') && !inputDisplayedFormula.contains('^') && !inputDisplayedFormula.contains('%')) {
if (!inputDisplayedFormula.contains('+') &&
!inputDisplayedFormula.substring(1).contains('-') &&
!inputDisplayedFormula.contains('*') &&
!inputDisplayedFormula.contains('/') &&
!inputDisplayedFormula.contains('^') &&
!inputDisplayedFormula.contains('%')) {
inputDisplayedFormula += getSign(operation)
} else {
moreOperationsInRaw = true
@ -160,6 +176,14 @@ class CalculatorImpl(calculator: Calculator, val context: Context) {
lastOperation = tempOp
} else if (lastKey == DIGIT && operation != ROOT && operation != FACTORIAL) {
handleResult()
if (inputDisplayedFormula.last() != '+' &&
inputDisplayedFormula.last() != '-' &&
inputDisplayedFormula.last() != '*' &&
inputDisplayedFormula.last() != '/' &&
inputDisplayedFormula.last() != '^' &&
inputDisplayedFormula.last() != '%') {
inputDisplayedFormula += getSign(operation)
}
}
resetValue = true
@ -170,8 +194,7 @@ class CalculatorImpl(calculator: Calculator, val context: Context) {
if (operation == ROOT) {
handleRoot()
resetValue = false
}
if (operation == FACTORIAL) {
} else if (operation == FACTORIAL) {
handleFactorial()
resetValue = false
}
@ -206,12 +229,7 @@ class CalculatorImpl(calculator: Calculator, val context: Context) {
newValue = formatString(newValue)
}
setValue(newValue)
if (newValue != "0") {
inputDisplayedFormula = newValue
} else {
inputDisplayedFormula = ""
}
inputDisplayedFormula = if (newValue != "0") newValue else ""
}
}
@ -229,7 +247,13 @@ class CalculatorImpl(calculator: Calculator, val context: Context) {
if (lastKey != DIGIT)
return
displayedNumber = displayedNumber!!.substring(displayedNumber!!.indexOfAny(operations, 0, false) + 1)
val numberToCheck = if (inputDisplayedFormula.startsWith("-")) {
inputDisplayedFormula.substring(1)
} else {
inputDisplayedFormula
}
displayedNumber = numberToCheck.substring(numberToCheck.indexOfAny(operations, 0, false) + 1)
secondValue = getDisplayedNumberAsDouble()
calculateResult()
lastKey = EQUALS