с++ и ASM

[breate]

помогите найти ошибку

#pragma once
#include <stdlib.h>
#include <iostream>
 
class bbyte
{
private:
        void *data;
public:
        bbyte(void)
        {
        data=malloc(sizeof(char));
        void *a=data;
        _asm{
        mov esi,0
        mov a, esi
        }
        }
 
        bbyte (const void *a){
                data=malloc(sizeof(char));
                void *b=data;
                _asm
        {
        mov esi, a
        cmp esi, 0
        je m1
        jne m2
m1:
        mov esi, 0
        mov b,esi
        jmp exit1
m2:
        mov esi,a
        mov b,esi
exit1:
        }
                }
        ~bbyte(void)
        {
                free((char*)data);
        }
friend bbyte add(bbyte &a,const bbyte &b)
        {
                void *c=a.data;
                void *d=b.data;
                void *e=malloc(sizeof(char));
                _asm
                {
                mov esi,c
                mov edi,d
                add esi,edi
                cmp esi,10
        jb m1
                sub esi,10
                mov e,1
        m1:
                mov a,esi
                }
                return bbyte(e);
        }
        bbyte &operator =(bbyte &b){
                void *c=this->data;
                void *d=b.data;
                _asm
                {
                mov esi,d
                mov c,esi               
                }
        return *this;
        }
 
        friend std::istream & operator >>(std::istream &is,bbyte &b)
        {
                is>>b.data;
        return is;
        }
        friend std::ostream & operator <<(std::ostream &os,bbyte &b){
                os<<(int)b.data;
        return os;
        }
};

main func

#include <iostream>
#include "bbyte.h"
using namespace std;
void main()
{
bbyte a,b,c;
cin>>a>>b;
c=add(a,b);
cout<<c;
cout<<a;
cout<<b;
}

ошибка такова переменную с закидывает мусором и потом не может удалить сам найти не могу

[netrino]

Запутанный, неэффективный, избыточный код... И что он делает? )

 friend std::ostream & operator <<(std::ostream &os,bbyte &b){
                os<<(int)b.data;

Вы выводите указатель, естественно он будет выглядеть как мусор

[breate]

Сообщение от netrino

Запутанный, неэффективный, избыточный код

необходимо реализовать класс BCD, предложи лучше

[netrino]

Сообщение от breate

необходимо реализовать класс BCD, предложи лучше

Не буду - Ваше задание )
Вы бы для начала разобрались с BCD, хоть на вики гляньте...
Ещё советую получше узнать про ассемблерные вставки, синтаксис и т.д.

        bbyte(void)
        {
        data=malloc(sizeof(char));
        void *a=data;
        _asm{
        mov esi,0
        mov a, esi
        }
        }

дошёл только до этого, дальше даже не смотрел. Этот код абсолютно лишён смысла. Во-первых Вы выделяете в куче 1 байт, зачем? Ну ладно, может нравится так, но потом Вы определяете указатель, который указывает на только что выделенную память. А потом записываете в указатель 0. Не в память по адресу, что в указателе, а в сам указатель. Более того, esi - 4 байта, а выделили Вы лишь 1
Чтобы положить значение по адресу, хранящемуся в указателе нужно писать так:

mov ebx, a
mov [ebx], esi

И тем не менее не вижу смысла использовать тут ассемблерную вставку.
Короче говоря: Ваша ошибка - класс bbyte, перепишите его правильно и всё будет работать.
На всякий случай...
http://ru.wikipedia.org/wiki/Двоично-десятичный_код

[breate]

Сообщение от netrino

mov ebx, a
mov [ebx], esi

огромное спасибо за этот фрагмент

на счет кода после того как дописал и отладил убедился что смысла в этом нет...