Тем что это прецедент который может лечь в основу неприятной цепочки:
1. Дети читерят, а давайте дадим возможность закрыть просмотр исходного кода.
2. Ой, эта фича такая популярная, давайте сделаем просмотр закрытым по умолчанию и вынесем его включение в настройки.
3. Ой, обычные пользователи его не включают, давайте вообще в about:config
4. Ой, оказывается об этой фиче знают только разработчики. Давайте же накатим DRM.
…
66. На этом шаге сайты отдаются в виде подписанных бинарников, пользователи рады скорости, бизнес рад тому что больше не палит код написанный разработчиками, гугл счастлив что его зонды/рекламу больше не вырезать.
Arc + Mutex используются чтобы раскидать ссылки на одну и ту же пямять по тредам и из каждого гарантировать владение этой памятью ровно одним тредом в конкретный момент. У вас же я не вижу ручного создания тредов, вы просто отдаете владение своим состоянием PlayGame в фреймворк а дальше из обработчика получаете &self этого состояния. Следовательно в self.world можете сразу хранить Mutex
Вы достаёте MutexGuard, через Deref вызываете as_ref преобразуя &Option в Option<&World> достаете &World и полностью клонируете чтобы получить World далее по коду вы просто читаете из него нужные вам поля. Лучшее что вы можете тут сделать без серьёзных изменений — убрать последний .clone(). Это архитектурная проблема возникающая из за того что Option используется как указатель в C постоянными проверками на NULL. Приведу пример, у вас количество игроков и сам мир находятся в двух независимых Option, хотя во время раунда очевидно что если есть игроки то они находятся в мире который не может быть None, однако вам придется делать unwrap. Вместо этого предлагаю заранее определить инварианты игры, выглядеть будет примерно так:
struct RoundState {/* Игроки, позиции обьектов, etc */}
struct RoundResult { /* Победители, очки, etc */ }
enum Round {
Lobby(/* кто в лобби и т.д*/),
Game(RoundState),
Finished(RoundResult),
}
struct Game {
round: Round
/* Тут всякие статические штуки которые не зависят от раунда
* Имя серевера, MOTD и прочее
*/
}
И когда вам придётся обработать запрос вам нужно просто проверить допустим ли он для этого инварианта, остальные проверки отпадут сами собой. Если раунд идёт то для него есть мир, если раунд закончился то есть статистика и т.д:
async fn fetch_map(&self, ...) -> ... {
...
match round {
Round::Game(RoundState{players, map, ...}) => {
/* Актуальные данные вы получили строкой выше
* осталось бех всяких проверок обработать запрос
*/
},
_ => {
/* Попытка получить карту когда игра не запущена */
}
}
}
let world = Arc::clone(&self.world).lock().unwrap().as_ref().unwrap().clone();
Знаменитое удобство многопоточности в расте, ну вы знаете. Я не могу разобраться зачем вы так делаете, но наверное не стоит клонировать world просто чтобы взять параметры и использовать везде Option/Result без обработки ошибок
Код на 40% состоит из преобразований FloatTuple <-> Vec2, просто реализуйте трейт From<...> для этих типов.
Плюс Option уже есть в std::prelude, можно использовать Some(...).
Три
let mut ball_velocity = 0f32;
if players_count >= 2 {
let num = rand::thread_rng().gen_range(0..2);
if num == 0 {
ball_velocity = -BALL_SPEED;
} else {
ball_velocity = BALL_SPEED;
}
}
Тут ball_velocity не должна быть мутабельной, вы просто инициализируете её одним из трёх значений, для этого есть специальный сахар, например:
let velocity = { if ... { value1 } else { if ... { ... } else { ... } } };
Ну а статья норм, приятно смотреть как другие изучают то что самому учить лень.
Пользователи из РФ теряют доступ к нежелательному контенту
Часть освободившейся от монополии ютюба аудитории приходит на отчественные площадки, рутуб или, вангую, созданный после блокировки сервис от Яндекса/Сбера который разовьется очень быстро потому что для контентмейкеров(и не только) деньги > совесть.
Заинтересованные в плюсе при любом раскладе. А градус накала метрика сомнительная, т.к. уже не раз было показано что прогулки по улицам можно игнорировать.
Да нет, вакансии открыты, руководители нормальные, зарплаты чуть выше чем у других специалистов по предприятию. Просто никто не хочет работать за обычную зарплату с технологиями о которых не пишут на хабре по 20 статей в день.
Ну и да, приезжего кодера оторвут с руками.
У вас не было разрыва шаблона когда оказалось что в стоковой прошивке нельзя поменять время без синхронизации с телефоном? Я не пользователь умных часов, возможно это обычная ситуация.
1. Дети читерят, а давайте дадим возможность закрыть просмотр исходного кода.
2. Ой, эта фича такая популярная, давайте сделаем просмотр закрытым по умолчанию и вынесем его включение в настройки.
3. Ой, обычные пользователи его не включают, давайте вообще в about:config
4. Ой, оказывается об этой фиче знают только разработчики. Давайте же накатим DRM.
…
66. На этом шаге сайты отдаются в виде подписанных бинарников, пользователи рады скорости, бизнес рад тому что больше не палит код написанный разработчиками, гугл счастлив что его зонды/рекламу больше не вырезать.
Arc + Mutex используются чтобы раскидать ссылки на одну и ту же пямять по тредам и из каждого гарантировать владение этой памятью ровно одним тредом в конкретный момент. У вас же я не вижу ручного создания тредов, вы просто отдаете владение своим состоянием PlayGame в фреймворк а дальше из обработчика получаете &self этого состояния. Следовательно в self.world можете сразу хранить Mutex
Вы достаёте MutexGuard, через Deref вызываете as_ref преобразуя &Option в Option<&World> достаете &World и полностью клонируете чтобы получить World далее по коду вы просто читаете из него нужные вам поля. Лучшее что вы можете тут сделать без серьёзных изменений — убрать последний .clone(). Это архитектурная проблема возникающая из за того что Option используется как указатель в C постоянными проверками на NULL. Приведу пример, у вас количество игроков и сам мир находятся в двух независимых Option, хотя во время раунда очевидно что если есть игроки то они находятся в мире который не может быть None, однако вам придется делать unwrap. Вместо этого предлагаю заранее определить инварианты игры, выглядеть будет примерно так:
И когда вам придётся обработать запрос вам нужно просто проверить допустим ли он для этого инварианта, остальные проверки отпадут сами собой. Если раунд идёт то для него есть мир, если раунд закончился то есть статистика и т.д:
Знаменитое удобство многопоточности в расте, ну вы знаете. Я не могу разобраться зачем вы так делаете, но наверное не стоит клонировать world просто чтобы взять параметры и использовать везде Option/Result без обработки ошибок
Код на 40% состоит из преобразований FloatTuple <-> Vec2, просто реализуйте трейт From<...> для этих типов.
Плюс Option уже есть в std::prelude, можно использовать Some(...).
Тут ball_velocity не должна быть мутабельной, вы просто инициализируете её одним из трёх значений, для этого есть специальный сахар, например:
Ну а статья норм, приятно смотреть как другие изучают то что самому учить лень.
Если гугл не прогибается(что не случится):
Заинтересованные в плюсе при любом раскладе. А градус накала метрика сомнительная, т.к. уже не раз было показано что прогулки по улицам можно игнорировать.
Навальныйэто плохо?Ну и да, приезжего кодера оторвут с руками.