Замена элементов в строке в ассемблерной вставке

@DarkLoS · Регистрация: 16.05.2015

Author24 — интернет-сервис помощи студентам

C++

void split_and_print(char s1[])
{
    _asm
    {
        xor ebx,ebx
            cld
            mov ecx,15
            mov al, '+'
            lea edi,s1
        m_beg:
        or al,0
            repne scasb
            jz m_replace
            jecxz m_end
            jmp m_beg
        m_replace:
        mov byte ptr[edi - 1], '-'
            jmp m_beg
            m_end:
    }
    cout << s1 << endl;
}
int main() 
{
    char s1[] = "+Iv+an+Iva+nov+";
    split_and_print(s1);
    system("pause");
    return 0;
}

Нужно было написать простенькую программу на с++ с ассемблерной вставкой. функция просто заменяет все знаки + на - и выводит получившуюся строку. Но она ничего не делает.Кто может,подскажите,в чём проблема

@R71MT · 10.03.2016, 05:39

Assembler

   mov   di,mess
   mov   cx,16
   mov   al,'+'
@@:
   repne scasb
   or    cx,cx
   jz    next
   dec   di
   mov   byte[di],'-'
   inc   di
   jmp   @b
next:

@Jin X · 06.04.2017, 10:51

R71MT, последний плюс не заменит такой код, т.к. там будет cx=0 (тем более, вопрос про 32 бита).

DarkLoS, потому что функции передаётся не строка, а ссылка на строку, т.е. надо так:

Assembler

9	mov edi,s1

И зачем ebx меняется? Вообще, ebx, esi, edi, ebp менять в Windows нежелательно (без сохранения/восстановления).

Ненужно никаких repne scasb здесь:

Assembler

            cld  // cld вообще уже и так должно быть выполнено (по соглашениям о вызовах), так что эту строку можно убрать
            mov esi,s1
        m_beg:
            lodsb
            test al,al
            jz m_end
            cmp al,'+'
            jne m_beg
            mov byte ptr [esi-1],'-'
            jmp m_beg
        m_end:

@liv · 06.04.2017, 11:15

DarkLoS[/nick], или так:

Assembler

void split_and_print(char s1[])
{
    __asm
    {
        mov ebx, s1
       m_beg:
        mov al,[ebx]
        inc ebx
        cmp al, 0
        je m_end
        cmp al, '+'
        jne m_beg
        mov byte ptr [ebx-1], '-'
        jmp m_beg
       m_end:
    }
    cout << s1 << endl;
}

[nick]
Jin X, регистры ebx, esi, edi сохраняются в стеке автоматически

@Jin X · 06.04.2017, 11:24

_liv_, да, или так. Так даже быстрее, lodsb работает медленнее (как и stosb, loop и даже repne scasb медленнее, чем через цикл).

Добавлено через 2 минуты

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

cmp al, 0

Вот на это заменять только смысла нет, это абсолютно равнозначно test al,al (и по скорости, и по размеру).

Добавлено через 1 минуту
Можно ещё заменить ebx на edx (если это как-то повлияет на сохранение ebx в стеке, т.е. отменит это).

@liv · 06.04.2017, 11:37

Jin X, кто бы спорил

Только каков смысл вылавливания даже не блох, а блошечек?

Разве что только ради стремления к идеалу...

@Jin X · 06.04.2017, 11:55

_liv_, в данном случае никакого. Забавы ради только

@Jin X Asm/C++/Delphi/Py/PHP/VBA 6747 / 1993 / 231 Регистрация: 14.12.2014 Сообщений: 4,186 Записей в блоге: 12
	06.04.2017, 11:24	5
	_liv_, да, или так. Так даже быстрее, `lodsb` работает медленнее (как и `stosb`, `loop` и даже `repne scasb` медленнее, чем через цикл). Добавлено через 2 минуты Сообщение от _liv_ cmp al, 0 Вот на это заменять только смысла нет, это абсолютно равнозначно `test al,al` (и по скорости, и по размеру). Добавлено через 1 минуту Можно ещё заменить `ebx` на `edx` (если это как-то повлияет на сохранение ebx в стеке, т.е. отменит это). 2

@liv 5118 / 4557 / 854 Регистрация: 07.10.2015 Сообщений: 9,462
	06.04.2017, 11:37	6
	Jin X, кто бы спорил Только каков смысл вылавливания даже не блох, а блошечек? Разве что только ради стремления к идеалу... 0

@Jin X Asm/C++/Delphi/Py/PHP/VBA 6747 / 1993 / 231 Регистрация: 14.12.2014 Сообщений: 4,186 Записей в блоге: 12
	06.04.2017, 11:55	7
	_liv_, в данном случае никакого. Забавы ради только 1