Skip to content

Conversation

@LudwigVonChesterfield
Copy link
Contributor

@LudwigVonChesterfield LudwigVonChesterfield commented Mar 16, 2023

Описание изменений

Убрал пропуск первой зарплаты:
0. (Почему это вообще было?)

  1. Уменьшает тайминг первой зарплаты, с 30 минут до 15, значит у игроков деньги будут поступать более постоянно а не первым рывком а потом постоянно
  2. Последняя зарплата в 1:15 сильно погоды не сыграет потому-что уже конец смены, считаем что её переместили на пораньше...

Сотрудники карго получили дивидендовые акции на счёт карго! А карго реформировалось в акционерское общество!

Что это значит на практике:

  • "Зарплата" сотрудников карго зависит от количества денег на счету карго
  • Каргонцы могут продавать акции другим членам экипажа чтобы быстренько поднять денег в обмен на потерю денег в долгой перспективе

Немножко математики: Независимо от наполненности карго на счету карго есть 260 акций (которые распределяются между работниками если они есть), ещё 10 акций у станции, ещё 10 у ЦК. Суммарно акций 280. Это значит что одна акция даёт 0.35 кредита за каждые 1000 очков карго при дефолтном дивиденд рейте в 10%.

У квартирмейстера 40 акций, значит он будет получать 14 кредитов за каждых 1000 очков
Грузчик 7 кредитов
Шахтёр и переработчик 3.5 кредита

image

В будущем я надеюсь перебалансить карго так чтобы при нормальных условиях у них было где-то 10к кредитов на счету.

Акции можно изымать на чип в ближайшем АТМ-е (а потом класть на свой счёт вставляя чип, как с деньгами).

image

image

Надеюсь что кто-то в будущем реализует:

  • У КМ-а консоль для дробления акций и изменения дивиденд рейта. По дефолту дивиденд рейт 10%
  • Акции можно трансферить с помощью Finance Management в ПДА
  • КМ может триггернуть внеплановую выплату дивидендов. К примеру чтобы уменьшить ярость инвесторов когда перед выплатой забыли ликвидировать все активы...

Для кодеров

spacecash/ewallet больше не наследуется от spacecash так как не имеет никакого общего поведения, теперь это карточка со счётом а не эээ... наличность.

Почему и что этот ПР улучшит

У грузчиков внятная мотивация таскать ящики, а не зарплата которая приходит им независимо от того что они делают. У шахтёров появилась мотивация не давать рнд за бесплатно ресурсы... Ну и в целом прикольно вроде бы.

Чеинжлог

🆑 Luduk

  • rscadd[link]: Карго переформировалось в акционерское общество, которое выплачивает дивиденды сотрудникам владеющим акциями карго.
  • rscdell: Убран первый пропуск зарплаты. Теперь первая зарплата выплачивается не через 30 минут а через 15!!!

@TauKitty
Copy link
Contributor

TauKitty commented Mar 16, 2023

Changelog status: ❌ Invalid changelog classes (rscdell)

@TauKitty TauKitty added the Feature Новая фича label Mar 16, 2023
@AndroBetel
Copy link
Member

лайк

@LudwigVonChesterfield
Copy link
Contributor Author

LudwigVonChesterfield commented Mar 17, 2023

Этот ПР ждёт помощи с добавлением кнопочек в ПДА, консоль КМ-а...

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

@LudwigVonChesterfield
Copy link
Contributor Author

Окей я попытался добавить снятие акции со счёта в АТМ-е и у меня не получилось, разрешаю кому-то другому попробовать... Остальное в ПР-е работает и я не против мержа с надеждой что кто-то в будущем закодит нужные интерфейсы для передачи акций другим игрокам.

@LudwigVonChesterfield LudwigVonChesterfield changed the title [DNM] Акционерское Общество с Ограниченной Ответственностью "Каргония Стронк" Акционерское Общество с Ограниченной Ответственностью "Каргония Стронк" Mar 18, 2023
@LudwigVonChesterfield
Copy link
Contributor Author

аааа ура я смог в АТМ интерфейс!!

if(!total_department_stocks[department])
LAZYSET(total_department_stocks, department, 0)

total_department_stocks[department] += amount
Copy link
Member

Choose a reason for hiding this comment

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

Было бы круче, если бы акция была бы датумом, а не интом

Copy link
Contributor Author

Choose a reason for hiding this comment

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

попробуй описать какой функционал должен быть у акции-датума.

Copy link
Member

@KIBORG04 KIBORG04 Mar 19, 2023

Choose a reason for hiding this comment

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

Можно психануть и сделать 1 акции = 1 датум, но я думаю такое будет даже хуже. Поэтому легче сделать датум-холдер этих акций и в его неймспейс кинуть все функции, связанные с обработкой акций.

Какую-то простую штуку, которая бы хранила: количество акций (инт), привязанный департамент, total_dividend_payouts . А так же в неймспейс этих акций кинуть: get_stocks_string, ну и всякие геттеры кол-ва акцией.

То есть буквально тот же функционал пишу, что у тебя сейчас есть, с этим проблем нет. Проблемы могут появиться, когда захочется к этим акциям что-то новое добавить. условно, отслеживание от кого у тебя в портфеле акции. К тому же, буквально удобней обращаться с каким-то апи акций, чем с листом и LAZYACCESS, LAZYADD и тд

Copy link
Contributor Author

Choose a reason for hiding this comment

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

опиши пожалуйста полный функционал того что ты хочешь видеть где-нибудь, я сейчас не понимаю кто в итоге этот датум хранит и где

сейчас роль этого датума выполняет счёт почти во всех случаях

ты хочешь чтобы каким-то образом контроль над акцией (дробление и полная их сумма) и владение (сколько у меня есть и чего) были в одном датуме

@LudwigVonChesterfield
Copy link
Contributor Author

Киборг очень просил чтобы у чипов был свой счёт и видимо мне придётся это закодить.

@LudwigVonChesterfield LudwigVonChesterfield added the Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже label Mar 18, 2023
@LudwigVonChesterfield
Copy link
Contributor Author

@KIBORG04 я в принципе не против переделать ewallet на счёт...

Но у нас кажется сейчас нигде счета не удаляются и я не знаю к каким багам это может привести...

если мы не боимся Б-га и ты не против дай знать и я закодю

@volas
Copy link
Member

volas commented Mar 20, 2023

конфлиты с твоим другим пр-ом

Copy link
Member

@KIBORG04 KIBORG04 left a comment

Choose a reason for hiding this comment

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

ладно

@LudwigVonChesterfield LudwigVonChesterfield merged commit 1098969 into TauCetiStation:master Mar 24, 2023
TauKitty added a commit to TauCetiStation/ClassicChangelog that referenced this pull request Mar 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Feature Новая фича Test Merge Candidate ПР с этим лэйблом будет или уже находится с тест мерже

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants