Титаев Андрей #19

18thday · 2025-12-24T09:14:57Z

No description provided.

Temporary Update

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

01_week/tasks/addition/addition.cpp

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


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

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

01_week/tasks/char_changer/char_changer.cpp

 #include <cstddef>
 #include <stdexcept>

+size_t CharChanger(char* array, size_t size, char delimiter) {


Не нужно изменять сигнатуру функции если не просят. В данном случае это некорректное изменение, так как теряется часть функциональности

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

01_week/tasks/char_changer/char_changer.cpp

+        // Приведения текущего символа к определенным типам: пробел, число или ASCII символ
+        bool isSpace = std::isspace(static_cast<unsigned char>(currentChar)) != 0;
+        bool isDigit = std::isdigit(static_cast<unsigned char>(currentChar)) != 0;
+        bool isAlpha = std::isalpha(static_cast<unsigned char>(currentChar)) != 0;


Зачем создавать переменные если переменная:

Используется единожды

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

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

01_week/tasks/char_changer/char_changer.cpp

+        // Подсчёт длины последовательности одинаковых символов
+        size_t nextPos = readPos + 1;
+        while (nextPos < size - 1 && array[nextPos] == currentChar) {
+            nextPos++;


Постфиксный инкремент создает ненужную копию

18thday · 2025-12-24T09:30:28Z

01_week/tasks/char_changer/char_changer.cpp

+
+        // Обработка пробелов
+        if (isSpace) {
+            array[writePos++] = delimiter;


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

18thday · 2025-12-24T09:31:56Z

01_week/tasks/check_flags/check_flags.cpp

-    throw std::runtime_error{"Not implemented"};
-}
+    uint8_t value = static_cast<uint8_t>(flags);
+    uint8_t all_bits = static_cast<uint8_t>(CheckFlags::ALL);


единственное место использования

18thday · 2025-12-24T09:36:13Z

01_week/tasks/check_flags/check_flags.cpp

+        return;
+    }
+
+    const std::pair<CheckFlags, const char*> flag_names[] = {


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

18thday · 2025-12-24T09:38:16Z

01_week/tasks/length_lit/length_lit.cpp

+
+
+constexpr long double  CM_TO_M = 0.01L;
+constexpr long double M_TO_CM = 1.0L / CM_TO_M;


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

18thday · 2025-12-24T09:38:26Z

01_week/tasks/length_lit/length_lit.cpp

+constexpr long double IN_TO_CM = IN_TO_M * 100.0L;
+constexpr long double CM_TO_IN = 1.0L / IN_TO_CM;
+
+


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

18thday · 2025-12-24T09:39:05Z

01_week/tasks/print_bits/print_bits.cpp


-void PrintBits(long long value, size_t bytes) {
-    throw std::runtime_error{"Not implemented"};
+ void PrintBits(long long value, size_t bytes) {


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

18thday · 2025-12-24T09:42:10Z

01_week/tasks/print_bits/print_bits.cpp

+
+    // Сдвиг, чтобы напечатать младшие байты
+    ULL mask = (bytes == BYTES_IN_LONG_LONG) 
+      ? ~0ULL 


Как-то неожиданно отступ стал 2 пробела вместо принятых 4. Обычно пишут в одном стиле, когда сами пишут

18thday · 2025-12-24T09:43:05Z

01_week/tasks/quadratic/quadratic.cpp

+    }
+
+    if(a != 0) { // two roots
+		int discriminant = b * b - 4 * a * c;


то 2, то 4 пробела

18thday · 2025-12-24T09:43:39Z

01_week/tasks/quadratic/quadratic.cpp

+
+		cout << setprecision(6);
+		if(discriminant > 0) {
+		  double discriminantRoot = sqrt(discriminant);


лучше использовать std::sqrt

18thday · 2025-12-24T09:44:40Z

01_week/tasks/quadratic/quadratic.cpp

+		cout << setprecision(6);
+		if(discriminant > 0) {
+		  double discriminantRoot = sqrt(discriminant);
+		  double x1 = (-b - discriminantRoot) / (2.0*a);


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

18thday · 2025-12-24T09:46:42Z

01_week/tasks/rms/rms.cpp

+    long double sumOfSquares = 0;
+
+    for(size_t i = 0; i < size; ++i) {
+      double &value = values[i];


const double&

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

02_week/tasks/func_array/func_array.cpp

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


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

18thday · 2025-12-24T09:48:35Z

02_week/tasks/func_array/func_array.cpp

+    double sum = 0.0;
+    for (size_t i = 0; i < size; ++i) {
+        if (ops[i] == nullptr) {
+			continue; // �� ��������� ��������, ���� ��������� ������� ����� nullptr


Вероятно нужно поправить кодировку на UTF-8

18thday · 2025-12-24T09:52:38Z

02_week/tasks/little_big/little_big.cpp

-}
+void PrintMemory(double value, bool reverse = false) {
+    using std::cout, std::hex, std::uppercase, std::setw, std::setfill;
+    const uint8_t* bytes = reinterpret_cast<const uint8_t*>(&value);


Большое дублирование кода после того как мы получили указатель и зная sizeof(value) далее можно было вызывать одну общую функцию, и переиспользовать код

18thday · 2025-12-24T09:54:27Z

02_week/tasks/longest/longest.cpp

+        previousCharacter = currentCharacter;
+    }
+
+    // �������� ���������� ��������


Проблемы с кодировкой

18thday · 2025-12-24T09:55:15Z

02_week/tasks/longest/longest.cpp

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


Хорошее применение

18thday · 2025-12-24T09:56:25Z

02_week/tasks/pretty_array/pretty_array.cpp

+    }
+
+
+    // ������ � �������������


Кодировка

18thday · 2025-12-24T09:56:29Z

02_week/tasks/pretty_array/pretty_array.cpp

+            ptr += reversed ? -1 : 1;
+        }
+
+        // �� ��� ������� ������ ���������� �� ����� ������, ���������


Кодировка

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

02_week/tasks/pretty_array/pretty_array.cpp

+    // ������ ��� ����������� �� ���������� ���������
+	cout << "[";
+    if (limit == 0 || limit >= count) {
+        if (reversed == false) {


!reversed - лучше подходит

18thday · 2025-12-24T10:01:14Z

02_week/tasks/pretty_array/pretty_array.cpp

+                if (ptr != first) cout << ", ";
+                cout << *ptr;
+            }
+        }


тут практически полное дублирование, можно вынести в функцию, либо производить проверку reversed ? --ptr : ++ptr

18thday · 2025-12-24T10:03:42Z

03_week/tasks/data_stats/data_stats.cpp

+
+    stats.avg = sum / data.size();
+    double variance = (sum_of_squares / data.size()) - (stats.avg * stats.avg);
+	variance = variance >= 0.0 ? variance : 0.0;


лишнее присваивание variance = variance

18thday · 2025-12-24T10:08:40Z

03_week/tasks/easy_compare/easy_compare.cpp

+        return lhs.course > rhs.course;
+    }
+
+    return lhs.birth_date < rhs.birth_date;


Всё можно переписать с помощью std::tie , просто в операндах можно связывать как lhs так и rhs, чтобы выразить меньше и больше, а сам оператор в выражении return останется <

18thday · 2025-12-24T10:26:04Z

03_week/tasks/enum_operators/enum_operators.cpp

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


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

18thday · 2025-12-24T10:30:06Z

03_week/tasks/minmax/minmax.cpp

+#include <vector>
+#include <utility>

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


в данном месте нанапрашивается псевдоним для std::vector<int>::const_iterator

18thday · 2025-12-24T10:31:34Z

03_week/tasks/minmax/minmax.cpp

+        ++it;
+    }
+
+    while ((it + 1) < vec.end()) {


лишние скобки для it + 1

18thday · 2025-12-24T10:34:25Z

03_week/tasks/os_overload/os_overload.cpp

+std::ostream& operator<<(std::ostream& os, const CircleRegionList& list) {
+    if (list.empty()) {
+        os << "{}";
+    } else {


лучше внутри if добавить return и не делать вложенность внутрь else. По возможности стараются избегать else

18thday · 2025-12-24T10:36:37Z

03_week/tasks/range/range.cpp

+        count = (to - from + step - 1) / step;
+    } else {
+        count = (to - from + step + 1) / step;
+    }


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

18thday · 2025-12-24T10:38:31Z

03_week/tasks/range/range.cpp

+    std::vector<int> result;
+    result.reserve(count);
+
+    bool isForward = step > 0;


тем более тут все равно используется отдельная переменная для этого.
Но можно и без неё если используется в одном месте

18thday · 2025-12-24T10:39:56Z

03_week/tasks/unique/unique.cpp

-}
+std::vector<int> Unique(const std::vector<int>& vec) {
+    if (vec.empty()) {
+        return std::vector<int>();


{} - более выразительно с учетом известного возвращаемого типа

18thday · 2025-12-24T10:45:25Z

03_week/tasks/unique/unique.cpp

+        if (vec[i] != vec[i - 1]) {
+            ++unique_count;
+        }
+    }


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

18thday

@Andrey23525, код понравился, аккуратно, выразитльно, задачи решены достаточно оптимально, указал некоторые моменты

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

04_week/tasks/ring_buffer/ring_buffer.hpp

+    size_t capacity_ = 0;
+    size_t size_ = 0;
+    size_t head_ = 0;  // индекс для добавления (Front)
+    size_t tail_ = 0;  // индекс для удаления (Back)


можно было обойтись меньшим количество позиций, например capacity можно брать у вектора,если его корректно поддерживать, не обязательно но можно было отказаться от head_ или tail_ зная size и capacity

18thday · 2026-01-21T10:37:53Z

04_week/tasks/ring_buffer/ring_buffer.cpp

+    , size_(0)
+    , head_(0)
+    , tail_(0) {
+    data_.resize(capacity_);


лучше сразу вызвать конструктор нужного размера в списке инициализации

18thday · 2026-01-21T10:38:22Z

04_week/tasks/ring_buffer/ring_buffer.cpp

+    , size_(capacity_)
+    , head_(0)
+    , tail_(0) {
+    data_.resize(capacity_, value);


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

18thday · 2026-01-21T10:39:18Z

04_week/tasks/ring_buffer/ring_buffer.cpp

+    size_t idx = 0;
+    for (int value : init) {
+        data_[idx++] = value;
+    }


лишний код, можно сконструировать вектор из init

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

04_week/tasks/queue/queue.cpp


-};
+Queue::Queue(const std::vector<int>& vec) {
+    input_stack_ = vec;


можно было перенести в список инициализации

18thday · 2026-01-21T10:44:41Z

04_week/tasks/queue/queue.cpp


-class Queue {
+Queue::Queue(size_t capacity) {
+    input_stack_.reserve(capacity);


почему бы не зарезервировать память под output_stack_

18thday · 2026-01-21T10:45:28Z

04_week/tasks/queue/queue.cpp

+        temp.push_back(s.top());
+        s.pop();
+    }
+    std::reverse(temp.begin(), temp.end());


почему сразу не складывать в otput_stack_ в подходящем порядке?

18thday · 2026-01-21T10:47:15Z

04_week/tasks/queue/queue.cpp

+}
+
+int& Queue::Back() {
+    if (input_stack_.empty() == false) {


empty() не принято явно сравнивать с булевой переменной, для этого есть !empty()

18thday · 2026-01-21T10:48:28Z

04_week/tasks/queue/queue.cpp

+}
+
+int& Queue::Front() {
+    TransferIfNeeded();


а почему нельзя поступить аналогично методу Back?

18thday · 2026-01-21T10:49:18Z

04_week/tasks/queue/queue.cpp

+    }
+
+    Queue a = *this;
+    Queue b = other;


мы знаем внутреннее устройство контейнера, создавать копии дорого

18thday · 2026-01-21T10:50:11Z

04_week/tasks/phasor/phasor.cpp

+    double new_real = real_ * other.real_ - imag_ * other.imag_;
+    double new_imag = real_ * other.imag_ + imag_ * other.real_;
+    real_ = new_real;
+    imag_ = new_imag;


зачем доролнттельные переменные?

18thday

@Andrey23525 отметил некоторые моменты:

хороший код
лишнее копирование при сравнении Queue, можно избежать
лишнии поля в RingBuffer, можно на 2 меньше

Andrey23525 and others added 8 commits November 27, 2025 22:28

add tasks

9f9fa2f

add char_changer and check_flags tasks

156abcf

Merge branch 'psds-cpp:main' into main

2c78ec6

add swap_ptr and last_of_us tasks

ad02168

add other tasks

2c633b8

Merge branch 'psds-cpp:main' into main

dd1a498

Update CMakeLists.txt

96c980f

Temporary Update

add 03_week tasks

12d3bc2

18thday commented Dec 24, 2025

View reviewed changes

02_week/tasks/pretty_array/pretty_array.cpp

}

// ��

Copy link

Contributor Author

18thday Dec 24, 2025

Choose a reason for hiding this comment

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

Кодировка

18thday commented Dec 24, 2025

View reviewed changes

02_week/tasks/pretty_array/pretty_array.cpp

ptr += reversed ? -1 : 1;

}

// �� , ��

Copy link

Contributor Author

18thday Dec 24, 2025

Choose a reason for hiding this comment

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

Кодировка

18thday commented Dec 24, 2025

View reviewed changes

03_week/tasks/minmax/minmax.cpp

++it;

}

while ((it + 1) < vec.end()) {

Copy link

Contributor Author

18thday Dec 24, 2025

Choose a reason for hiding this comment

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

лишние скобки для it + 1

18thday commented Dec 24, 2025

View reviewed changes

Andrey23525 and others added 3 commits December 26, 2025 23:49

add 04_week tasks

2e93014

delete a.out file

3d65f3a

forgotten updates

cea1c41

18thday commented Jan 21, 2026

View reviewed changes



		constexpr long double CM_TO_M = 0.01L;
		constexpr long double M_TO_CM = 1.0L / CM_TO_M;

		constexpr long double IN_TO_CM = IN_TO_M * 100.0L;
		constexpr long double CM_TO_IN = 1.0L / IN_TO_CM;

Титаев Андрей #19

Are you sure you want to change the base?

Титаев Андрей #19

Uh oh!

Conversation

18thday commented Dec 24, 2025

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

18thday Dec 24, 2025 • 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!

18thday Dec 24, 2025 • 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!

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!

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.

18thday Dec 24, 2025 •

edited

Loading

18thday Dec 24, 2025 •

edited

Loading

18thday left a comment •

edited

Loading

18thday Jan 21, 2026 •

edited

Loading