search

emotz / kladovka

Optimizer of inventory for games
http://kladovka.herokuapp.com/
MIT License
0 stars 0 forks source link

add crits to weapon #10

Closed Eoksni closed 7 years ago

Eoksni commented 7 years ago
Eoksni commented 7 years ago

dps increase from crits = critchance*critdamage

Eoksni commented 7 years ago

например если 20% крита с +100% крит дамагом, то итоговый дамаг увеличится от критов на 20%

Eoksni commented 7 years ago

если 40% крита с +150% крит дамаги, то итоговый дамаг увеличится от критов на 60%

ITarako commented 7 years ago

крит присутсвует по желанию или маст хев?

Eoksni commented 7 years ago

по желанию

ITarako commented 7 years ago

мб ошибкуmustBePositive переделать в mustNotBeNegative 0 крита, вполне легальное отсутствие крита 0 дмг.. палка х)

Eoksni commented 7 years ago

эм... не переделать, а добавить новую? сам же написал что валидация различная. там 0 невалидное значение, а здесь - валидное.

ITarako commented 7 years ago

имел ввиду что палка с миндмг 0 и махдмг 1 выглядит легально. ну да ладно, пусть будет еще проверка хм..и как лучше назвать ф-цию для нахождения отрицательный чисел, чтобы была консистента с filterNotPositive, filterNotNumbers

Eoksni commented 7 years ago

нет, палка с миндмг 0 и махдмг 1 выглядит не очень легально если честно.

filterNegative ?

Eoksni commented 7 years ago

потому что у нас легальным является махдмг равный миндмг, верно? а палка с 0-0 дамагом слегка ущербна

Eoksni commented 7 years ago

кстати statControl во фронте это не отражает. даже в таком простом случае появился рассинхрон валидаций. поэтому, по хорошему, все валидации надо как-то централизировать.

ITarako commented 7 years ago

как раз фиксил statControl, чтобы мин и мах могли быть равными х) похоже намечается исус "Validation come back"

ITarako commented 7 years ago

closed #10