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

m'mrm'mlrr DNA project #117

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

m'mrm'mlrr DNA project #117

wants to merge 7 commits into from

Conversation

mmrmmlrr
Copy link

@mmrmmlrr mmrmmlrr commented Nov 3, 2012

No description provided.

@Sachs
Copy link

Sachs commented Nov 3, 2012

  • Мутацию элемента ДНК можно сделать без лишнего перебора в while
  • Категорию лучше вынести в отдельные файлы (New->File->"Objective-C category", там же выбрать от какого класса категория и как её назвать)

Теперь нуклеотид для мутации не выбирается случайно "до победы", а
выбирается "наверняка" из усеченного массива.

Также вынес категорию в отдельный файл. Красиво!
@Sachs
Copy link

Sachs commented Nov 5, 2012

Каждый раз в цикле создавать усечённый массив... ну вообще так себе решение, если представить, что массив в реальной задаче может быть не из 4х элементов, да и цепочка ДНК длиной явно не 100, тогда эта лишняя операция с памятью съест очень много времени в сумме

@Sachs
Copy link

Sachs commented Nov 5, 2012

Вместо random(), посмотри описание функций arc4random(void) и arc4random_uniform(u_int32_t upper_bound), последняя выбирает случайное из диапазона [0, upper_bound - 1]; они не требуют рандомайза

@Sachs
Copy link

Sachs commented Nov 5, 2012

(Не по теме кода) у тебя ссылка в "Assignment" ведёт на коммит, а не на пулл реквест, поправь

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.

None yet

3 participants