jediserg
  • Joined on 2021-06-19
jediserg commented on pull request alexander/geekbrains_oop_cpp#1 2021-07-02 08:18:24 +00:00
lesson 1

Да с привидение не очень, если честно я когда писал, не подумал что поток для uint8_t тоже переопределен и как для…

jediserg commented on pull request alexander/geekbrains_oop_cpp#4 2021-07-01 16:20:15 +00:00
lesson 4

ну оно же все из методички))

jediserg commented on pull request alexander/geekbrains_oop_cpp#4 2021-07-01 16:15:15 +00:00
lesson 4

самое эфективное memcpy ))

jediserg commented on pull request alexander/geekbrains_oop_cpp#3 2021-07-01 15:24:32 +00:00
lesson_3

вообще насколько я помню если явно не прописанно значение, то гарантии нет что значения будут нужные. Если спользуются значения enum, лучше все забить. Но я бы не использовал чтобы сущности не путались. А сделать метод например

jediserg commented on pull request alexander/geekbrains_oop_cpp#3 2021-07-01 15:19:36 +00:00
lesson_3

а туз может два значения иметь)

jediserg commented on pull request alexander/geekbrains_oop_cpp#3 2021-07-01 15:17:40 +00:00
lesson_3

Не вижу смысл инициализации по умолчанию первых двух параметров, это не часто используемое и не основное значение

jediserg commented on pull request alexander/geekbrains_oop_cpp#3 2021-07-01 15:15:37 +00:00
lesson_3

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

jediserg commented on pull request alexander/geekbrains_oop_cpp#3 2021-07-01 15:11:40 +00:00
lesson_3

const std::string& или m_company(std::move(company))

jediserg commented on pull request alexander/geekbrains_oop_cpp#3 2021-07-01 15:05:09 +00:00
lesson_3

площадь ромба произведение сторон, или это уже здесь диогонали)

jediserg commented on pull request alexander/geekbrains_oop_cpp#3 2021-07-01 15:04:22 +00:00
lesson_3

можно сторона а в квадрате

jediserg commented on pull request alexander/geekbrains_oop_cpp#2 2021-06-28 15:34:46 +00:00
lesson_2

я бы конструктор сделал и в дочерних его вызывал из списка инициализации

jediserg commented on pull request alexander/geekbrains_oop_cpp#2 2021-06-28 15:34:07 +00:00
lesson_2

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

jediserg commented on pull request alexander/geekbrains_oop_cpp#2 2021-06-28 15:33:36 +00:00
lesson_2

ну тогда в значения параметра по умолчанию сразу green

jediserg commented on pull request alexander/geekbrains_oop_cpp#2 2021-06-28 15:32:48 +00:00
lesson_2

Отлично! Но в идеале еще конструктор перемещения))

jediserg commented on pull request alexander/geekbrains_oop_cpp#2 2021-06-28 15:31:40 +00:00
lesson_2

Я бы объявил enum в классе и кстати можно enum class использовать

jediserg commented on pull request alexander/geekbrains_oop_cpp#1 2021-06-19 11:00:30 +00:00
lesson 1

тут можно без каста