Оценка строки в kotlin

В случае этого конкретного ввода есть две ошибки.

Первая ошибка связана с тем, что вы сначала обрабатываете +, а затем -, но в математике мы должны обрабатывать их вместе, слева направо. В результате ваш алгоритм обрабатывает это: 1-1+1 (правильный ответ: 1) как: 1-(1+1) (ответ: -1). В качестве альтернативы, я считаю, что обработка - сначала, а затем + должна работать правильно.

Вторая ошибка заключается в том, что в sub() вы начинаете с 0 и вычитаете даже первое число, но на самом деле первое число должно быть добавлено или использовано в качестве начального значения.

Я считаю, что после исправления этих двух ошибок он должен дать правильный ответ для этого конкретного ввода, но, вероятно, таких ошибок больше. Например, вы неправильно обрабатываете цепочки одного и того же оператора, вас интересует только первая пара, поэтому 1+1+1 становится 2 (а затем превращается в -2 из-за ранее упомянутой ошибки).

Общий совет: научитесь пользоваться отладчиком и анализируйте, как меняются ваши данные, пока код их обрабатывает. Даже если вы просто распечатаете свои промежуточные результаты: divanswer, mulanswer и addanswer, а затем пропустите их через Google для обработки, вы заметите, что где-то на шаге add() есть проблема. Затем промойте и повторяйте, пока все не будет работать правильно.

Спасибо @broot за то, что заметил это, проблема заключалась в части сложения и вычитания, теперь, когда я объединил их в один и получил правильные ответы для строк, которые я ввел, вот исправленный полный код.

var x : String = "5+4-10x20-40/30x20+34/20"

fun div(x: String): String {
    val lis1 = x.split("-").toMutableList()
    for (ele1 in lis1) {
        if ("/" in ele1) {
            val pos1 = lis1.indexOf(ele1)
            val lis2 = ele1.split("+").toMutableList()
            for (ele2 in lis2) {
                if ("/" in ele2) {
                    val pos3 = lis2.indexOf(ele2)
                    val lis3 = ele2.split("x").toMutableList()
                    for (ele3 in lis3) {
                        if ("/" in ele3) {
                            val pos4 = lis3.indexOf(ele3)
                            val lis4 = ele3.split("/")
                            val div = (lis4[0].toDouble() / lis4[1].toDouble()).toString()
                            lis3[pos4] = div
                        }
                    }
                    lis2[pos3] = lis3.joinToString("x")
                }
            }
            lis1[pos1] = lis2.joinToString("+")

        }
    }
    return (lis1.joinToString("-"))
}

fun mul(x : String) : String{
    val lis1 = x.split("-").toMutableList()
    for (ele1 in lis1) {
        if ("x" in ele1){
            val pos1 = lis1.indexOf(ele1)
            val lis2 = ele1.split("+").toMutableList()
            for(ele2 in lis2){
                if ("x" in ele2){
                    val pos2 = lis2.indexOf(ele2)
                    val lis3 = ele2.split("x")
                    val mul = (lis3[0].toDouble() * lis3[1].toDouble()).toString()
                    lis2[pos2] = mul
                }
            }
            lis1[pos1] = lis2.joinToString("+")
        }
    }
    return (lis1.joinToString("-"))
}

fun final(x: String): Double {
    val substr = StringBuffer()
    var answer = 0.0
    var opr = '+'
    for(ele in x.indices){
        var current = x[ele]
        if (current in '0'..'9' || current == '.'){
            substr.append(current)
        }
        if (current == '-' || current == '+'){
            var operand = substr.toString().toDouble()
            substr.setLength(0)
            when (opr){
                '+' -> answer += operand
                '-' -> answer -= operand
            }
            opr = current
        }
    }
    val num = substr.toString().toDouble()
    when(opr){
        '+' -> answer += num
        '-' -> answer -= num
    }
    return answer
}


fun eval(x: String): Double {
    val divanswer = div(x)
    return final(mul(divanswer))
}

fun main(args: Array<String>){
    println(eval(x))
}