Assembly80x86 filling array with user input(names), but no repeating them

Question

I need to fill an array with names from user input, but keep out names that are already stored in the array(ex, john,jaina,tom,bob). I got this, but it's not working.

data segment
numbers db 0
names db 220 dup (?)
buffer db 10 dup (?)

code segment
start:
 mov ah, 1;
 int 21h
 mov numbers, ax
 mov cx, numbers
 lea bx, [names]
 names:
       onename: 
        lea si, [buffer]
        mov ah,1;character by character
        int 21h 
        mov [bx], al
        inc bx
        mov [si], al
        inc si
        cmp al, ',' ;end of name
        je compare;
       loop onename
       compare:;buffer with names
       lea di, [names]
            check:
            lea si, [buffer]
            cmp si, di
            jne nextname
            inc si
            inc di
            jmp check
       nextname:
       cmp di, ','
       je check
       inc di
       jmp nextname 
 loop names

In cmp di, ',' you're comparing a pointer with a character. That won't work. — Sep Roland
– Sep Roland, Commented Feb 28, 2016 at 21:57

Fifoernik · Accepted Answer · 2016-01-16 14:43:01Z

1

mov ah, 1;
int 21h
mov numbers, ax

You don't initialize the counter correctly. DOS function 01h only gives you an ASCII code in AL, but you treat it as a number in AX.
You've also defined the variable numbers of type byte but later on you process it as if it were a word.
The instruction loop onename should have been jmp onename
The label onename must be placed 1 line lower, just under the assignment to SI.

answered Jan 16, 2016 at 14:43

Fifoernik

9,9191 gold badge23 silver badges27 bronze badges

Sign up to request clarification or add additional context in comments.

Collectives™ on Stack Overflow

Assembly80x86 filling array with user input(names), but no repeating them

1 Answer 1

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

1 Answer 1

Comments

Your Answer

Sign up or log in

Post as a guest

Related