Homework#34
Conversation
|
Готовы задания 1, 2, 4, 5, 7 |
|
|
||
| using namespace std; | ||
|
|
||
| pair<int, int> get_terms(vector<int> v, int S) { |
There was a problem hiding this comment.
GetTerms(const std::vector<int>& array, int sum) {...}
| using namespace std; | ||
|
|
||
| pair<int, int> get_terms(vector<int> v, int S) { | ||
| int N = v.size(); |
|
|
||
| int l{0}, r{N - 1}; | ||
|
|
||
| int s; |
| else | ||
| cout << result.first << " " << result.second << endl; | ||
| /* | ||
| python3 ./scripts/run_cases.py --tasks task_01 |
There was a problem hiding this comment.
хорошо, но я бы перенес в начало файла (что бы при открытии не крутить вниз, в поисках этой строки)
|
|
||
| using namespace std; | ||
|
|
||
| int get_border_index(vector<int> v) { |
There was a problem hiding this comment.
то же что и в первой задаче
| ASSERT_EQ(get_border_index(v13), 0); | ||
| } | ||
|
|
||
| void f(int i, vector<int> v) { |
| using namespace std; | ||
|
|
||
| vector<int> temperature_rise(vector<int> v1) { | ||
| vector<int> stack, v2(v1.size(), 0); |
There was a problem hiding this comment.
давай 1 строка - одно объявление
|
Добавлены решения и тесты к задачам 3, 10, 12. Добавлены тесты к задаче 9. Исправлены замечания. |
|
Задания 8, 11, 13 добавлены |
Fixed error in task11 basic.test (DafeCpp#37)
|
задание 14 тоже готово теперь |
| @@ -0,0 +1,20 @@ | |||
| #include <vector> | |||
|
|
|||
| int GetBorderIndex(std::vector<int> v) { | |||
| int GetBorderIndex(std::vector<int> v) { | ||
| int N = v.size(); | ||
|
|
||
| if (v[1] == 1) return 0; |
There was a problem hiding this comment.
кажется и без этих строк должно работать, если и без них работает, то зачем они?
| TEST(Test, Single) { | ||
| // Одна цифра | ||
| std::vector<std::string> empty{}; | ||
| std::vector<std::string> v2{"a", "b", "c"}; |
There was a problem hiding this comment.
это можно не исправлять, но если бы тест был больше, то название переменной хорошо бы было сделать понятнее, например expected2
| void MinStack::Push(int value) { data_.push_back(value); } | ||
| void MinStack::Push(int value) { | ||
| stack_.push_back(value); | ||
| if (min_stack_.size() == 0 || value < min_stack_.back()) |
There was a problem hiding this comment.
лучше заменить min_stack_.size() == 0 на min_stack_.empty()
| } | ||
| min_costs.push_back(std::min(fish_cost[i], min_costs.back())); | ||
| min_sum += min_costs.back(); | ||
| // std::cout << min_sum << " "; |
There was a problem hiding this comment.
в коде не нужно оставлять закоменчиный код, если только это не часть комментария, например пример использования
и код пониже тоже удали
| @@ -0,0 +1,46 @@ | |||
| #include <vector> | |||
|
|
|||
| using namespace std; | |||
There was a problem hiding this comment.
в других местах не использовали, давай и тут не будем)
|
|
||
| using namespace std; | ||
|
|
||
| int get_k_element(vector<int> v, int K, int left, int right) { |
| TEST(TopologySort, Simple) { | ||
| #include <get_min_cost.hpp> | ||
|
|
||
| TEST(Test, Simple) { |
There was a problem hiding this comment.
хорошо бы добавить тестов
|
|
||
| #include <string> | ||
|
|
||
| void SplayTree::RotateLeft(Node* n) { |
There was a problem hiding this comment.
слишком короткое имя аргумента n
| int main() { return 0; } | ||
| int main() { | ||
| int N; | ||
| std::vector<int> v; |
There was a problem hiding this comment.
cin, cout, endl без std:: — код не скомпилируется. Нужно добавить std:: везде по аналогии с другими задачами.
| @@ -1,3 +1,20 @@ | |||
| #include <iostream> | |||
| #include <merge_sort.hpp> | |||
| #include <vector> | |||
There was a problem hiding this comment.
то же что и в merge_sort.hpp — в других задачах using namespace std; не используется, лучше убрать и писать std:: явно
| @@ -1,3 +1,20 @@ | |||
| #include <iostream> | |||
| #include <merge_sort.hpp> | |||
There was a problem hiding this comment.
лучше инклуды разделять по группам (с начала стандартные библиотеки потом другие библиотеки и в конце файлы из проекта, примерно так должно выглядеть:
#include <iostream>
#include <vector>
#include <merge_sort.hpp>
| #include <string> | ||
|
|
||
| int main() { return 0; } | ||
| #include "substring_search.hpp" |
There was a problem hiding this comment.
переменная string совпадает с именем типа std::string — лучше переименовать, например в text и pattern
| } | ||
| }; | ||
|
|
||
| std::vector<Lesson> GetSelectedLessons(std::vector<Lesson>& lessons) { |
There was a problem hiding this comment.
функция принимает вектор по неконстантной ссылке и сортирует его внутри — это неожиданно меняет данные у вызывающего кода. Лучше передавать по значению: std::vector<Lesson> GetSelectedLessons(std::vector<Lesson> lessons)
There was a problem hiding this comment.
тут или передавать по ссылке и ничего не возвращать, или сделать константную ссылку и работать с копией. можно конечно еще передавать rvalue ссылку (&&), но не уверен что проходили в плюсах это
|
так же тесты упали |
2 participants
No description provided.