Как применить принцип DRY к коду?

Question

[bimka]

У меня есть код, который отрисовывает геометрические фигуры:

def drawing_rectengle(a, b):
    figure = models.Rectangle(a, b)
    plt.plot([0, figure.a, figure.a, 0, 0], [0, 0, figure.b, figure.b, 0])
    plt.axis('equal')
    plt.show()   

def drawing_rhomb(h1, h2):
    figure = models.Rhomb(h1, h2)
    plt.plot([0, figure.h2 / 2, figure.h2, figure.h2 / 2, 0], [0, figure.h1 / 2, 0, -figure.h1 / 2, 0])
    plt.axis('equal')
    plt.show()   

def drawing_rhomb(h1, h2):
    figure = models.Rhomb(h1, h2)
    plt.plot([0, figure.h2 / 2, figure.h2, figure.h2 / 2, 0], [0, figure.h1 / 2, 0, -figure.h1 / 2, 0])
    plt.axis('equal')
    plt.show()

Как видно выше код часто повторяется. Как мне переделать отрисовку фигур, используя принцип DRY? Я думал сделать что-то вроде:

rhomb = [0, figure.h2 / 2, figure.h2, figure.h2 / 2, 0], [0, figure.h1 / 2, 0, -figure.h1 / 2, 0]
square = [0, figure.a, figure.a, 0, 0],[0, 0, figure.a, figure.a, 0]

def drawing_figure(a):
    figure = models.Square(a) # Как создавать разные объекты для разных классов, 
    plt.plot(square)
    plt.axis('equal')
    plt.show()

Comments

[Andrey]

имхо, dry надо понимать не сколько в повторении а сколько в изменении т.е. при изменении чего-то одного нужно менять и что-то другое в других местах

так же надо помнить о единстве ответственности и тут вопрос - что хуже или лучше - универсальность либо простота и понятность

в вашем варианте вы вроде как бы реализуете dry но по моему мнению его тут нет но явно усложняете код нарушая принцип единой ответственности

к тому же возникает вопрос - как вы поступите если потребуется отображение еще каких-либо фигур ?

Answer 1

[Владимир Куц]

Ну так передавайте класс:

def drawing_figure(mymodel, coords):
    figure = mymodel(coords) # Как создавать разные объекты для разных классов, 
    ...

drawing_figure(models.Square, [...])

А вообще тут ООП напрашивается...

Answer 2

[LXSTVAYNE]

В matplotlib уже реализованы базовые фигуры: https://matplotlib.org/stable/api/patches_api.html...