長い関数を小さな単位に分割すると、読みやすさとモジュール性の両方が向上します。この演習では、ある関数をより小さな単位にリファクタリングします。リファクタリング対象の関数を以下に示します。なお、この演習ではスペースの都合上 docstring を省略していますが、実際のアプリケーションではドキュメントを必ず含めるようにしましょう。

def polygon_area(n_sides, side_len):
    """Find the area of a regular polygon

    :param n_sides: number of sides
    :param side_len: length of polygon sides
    :return: area of polygon

    >>> round(polygon_area(4, 5))
    25
    """
    perimeter = n_sides * side_len

    apothem_denominator = 2 * math.tan(math.pi / n_sides)
    apothem = side_len / apothem_denominator

    return perimeter * apothem / 2

Readability counts～読みやすいことは重要だ

Create Your Free Account