комменты по текущему «уровню» - приветствуются:
https://play.rust-lang.org/?version=stable&mode=debug&edition=2018&gist=7f1ad4d7a64bce5c69bd1f535efee30c
1. Абсолютно бесполезные комментарии. 2. Лучше не надо выводить ничего в summary, пусть она просто отдает стрингу со своим описанием. 3. Зачем нужна book_factory? Может просто метод добавить в Book? 4. Иф на три условия можно заменить на матч. 5. @rust_begginers_ru
Вроде норм, чистенько
Вместо аргумента Vec<что-то> лучше принимать &[что-то]
В коде convert_to_lines вместо пуша в строку можно сделать join элементов с разделителем (“-“). Тогда не нужно будет убирать последний дефис. Либо собрать в строку через макрос format!
Я бы использовал enum для состояния прочитано-непрочитано. Во-первых, его можно расширять. :) Во-вторых, логические значения, когда их в списке аргументов становится больше одного, путь к ошибкам.
Обсуждают сегодня