Skip to content

Многовариативное голосование и больше пресетов#242

Draft
vanomorodellefake wants to merge 1 commit intodead-space-server:masterfrom
vanomorodellefake:MultipleVotingSystem
Draft

Многовариативное голосование и больше пресетов#242
vanomorodellefake wants to merge 1 commit intodead-space-server:masterfrom
vanomorodellefake:MultipleVotingSystem

Conversation

@vanomorodellefake
Copy link
Contributor

Описание PR

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

Ну и я немного устал, поэтому пр не особо сильно расписываю

Почему / Зачем / Баланс

Открывает возможности для крупных голосований, в которых игроки могли бы выбирать сразу несколько желаемых вариантов. Удобно подходит, если игроки хотят условных оперативников.

Технические детали

Изменено количество голосов для игрока с int на List
Для Multivariate голосований не создаётся отдельная группа

Медиа

image

Требования

  • PR полностью завершён и мне не нужна помощь чтобы его закончить.
  • Я внимательно просмотрел все свои изменения и багов в них не нашёл.
  • Я запускал локальный сервер со своими изменениями и всё протестировал.
  • Я добавил скриншот/видео демонстрации PR в игре, или этот PR этого не требует.

Критические изменения

int OurVote -> List OurVotes

  • bool Multivariate

@Vecortys
Copy link
Contributor

Vecortys commented Feb 9, 2026

Невозможность делать голосования с одним вариантом ответа это бредик. А ещё геймрулы основной принцип текущей системы пресетов нарушает

@vanomorodellefake
Copy link
Contributor Author

Невозможность делать голосования с одним вариантом ответа это бредик. А ещё геймрулы основной принцип текущей системы пресетов нарушает

Можно делать голосования с одним вариантом, настраивается через класс VoteOptions
Цель пра - дать игрокам свободу выбора режима, который они непосредственно хотят

@Vecortys
Copy link
Contributor

Vecortys commented Feb 9, 2026

Или можно, я код смотрю играя в доту. Хз. Все равно лишняя нагрузка

@Vecortys
Copy link
Contributor

Vecortys commented Feb 9, 2026

Позор конечно. Я все равно сделаю новую систему

@Sidzaru
Copy link
Collaborator

Sidzaru commented Feb 9, 2026

Позор конечно. Я все равно сделаю новую систему

Мицелия угомони

@vanomorodellefake
Copy link
Contributor Author

да кто этот ваш мицелий

@Sidzaru
Copy link
Collaborator

Sidzaru commented Feb 9, 2026

да кто этот ваш мицелий

Можем просвятить, но тогда придётся как и мы не спать ночами

@vanomorodellefake
Copy link
Contributor Author

да кто этот ваш мицелий

Можем просвятить, но тогда придётся как и мы не спать ночами

Ладно, всему своё время

{
_voteButtons[i].Pressed = false;
}
// Erida-end
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Можно сократить до
if (_vote.OurVotes != null)
_voteButtons[i].Pressed = _vote.OurVotes.Contains(i);

id: LightDynamic
name: light-dynamic-title
showInVote: false
showInVote: true
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Нет смысла смешивать все голосования воедино, включая и всё сразу

@github-actions
Copy link
Contributor

github-actions bot commented Mar 6, 2026

This pull request has conflicts, please resolve those before we can evaluate the pull request.

@vanomorodellefake vanomorodellefake marked this pull request as draft March 17, 2026 16:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants