[?] Inicializar vector y trabajar con el

[MeCraniDOS]

Hola  

Estoy intentando calcular la media de unos numeros para practicar un poco, pero tengo algunos fallos de concepto, he trabajado antes con ARM, pero las instrucciones en ASM son un poco diferente..

el codigo es este

Código (asm) [Seleccionar] Expandir

include 'FASM\INCLUDE\WIN32AX.INC'

.data
vector rd 8 ; Reservamos 8 bytes

.code
start:
mov eax, 0 ;Media
mov ecx, 0 ;Contador

;Inicializamos el vector
mov [vector], 7
mov [vector+1], 8
mov [vector+2], 5
mov [vector+3], 2
mov [vector+4], 9
mov [vector+5], 1
mov [vector+6], 0
mov [vector+7], 3

for:
add eax, [vector] ;Aqui esta el error
inc ecx
cmp ecx, 8
jne for

       ; Aqui divido para obtener la media

invoke ExitProcess, 0

.end start

Se que para los que sepan, es algo muy basico... hay alguna manera de inicializar el vector de una manera mas sencilla? y otra cosa, para ir sumando el valor del vector en un bucle com seria? Intentaba simular esto

Código (cpp) [Seleccionar] Expandir

for(int i=0;i<8;i++)
{
   eax += vector[i];
}

No quiero codigos completos, solo una ayudita con las instrucciones  

Saludos

[Eternal Idol]

No se si entendi muy bien:

Código (asm) [Seleccionar] Expandir

vector db 1, 2, 3, 4, 5
...
movzx edx, byte [vector+ecx]
add eax, edx

[MeCraniDOS]

No se si entendi muy bien:

Le he hecho en C para que se entienda mejor

Código (c) [Seleccionar] Expandir

   int eax=0, ecx=0;
   int Vector[] = {7,8,5,2,9,1,0,3};

   for(ecx=0;ecx<8;ecx++)
   {
        eax += Vector[ecx];
   }

   eax /= ecx;

Entonces lo que quería saber era si se podía inicializar como lo he hecho en C

Código (asm) [Seleccionar] Expandir

;Inicializamos el vector
mov [vector], 7
mov [vector+1], 8
mov [vector+2], 5
mov [vector+3], 2
mov [vector+4], 9
mov [vector+5], 1
mov [vector+6], 0
mov [vector+7], 3

Eso equivale a esto en C

Código (c) [Seleccionar] Expandir

   int Vector[8];

   Vector[0] = 7;
   Vector[1] = 8;
   Vector[2] = 5;
   Vector[3] = 2;
   Vector[4] = 9;
   Vector[5] = 1;
   Vector[6] = 0;
   Vector[7] = 3;

Y hacerlo asi en un  



Al final el código me ha quedado así con las modificaciones que me has dicho Eternal

Código (asm) [Seleccionar] Expandir

include 'FASM\INCLUDE\WIN32AX.INC'

.data
       vector db 7, 8, 5, 2 ,9 ,1 ,0, 3

.code
       start:
               mov eax, 0      ;Media
               mov ecx, 0      ;Contador

               for_add:
                       movzx edx, byte [vector+ecx]
                       add eax, edx
                       inc ecx
                       cmp ecx, 8
                       jne for_add

               mov ecx, 0 ;Reiniciamos ecx

               for_div:
                       sub eax, 8
                       cmp eax, 8
                       inc ecx
                       jae for_div

               invoke ExitProcess, 0

.end start

Por cierto, dos cosas, la rutina for_div se puede simplificar? Es que en arm lo tenia que hacer asi porque no hay operacion de division, pero en asm he visto que si, pero solo pones el registro a dividir, no entre que valor lo quieres dividir  

Y la otra, que diferencia hay entre esto?

Código (asm) [Seleccionar] Expandir

vector db 7, 8, 5, 2 ,9 ,1 ,0, 3

Código (asm) [Seleccionar] Expandir

vector rb 8

Pensaba que db era para una unica variable, y rb (reserved byte) para vectores, ya que especificas cuanto espacio quieres reservar  

[Eternal Idol]

Le he hecho en C para que se entienda mejor

Código (c) [Seleccionar] Expandir

   int eax=0, ecx=0;
   int Vector[] = {7,8,5,2,9,1,0,3};

   for(ecx=0;ecx<8;ecx++)
   {
        eax += Vector[ecx];
   }

   eax /= ecx;

Entonces lo que quería saber era si se podía inicializar como lo he hecho en C

Evidentemente entonces lo habia entendido bien.

Por cierto, dos cosas, la rutina for_div se puede simplificar? Es que en arm lo tenia que hacer asi porque no hay operacion de division, pero en asm he visto que si, pero solo pones el registro a dividir, no entre que valor lo quieres dividir  

Código (asm) [Seleccionar] Expandir

mov ECX, 25 ;dividimos por 25
cdq ;limpiamos EDX
idiv ECX ;EDX:EAX es dividido por ECX
;EAX=COCIENTE EDX=RESTO

Y la otra, que diferencia hay entre esto?

Código (asm) [Seleccionar] Expandir

vector db 7, 8, 5, 2 ,9 ,1 ,0, 3

Código (asm) [Seleccionar] Expandir

vector rb 8

Pensaba que db era para una unica variable, y rb (reserved byte) para vectores, ya que especificas cuanto espacio quieres reservar  

Notese que estas usando char (byte, db/rb), si queres usar un int usa dword (y el add puede ser directo).

db:
char Vector[] = {7,8,5,2,9,1,0,3};
rb:
char Vector[8];

[MeCraniDOS]

Notese que estas usando char (byte, db/rb), si queres usar un int usa dword (y el add puede ser directo).

Al utilizar dword ocupo 4 bytes en la memoria, por lo que tendria que incrementar el contador de 4 en 4  

00401000  07 00 00 00 08 00 00 00  ......
00401008  05 00 00 00 02 00 00 00  ......
00401010  09 00 00 00 01 00 00 00  ....
...
00401018  00 00 00 00 03 00 00 00  .......

Lo de la division lo veo raro  

Poniendo el codigo como me has dicho, los registros quedan de esta manera una vez realizada la operación:

EAX 00000001
ECX 00000008
EDX 00000007
EBX 7FFDF000
ESP 0007FF90
EBP 0007FF98
ESI 00000000
EDI 00000000
EIP 00402021 2_-_copi.00402021

Código (asm) [Seleccionar] Expandir

include 'FASM\INCLUDE\WIN32AX.INC'
   
.data
vector dd 7, 8, 5, 2 ,9 ,1 ,0, 3

.code
start:
mov eax, 0 ;Media
mov ecx, 0 ;Contador
   
for_add:
movzx edx, byte [vector+ecx]
add eax, edx
inc ecx
cmp ecx, 8
jne for_add
   
mov edx, 0 ;Inicializamos edx
idiv ecx ;edx:eax es dividido por ecx
 
;eax = cociente
;edx = resto
   
invoke ExitProcess, 0
   
.end start

No saca muy bien el resultado...  

En que estoy metiendo la pata?  

Código (asm) [Seleccionar] Expandir

vector db 7, 8, 5, 2 ,9 ,1 ,0, 3

EAX 00000004
ECX 00000008
EDX 00000003
EBX 7FFDF000
ESP 0007FF8C
EBP 0007FF98
ESI 00000000
EDI 00000000
EIP 00402022 2_-_copi.00402022

Solucionado

[Eternal Idol]

No se cual seria el problema al dividir que tenias 

Y si usas DWORD:

Código (asm) [Seleccionar] Expandir

add eax, [vector + ecx * 4]

[MeCraniDOS]

No se cual seria el problema al dividir que tenias  

Y si usas DWORD:

Código (asm) [Seleccionar] Expandir

add eax, [vector + ecx * 4]

El problema era que estaba utilizando DWORD, y solo incementaba el contador en 1, por lo que sumaba el valor y luego 3 ceros hasta el siguiente numero, pero se soluciona multiplicando como has hecho  

Otra cosita, utilizando DWORD, no malgastarias memoria? (En este caso, ya que los numeros no son de mas de dos cifras)  



En este caso podría utilizar WORD? que ocupa 2 bytes y es numerico

Código (asm) [Seleccionar] Expandir

vector dw 7, 8, 5, 2 ,9 ,1 ,0, 3

; ....

add eax, [vector + ecx * 2]

[Eternal Idol]

El problema era que estaba utilizando DWORD, y solo incementaba el contador en 1, por lo que sumaba el valor y luego 3 ceros hasta el siguiente numero, pero se soluciona multiplicando como has hecho  

Otra cosita, utilizando DWORD, no malgastarias memoria? (En este caso, ya que los numeros no son de mas de dos cifras)  

En el codigo de C estabas usando int, que es un DWORD con signo.

En este caso podría utilizar WORD? que ocupa 2 bytes y es numerico

Un BYTE puede representar valores entre 0 y 255 asi que en este caso no necesitas un WORD tampoco ... y no, no podes usar add directamente con un WORD, vas a tener que pasarlo a un registro como con BYTE anteriormente; o usar el registro correspondiente en su lugar (al, ax, eax, rax).

[MeCraniDOS]

En el codigo de C estabas usando int, que es un DWORD con signo.

Un BYTE puede representar valores entre 0 y 255 asi que en este caso no necesitas un WORD tampoco ... y no, no podes usar add directamente con un WORD, vas a tener que pasarlo a un registro como con BYTE anteriormente; o usar el registro correspondiente en su lugar (al, ax, eax, rax).

De acuerdo, muchas gracias Eternal  

Saludos

[Eternal Idol]

De nadas