Homework 1

[ghost]

No description provided.

[megavolt605]

приветствую! меня выбрали для оценки Вашей работы.

вот мои замечания:

1. при компиляции компилятор выдает предупреждение о незаконченности категории:
   osx-project-1 Group
   /.../osx-project-1-master/osx-project-1/osx-project-1/Cell+mutator.m
   /.../osx-project-1-master/osx-project-1/osx-project-1/Cell+mutator.m:11:17: Incomplete implementation

это связано с тем, что Вы оставили "лишние" декларации свойства и метода в интерфейсной части категории. я предполагаю что это просто классическая ошибка копи+паста. :)

@Property NSMutableArray* DNA;

• (id) init;
• (int) hammingDistance:(Cell*) otherCell;

они не нужны в интефейсе категории.

2. Cell+mutator.m (13):

• (void) mutate:(int)percent

алгоритм мутации с использованием поиска еще неотмутировавшего элемента алгоритмически верен, но неэффективен. (не линеен, зависит от случая). я бы рекомендовал создать массив из индексов всех элементов, выбирать оттуда случайный, мутировать его, удалять из массива индексов.

3. я бы посоветовал заменить код в Cell.m(21)
   NSInteger i = 0;
   while(i<100)
   {
   ...
   i++;
   }
   на конструкцию for - более очевидно для читающего Ваш код, что описан итерационный цикл. тоже самое и в Cell+mutator.m(24). ну и если уж вынесли размер днк в #define (Cell.h(10)), то почему бы его тут не использовать? :)

4. Cell.m (26) и Cell+mutator.m(43): очень схожий код, я бы рекомендовал вынести его в отдельный метод объекта.

5. практически отсутствие комментариев в коде.

не сочтите за придирки, просто я постарался изложить свое мнение по Вашему коду.

удачи!

[ghost]

Прикольно, а я оценивал вашу версию)

1. исправил
2. исправил

№3 исправлю, если будет время)