commentsEdit
I definitely prefer option 2 over option 1 -- option 1 is just a big unreadable mess. The formatting of the variable explanations under option 2 is nice, but it loses having the default values right with each factor.
Also, I'm not sure why you changed ArmPenalty to ArmFactor, since it's additive. I put it at the end of the formula because it's the least important part of the formula. The ideal ordering of variables would be from most important to least important, since that's the order they'll be read in.
As for Variables Old Formatting vs Variables New Formatting, I still prefer the 'Old' Formatting, as the New Formatting is too monochrome, which makes it harder to read and interpret.
--Quetzilla 04:24, 22 December 2008 (UTC)
- I used "ArmFactor" to be consistent with all other names used for the formula. It was the only one that didn't have a "Factor" suffix. You are right that the arm factor/condition/penalty part of the formula should be listed later as it is less important than the ironsights and crouch factors.
- I'm looking at the old variable formatting combined with option 2... Perhaps instead of having a separate variables segment, list the complete formula first (the one with the max() function), then the perk modifier part, then the primary calculation part, and follow the calculation part by saying "Where:" and then fall into the old variable formatting style without a header. Here is a rough example.
The gun spread formula is
GunSpread = Max(WeaponMinSpread, PerkModifiers(CalculatedSpread))
The Max() function does ....
PerkModifiers() is a complex function that...
CalculatedSpread = FooFactor + BarFactor * ZedFactor
Where: FooFactor = a + b * c
- a = 1
- b = 2
- c = 0
etc
--SnakeChomp 05:09, 22 December 2008 (UTC)
- That formatting looks good. I still think you should start with calculated spread and work towards the final result, as it puts the most relevant info at the beginning (the final result isn't the most relevant info, even though that sounds contradictory).
- --Quetzilla 06:06, 22 December 2008 (UTC)