2 ternary numbers#27
2 ternary numbers#27ZakKurasov wants to merge 23 commits intodriverdevteam:masterfrom ZakKurasov:2-ternary-numbers
Conversation
| return std::stoi(ternary); | ||
| } | ||
|
|
||
| TEST(PowerTest, ItReturnSameNumberForIndexOne) |
There was a problem hiding this comment.
по канонам TDD у теья не должно висеть красных тестов на протяжении долгого времени. Если ты написал тест, который требует доп реализации и прохождения других тестов сначала - тест следует убрать или задизэйблить. У Google Test есть такая фича - https://stackoverflow.com/a/7208119/5607187
| @@ -19,7 +19,12 @@ If your language provides a method in the standard library to perform the conver | |||
|
|
|||
| int Power(int number, int index) | |||
There was a problem hiding this comment.
std::pow уже сделал это за нас :)
There was a problem hiding this comment.
If your language provides a method in the standard library
Слишком буквально это воспринял)))
| { | ||
| result.push_back(ch - '0'); | ||
| } | ||
| return result; |
There was a problem hiding this comment.
тут также можно было бы использовать std::transform для наполения вектора:
http://www.cplusplus.com/reference/algorithm/transform/
std::string in = "123";
std::vector<int> out(3);
std::transform(in.cbegin(), in.cend(), out.begin(), [] (char ch) { return ch - '0'; });
EXPECT_EQ(std::vector<int>({1,2,3}), out);
There was a problem hiding this comment.
Такая же ситуация как и выше)
2 participants
No description provided.