2015-01-11 44 views
1

我有这个代码(having a string of length n, build another of length n-2 as it follows: sir2[i]=(sir[i]+sir[i+1]+sir[1+2])/3),我不知道它为什么冻结。没有错误,我已经调试了一段时间,但我无法弄清楚问题是否与算法有关。x86程序冻结[tasm]

prints macro number 
local decompose, pops 
    mov bx,10 
    mov al, number 
    mov cx,0 
    decompose: ;pushing digits to the stack 
     inc cx 
     mov ah,0 
     div bl 
     mov dl,ah ; remainder - last digit 
     add dx,48 ; to transform it in its char version 
     push dx 
     cmp al,0 
     jnz decompose 
    pops:    ; pop digits off the stack 
     pop dx 
     mov ah,2h 
     int 21h 
    loop pops 
     ; pretty spacing 
     mov dl,' ' 
     mov ah,2h 
     int 21h 

endm 

data segment para public 'data' 
    sir db 5, 10, 12, 4, 3 
    n equ $-sir 
    sir2 db n-2 dup(0) 
data ends 

code segment para public 'code' 
start proc far 
    assume cs:code,ds:data 
    push ds 
    xor ax,ax 
    push ax 
    mov ax,data 
    mov ds,ax 

    mov si,0 
    mov dx,3 
    mov cx,n 
    sub cx,2 

l1: 
    mov ah,0 
    mov al,sir[si] 
    add al,sir[si+1] 
    add al,sir[si+2] ;sum of 3 numbers 
    div dx     ;sir2[i]=(sir[i]+sir[i+1]+sir[1+2])/3 
    mov sir2[si],al 
    inc si 
    cmp si,cx 
jbe l1 

    mov si,0 
l2: 
    prints sir2[si] 
    inc si 
loop l2 

ret 
start endp 
code ends 
end start 

任何人都可以帮忙吗?

+1

你做了什么努力来发现它为什么会冻结,你卡在哪里?换句话说,**你是否调试过你的代码**?这个网站不是一个免费的调试服务,因此,就你的问题而言,最有可能的是,它会脱离主题。 –

+0

是的,我做了+我以为我被困在一个循环,所以我改变它为一个jmp。 –

+1

您在* decompose *例程中清除了CX!将'mov cx,0'改成'mov ah,0' –

回答

1

变化

div dx     ;sir2[i]=(sir[i]+sir[i+1]+sir[1+2])/3 

div dl     ;sir2[i]=(sir[i]+sir[i+1]+sir[1+2])/3 

(见迈克尔的答案)。

宏是一段代码,它在调用宏的地方按原样插入。宏prints更改CX,您需要保持loop l2不变。重写循环:

mov si, 0 
    mov di, cx 
l2: 
    prints sir2[si] 
    inc si 
    dec di 
    jnz l2 

BTW:不要忘记定义堆栈:

_STACK SEGMENT PARA STACK 'STACK' 
    dw 1024 dup (?) 
_STACK ENDS 
+0

它允许写'cmp si,n-2'吗? –

+0

@LorenaSfăt:是的。 TASM在编译时将'n-2'转换为一个常数。 – rkhb

+0

好,所以现在我想它一定是算法,因为它打印一些值(不正确的),然后它冻结... –

1

DIV r/m16的描述为“Unsigned divide DX:AX by r/m16。换句话说,分子是由组合的DXAX中的位形成的32位值。
如果商数变得太大而不适合AX,则会发生异常。为了避免这种情况,您应该在划分之前清除DX(即mov dx,0xor dx,dx)。显然这也意味着你不能使用DX作为分母。

1

你的程序可能会冻结,因为你正在做一个迭代太多了!

cmp si,cx 
jbe l1 

这给了4次迭代,而SIR2只有3个字节的存储空间!使用jb l1

+0

我希望就是这样。我做了你们所有人都说的,现在它打印价值(不是正确的),然后它冻结。 –

+0

我希望这个程序只用'ret'退出,那么你应该删除'push ds'和'push ax'指令。或者使用'mov ah,4Ch'和'int 21h'作为程序的最后一步。 –

+0

但是老师说这是.exe程序主体的一部分,我们应该照原样使用它:/ –