Вставка Assembler.Работа с символами. Проблема с пересылкой символа

[Jawner]

Доброго времени суток.
Пишем на C+ Builder 6
По заданию нужно постороить вторую строку символов из матрицы символов 3х4.

Код:

void __fastcall TForm1::Button1Click(TObject *Sender)
{
char s[3][4];
AnsiString c;
c.SetLength(4);
for(int i=0; i<3;i++)
for(int j=0; j<4;j++)
s[i][j]=Memo1->Lines->Strings[i][j+1];
asm
{
pusha
mov edi, dword ptr [s]
lea edi, [edi+4]
mov esi, dword ptr [c]
mov ecx,4
next:
mov al,byte ptr [edi]  <-- в этом месте вылетает ошибка(скрин прикрепил)
mov byte ptr[esi],al
lea edi, [edi+1]
inc esi
loop next
popa
}
Edit1->Text=c;
}

[FataLL]

lea edi, [edi+4]

Что вот здесь ты пытаешься сделать?

[Jawner]

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

lea edi, [edi+4]

Что вот здесь ты пытаешься сделать?

Смотри, допустим у меня матрица символов такая:
йцук
фыва
ячсм

в команде выше я пытаюсь перейти на символ "ф"

[FataLL]

Фактически, можно просто прибавить edi 4. Но ошибка в другом

mov edi, dword ptr [s]

Здесь ты загружаешь не адрес начала массива, как, видимо, хотел сделать, а двойное слово, находящееся по адресу s. Т.е., у тебя в edi загружаются коды символов "йцук" Будь проще

lea edi, s // правильный вариант

В случае с AnsiString всё правильно

mov esi, dword ptr [c] // адрес начала строки

Хотя угловые скобки ставить необязательно, они сбивают с толку...

[FataLL]

Кстати, зачем тебе ассемблер? Спортивный интерес?

AnsiString c = s[1];
c.SetLength( 4 );
Edit1->Text = c;

И никакого гимора...

[Jawner]

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

Фактически, можно просто прибавить edi 4. Но ошибка в другом

mov edi, dword ptr [s]

Здесь ты загружаешь не адрес начала массива, как, видимо, хотел сделать, а двойное слово, находящееся по адресу s. Т.е., у тебя в edi загружаются коды символов "йцук" Будь проще

lea edi, s // правильный вариант

В случае с AnsiString всё правильно

mov esi, dword ptr [c] // адрес начала строки

Хотя угловые скобки ставить необязательно, они сбивают с толку...

Действительно, поменял на lea и все нормально.
ТОлько я что то не понял, почему с AnsiString проблем не было, а c char проблемы ?
А насчет угловых скобок - насколько я понял в них указывается адрес, а в круглых само значение.

[Jawner]

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

Кстати, зачем тебе ассемблер? Спортивный интерес?

AnsiString c = s[1];
c.SetLength( 4 );
Edit1->Text = c;

И никакого гимора...

Ну вообще то мы изучаем ассемблер.
Еще если не сложно объясни пожалуйста зачем SetLength. Насколько я понял это я просто обрезаю строку до 4ех символов для результата, что бы не было всякого мусора из ОЗУ.
Еще момент с s[i][j]=Memo1->Lines->Strings[i][j+1];
зачем j+1 ? не могу понять.
А так же еще не совсем понимаю команды pusha и popa. Это загрузка из стека в флаги, а в конце выгрузка - зачем это делать ?

[FataLL]

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

ТОлько я что то не понял, почему с AnsiString проблем не было, а c char проблемы ?
А насчет угловых скобок - насколько я понял в них указывается адрес, а в круглых само значение.

AnsiString - это указатель на указатель. Указатель на экземпляр класса AnsiString, из которого надо взять указатель на строку. Поэтому, dword ptr с означает: "взять адрес, находящийся по адресу с". В случае с массивом ты имеешь просто указатель на начало области памяти 3х4.
Про скобки: ты же не на чистом ассемблере пишешь, а на встроенном. Сначала эти строки обрабатываются компилятором С++. Поэтому есть некоторые расхождения с чистым ассемблером. В данном случае, ставишь скобки или нет - код генерируется один. Почему так? Точнее не могу сказать, экспериментируй.

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

Еще если не сложно объясни пожалуйста зачем SetLength. Насколько я понял это я просто обрезаю строку до 4ех символов для результата, что бы не было всякого мусора из ОЗУ.

Всё правильно. Хотя и не совсем корректно, так как изначально мусора из ОЗУ может быть очень много.

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

Еще момент с s[i][j]=Memo1->Lines->Strings[i][j+1];
зачем j+1 ? не могу понять.

Без понятия. Вопрос в том, что хотелось сделать? В данном случае получается, что у тебя копируется 2, 3, 4 и 5 символ строки Memo. Я ж не знаю, что у тебя там было изначально. Но ясно одно, если в Memo строки короче 4 символов, то могут быть проблемы типа EAccessViolation...

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

А так же еще не совсем понимаю команды pusha и popa. Это загрузка из стека в флаги, а в конце выгрузка - зачем это делать ?

Эти две команды сохраняют (pusha) и восстанавливают (popa) содержимое всех регистров и состояние флагов в (из) стека. Видимо, это сделано для страховки, чтоб ничего не "испортить" для функции. На самом деле, без них можно обойтись, главное не испортить регистры bp и sp. Если предполагается менять их содержимое, то надо позаботиться о восстановлении, как было до входа в твой код.