Как правильно исправить код для вывода if\else?

Question

[Joulence]

https://repl.it/@Joulence/vectorsfigure
Есть такой код. Работает +- неплохо, но есть недочёт. Код определяет вид треугольника по сторонам и часто выводит два результата или иногда три. Как улучшить, изменить или даже переписать код, чтобы выводило всегда одно значение, а не больше одного, как на картинке:

Всё считай написано на if \ elif

Answer 1

[dmshar]

Странный вопрос. Как раз такой вывод - правильный. Согласно вашему-же определения ваш треугольник есть одновременно и простым (3 < 4+5; 4<3+5;5<3+4) и прямоугольным 25= 9+16. А что код должен делать в таких случаях?
Что бы выводило "ТОЛЬКО ОДНО" - ваши определения (не код, а именно не определения!) должны быть взаимоисключающими. У вас это не так.

Comments

[Joulence]

Ну как можно было бы правильно переписать код, чтобы определения были взаимоисключающими?

[dmshar]

Вы наверное не дочитали мой ответ.
"Ваши определения (не код, а именно не определения!) должны быть взаимоисключающими."
Т.е. сначала составляем взаимоисключающие определения, а потом уже пишем код. И это никакого отношения ни к Python, ни к программированию вообще не имеет. Это геометрия. Школьная. 6-й класс.

Answer 2

[EzikBro]

Если вы хотите исправить конкретно проблему множественного вывода, то замените все if, кроме первого на elif.
Но тогда сразу появится проблема с неправильным определением треугольника: для любого существующего функция выведет "Простой треугольник". Чтобы избежать этого, нужно переставить проверки местами (правильный, прямоугольный, равнобедренный, простой, не существует (равнобедренный и прямоугольный можно поменять местами - смотря, что важнее)).

Чтобы не исправлять много разных ошибок и упростить код для двух разных языков вам лучше написать отдельные функции вида is_right(a, b, c).
Тогда можно написать что-то вроде:

def triangle(a, b, c):
    #Отсортируем стороны в порядке возрастания для удобства
    a, b, c = sorted([a, b, c])

    if c < a + b:
        if is_equilateral(a, b, c):
            print("Equilateral")
        elif is_right(a, b, c) and is_isosceles(a, b, c):
            print("Right isosceles")
        elif is_right(a, b, c):
            print("Right")
        elif is_isosceles(a, b, c):
            print("Isosceles")
        else:
            print("Simple")
    else:
        print("Triangle doesn't exist")

    repeat = input("Repeat - y\nExit - n\n")
    if repeat == "y":
        triangle1()
    elif repeat == "n":
        print("Shutting down")

start = input("1 - English\n2 - Русский\n")

if start == "1":
    triangle1()
elif start == "2":
    triangle2()