2

I am writing this assembly program in 8086, but it is not working properly. The quotient and remainder prints out as some random symbols even though I use single digit numbers. Can someone please point out the errors/mistakes in the program? Thank you.

.model small
.stack 50h

.data
Divisor db ?
Dividend db ?
Quotient db ?
Remainder db ?

.code
main_method   proc
              mov    ax, @data
              mov    ds, ax

              mov    ah, 01
              int    21h
              sub    al, 48
              mov    Divisor, al

              mov    ah, 01
              int    21h
              sub    al, 48
              mov    Dividend, al
              mov    bl, 00
              mov    al, 00
              mov    bl, Divisor
              mov    al, Dividend
              div    bl

              mov    Quotient, al
              mov    Remainder, ah

              mov    dl, Quotient
              add    dl, 48
              mov    ah, 02
              int    21h

              mov    dl, Remainder
              add    dl, 48
              mov    ah, 02
              int    21h
              mov    ah, 4ch
              int    21h
main_method   endp
              end    main_method
Pyromaster
  • 147
  • 6
user3226056
  • 275
  • 3
  • 4
  • 9
  • 1
    _"it is not working properly"_ could mean anything. Be more specific. – Michael Feb 09 '14 at 07:41
  • The quotient and remainder prints out as some random symbols even I use single digit numbers. – user3226056 Feb 09 '14 at 07:50
  • It would help if you comment your code!! Especially since this is code many have not used in many years!!! Plus, it will help you catch errors. – Gunner Feb 09 '14 at 07:59
  • `sub al, '0'` would be make the conversion from ASCII to integer semantically clear in the source code, instead of using `48`. – Peter Cordes Oct 31 '17 at 13:41

2 Answers2

5

DIV BL divides the 16-bit value in AX by BL, so you should clear those bits of AX that you're not using (in this case the entire upper byte). So right before the DIV, add either:

MOV AH,0

or

XOR AH,AH  ; XORing something with itself clears all bits


Or, if you're targetting 80386 or above you can replace Mov Al, Dividend with MOVZX AX, BYTE PTR Dividend

Michael
  • 57,169
  • 9
  • 80
  • 125
  • Oh! Thank you very much, Michael! Resetting (clearing) the Ah solved the problem. I didn't clear it before thinking that it would replace the existing value, though I was wrong. Thank you. – user3226056 Feb 09 '14 at 08:10
1

Basically DIV function divide AX and then put quotient in AL and remainder in AH. AX consists of AH and AL. So if you only want to divide AL then you have to make sure that AH is 0 . You can do below method to make something zero.

MOV AH, 0

OR

AND AH, 0

OR

XOR AH, AH
arafatkn
  • 117
  • 4