Бобровских Сергей #38

18thday · 2026-01-21T05:43:30Z

No description provided.

WIP: week_01

WIP: week 02

…asy_compare

18thday · 2026-01-21T05:58:06Z

01_week/tasks/quadratic/quadratic.cpp

+
+    double a_ = static_cast<double>(a);
+    double b_ = static_cast<double>(b);
+    double c_ = static_cast<double>(c);


это лишнее, вариант выше был лучше, дальше также

18thday · 2026-01-21T05:59:34Z

01_week/tasks/quadratic/quadratic.cpp

+            // Один корень линейного уравнения
+            PrintRoot(-c_ / b_);
+        }
+        return;


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

18thday · 2026-01-21T06:00:40Z

01_week/tasks/quadratic/quadratic.cpp

+        return;
+    }
+
+    double D = b_*b_ - 4*a_*c_;


в данном случае лучше здесь сделать static_cast для одного b, а вместо 4 использовать 4.0

18thday · 2026-01-21T06:01:18Z

01_week/tasks/quadratic/quadratic.cpp

+            std::cout << "no solutions";
+        } else {
+            // Один корень линейного уравнения
+            PrintRoot(-c_ / b_);


лучше static_cast одной переменной, без создания

18thday · 2026-01-21T06:04:04Z

01_week/tasks/quadratic/quadratic.cpp

+        PrintRoot(x);
+    }
+    else {
+        std::cout << "no solutions";


лучше начать с коротких ветвей (D < 0) с return а основной случай D > 0 вынести из под условия, чтобы не было лишней вложенность

18thday · 2026-01-21T06:05:35Z

02_week/tasks/func_array/func_array.cpp


-double ApplyOperations(double a, double b /* other arguments */) {
-    throw std::runtime_error{"Not implemented"};
+double ApplyOperations(double a, double b, double (*ptr[])(double, double), size_t size) {


ptr не удачно название здесь, лучше func_arr например, более соответствует действительности

18thday · 2026-01-21T06:08:13Z

02_week/tasks/last_of_us/last_of_us.cpp

-/* return_type */ FindLastElement(/* ptr_type */ begin, /* ptr_type */ end, /* func_type */ predicate) {
-    throw std::runtime_error{"Not implemented"};
+const int* FindLastElement(const int* begin, const int* end, bool (*predicate)(const int)) {
+    if((begin > end) || (begin == nullptr) || (end == nullptr)) return end;


для указателей часто принято не использовать явное сравнение с nullptr, а использовать неявное приведение к bool !begin || !end будет короче

18thday · 2026-01-21T06:09:08Z

02_week/tasks/last_of_us/last_of_us.cpp

+const int* FindLastElement(const int* begin, const int* end, bool (*predicate)(const int)) {
+    if((begin > end) || (begin == nullptr) || (end == nullptr)) return end;
+
+    for(const int* ptr = end - 1; ptr >= begin; --ptr) {


пробел после if и for ставится

18thday · 2026-01-21T06:09:57Z

02_week/tasks/little_big/little_big.cpp

+        for (size_t i = 0; i < sizeof(int); ++i) {
+            std::cout << std::hex << std::uppercase << std::setw(2) << std::setfill('0') << static_cast<int>(bytes[i]);
+        }
+    }


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

18thday · 2026-01-21T06:11:25Z

02_week/tasks/little_big/little_big.cpp

-void PrintMemory(double /* write arguments here */) {
-    throw std::runtime_error{"Not implemented"};
+void PrintMemory(double value, bool reverseBytes = false) {
+    unsigned char* bytes = reinterpret_cast<unsigned char*>(&value);


это полное дублирование кода, лучше было из данных функций после получения указателя и зная sizeof от типа вызыватьтретью функцию которая как раз выполняет работу

18thday · 2026-01-21T06:12:29Z

02_week/tasks/longest/longest.cpp

+
+    for (const char* ptr = begin + 1; ptr < end; ++ptr) {
+        if (*ptr == *(ptr - 1)) {
+            current_count++;


это должен быть префиксный инкременет

18thday · 2026-01-21T06:13:17Z

02_week/tasks/longest/longest.cpp

+    for (const char* ptr = begin + 1; ptr < end; ++ptr) {
+        if (*ptr == *(ptr - 1)) {
+            current_count++;
+        } else {


лучше выше написать continue, а тут избавится от вложенности лишнего else

18thday · 2026-01-21T06:16:15Z

02_week/tasks/longest/longest.cpp

+        longest_ptr = current_ptr;
+    }
+    count = max_count;
+    return const_cast<char*>(longest_ptr);


это UB, так как мы можем принять изначально указатель на константный объект, нужно было написать две функции у одной принимаемые и возвращаемые указатели на константу, а другого простые, и вот уже из функции без константости корректно было бы вызывать константную версию и снимать константность с результата const_cast, так как была бы увереность, что объект изначально не константный

18thday · 2026-01-21T06:17:19Z

02_week/tasks/pretty_array/pretty_array.cpp

+            if(ptr == end) { break; }
+            ++count;
+        }
+        std::cout << "]\n";


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

18thday · 2026-01-21T06:19:19Z

02_week/tasks/swap_ptr/swap_ptr.cpp

+template <typename T>
+void SwapPtr(T*& ptr1, T*& ptr2) {
+    // Проверяем, указывают ли ptr1 и ptr2 на указатели
+    if constexpr (std::is_pointer_v<std::remove_pointer_t<T>>) {


как будто это уже лишнее, и вывод типов шаблона справился бы без этого

18thday · 2026-01-21T06:19:36Z

02_week/tasks/swap_ptr/swap_ptr.cpp


-void SwapPtr(/* write arguments here */) {
-    throw std::runtime_error{"Not implemented"};
+template <typename T>


можно было и без шаблона получить решение

18thday · 2026-01-21T06:21:04Z

03_week/tasks/data_stats/data_stats.cpp

+        stats.sd += static_cast<long long>(num) * num;
+    }
+    // Вычисляем среднее значение
+    stats.avg = static_cast<double>(stats.avg) / numbers.size();


stats.avg уже имеет тип double поэтому каст тут лишний и можно было использовать /=

18thday · 2026-01-21T06:22:55Z

03_week/tasks/data_stats/data_stats.cpp

+    }
+    else {
+        stats.sd = 0;
+    }


как будто это лишнее, либо можно было компактнее переписать

18thday · 2026-01-21T06:25:14Z

03_week/tasks/easy_compare/easy_compare.cpp

+    static const std::map<char, int> priority = {
+        {'Z', 0}, {'D', 1}, {'C', 2}, {'B', 3}, {'A', 4}
+    };
+    return priority.at(mark);


это плохой вариант, так как символы оценок уже упорядочены, иможно было использовать напрямую коды символов, данные подход не гибкий, в случае добавления буков например Y или F, любой, придется переписыватть все начения, очень неудобно

18thday · 2026-01-21T06:27:02Z

03_week/tasks/easy_compare/easy_compare.cpp

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


это тоже можно сделать компактно с std::tie так как в если менять местами поля для rhs lhs в правом и левом операнде, при одном знаке <, можно некоторые поля сравнивать на <, а некотоыре на >

18thday · 2026-01-21T06:28:37Z

03_week/tasks/filter/filter.cpp

+
+    for (size_t i = 0; i < vec.size(); ++i) {
+        if (predicate(vec[i])) {
+            if (i != write_index) {


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

18thday · 2026-01-21T06:31:29Z

03_week/tasks/minmax/minmax.cpp

 #include <stdexcept>
+#include <vector>

+using IntVecIterator = std::vector<int>::const_iterator;


лучше отметить что он константный, cit, как вариант можно было сделать два псевдонима, чтобыбыло понятнее какой min какой max, тут понятно по названию функции, но в CitMin CitMax например тоже можно

18thday · 2026-01-21T06:34:24Z

03_week/tasks/minmax/minmax.cpp

+        }
+
+        if (*local_min < *min_it) min_it = local_min;
+        if (*local_max >= *max_it) max_it = local_max;


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

18thday · 2026-01-21T06:36:31Z

03_week/tasks/os_overload/os_overload.cpp

 struct Coord2D {
    int x;
    int y;
+    Coord2D(int x_val = 0, int y_val = 0) : x(x_val), y(y_val) {}


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

18thday · 2026-01-21T06:37:38Z

03_week/tasks/os_overload/os_overload.cpp

+    unsigned radius = 1;
+    Circle() : coord(), radius(1) {}    // Default
+    explicit Circle(Coord2D c, unsigned r = 1) : coord(c), radius(r) {}
+    Circle(unsigned r) : coord(), radius(r) {}


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

18thday · 2026-01-21T06:40:16Z

03_week/tasks/os_overload/os_overload.cpp

+}
+
+std::ostream& operator<<(std::ostream& os, const CircleRegionList& circle_vector) {
+    os << '{';


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

18thday · 2026-01-21T06:48:49Z

03_week/tasks/range/range.cpp

+            sequence.push_back(current);
+            current += step;
+        }
+    }


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

18thday · 2026-01-21T06:49:18Z

03_week/tasks/unique/unique.cpp

-}
+std::vector<int> Unique(std::vector<int> sorted_nums) {
+    std::vector<int> unique_nums;
+    if(sorted_nums.empty()) {return unique_nums;}


что- то тут стилькода поменялся

18thday · 2026-01-21T06:49:55Z

03_week/tasks/unique/unique.cpp

+    int previous = *(sorted_nums.begin());
+    unique_nums.push_back(previous);
+    for(auto num : sorted_nums) {
+        if(previous != num) {


лучше убрать вложенность previous == num - > continue

18thday

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

дублирование кода
UB const_cast
местами лишние переменные
не все задачи решены

Sergey Bobrovskikh and others added 30 commits November 22, 2025 22:49

add solution: addition, rms, print_bits tasks

786c587

Merge branch 'main' into week_01

e47e779

add (solution): add check_flags task

2be84bb

add (solution) add quadratic tast

73ae725

add solution: add length_lit task

3c312a5

Merge remote-tracking branch 'origin/main' into week_01

97ebf04

Merge branch 'main' into week_01

5eb2252

add [solution]: add task swap_ptr

b53ca2a

add [solution]: add task func_array

3e5a80a

add [solution]: add task longest

9159eeb

Merge pull request #1 from WhoAmI1909/week_01

b20c4ff

WIP: week_01

Merge branch 'main' into week_02

097fc7d

add [solution]: add task last_of_us

9f39dfa

add [solution]: add task little_big

4f48105

add [solution]: add task pretty_array

b6f99dc

Merge branch 'psds-cpp:main' into main

78363af

add solution: add task [data_stats]

e8920e7

add solution: add task [unique]

56befd7

add solution: add task [range]

27abbaa

add solution: add task [minmax]

a5f6e06

add solution: add task [find_all]

dca960a

add solution: add task [os_overload]

81a216b

add solution: add task [easy_compare]

99c2021

add solution: add task [filter]

0ecaae7

Merge branch 'psds-cpp:main' into main

07c7889

Merge branch 'main' into week_03

8ecb3cb

Merge pull request #2 from WhoAmI1909/week_02

7658216

WIP: week 02

Merge branch 'main' into week_03

17ee479

Для тестов без 4 недели

5fda3fb

Дополнены данные для конструктора по умолчанию: Date: task week_03: e…

bd5dc5a

…asy_compare

18thday commented Jan 21, 2026

View reviewed changes

Бобровских Сергей #38

Are you sure you want to change the base?

Бобровских Сергей #38

Uh oh!

Conversation

18thday commented Jan 21, 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!

18thday Jan 21, 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!

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!

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 • 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!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

18thday Jan 21, 2026 •

edited

Loading

18thday left a comment •

edited

Loading