Михайлов Илья #27

18thday · 2026-01-05T19:18:28Z

No description provided.

18thday · 2026-01-05T19:20:43Z

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-05T19:22:27Z

01_week/tasks/char_changer/char_changer.cpp

-}
+size_t CharChanger(char array[], size_t, char delimiter = ' ') {
+    int counter = 0; // Счётчик повторяющихся символов
+    int write = 0;   // Указатель для записи обработанного символа


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

18thday · 2026-01-05T19:22:59Z

01_week/tasks/char_changer/char_changer.cpp

-    throw std::runtime_error{"Not implemented"};
-}
+size_t CharChanger(char array[], size_t, char delimiter = ' ') {
+    int counter = 0; // Счётчик повторяющихся символов


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

18thday · 2026-01-05T19:24:02Z

01_week/tasks/char_changer/char_changer.cpp

+
+    while (repeating_symbol != '\0'){
+        if (repeating_symbol == array[read]){
+            counter++;


здесь и далее корректней использовать префиксный инкремент

18thday · 2026-01-05T19:28:06Z

01_week/tasks/char_changer/char_changer.cpp

+    while (repeating_symbol != '\0'){
+        if (repeating_symbol == array[read]){
+            counter++;
+        } else {


лучше для if сделать короткую ветвь с continue:

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

Можно будет убрать else и лишний уровень вложенности

18thday · 2026-01-05T19:29:52Z

01_week/tasks/char_changer/char_changer.cpp

+            }
+
+            if (counter >= 10){
+                counter = 0;


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

18thday · 2026-01-05T19:32:44Z

01_week/tasks/check_flags/check_flags.cpp

+        return;
+    }
+
+    std::vector<std::string> set_flags;


Выглядит очень не оптимально, во первых вектор будет вызывать реалокацию при push_back.
Да и будем перекладывать строки из вектора в поток

18thday · 2026-01-05T19:33:33Z

01_week/tasks/length_lit/length_lit.cpp

+// Константы для преобразования
+constexpr long double IN_TO_CM = 2.54L;
+constexpr long double FT_TO_IN = 12.0L;
+constexpr long double M_TO_CM = 100.0L;


Это правильный подход, добавить константы

18thday · 2026-01-05T19:35:26Z

01_week/tasks/print_bits/print_bits.cpp

+            std::cout << 1;
+        } else {
+            std::cout << 0;
+        }


Можно 5 строк заменитьтернарным оператор выводить в поток по условию 1 или 0, будет компактнее

18thday · 2026-01-05T19:37:29Z

01_week/tasks/quadratic/quadratic.cpp

+        } else {
+            double x = static_cast<double>(-c) / b;
+            std::cout << std::defaultfloat << std::setprecision(6) << x;
+        }


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

18thday · 2026-01-05T19:39:45Z

01_week/tasks/quadratic/quadratic.cpp

+            std::cout << std::defaultfloat << std::setprecision(6) << x;
+        }
+    } else {
+        long long discriminant = static_cast<long long>(b) * b - 4 * static_cast<long long>(a) * c;


Понятно для чего так сделано, но не лучше хранить дискриминант в double в данном случае

18thday · 2026-01-05T19:42:15Z

01_week/tasks/rms/rms.cpp


 double CalculateRMS(double values[], size_t size) {
-    throw std::runtime_error{"Not implemented"};
+    if (size <= 0 || values == nullptr){ return 0.0; }


отсутствует пробел перед {

18thday · 2026-01-05T19:42:51Z

01_week/tasks/rms/rms.cpp

+
+    double sum = 0;
+    for (size_t i=0; i < size; i++){
+        sum+=std::pow(values[i], 2);


нет пробелов вокруг арифметических операторов и инициализации i

18thday · 2026-01-05T19:43:06Z

01_week/tasks/rms/rms.cpp

+    if (size <= 0 || values == nullptr){ return 0.0; }
+
+    double sum = 0;
+    for (size_t i=0; i < size; i++){


Нужно использовать префиксный инкремент

18thday · 2026-01-05T19:44:57Z

02_week/tasks/func_array/func_array.cpp


-double ApplyOperations(double a, double b /* other arguments */) {
-    throw std::runtime_error{"Not implemented"};
+double ApplyOperations(const double a, const double b, funcPtr* arr, size_t funcArraySize) {


наглядней было быиспользовать массив указателей на функцию, вместо указателя на указатель на функцию

18thday · 2026-01-05T19:45:19Z

02_week/tasks/func_array/func_array.cpp

+    }
+
+    double cumulative_result = 0;
+    for (unsigned int i=0; i < funcArraySize; ++i){


пробелы вокруг =

18thday · 2026-01-05T19:45:54Z

02_week/tasks/func_array/func_array.cpp

+        if (arr[i] == nullptr){
+            continue;
+        }
+        cumulative_result+=arr[i](a, b);


пробелы вокруг оператора +=

18thday · 2026-01-05T19:47:19Z

02_week/tasks/last_of_us/README.md

 указателя на начало и конец целочисленного массива, а также указатель на 
 функцию предикат, и возвращает указатель на найденный элемент. Если элемент 
-не найден, то возвращается указатель за последний элемент.
+не найден, то возвращается указатель на последний элемент.


Это некорректное исправление так как, end не указывает на элемент и его нельзя разыменовывать

18thday · 2026-01-05T19:51:44Z

02_week/tasks/last_of_us/last_of_us.cpp

+
+    if (last_element == nullptr){
+        return end;
+    }


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

18thday · 2026-01-05T19:53:10Z

02_week/tasks/last_of_us/last_of_us.cpp

+    {
+        if (predicate(*begin)){
+            last_element = begin;
+        }


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

18thday · 2026-01-05T19:59:04Z

02_week/tasks/little_big/little_big.cpp

+                  << std::setfill('0') << std::setw(2) 
+                  << static_cast<unsigned int>(bytes[i]);
+    }
+    std::cout << std::dec << std::endl;


Это задание было конечно на reinterpret_cast, что позволило бы не создавать буфер копию текущего значения. Но даже так, зачем в данном случае полное дублирование?
Лучше было написать ещё одну функцию, которая бы принимала буффер, размер и флаг и вызывалась бы из данных функций

18thday · 2026-01-05T20:12:14Z

02_week/tasks/pretty_array/pretty_array.cpp

+            }
+        }
+    }
+    std::cout << "]\n";


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

18thday · 2026-01-05T20:12:50Z

02_week/tasks/swap_ptr/swap_ptr.cpp

+    T* temp = ptr2;
+    ptr2 = ptr1;
+    ptr1 = temp;
 }


Можнобыло практически тоже самое сделать без шаблона

18thday · 2026-01-05T20:13:13Z

03_week/tasks/data_stats/data_stats.cpp

+    double average = 0.0;
+    double standardDeviation = 0.0;
+
+    if (!numArray.empty()){


пробела не хватает перед {

18thday · 2026-01-05T20:14:26Z

03_week/tasks/data_stats/data_stats.cpp

-/* return_type */ CalculateDataStats(/* args */) {
-    throw std::runtime_error{"Not implemented"};
-}
+DataStats CalculateDataStats(std::vector<int> numArray) {


Зачем мы копируем весь вектор? Там может быть очень много элемментов.

18thday · 2026-01-05T20:16:49Z

03_week/tasks/data_stats/data_stats.cpp

+            average += delta / count;
+            delta2 = num - average;
+            standardDeviation += delta * delta2;
+        }


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

18thday · 2026-01-05T20:19:27Z