170 похожих чатов

Всем привет! Интересно мнение людей по програмке, которую написал

для универа.
Может что-то стоит переписать по-другому?
https://github.com/testuser-0/university/tree/master/2/KP/6_IPW1

19 ответов

18 просмотров

https://github.com/testuser-0/university/blob/master/2/KP/6_IPW1/book.hpp#L19 std::string не нужно по значению передавать https://github.com/testuser-0/university/blob/master/2/KP/6_IPW1/book.cpp#L13 this не нужно явно указывать

Ivanov-Ivan Автор вопроса
Ivanov Ivan
Передавать по укателю?

зачем? передавайте по const ссылке

Ivanov-Ivan Автор вопроса
Ivanov Ivan
т.е. просто const std::string title ?

нет, рекомендую почитать про ссылки

https://github.com/testuser-0/university/blob/4f1e319aecac5e489386f37a4e1fe387eb05eb85/2/KP/6_IPW1/main.cpp#L36 ты нашел книгу. может не надо дальше их просматривать?

Ivanov-Ivan Автор вопроса
Michael Chambers
нет, рекомендую почитать про ссылки

Окей, а ссылки я объявляю в хэдере? std::string separator; std::string title; std::string author; std::string genre; int year; double price; std::string &separatorRef = separator; std::string &titleRef = title; std::string &authorRef = author; std::string &genreRef = genre;

Ivanov-Ivan Автор вопроса
Ivanov Ivan
Не понял?

может ли быть более одной искомой книги?

Ivanov-Ivan Автор вопроса
Ivanov Ivan
Окей, а ссылки я объявляю в хэдере? std::strin...

и снова нет) но вы всё ближе std::string separator; std::string title; std::string author; std::string genre; int year; double price; вот эту часть трогать не нужно

Ivanov Ivan
В рамках задания - нет

то есть, если искомая книга нашлась, вы всё равно пройдётесь циклом по остальным книгам, сравнивая названия. Зачем?

Ivanov-Ivan Автор вопроса
Michael Chambers
и снова нет) но вы всё ближе std::string sepa...

т.е. в конструктор передаю значение const &title и больше ничего не делаю, верно?

Ivanov-Ivan Автор вопроса
Ivanov-Ivan Автор вопроса
Michael Chambers
в точку, только тип потеряли куда-то

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

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

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

Ivanov-Ivan Автор вопроса
Michael Chambers
простые типы не имеет особого смысла так передават...

Спасибо, сейчас поправлю. А почему this явно не стоит указывать?

Ivanov Ivan
Спасибо, сейчас поправлю. А почему this явно не ст...

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

Похожие вопросы

Обсуждают сегодня

30500 за редактор? )
Владимир
47
а через ESC-код ?
Alexey Kulakov
29
Чёт не понял, я ж правильной функцией воспользовался чтобы вывести отладочную информацию? но что-то она не ловится
notme
18
У меня есть функция где происходит это: write_bit(buffer, 1); write_bit(buffer, 0); write_bit(buffer, 1); write_bit(buffer, 1); write_bit(buffer, 1); w...
~
13
Недавно Google Project Zero нашёл багу в SQLite с помощью LLM, о чём достаточно было шумно в определённых интернетах, которые сопровождались рассказами, что скоро всех "ибешни...
Alex Sherbakov
5
Ребят в СИ можно реализовать ООП?
Николай
33
Как передать управляющий символ в открытую через CreateProcess консоль? Собсна, есть процедура: procedure TRedirectThread.WriteData(Data: OEMString); var Written: Cardinal;...
Serjone
6
в JclConsole объявлено так: function CtrlHandler(CtrlType: DWORD): BOOL; stdcall; - где ваше объявление с stdcall? у вас на картинке нет stdcall
Karagy
8
https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_h_common.erl#L174 https://github.com/erlang/otp/blob/OTP-27.1/lib/kernel/src/logger_olp.erl#L76 15 лет назад...
Maksim Lapshin
20
Вот еще странный косяк, подскажите как бороться. Я git clone сделал себе всего embassy и примеры там запускаю. Всё хорошо. Но вот решил в cargo.toml зависимости не как в приме...
Lukutin R2AJP
2
Карта сайта