Skip to content
This repository has been archived by the owner on Jan 12, 2022. It is now read-only.

Project-1-DNA #416

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Project-1-DNA #416

wants to merge 2 commits into from

Conversation

evlogiy
Copy link

@evlogiy evlogiy commented Nov 13, 2012

No description provided.

- (int) getHammingDistance: (Cell *) cell { //получить дистанцию хамминга
int count = 0;
for (int i=0; i<100; i++) {
if ([self->DNA objectAtIndex:i]==[cell->DNA objectAtIndex:i]) count++;

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

У вас неправильно реализован hammingDistance, в описании написано: "возвращать количество позиций где символы ДНК не совпадают." , в вашем случае же возвращается количество позиций где ДНК совпадают. Вместо == нужно было написать !=
Второй момент которым вы меня поставили в тупик это этот код [self->DNA objectAtIndex:i] я уже некоторое время программирую на Objective C и впервые встречаю такую запись. Насколько я понимаю вы таким образом напрямую обращаетесь к переменной. С точки зрения OOP это не является верным и все обращения к переменной должны проходить через getter и setter. Вам стоило объявить @Property NSArray *DNA; в header файле и тогда вы смогли бы обращаться к переменной через self.DNA или cell.DNA

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants