请大家帮忙看看乘除法结果数据跑飞的原因是什么 | 已解决，见8楼

hsr***

kpf*** 发表于 2025-12-2 23:18
谢谢老机长的解答。我按照你的办法试过了还是不行，不管是用8位编译还是32位编译都不行，实在是想不通问题 ...

sum1 = ((u32)ADC_VAL) * ((u32)ADC_VAL);
复制代码

hsr***

这是原始的C代码：

uint16_t calc1(uint16_t ADC_VAL)
{
    uint32_t sum1 = ADC_VAL * ADC_VAL;
    return (uint16_t)(sum1 / diff);
}

uint16_t calc2(uint16_t ADC_VAL)
{
    uint32_t sum1 = ((uint32_t)ADC_VAL) * ADC_VAL;
    return (uint16_t)(sum1 / diff);
}

这是生成的汇编：

;------------------------------------------------------------
;Allocation info for local variables in function 'calc1'
;------------------------------------------------------------
;ADC_VAL                   Allocated to registers r6 r7 
;sum1                      Allocated to registers 
;------------------------------------------------------------
;	.\src\main.c:27: uint16_t calc1(uint16_t ADC_VAL)
;	-----------------------------------------
;	 function calc1
;	-----------------------------------------
_calc1:
;	.\src\main.c:29: uint32_t sum1 = ADC_VAL * ADC_VAL;
	mov	r6,dpl
	mov	r7,dph
	mov	__mulint_PARM_2,r6
	mov	(__mulint_PARM_2 + 1),r7
	lcall	__mulint
;	.\src\main.c:30: return (uint16_t)(sum1 / diff);
	mov	r4,_diff
	mov	r5,(_diff + 1)
	mov	__divsint_PARM_2,r4
	mov	(__divsint_PARM_2 + 1),r5
;	.\src\main.c:31: }
	ljmp	__divsint

;------------------------------------------------------------
;Allocation info for local variables in function 'calc2'
;------------------------------------------------------------
;ADC_VAL                   Allocated to registers r6 r7 
;sum1                      Allocated to registers r4 r5 r6 r7 
;------------------------------------------------------------
;	.\src\main.c:33: uint16_t calc2(uint16_t ADC_VAL)
;	-----------------------------------------
;	 function calc2
;	-----------------------------------------
_calc2:
	mov	r6, dpl
	mov	r7, dph
;	.\src\main.c:35: uint32_t sum1 = ((uint32_t)ADC_VAL) * ADC_VAL;
	mov	r5,#0x00
	mov	r4,#0x00
	mov	__mullong_PARM_2,r6
	mov	(__mullong_PARM_2 + 1),r7
	mov	(__mullong_PARM_2 + 2),r5
	mov	(__mullong_PARM_2 + 3),r4
;	.\src\main.c:36: return (uint16_t)(sum1 / diff);
	mov	dpl, r6
	mov	dph, r7
	mov	b, r5
	mov	a, r4
	lcall	__mullong
	mov	__divulong_PARM_2,_diff
	mov	(__divulong_PARM_2 + 1),(_diff + 1)
	mov	(__divulong_PARM_2 + 2),#0x00
	mov	(__divulong_PARM_2 + 3),#0x00

肉眼可见的差异巨大。虽然我用的sdcc可能和keil有区别，但不会太离谱

kpf***

hsr*** 发表于 2025-12-2 23:25

        diff = 301;
    sum1 = ((u32)diff)*((u32)diff);
    rms = (u16)sum1/diff;         //        
//        rms = 301;
复制代码

如果使用后面的rms = 301，显示的返回值是12D，说明显示子程序是没问题的，但如果屏蔽这条语句，平方再开方后显示的运算结果居然是0X0053。

Lkck***

kpf*** 发表于 2025-12-2 23:46
如果使用后面的rms = 301，显示的返回值是12D，说明显示子程序是没问题的，但如果屏蔽这条语句，平方再开 ...

我很好奇你为啥偏要加个(u16)在sum1前？
sum1 = 301 * 301 =  90601 = 0x161E9
(u16) sum1 = 0x61E9
rms = 0x61E9 / 0x012D = 0x0053
没毛病

hsr***

[quote][size=2][url=forum.php?mod=redirect&goto=findpost&pid=197329&ptid=21113][color=#999999]kpffq 发表于 2025-12-2 23:46[/color][/url][/size] 如果使用后面的rms = 301，显示的返回值是12D，说明显示子程序是没问题的，但如果屏蔽这条语句，平方再开 ...[/quote]

diff = 301;
sum1 = ((u32)diff) * ((u32)diff);
rms = (u16) (sum1 / diff);

大哥，当你看到我第二次回复，就应该勤点打括号了呀。

我也记不住那么多类型隐式转换的细节，但知道肯定是这个隐式类型转换的锅。
diff = 301 (0x012D)
sum1 = 90601(0x000161E9)
没括号时：
(u16)sum1 = 0x61E9(25065)
rms = 25065 / 301 = 83(0x0053)
这不是很符合你的结果么？

Ayb_***

计算结果不管怎样，应该不至于跑飞

lcws***

解决了吗？
你这代码里能看到u8、u16、u32的定义吗？建议确认一下。

kpf***

lcws*** 发表于 2025-12-3 11:19
解决了吗？
你这代码里能看到u8、u16、u32的定义吗？建议确认一下。

解决了，方法在8楼