danmilko / studyProgramming

A repository for homework for programming
0 stars 0 forks source link

asm #1

Open akudrinsky opened 5 years ago

akudrinsky commented 5 years ago

1) Немного не хватает описания работы функций, иногда - что обозначает та или иная переменная. Как бы разобраться можно, но читается не так легко. Как мне кажется, перед сдачей деду стоит добавить комменты с пояснениями (например, строка 103 ассемблера выглядит загадочно, если не знать, в каком порядке идут команды) 2) Очень красивое решение - преобразовывать файлы в char*, чтобы работать с командой как с единой структурой, мне это правда очень понравилось

Shishqa commented 5 years ago
  1. нигде не освобождается result
  2. функция могла бы возвращать что-нибудь (не зря же она не void) 65, 83. А точно надо делать аборт? можно же возвращать код ошибки
  3. ору
  4. может лучше объявить статический массив? этот res еще и не очищается - побеспокоился бы о памяти

динамика много где не очищается