Комментарии 23
1. Я надеюсь, для бенчмарков проект компилируется с оптимизациями:
Ещё возможно вам будет интересно: у Rust есть встроенный движок бенчмарков: doc.rust-lang.org/book/benchmark-tests.html
2.
Во-первых, вместо
Во-вторых, не прививайте себе привычку везде тыкать
3.
cargo build --release?Ещё возможно вам будет интересно: у Rust есть встроенный движок бенчмарков: doc.rust-lang.org/book/benchmark-tests.html
2.
pub fn new(file_path: &str) -> ModelВо-первых, вместо
&str правильнее передавать AsRef<Path>: см. std::fs::File::open. Некоторые пути непредставимы в валидном utf-8, коими являются все строки в Rust.Во-вторых, не прививайте себе привычку везде тыкать
.unwrap(). Почитайте прекрасную статью об обработке ошибок в Rust: blog.burntsushi.net/rust-error-handling. Да, большая, но она раскладывает всё по полочкам.3.
или использовать стандартный типаж std::num::ZeroЕщё есть стабильный std::default::Default. Но
Zero, конечно, по смыслу подходит больше.
Я думаю, что для кода приложения, а не библиотеки, использовать &str для имени файла — вполне нормально. Если понадобится что-то другое, всегда можно будет исправить; а использование AsRef никак не поможет, если опять же в самом начале передавать строку.
То же самое, в общем-то, справедливо и для обработки ошибок, с учётом «игрушечности» рендерера и целей, с которыми он делается.
То же самое, в общем-то, справедливо и для обработки ошибок, с учётом «игрушечности» рендерера и целей, с которыми он делается.
То же самое, в общем-то, справедливо и для обработки ошибокЕсли знаете язык — да. Но пока автор его только изучает, как по мне лучше сразу приучаться правильно. «Игрушечность» рендера наоборот только тому способствует: кода мало, соответственно, и переделывать тоже немного. Представьте, как в большом проекте искать и исправлять .unwrap(), написанные по старой привычке, исправлять возвращаемые значения функций и т.д.
На самом деле, жаль, что Rust так легко позволяет обойти свою модель обработки ошибок. Это ведь даже не unsafe код.
Да. Для бенчмарков компилировалось release в Rust и -O3 в gcc.
Некоторые пути непредставимы в валидном utf-8, коими являются все строки в Rust.
А можно подробнее об этом? Ни разу не слышал о не текстовых путях. Какая-то специфическая область?
Например, вот про Linux: github.com/rust-lang/rust/issues/7225
А вот про Windows: github.com/rust-lang/rust/issues/12056
А вот про Windows: github.com/rust-lang/rust/issues/12056
Разве что только поиск информации, как сделать ту или иную штуку в расте осложняется тем, что язык быстро меняется. Подчас находишь какие-то ответы, пробуешь их, а они не работают, потому-что оказывается буквально несколько недель назад в 1.1 этот метод переименовали и т. п. Столкнулся с этим на примере метода from_str(), который удалили из Rust 1.1.Очень мило.
НЛО прилетело и опубликовало эту надпись здесь
from_str() удалили не из Rust 1.1, его нет уже давным-давно. Стандартным способом сконвертировать строку во что-то как минимум полгода является метод parse().
Удалили только отдельную функцию из Trait std::str::FromStr, еще до того как появился стабильный Rust (коммит).
Метод from_str() для разных типов вполне существует и работает.
Метод from_str() для разных типов вполне существует и работает.
std::vector<int> face = model->face(i);Копирование массива вершин треугольника на каждой итерации. Вот откуда тормоза в С++ версии наверное.
НЛО прилетело и опубликовало эту надпись здесь
Да, таки копируется. По уму, нужно что-то вроде
std::vector<int> const& Model::face(int idx) const {
return faces_[idx];
}
...
auto const& face = model->face(i);
Такие вещи с -O3 оптимизируются )
Современный gcc даже без C++11 режима не делает никакого копирования и мува при такой передаче. То есть конструктор копирования для вектора не вызывается вообще он создается ровно 1 раз и заполняется и потом используется.
Причём не только GCC. RVO давно уже поддерживается большей частью современных компиляторов.
Смысл в том что заполнять вектор на стеке не нужно. В данном случае интерфейс должен проектироваться таким образом, чтобы предоставлять прямой доступ к внутреннему массиву. Из такого интерфейса же явно следует что создаётся копия внутреннего массива и копирование таки будет.
можно дописать
и это будет валидный код — работа с копией массива вершин.
Так что правильно возвращать всё же константную ссылку а не значение.
можно дописать
face.push_back(-1);
и это будет валидный код — работа с копией массива вершин.
Так что правильно возвращать всё же константную ссылку а не значение.
Зарегистрируйтесь на Хабре, чтобы оставить комментарий
Пишем свой упрощенный OpenGL на Rust — часть 2 (проволочный рендер)