Войкина Варвара #13

VaryVA · 2025-12-07T13:15:50Z

No description provided.

…adratic, rms tasks

18thday · 2026-01-20T05:55:15Z

01_week/tasks/addition/addition.cpp

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


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

18thday · 2026-01-20T05:57:28Z

01_week/tasks/check_flags/check_flags.cpp

+        {CheckFlags::CERT, "CERT"},
+        {CheckFlags::KEYS, "KEYS"},
+        {CheckFlags::DEST, "DEST"}
+    };


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

18thday · 2026-01-20T05:58:51Z

01_week/tasks/length_lit/length_lit.cpp

+}
+
+constexpr double operator"" _ft_to_in(long double feet) {
+    return static_cast<double>(feet) * 12.0;


12.0 это тоже magic value, которое не принято оставлять в коде

18thday · 2026-01-20T05:59:29Z

01_week/tasks/length_lit/length_lit.cpp

+constexpr double METER_TO_FOOT = 1.0 / FOOT_TO_METER;
+constexpr double METER_TO_INCH = 1.0 / INCH_TO_METER;
+constexpr double CM_TO_METER = 0.01;
+constexpr double METER_TO_CM = 100.0;


лучше поместить в namespace, можно безымянный

18thday · 2026-01-20T06:00:45Z

01_week/tasks/length_lit/length_lit.cpp

+    return static_cast<double>(feet) * FOOT_TO_METER;
+}
+
+constexpr double operator"" _ft_to_m(unsigned long long feet) {


в данном случае это излишнее дублирование, так как неявно сконвертировался бы long double и достаточно одной функции в нашем случае

18thday · 2026-01-20T06:01:55Z

01_week/tasks/print_bits/print_bits.cpp

+    std::cout << "0b";
+
+    int total_bits = static_cast<int>(bytes) * 8;
+    for (int i = total_bits - 1; i >= 0; i--) {


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

18thday · 2026-01-20T06:03:27Z

01_week/tasks/quadratic/quadratic.cpp

+                std::cout << "infinite solutions";
+            } 
+            else {
+                std::cout << "no solutions";


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

18thday · 2026-01-20T06:36:00Z

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-20T06:36:39Z

02_week/tasks/func_array/func_array.cpp

@@ -1,6 +1,12 @@
 #include <stdexcept>

+typedef double (*MathOperation)(double, double);


в С++ принято использовать using

18thday · 2026-01-20T06:38:32Z

02_week/tasks/last_of_us/last_of_us.cpp

-    throw std::runtime_error{"Not implemented"};
-}
+const int* FindLastElement(const int* begin, const int* end, Predicate predicate) {
+    if (begin == nullptr || end == nullptr || begin > end || predicate == nullptr) {


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

18thday · 2026-01-20T06:39:22Z

02_week/tasks/little_big/little_big.cpp

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


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

18thday · 2026-01-20T06:40:43Z

02_week/tasks/pretty_array/pretty_array.cpp

+        count = static_cast<int>(end - begin);
+    } else {
+        count = static_cast<int>(begin - end);
+    }


тернарный оператор при инициализации count сократил бы 6 строк до 1

18thday · 2026-01-20T06:41:31Z

02_week/tasks/pretty_array/pretty_array.cpp

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


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

18thday · 2026-01-20T07:59:03Z

02_week/tasks/pretty_array/pretty_array.cpp

+            current = begin + i;
+        } else {
+            current = begin - i;
+        }


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

18thday · 2026-01-20T07:59:46Z

02_week/tasks/pretty_array/pretty_array.cpp

+        }
+
+        std::cout << *current;
+        elements_in_current_line++;


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

18thday · 2026-01-20T08:00:29Z

02_week/tasks/longest/longest.cpp

+}
+
+char* FindLongestSubsequence(char* begin, char* end, size_t& count) {
+    return FindLongestSubsequenceImpl(begin, end, count);


зачем тогда писать Impl ? если можно сразу ссделать шаблон

18thday · 2026-01-20T08:02:39Z

04_week/tasks/queue/queue.cpp

+            tmp.push_back(st.top());
+            st.pop();
+        }
+        for (auto it = tmp.rbegin(); it != tmp.rend(); ++it) {


зачем два цикла? почему сразу не писать в out_?

18thday · 2026-01-20T08:04:04Z

04_week/tasks/queue/queue.cpp

+        if (a.Size() != b.Size()) {
+            return false;
+        }
+        return a.ToVector() == b.ToVector();


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

18thday · 2026-01-20T08:10:55Z

04_week/tasks/queue/queue.cpp

+            return;
+        }
+
+        Queue* self = const_cast<Queue*>(this);


это нарушение семантики, константный метод далее модифицирует объект, к тому же это UB const_cast при вызове от константного объекта. Пересмотреть дизайн метода Front()

18thday · 2026-01-20T08:45:10Z

04_week/tasks/ring_buffer/ring_buffer.cpp

+        size_ = buffer_.size();
+        if (size_ == 0) { 
+            buffer_.resize(1);
+            size_ = 0;


при выполнении условия известно, что размер 0

18thday · 2026-01-20T08:45:34Z

04_week/tasks/ring_buffer/ring_buffer.cpp

+
+    bool TryPush(int value) {
+        if (Full()) return false;
+        Push(value);


дважды проверяется Full

18thday · 2026-01-20T08:47:16Z

04_week/tasks/ring_buffer/ring_buffer.cpp

+    bool Empty() const { return size_ == 0; }
+    bool Full() const { return size_ == buffer_.size(); }
+    size_t Size() const { return size_; }
+    size_t Capacity() const { return buffer_.size(); }


может лучше capacity(), чтобы не удивляло

18thday

@VaryVA основные моменты:

нет задач 3 недели
C-style cast
неоптимальное сравнение Queue
UB const_cast в Queue, нарушение семантики const метод модифицирует объект

VaryVA and others added 26 commits November 28, 2025 13:12

add (solution): add addition, check_flags, lenght_lit, print_bits, qu…

318cf97

…adratic, rms tasks

add (solution): add check_flags task

190ea20

add (solution): add length_lit task

58fbe23

add (solution): add print_bits task

3573ba1

add (solution): add quadratic task

15ec4a4

add (solution): add rms task

3fcc0bc

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

841c809

Merge branch 'psds-cpp:main' into main

7e86b75

add (solution): add func_array task

f174bae

Надо, я чего-то не того наделала (

2328946

add (solution): add last_of_us task�[C

fc312f9

add (solution): add little_big task

837fe51

add (solution): add longest task

6ea5c9f

add (solution): add pretty_array task

98e5d59

add (solution): add swap_ptr task

3a9a419

Merge branch 'psds-cpp:main' into main

ed70f99

Merge remote-tracking branch 'upstream/main'

75cb8e6

Merge branch 'psds-cpp:main' into main

fc3abfa

add (solution): add stack task

3d5c858

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

8a5e0bd

add (solution): add phasor task

c52f90c

add (solution): add queue task

7dd4d11

add (solution): add ring_buffer task

a6fea19

add (solution): add stack task

0693b53

Trigger CI

39a594d

dont push build folder please

4d40cd2

18thday changed the title 1 Войкина Варвара Jan 20, 2026

18thday reviewed Jan 20, 2026

View reviewed changes


		double mean_of_squares = sum_of_squares / size;

		return std::sqrt(mean_of_squares);

		@@ -1,6 +1,12 @@
		#include <stdexcept>

		typedef double (*MathOperation)(double, double);

Войкина Варвара #13

Are you sure you want to change the base?

Войкина Варвара #13

Uh oh!

Conversation

VaryVA commented Dec 7, 2025

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!

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 left a comment •

edited

Loading