Шайхуллин Евгений #18

18thday · 2025-12-24T06:28:52Z

No description provided.

18thday · 2025-12-24T06:39:15Z

01_week/tasks/addition/addition.cpp

-    throw std::runtime_error{"Not implemented"};
-}
+int64_t res = static_cast<int64_t>(a);    // Присвоение и приведение типов.
+res = res + static_cast<int64_t>(b);      // Вычисление суммы и приведение типов.


В данном случае это лишнее приведение типов, поскольку тип int64_t res то сработает implicit cast b к данному типу

18thday · 2025-12-24T06:40:50Z

01_week/tasks/addition/addition.cpp

-}
+int64_t res = static_cast<int64_t>(a);    // Присвоение и приведение типов.
+res = res + static_cast<int64_t>(b);      // Вычисление суммы и приведение типов.
+    return res;


Желательно написать вместо res сразу все выражение return a + b (с учетом каста одного из операндов), это будет нагляднее

18thday · 2025-12-24T06:46:19Z

01_week/tasks/rms/rms.cpp

+    double sum = 0.0;
+
+    if(0 == size) return 0.0;//
+    if(&values[0] == nullptr) return 0.0;


нет пробела после if

18thday · 2025-12-24T06:47:02Z

01_week/tasks/rms/rms.cpp

+    if(0 == size) return 0.0;//
+    if(&values[0] == nullptr) return 0.0;
+
+    for(size_t i = 0; i < size; i++){


нет пробела после for и перед {

18thday · 2025-12-24T06:47:24Z

01_week/tasks/rms/rms.cpp

+    if(&values[0] == nullptr) return 0.0;
+
+    for(size_t i = 0; i < size; i++){
+        sum += values[i]*values[i];


нет пробелов вокруг оператора *

18thday · 2025-12-24T06:49:31Z

01_week/tasks/rms/rms.cpp

-}
+    double sum = 0.0;
+
+    if(0 == size) return 0.0;//


Лучше использовать литерал для беззнаковых 0u

18thday · 2025-12-24T06:52:26Z

01_week/tasks/rms/rms.cpp

 double CalculateRMS(double values[], size_t size) {
-    throw std::runtime_error{"Not implemented"};
-}
+    double sum = 0.0;


В данном случае либо sum создается рано, так как далее две короткие ветви выхода из функции, в которых sum не участвует и возвращается литерал, либо возвращать не литерал

18thday · 2025-12-24T06:53:28Z

01_week/tasks/rms/rms.cpp

+    if(0 == size) return 0.0;//
+    if(&values[0] == nullptr) return 0.0;
+
+    for(size_t i = 0; i < size; i++){


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

18thday · 2025-12-24T06:59:49Z

01_week/tasks/rms/rms.cpp

+    double sum = 0.0;
+
+    if(0 == size) return 0.0;//
+    if(&values[0] == nullptr) return 0.0;


Это некорректная проверка приводящая к UB, values это уже указатель, поэтому если он nullptr то values[0] разыменовывает нулевой указатель.

Если быбыли соответствующие проверки в тестах, то этот код бы упал

18thday · 2025-12-24T07:07:26Z

01_week/tasks/quadratic/quadratic.cpp

+    float fA = static_cast<float>(a);   // Явное приведение типа int к float
+    float fB = static_cast<float>(b);   // Явное приведение типа int к float
+    float fC = static_cast<float>(c);   // Явное приведение типа int к float
+    float disc = static_cast<float>(c); // Явное приведение типа int к float


Слишком рано определяется множество переменных, дальше идут короткие ветки выхода из функций и если они срабатывают, то бесполезно создается множество переменных с преобразованием во float, во-вторых желательно было их не создавать, а выполнить некоторые преобразования по месту в выражениях и довериться неявному приведению

18thday · 2025-12-24T07:11:54Z

01_week/tasks/quadratic/quadratic.cpp

+    }
+
+    // Если x = 0 
+    if(((a == 0) && (c == 0)) || ((b == 0) && (c == 0)))


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

18thday · 2025-12-24T07:12:26Z

01_week/tasks/quadratic/quadratic.cpp

+
+    // Если a, b, c равны нулю – решений бесконечно много.
+    if((a == 0) && (b == 0) && (c == 0)){
+        std::cout <<"infinite solutions";


Вокруг операторов с двух сторон пробел по style guide

18thday · 2025-12-24T07:16:51Z

01_week/tasks/quadratic/quadratic.cpp

+
+    if(a == 0)  // Своего рода ускорение вычисления при a равном нулю
+    {
+        x1 = (-fC)/fB;


лишние скобки, нет пробелов

18thday · 2025-12-24T07:18:08Z

01_week/tasks/quadratic/quadratic.cpp

+    }
+    else    // Решение через дискрименант
+    {
+        disc = fB * fB - 4 * fA * fC;


Достаточно было сделать float disc, а лучше double и не заводить переменные fA, fB, fC

18thday · 2025-12-24T07:23:22Z

01_week/tasks/quadratic/quadratic.cpp

+        // определяем вывод в зависимости от X1 и X2
+        if (x1 == x2)    std::cout << std::setprecision(6) << x1 ;
+        else if(x1 < x2) std::cout << std::setprecision(6) << x1 << ' ' << x2;
+        else             std::cout << std::setprecision(6) << x2 << ' ' << x1;


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

18thday · 2025-12-24T07:25:18Z

01_week/tasks/quadratic/quadratic.cpp

+        }
+
+        x1 = (negativeUnit * fB - sqrt(disc))/(2*fA);
+        x2 = (negativeUnit * fB + sqrt(disc))/(2*fA);


Абсолютно не понял назначение целой переменно negativeUnit если можно использовать унарный минус. Нет std:: у sqrt. Нет пробелов вокруг операторов

18thday · 2025-12-24T07:26:48Z

01_week/tasks/length_lit/length_lit.cpp

+// Константы для преобразования
+#define IN_TO_CM 2.54
+#define FT_TO_IN 12.0
+#define M_TO_CM 100.0


Лучше использовать constexpr переменные

18thday · 2025-12-24T07:27:23Z

01_week/tasks/length_lit/length_lit.cpp

+}
+
+constexpr double operator"" _m_to_cm(long double x) {
+    return static_cast<double>( x * M_TO_CM); // 


лишний пробел перед x

18thday · 2025-12-24T07:39:11Z

01_week/tasks/length_lit/length_lit.cpp

+
+
+
+


Миллион лишних новых строк

18thday · 2025-12-24T07:39:31Z

01_week/tasks/length_lit/length_lit.cpp

+    return static_cast<double>( x * FT_TO_IN); // преобразование фт в дюймы 
+}
+
+


Если есть комментарий достаточно одной новой строки

18thday · 2025-12-24T07:41:47Z

01_week/tasks/print_bits/print_bits.cpp

-    throw std::runtime_error{"Not implemented"};
+
+// Защита от неверного значения размера в байтах. Размер в байтах должен быть от 1 до 8.
+if(bytes == 0)  return;    


отсутствие пробела и лишний пробел

18thday · 2025-12-24T07:45:07Z

01_week/tasks/print_bits/print_bits.cpp

+if(bytes == 0)  return;    
+if(bytes > 8) return;
+
+char buf[90] = {'\0'};    // Объявление и инициализация массива '\0'.


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

18thday · 2025-12-24T07:46:49Z

01_week/tasks/print_bits/print_bits.cpp

+        {
+            if(sizeBit != count)
+            {
+                buf[countBuf + count] = 39;


это magic value - от этого в коде принято избавляться, для представления данного символа нужно использовать escape-последовательности

18thday · 2025-12-24T07:47:14Z

01_week/tasks/print_bits/print_bits.cpp

+        tempBit = 0x01;
+        while(temp > countBit)
+        {
+            countBit ++;


Пробел не ставится, и постфиксный инкремент создает копию

18thday · 2025-12-24T07:48:33Z

01_week/tasks/print_bits/print_bits.cpp

+unsigned long long sizeBit = bytes*8;    // количество бит для вывода
+unsigned long long countFour = 0;
+unsigned long long countBuf = 0;    // счетчик для элементов массива
+unsigned long long countBit = 0;


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

18thday · 2025-12-24T07:50:41Z

01_week/tasks/check_flags/check_flags.cpp

+    if(data > static_cast<uint8_t>(CheckFlags::ALL))        return;
+    if(data != 0) 
+    {
+            if(data & static_cast<uint8_t>(CheckFlags::TIME)){


двойной отступ во вложенном if

18thday · 2025-12-24T07:51:50Z

01_week/tasks/check_flags/check_flags.cpp

 void PrintCheckFlags(CheckFlags flags) {
-    throw std::runtime_error{"Not implemented"};
+
+    char buf[40] = {'\0'};


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

18thday · 2025-12-24T07:52:51Z

01_week/tasks/check_flags/check_flags.cpp

+                    i++;
+                } 
+
+                buf[i++] = 'T'; buf[i++] = 'I'; buf[i++] = 'M'; buf[i++] = 'E';


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

18thday · 2025-12-24T07:57:19Z

01_week/tasks/char_changer/char_changer.cpp

 #include <stdexcept>

+// !!!! ВАЖНО !!!!  Изначальное объявление функции изменено в части инициализации передаваемого элемента с «char delimiter = ' '» на «char delimiter».
+size_t CharChanger(char array[], size_t size, char delimiter) {


Это некорректное исправление!
В тестах не было проверки на разделитель по умолчанию, но он должен быть задан в соответствии с сигнатурой функции по заданию

18thday · 2025-12-24T08:51:29Z

03_week/tasks/find_all/find_all.cpp

-/* return_type */ FindAll(/* args */) {
-    throw std::runtime_error{"Not implemented"};
-}
+std::vector<size_t>  FindAll(const std::vector<int>& data, bool(*op)(int))


лишний пробел

18thday · 2025-12-24T08:52:52Z

03_week/tasks/minmax/minmax.cpp


-/* return_type */ MinMax(/* args */) {
-    throw std::runtime_error{"Not implemented"};
+std::pair<std::vector<int>::iterator, std::vector<int>::iterator> MinMax(std::vector<int>& data) {


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

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

18thday · 2025-12-24T08:54:14Z

03_week/tasks/minmax/minmax.cpp

+    // проверка, что контейнер не пуст
+    if(data.size() == 0){    // контейнер пуст
+//        return {begin, begin};  // Используем uniform initialization
+        return std::pair<std::vector<int>::iterator, std::vector<int>::iterator>(begin, begin);


длинно

18thday · 2025-12-24T08:57:34Z

03_week/tasks/minmax/minmax.cpp

+}
+
+// перегрузка функции
+std::pair<std::vector<int>::const_iterator, std::vector<int>::const_iterator> MinMax(const std::vector<int>& data) {


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

18thday · 2025-12-24T09:00:17Z

03_week/tasks/os_overload/os_overload.cpp

+// Оператор вывода для CircleRegion
+std::ostream& operator<<(std::ostream& os, const CircleRegion& сircleRegion) {
+    // работа с контейнером "pair"
+    // первый элемент типа Circle, второй bool


в целом такие комментарии излишние, поскольку CircleRegion это пара соответствующих занчений

18thday · 2025-12-24T09:01:12Z

03_week/tasks/range/range.cpp

-    throw std::runtime_error{"Not implemented"};
+std::vector<int> Range(int from, int to, int step = 1) {
+
+    if(from == to)   return std::vector<int>();     // проверка равенства начала и конца интервала


{} - более короткий вариант пердпочтительнее

18thday

@9617771614, основные проблемы

Разыменование nullptr из-за некорректной проверки на nullptr (01_week)
Использование постфиксного инкремента в циклах
Излишние явные приведения типов, не используются неявные касты
Создание избыточных переменных, неэффективный код
Неаккуратный код: проблемы со стилем, лишние строки и пробелы, разный стиль
Дублирование функциональности

Stack

Временно отключаю контроль задачь недели 3

Временно отключаю контроль задачь недели 2 иначе в проверке куча строк.

Временно отключаю контроль задачь недели 1 иначе в проверке куча строк.

18thday · 2026-01-21T09:56:43Z

04_week/tasks/stack/stack.cpp

+
+    void Swap(Stack& other);  // меняем местами элементы
+
+    // определение для ==


лишний комментарий

18thday · 2026-01-21T09:58:36Z

04_week/tasks/stack/stack.cpp

+
+    // определение для !=
+    bool operator!=(const Stack& x) const {return !(data == x.data);}
+


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

18thday · 2026-01-21T10:00:36Z