Почему неправильно выводит максимальное число?

Question

[dzzaad]

При заполнении numbers 10 9 8 7 6 5 4 3 2 1, как максимально число выводит 9. Почему?

numbers = input().split()
flag = 0
for i in range(len(numbers)):
    if numbers[i].isdigit() == True:
        flag += 1
if flag == len(numbers):
    for i in range(len(numbers)-1):
        print(numbers[i], numbers[i+1])
        if numbers[i] < numbers[i+1]:
            mx = numbers[i+1]
print(mx)

Answer 1

[Wataru]

Код работает ровно так, как написан.

Можете строчка за строчкой объяснить, что происходит в коде? Я начал, вы дополните в остальных строчках. Сразу должно стать понятно, где ошибка в логике:

numbers = input().split()               # получаем в numbers массив из отдельных слов в введенной строке
flag = 0                                # инициализируем счетчик нулем
for i in range(len(numbers)):           # проходимся по всей длине массива (по всем словам)
    if numbers[i].isdigit() == True:    # Если текущее слово состоит только из цифр (т.е. оно число)
        flag += 1                       # Увеличиваем счетчик
if flag == len(numbers):                # если все слова в строке - числа
    for i in range(len(numbers)-1):     # ???
        print(numbers[i], numbers[i+1]) # ???
        if numbers[i] < numbers[i+1]:   # ???
            mx = numbers[i+1]           # ???
print(mx)

Comments

[dzzaad]

for i in range(len(numbers)-1): # проходим по длине - 1, чтобы проверить максимально ли число
print(numbers[i], numbers[i+1]) # выводим i-тый и i-1 элементы
if numbers[i] < numbers[i+1]: # сравниваем их
mx = numbers[i+1] # присваиваем к mx максимальное из i-тый и i-1 элементы

[dzzaad]

Можно добавить условие else, но все равно от этого ничего не изменилось.

numbers = input().split()
flag = 0
for i in range(len(numbers)):
    #print(numbers[i])
    if numbers[i].isdigit() == True:
        flag += 1
# print(flag)
# print(len(numbers))
if flag == len(numbers):
    for i in range(len(numbers)-1):
        # print(numbers[i], numbers[i+1])
        if numbers[i] < numbers[i+1]:
            mx = numbers[i+1]
        else:
            mx = numbers[i]
print(mx)

[Wataru]

dzzaad, Еще раз медленно.

проходим по длине - 1, чтобы проверить максимально ли число

Пока ладно, но важно, что там внутри делается. Действительно ли в этом цикле вы находите максимальное число?

выводим i-тый и i-1 элементы

Ok.

сравниваем их

Да, сравниваем.

mx = numbers[i+1] # присваиваем к mx максимальное из i-тый и i-1 элементы

Нет. Вы не присваиваете максимум из двух чисел в mx. Вы присваиваете mx второе число, только если оно больше. Иначе ничего не происходит.

После вашего добавления else ваши комментарии действительно становятся верными, кроме одного к циклу. Почему из "присваиваем к mx максимальное из i-тый и i-1 элементы " следует, что цикл "чтобы проверить максимально ли число"?

Вы внутри не ищите максимум в массиве, а много раз присваиваете переменной максимальное из двух чисел. В переменной будет в итоге результат последней операции - а именно максимум из двух последних чисел. Если, как в примере в вопросе, максимум во всем массиве не в конце - вы его не найдете.

Подсказка: Результат работы прошлых итераций как-то должен учитыватся в следующих. Вы этот результат кладете в mx. Значит записанное в mx значение должно как-то влиять на вычисление mx дальше.

[dzzaad]

Wataru, спасибо большое.