Как можно ещё сократить количество строк в коде?

Question

[Александр]

Только изучаю Python.
Собственно вопрос в заголовке.
Вопрос возник потому, что изначаль кода было в два раза больше.

# coding=utf8
from math import sqrt

def fu(a, b):
    c = sqrt(a ** 2 + b ** 2)
    b = (a * b) / 2
    print c, "гипотенуза"
    print b, "площадь"

def fu1():
    'Функция проверяет чтобы введённые данные были числом'
    a = raw_input("Введите число: ")
    while a == str(a):
        try:
            a = int(a)
        except ValueError:
            a = raw_input("Введите число: ")
    return a

def fu2():
    'Функция проверяет чтобы введённое число было больше нуля'
    a = fu1()
    while a <= 0:
        a = fu1()
    return a

fu(fu2(), fu2())

Answer 1

[lega]

fu1 и fu2 можно объединить в

def fu2():
    while True:
        try:
            a = int(raw_input("Введите число: "))
            assert a > 0
            return a
        except (ValueError, AssertionError):
            pass

Comments

[Александр]

Это как раз то что я не мог никак сделать

Answer 2

[Oscar Django]

Работает только для целых неотрицательных чисел

from math import sqrt

def input_int(count):
    cathetus = raw_input('Введите число {}: '.format(count)) # python2
    # cathetus = input('Введите число {}: '.format(count)) # python3
    return int(cathetus) if cathetus.isdigit() else input_int(count)

def calculation(a, b):
    hypotenuse = sqrt(a ** 2 + b ** 2)
    square = float(a * b)/2 # updated, old: square = (a * b)/2
    return {'hypotenuse': hypotenuse, 'square': square}

result = calculation(input_int(1), input_int(2))
print("\nгипотенуза: {hypotenuse}\nплощадь: {square}".format(**result))

И функции все таки по человечески надо называть.
PS по поводу того что только с целочисленными работает - исходный вариант также работал только с целыми числами (a = int(a), а не a = float(a))

Comments

[Andrey Dugin]

В Вашем коде как минимум 2 ошибки:
1) Использование input() вместо предполагавшегося raw_input()
2) Целочисленная площадь (закроем глаза на то, что стороны только целочисленные).

И
3) А рекурсивный вызов функции здесь правда нужен?

[Oscar Django]

1 у меня python3. В python2 действительно raw_input
2 это строки автора
3 так строк меньше. Недостатков перед циклом не

[Oscar Django]

... не вижу

[Артём Клименко]

Oscar Django: ну как, если пользователь будет упорен, очень упорен, то он упрётся в ограничение глубины рекурсии xD

[Oscar Django]

Если он будет упор(от/ен) ему ничего не поможет.
А мы садисты и sys.setrecursionlimit(limit) нам в помощь xD

Answer 3

[MiiNiPaa]

Адекватно назовите функции и удалите комменты.

Comments

[Александр]

Я не про названия. Я имею в виду сократить количество строк.

[MiiNiPaa]

Ну так -2 строки с комментами.

Answer 4

[Илья]

fu1, сократил на одну строку, символы не считал)

def fu1():                                                                      
    while True:
        try:
            a = int(raw_input('Введите число: '))
            return a
        except ValueError:
            a = int(raw_input('Введите число: '))

Comments

[Александр]

Спасибо. Только в except a = int(raw_input('Введите число: ')) нужно изменить на a = raw_input('Введите число: ')

[Илья]

Хотя нет, в except вообще нужно написать pass. Тк в новой итерации опять же нужно будет вводить число.

[Oscar Django]

в случае исключения оно будет два раза спрашивать число (один в теле except'a, второй - на новом витке цикла). Поэтому:

except ValueError:
    pass

[Илья]

Да, я выше уже это и написал)

Answer 5

[Дмитрий]

Александр
1. Как уже говорили в одном из ответов Вам надо дать отличные названия и убрать комментарии. Есть такое правило, если есть кусок кода, который хочется прокомментировать, то стоит этот кусок выделить в отдельную функцию и дать ей хорошее названии
2. Функции нужно стремиться оформлять так чтоб они меньше побочных эффектов производили. Должна одна что-то спрашивать у юзера и выводить ему результат, а другие должны для нее выполнять вычислительную работу. У Вас же raw_input в одном месте, print в другом. Не зачем так писать.
3. Не обязательно в маленькой функции писать обработчик исключений. Вынесите это в высокоуровневый код.

Приведу аналогию с начальник-подчиненный. Начальник дает задачу , подчиненный берет ее и убеждается что понял ее, если что спрашивает уточняющие или сразу говорит "не могу выполнить, не хватает ...". Когда же он что-то делает, он не парится а хорошо ли плохо. Зато если вдруг что-то серьезное произошло, то он не парится на тему, а как бы это исправить, а просто уведомляет "Иван Иваныч, подшипники надо истерлись, надо бы купить новые" и это проблема начальника обратиться в другой отдел. От подчиненного все что было нужно уже имеется!

Answer 6

[Александр]

А так считается хорошо?
вместо

print c, "гипотенуза"
print b, "площадь"

Писать
print c, "гипотенуза","\n", b, "площадь"

Comments

[Oscar Django]

Полагаю хорошо писать через форматирование строк. Через format, как у меня было, или через '%s' % variable