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

Привет всем! Ищу Peer-программиста, чтобы помочь с изучением раста. Любые

комменты по текущему «уровню» - приветствуются:


https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=7f1ad4d7a64bce5c69bd1f535efee30c

5 ответов

14 просмотров

1. Абсолютно бесполезные комментарии. 2. Лучше не надо выводить ничего в summary, пусть она просто отдает стрингу со своим описанием. 3. Зачем нужна book_factory? Может просто метод добавить в Book? 4. Иф на три условия можно заменить на матч. 5. @rust_begginers_ru

Вроде норм, чистенько

Вместо аргумента Vec<что-то> лучше принимать &[что-то]

В коде convert_to_lines вместо пуша в строку можно сделать join элементов с разделителем (“-“). Тогда не нужно будет убирать последний дефис. Либо собрать в строку через макрос format!

Я бы использовал enum для состояния прочитано-непрочитано. Во-первых, его можно расширять. :) Во-вторых, логические значения, когда их в списке аргументов становится больше одного, путь к ошибкам.

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

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

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
Карта сайта