C#中含申诉/折扣及销售折扣的最终百分比计算问题

场景与问题 #

基础价格有两种调整类型：GRIEVANCE（申诉，即涨价）或DISCOUNT（折扣，即降价），来自矩阵的调整百分比始终为正数，之后需要叠加销售折扣百分比。目标是计算经过初始调整后再应用销售折扣的最终百分比调整值。

当前遇到的问题：当设置100%申诉 + 10%销售折扣时，预期最终百分比为80%，但现有代码计算结果不符合预期。

现有代码 #

// Determine if the adjustment is a discount
bool isMatrizDiscount = matrizDescription == "DISCOUNT";

// Percentages (always positive)
int matrizPercentage = 100; // Grievance or discount from matriz
int saleDiscountPercentage = 10; // Discount from sale

// Convert percentages to decimal for calculation
double matrizPercentageDivided = matrizPercentage / 100.0;
double saleDiscountPercentageDivided = saleDiscountPercentage / 100.0;

// Calculating final percentage
var finalPercentual = isMatrizDiscount
    ? 1 - ((1 - saleDiscountPercentageDivided) * (1 - Math.Abs(matrizPercentageDivided)))
    : 1 / ((1 - saleDiscountPercentageDivided) * (1 + Math.Abs(matrizPercentageDivided))) - 1;

finalPercentual = Math.Floor(finalPercentual * 100);

// Determine the description based on final percentage
var description = finalPercentual <= 0 ? "GRIEVANCE" : "DISCOUNT";

// Expecting 80, due to a 100% grievance followed by a 10% sale discount
finalPercentual = Math.Abs(finalPercentual);

错误分析 #

针对GRIEVANCE场景的计算逻辑完全错误：

• 正确的价格计算逻辑应为：原价 × (1 + 申诉百分比) × (1 - 销售折扣百分比)，最终百分比调整值为**(最终价格倍数 - 1) × 100**。
• 现有代码中else分支使用了1 / ((1 - salePct) * (1 + matrizPct)) - 1，这完全颠倒了价格倍数的计算逻辑，导致结果偏差。

以测试案例代入：

• 申诉100%：价格变为原价的1 + 1 = 2倍
• 叠加10%销售折扣：价格变为2 × 0.9 = 1.8倍
• 最终调整百分比应为(1.8 - 1) × 100 = 80%

而现有代码计算：
1 / (0.9 × 2) - 1 = 1/1.8 -1 ≈ -0.444，乘100后为-44，取绝对值得到44，与预期不符。

修正方案 #

将GRIEVANCE场景的计算公式改为价格倍数减1，而非1/价格倍数减1，同时可移除多余的Math.Abs（因为matrizPercentage本身是正数）：

// Calculating final percentage
var finalPercentual = isMatrizDiscount
    ? 1 - ((1 - saleDiscountPercentageDivided) * (1 - matrizPercentageDivided))
    : ((1 + matrizPercentageDivided) * (1 - saleDiscountPercentageDivided)) - 1;

修正后测试案例的计算：
(1 + 1) × (1 - 0.1) -1 = 2×0.9 -1 = 1.8 -1 = 0.8，乘100后为80，Floor后仍为80，取绝对值后得到预期的80。

额外提示 #

当前description的判定逻辑可能存在混淆：

• GRIEVANCE是涨价，最终调整百分比应为正数，但现有代码中finalPercentual <=0才判定为GRIEVANCE，这与实际场景矛盾，建议根据业务需求调整该逻辑。

内容的提问来源于stack exchange，提问作者Luiz Kohler