Кайгородов Александр #28

18thday · 2026-01-12T05:50:14Z

No description provided.

github.com:psds-cpp/psds-cpp-2025

18thday · 2026-01-12T05:52:26Z

01_week/tasks/addition/addition.cpp

-    throw std::runtime_error{"Not implemented"};
+    int64_t c = static_cast<int64_t>(a);
+    int64_t d = static_cast<int64_t>(b);
+    return c + d;


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

18thday · 2026-01-12T05:54:16Z

01_week/tasks/check_flags/check_flags.cpp

+            }
+            std::cout << "DEST";
+            prev_print = 1;
+        }


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

18thday · 2026-01-12T05:54:49Z

01_week/tasks/check_flags/check_flags.cpp

+    if (flags <= CheckFlags::ALL && flags >= CheckFlags::NONE){
+        std::cout << "[";
+        bool prev_print = 0;
+        if (static_cast<uint8_t>(flags) & static_cast<uint8_t>(CheckFlags::TIME)){


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

18thday · 2026-01-12T07:42:34Z

03_week/tasks/easy_compare/easy_compare.cpp

+}
+
+bool operator>(Date date1, Date date2) {
+    return !(date1 < date2) and !(date1 == date2);


принято использовать операторы && и ||

18thday · 2026-01-12T07:43:49Z

03_week/tasks/easy_compare/easy_compare.cpp

+    return std::tie(st1.mark, st1.score) == std::tie(st2.mark, st2.score);
+}
+bool operator!=(StudentInfo st1, StudentInfo st2) {
+    return !(std::tie(st1.mark, st1.score) == std::tie(st2.mark, st2.score));


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

18thday · 2026-01-12T07:44:24Z

03_week/tasks/easy_compare/easy_compare.cpp

+    if (st1.birth_date != st2.birth_date) {
+        return st1.birth_date < st2.birth_date;  
+    }
+    else


else тут лишний

18thday · 2026-01-12T07:44:46Z

03_week/tasks/easy_compare/easy_compare.cpp

+    }
+    else
+        return false;
+


лишняя строка

18thday · 2026-01-12T07:47:56Z

03_week/tasks/easy_compare/easy_compare.cpp

+    return !(std::tie(st1.mark, st1.score) == std::tie(st2.mark, st2.score));
+}
+
+bool operator<(StudentInfo st1, StudentInfo st2) {


Так не плохо, но в данном случае также можно было бы воспользоваться std::tie в левом операнде аргументом могут быть как st1 так и st2 что позволит выразить как < так и > используя в конечном счете выражение std::tie(st1..., st2....) < std::tie(st1..., st2....)

18thday · 2026-01-12T07:59:33Z

03_week/tasks/filter/filter.cpp

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


зачем возвращать копию? в данном случае функция меняет входной аргумент vec, следовательно возвращать должна void, иначе излишне копируем

18thday · 2026-01-12T08:00:49Z

03_week/tasks/find_all/find_all.cpp

+    }
+    vec_out.shrink_to_fit();
+    return vec_out;
+}


Достаточно одной функции которая принимает константный вектор и возвращает неконстантный вектор позиций,

18thday · 2026-01-12T08:02:03Z

03_week/tasks/find_all/find_all.cpp

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


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

18thday · 2026-01-12T08:03:15Z

03_week/tasks/find_all/find_all.cpp

+    return vec_out;
+}
+
+const std::vector<size_t> FindAll(const std::vector<int>& vec, bool (*func) (int)) {


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

18thday · 2026-01-12T08:06:09Z

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>& vec) {


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

18thday · 2026-01-12T08:07:03Z

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>& vec) {
+    std::pair<std::vector<int>::iterator, std::vector<int>::iterator> rez;


принято писать result на английском, либо res

18thday · 2026-01-12T08:08:41Z

03_week/tasks/minmax/minmax.cpp

+    std::pair<std::vector<int>::iterator, std::vector<int>::iterator> rez;
+    if (vec.begin() == vec.end()){
+        rez.first = vec.begin();
+        rez.second = vec.begin();


можно проинициализировать пару сразу

18thday · 2026-01-12T08:09:04Z

03_week/tasks/minmax/minmax.cpp

 }
+
+std::pair <std::vector<int>::const_iterator, std::vector<int>::const_iterator> MinMax(const std::vector<int>& vec) {
+    std::pair<std::vector<int>::const_iterator, std::vector<int>::const_iterator> rez;


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

либо создавать пару непосредственно при возврате из функции, чтобы не создавать лишние переменные, сейчас под одно назначение выделена двойная память под res и под min_it и max_it

18thday · 2026-01-12T08:13:02Z