Коптелов Константин by 18thday · Pull Request #39 · psds-cpp/psds-cpp-2025

18thday · 2026-01-21T07:03:56Z

No description provided.

18thday · 2026-01-21T07:05:34Z

01_week/tasks/addition/addition.cpp


 int64_t Addition(int a, int b) {
-    throw std::runtime_error{"Not implemented"};
+    return static_cast<int64_t>(a) + static_cast<int64_t>(b);


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

18thday · 2026-01-21T07:06:55Z

01_week/tasks/char_changer/char_changer.cpp

+        char new_char;
+
+        // Преобразуем символ по правилам
+        if (c >= '0' && c <= '9') {


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

18thday · 2026-01-21T07:07:32Z

01_week/tasks/char_changer/char_changer.cpp

+                next_new_char = delimiter;
+            } else {
+                next_new_char = '_';
+            }


дублирование всех проверок, не очень хорошо делать такой копи паст

18thday · 2026-01-21T07:08:12Z

01_week/tasks/char_changer/char_changer.cpp

+            }
+
+            if (next_new_char == new_char) {
+                count++;


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

18thday · 2026-01-21T07:09:01Z

01_week/tasks/char_changer/char_changer.cpp

+            if (next_new_char == new_char) {
+                count++;
+                next++;
+            } else {


это короткая ветвь корректней сделать if (next_new_char != new_char) break и убрать else

18thday · 2026-01-21T07:10:12Z

01_week/tasks/check_flags/check_flags.cpp

+    }
+
+    // Начинаем вывод
+    printf("[");


printf это не C++

18thday · 2026-01-21T07:10:54Z

01_week/tasks/check_flags/check_flags.cpp

+        if (!first) printf(",");
+        printf("DEST");
+        first = false;
+    }


в данном случае это тоже дублирование, много однотипных действий с разными аргументами

18thday · 2026-01-21T07:13:03Z

01_week/tasks/length_lit/length_lit.cpp

+}
+
+constexpr double operator"" _m_to_cm(long double meters) {
+    return static_cast<double>(meters * 100.0L);


использование magic value не принято в коде, удобнее сделать константы с понятными именами и уже их использовать в коде, безымянный namespace для их объединения как вариант использвоания

18thday · 2026-01-21T07:13:25Z

01_week/tasks/print_bits/print_bits.cpp

+            printf("1");
+        } else {
+            printf("0");
+        }


это должен быть тернарный оператор

18thday · 2026-01-21T07:14:21Z

01_week/tasks/quadratic/quadratic.cpp

+            // b*x + c = 0 → x = -c/b
+            double x = -static_cast<double>(c) / b;
+            std::cout << x << std::endl;
+        }


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

18thday · 2026-01-21T07:15:31Z

01_week/tasks/quadratic/quadratic.cpp

+
+    if (discriminant < 0) {
+        // Отрицательный дискриминант - нет действительных корней
+        std::cout << "no solutions" << std::endl;


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

18thday · 2026-01-21T07:16:26Z

01_week/tasks/rms/rms.cpp

+
+    // Корень из среднего значения квадратов
+    return std::sqrt(mean_of_squares);
 }


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

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

01_week/tasks/rms/rms.cpp

+    double mean_of_squares = sum_of_squares / size;
+
+    // Корень из среднего значения квадратов
+    return std::sqrt(mean_of_squares);


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

18thday · 2026-01-21T07:33:16Z

02_week/tasks/little_big/little_big.cpp

+    if (!reverseBytes) {
+        // Little-endian порядок (по умолчанию)
+        std::cout << "0x";
+        for (int i = 0; i < static_cast<int>(sizeof(temp)); i++) {


префиксный инкремент

18thday · 2026-01-21T07:33:30Z

02_week/tasks/little_big/little_big.cpp

+            unsigned char byte = (temp >> (i * 8)) & 0xFF;
+            std::cout << std::hex << std::setw(2) << std::setfill('0') 
+                      << static_cast<unsigned int>(byte);
+        }


дублирование тела функций

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

02_week/tasks/little_big/little_big.cpp

+            std::cout << std::hex << std::setw(2) << std::setfill('0') 
+                      << static_cast<unsigned int>(byte);
+        }
+    }


полное дублирвоание тела, нужно было после memcpy вызывать функцию выполняющую работу, передавая например указатель или сам temp и sizeof от типа

18thday · 2026-01-21T07:35:48Z

02_week/tasks/longest/longest.cpp

+    // Проверка на пустой диапазон
+    if (begin == end) {
+        count = 0;
+        return nullptr;


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

18thday · 2026-01-21T07:36:39Z

02_week/tasks/longest/longest.cpp

+            current_start = current;
+            current_length = 1;
+        }
+        current++;


префиксный инкремент должен быть + если ++current выполняется всегда, может стоило переписать на for

18thday · 2026-01-21T07:37:15Z

02_week/tasks/pretty_array/pretty_array.cpp

+    if (begin == end) {
+        std::cout << "[]\n";
+        return;
+    }


стоит объединить условия

18thday · 2026-01-21T07:38:06Z

02_week/tasks/pretty_array/pretty_array.cpp

+            last--;
+            if (last <= end) break;
+        }
+    }


неудобно читать код с множеством if else можно было написать компактнее

18thday · 2026-01-21T07:38:51Z

02_week/tasks/swap_ptr/swap_ptr.cpp


-void SwapPtr(/* write arguments here */) {
-    throw std::runtime_error{"Not implemented"};
+void SwapPtr(const int*& a, const int*& b) {


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

18thday · 2026-01-21T07:39:10Z

02_week/tasks/swap_ptr/swap_ptr.cpp

+    int** temp = a;
+    a = b;
+    b = temp;
 }


можно было реализовать одной функцией

18thday

@winnermrco-boop основные моменты:

мало решено заданий
дублирование кода
нет использования стандартных функций првоерки диапазонов символов в CharChanger

winnermrco-boop and others added 16 commits November 28, 2025 13:16

add (solution): add addition task

de47f80

add (solution): add char_changer task

05792e8

add (solution): add check_flags task

3bb2c01

add (solution): add length_lit task

e44d185

add (solution): add print_bits task

b7e7a72

add (solution): add quadratic task

843b8a0

add (solution): add rms task

8b3ab9b

Merge branch 'psds-cpp:main' into main

30a7215

add (solution): add funck_array task

17a6853

add (solution): add last_of_us task

10f35a2

add (solution): add little_big task

18e2690

add (solution): add longest task

58d1317

add (solution): add pretty_array task

4c2f08b

add (solution): add swap_ptr task

d939aec

add (solution): add longest task

9cd77eb

Merge branch 'main' of github.com:psds-cpp/psds-cpp-2025

52ef35e

18thday commented Jan 21, 2026

View reviewed changes

01_week/tasks/check_flags/check_flags.cpp

}

// Начинаем вывод

printf("[");

Copy link

Contributor Author

18thday Jan 21, 2026

Choose a reason for hiding this comment

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

printf это не C++

18thday commented Jan 21, 2026

View reviewed changes

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!

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

18thday left a comment

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