Skip to content

barisik.batuhan/T4#287

Open
batuhandb wants to merge 7 commits into
ov-ale:mainfrom
batuhandb:barisik.batuhan/T4
Open

barisik.batuhan/T4#287
batuhandb wants to merge 7 commits into
ov-ale:mainfrom
batuhandb:barisik.batuhan/T4

Conversation

@batuhandb

Copy link
Copy Markdown
Contributor

T4: Add Circle, IsoscelesTrapezoid and CompositeShape implementation

@ov-ale ov-ale left a comment

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

нет класса Rectangle

Comment thread barisik.batuhan/T4/circle.cpp Outdated
IsoscelesTrapezoid::IsoscelesTrapezoid(const Point& bl, double bottomBase,
double topBase, double height)
: bottomLeft(bl), bottomBase(bottomBase), topBase(topBase), height(height) {
if (bottomBase <= 0 || topBase <= 0 || height <= 0) {

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

тип double

return (bottomBase + topBase) * height / 2.0;
}
Point IsoscelesTrapezoid::getCenter() const {
double centerX = bottomLeft.x + bottomBase / 2.0;

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Для равнобедренной трапеции центр должен быть на средней линии

Comment thread barisik.batuhan/T4/main.cpp Outdated
std::cout << "=== BEFORE SCALING ===" << std::endl;
printShapesInfo(shapes);

double factor = 2.0;

Copy link
Copy Markdown
Owner

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

коэффициент масштабирования нужно читать , у Вас жестко задан

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants