Карсканова Светлана by 18thday · Pull Request #29 · psds-cpp/psds-cpp-2025

18thday · 2026-01-12T08:29:30Z

No description provided.

* add 1: add addition task * Add addition.cpp only * add 1: add char_changer task * add : add check_flags task

18thday · 2026-01-12T10:55:35Z

03_week/tasks/data_stats/data_stats.cpp

-    double sd = 0.0;
-};
+    double avg;
+    double sd;};


вот как раз оставлять неинициализированные поля не принято

18thday · 2026-01-12T10:56:26Z

03_week/tasks/data_stats/data_stats.cpp

+    double mean = sum / data.size();
+    double variance_sum = 0.0;
+    for (int x : data) {
+        variance_sum += (x - mean) * (x - mean);}


два прохода. Можно реализовать эффективнее и считать оба параметра на одном походе

18thday · 2026-01-12T10:57:49Z

03_week/tasks/easy_compare/easy_compare.cpp

+    unsigned day = 0;
+    Date() = default;
+    Date(unsigned y, unsigned m, unsigned d)
+        : year(y), month(m), day(d) {}};


конструкторы для структур излишни в данном случае, поскольку они тривиальные, компилятор бы справился и принято использовать данную возможность

18thday · 2026-01-12T10:58:39Z

03_week/tasks/easy_compare/easy_compare.cpp

+bool operator>(const Date& lhs, const Date& rhs) {
+    return rhs < lhs;}
+bool operator>=(const Date& lhs, const Date& rhs) {
+    return !(lhs < rhs);}


очень не приятное глазу оформление } на новой строке

18thday · 2026-01-12T10:59:04Z

03_week/tasks/easy_compare/easy_compare.cpp

    unsigned course;
-    Date birth_date;
-};
+    Date birth_date;};


18thday · 2026-01-12T11:00:57Z

03_week/tasks/easy_compare/easy_compare.cpp

+        return lhs.course > rhs.course;      
+    if (lhs.birth_date != rhs.birth_date)
+        return lhs.birth_date < rhs.birth_date; 
+    return false;


это не плохо но можно выполнить с помощью std::tie() < std::tie() в левом операнде можно располагать не только поля lhs но и rhs, тем самым добиваясь, что они сравниваются на <, а другие на >

18thday · 2026-01-12T11:02:15Z

03_week/tasks/enum_operators/enum_operators.cpp

+        static_cast<uint8_t>(rhs);
+    result &= static_cast<uint8_t>(CheckFlags::ALL);
+    return static_cast<CheckFlags>(result);}
+bool operator&(CheckFlags lhs, CheckFlags rhs) {


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

18thday · 2026-01-12T11:03:26Z

03_week/tasks/filter/filter.cpp

-/* return_type */ Filter(/* args */) {
-    throw std::runtime_error{"Not implemented"};
-}
+void Filter(std::vector<int>& v, bool (*pred)(int)) {


v не подходит в качестве названия ) data, values будет лучше

18thday · 2026-01-12T11:05:48Z

03_week/tasks/minmax/minmax.cpp

+        if (*it >= *max_it) {
+            max_it = it;}
+    }
+    return std::make_pair(min_it, max_it);


не уверен, что нужно давать возможность изменять по итераторам вне функции минимальный и максимальный элемент, а не только читать их, если этого не оговорено в ТЗ отдельно

18thday · 2026-01-12T11:06:26Z

03_week/tasks/os_overload/os_overload.cpp

+    int x = 0;
+    int y = 0;
+    Coord2D() = default;
+    Coord2D(int x_, int y_) : x(x_), y(y_) {}};


лишние конструкторы

18thday · 2026-01-12T11:07:27Z

03_week/tasks/os_overload/os_overload.cpp

+    if (c.isEmpty()) {
+        os << "circle[]";} 
+        else {
+        os << "circle[" << c.coord << ", r = " << c.radius << "]";}


в данном случае подходит тернарный оператор

18thday · 2026-01-12T11:08:14Z

03_week/tasks/os_overload/os_overload.cpp

-using CircleRegion = std::pair<Circle, bool>;
+    unsigned int radius = 1;
+    Circle() = default;
+    Circle(const Coord2D& c, unsigned int r = 1) : coord(c), radius(r) {}


лишние конструкторы, компилятор сам справится с тривиальными конструкторами, для структур, их не принято писать

18thday · 2026-01-12T11:09:53Z

03_week/tasks/os_overload/os_overload.cpp

+std::ostream& operator<<(std::ostream& os, const CircleRegionList& list) {
+    if (list.empty()) {
+        os << "{}";
+        return os;}


нужно } напистаь на новой строке и даже оставить потом пустую строку, чтобы отделить особый слуай от основной программы, также можно сразу писать return os << "{}";

18thday · 2026-01-12T11:11:36Z

03_week/tasks/range/range.cpp

+    } else {
+        for (int i = from; i > to; i += step) {
+            ++count;}
+    }


Это лишний проход, можно предпосчитать правильное посчитать элементов по формуле, без прохода

18thday · 2026-01-12T11:12:17Z

03_week/tasks/range/range.cpp

+    } else {
+        for (int i = from; i > to; i += step) {
+            result.push_back(i);}
+    }


можно по-другому переписать условия использовать тернарный оператор, но в данном случае дублирование кода, который можно избежать

18thday

@karskanovas основные моменты

C-style каст не используют в C++ кода
плохо читаемый стиль кода (} - принято на новой строке, иногда не хватает пустых строк для удобного оттеделния логических частей функйии )
UB const_cast
дублирование кода

18thday · 2026-01-22T08:36:37Z

04_week/tasks/stack/stack.cpp

+    bool operator==(const Stack& other) const {
+        return data == other.data;}
+    bool operator!=(const Stack& other) const {
+        return data != other.data;}


было требование к вынесению определений вне класса, так неодобно ситать код, когда } не заканчивается на новой строке

18thday · 2026-01-22T08:37:28Z

04_week/tasks/ring_buffer/ring_buffer.cpp

+            start = other.start;
+            end = other.end;
+            count = other.count;}
+        return *this;}


конструкторы и операторы присваивания пишутся с другими конструкторами

18thday · 2026-01-22T08:37:54Z

04_week/tasks/queue/queue.cpp

+public:
+    Queue() = default;
+    Queue(const std::stack<int>& s) {
+        std::stack<int> temp = s;


можно тогда сразу принять по значению

18thday · 2026-01-22T08:38:31Z

04_week/tasks/queue/queue.cpp

+        Transfer();
+        return outStack_.front(); }
+    const int& Back() const {
+        return const_cast<Queue*>(this)->Back();}


UB const_cast так как можетприменяться для константного объекта

18thday · 2026-01-22T08:39:08Z

04_week/tasks/queue/queue.cpp

+        std::swap(outStack_, other.outStack_);}
+    bool operator==(const Queue& other) const {
+        Queue copy1 = *this;
+        Queue copy2 = other;


неоптимальное сравнение, копирование излишне, причем мы копируем даже когда размеры разные

18thday

@karskanovas следующие моменты:

плохо читаемый стиль кода
UB const_cast
дублирование кода
неоптимальный оператор сравнения Queue с безусловным копированием двух очереденей

…отал изначально

karskanovas and others added 30 commits November 28, 2025 19:47

Update addition.cpp

aa06b1f

Addition task (#1)

240b2e7

* add 1: add addition task * Add addition.cpp only * add 1: add char_changer task * add : add check_flags task

ignore build directory

8374322

add: add length_lit task

73267df

add: add print_bits task

4b96e10

add: add quadratic task

71bc667

add: add rms task

b570327

fix: add addition task

7be7e34

fix: add char_changer task

15175f3

fix: add check_flags task

392bcb2

fix: add length_lit task

8c32ee1

fix: add print_bits task

620df85

fix: add quadratic task

49b333b

fix: add rms task

2171651

fix: add addition task

46982b6

Auto-sync: 2025-12-01 19:19

c8ec7eb

Auto-sync: 2025-12-02 09:46

a7f167e

Auto-sync: 2025-12-03 07:00

2e8f4d9

Auto-sync: 2025-12-03 12:28

a67e355

func_array

bd129ad

last of us

0c30f21

little big

edfd91a

longer

7450931

pretty_array

319b6f3

swap_ptr

5e9f0cf

Merge upstream/main

ed641e9

data_stats

43dcac3

Fix CMakeLists.txt for 04_week temporarily

465b6f1

easy_compar

3e84f50

enum_operators.cpp

e805cf1

18thday commented Jan 12, 2026

View reviewed changes

18thday commented Jan 22, 2026

View reviewed changes

karskanovas added 8 commits February 16, 2026 23:00

Восстановлена папка 05_week из оригинального репозитория

dac3bc0

05_week всё

29b0871

конфликты

9f6d1f9

конфликт 2

b5a088d

conflicts_3

874b6fc

Merge remote-tracking branch 'upstream/main'

b9e3198

7 неделя

e9034f8

Когда решала конфликты, случайно удалила конечную собку, а так он раб…

46275fb

…отал изначально

Conversation

18thday commented Jan 12, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

18thday Jan 12, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

18thday left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

18thday Jan 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

18thday left a comment

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

3 participants

18thday Jan 12, 2026 •

edited

Loading

18thday Jan 22, 2026 •

edited

Loading