Как можно сократить / оптимизировать решение задачи на Python?

Question

[Dmitry]

Есть задача:
Если бы вы сейчас были на Луне, ваш вес составил бы 16,5 процентов от земного. Если бы каждый год в течение следующих 15 лет вы прибавляли по одному килограмму веса, каким бы оказался ваш лунный вес в каждый из ежегодных визитов на Луну вплоть до 15-го года? Напишите программу, которая с помощью цикла for печатает на экране ваш лунный вес в каждом году.

Мое решение:

earth_weight = 70
moon_index = 0.165
moon_weight_2018 = earth_weight * moon_index
print('Ваш вес в 2018 году: ' + str(moon_weight_2018))

for i in range(1, 16):
    print(i, round(earth_weight * moon_index, 2))
    earth_weight += 1

Вопрос: какие косяки есть / что не верно, что можно упростить?
Сразу замечу, что до функций еще не дошел.
Спасибо.

Answer 1

[Станислав Некрасов]

Как-то так:

weight, moon = 70, 0.165
print("Ваш вес в 2018 году:", weight * moon)
print(*[str(i) + " " + str(round((weight + i-1) * moon, 2)) for i in range(1, 16)], sep="\n")

P. S. Только не знаю, насколько сложный для понимания этот код.

Comments

[Dmitry]

Спасибо, но да, посл строка не очень понятна:
- что такое * которая перед списком?
- weight + i-1 - не ясна запись...
- что такое sep="\n" ?

[Станислав Некрасов]

* перед списком - это Распаковка списка.
weight + i-1 это чтобы как у вас - сперва Вычисляем вес, потом увеличиваем i.
sep="\n" - это сепаратор, легче говоря разделитель строк, т. е. распакованный список печатается с разделителем - Новая Строка

[Станислав Некрасов]

Я тут кое-что заметил...
На счёт распаковки списка:

print(*[10, 20, 30])  # 10 20 30
# Идентично
print(10, 20, 30)  # 10 20 30

Далее, я заметил что вес спустя 15 лет не печатается. Для печати от 1 года до 15 нужно написать так:

weight, moon = 70, 0.165
print("Ваш вес в 2018 году:", weight * moon)
print(*[str(i) + " " + str(round((weight + i) * moon, 2)) for i in range(1, 16)], sep="\n")

[Станислав Некрасов]

Dmitry, Что сейчас непонятно?) Или всё понятно?

[Станислав Некрасов]

На счёт sep

[Dmitry]

Metalofon NT, спасибо, но не очень понял почему в предыдущем варианте было weight + 1 - i (c i не понятно).

[Dmitry]

Dmitry, ошибка была? надо без i - 1?

[Станислав Некрасов]

Dmitry, i - это сколько лет прошло - т. е. сколько пользователь набрал веса.
Мы прибавляем
weight + i
И получаем текущий вес.
С
weight + i -1
Было для того, чтобы код соответствовал вашему)).

[Станислав Некрасов]

Dmitry, Если хотите чтобы вывод был как у Вас, то пишите с i - 1, если хотите правильный вывод:
(Количество пройденных лет, Текущий вес на луне), то - 1 писать не надо.

[Dmitry]

Metalofon NT, ок, ясно)

[Станислав Некрасов]

Dmitry, Если помогло, то тогда пожалуйста, отметьте решением.)

[Станислав Некрасов]

Dmitry, Спасибо.) Удачи Вам.

[Dmitry]

Metalofon NT, взаимно) я скоро опять буду тут что-нибудь спрашивать...

[Станислав Некрасов]

Dmitry, Спрашивайте. Если буду в сети - отвечу.)

Answer 2

[Juhani Lahtinen]

Если слегка оптимизировать то

earth_weight = 70
moon_index = 0.165
year_now = datetime.date.today().year

for i in range(1, 16):
	print(f"Ваш вес в {year_now} году: {round(earth_weight * moon_index, 2)} кг" )
	earth_weight += 1
	year_now +=1

Comments

[Dmitry]

Спасибо

Answer 3

[F2pgg]

a=int(input('Введите свой вес '))
b=0.165
for i in range (1,16):
c=a*b
a=a+1
print(f'{i}, {c}')