Gxoct request try number 1

[GxocT]

Gxoct request try number 1

[Sachs]

1. Зачем каждый раз заново создавать atcgArray?
2. NSString *randomAcid = [atcgArray objectAtIndex:arc4random_uniform(4)];
   может дать индекс равный существующему, а в задании сказано, что он должен его обязательно изменить

[GxocT]

1. Зачем каждый раз заново создавать atcgArray?

Создал (readonly) property.

1. NSString *randomAcid = [atcgArray objectAtIndex:arc4random_uniform(4)];
   может дать индекс равный существующему, а в задании сказано, что он должен его обязательно изменить

Этот рэндом берёт из массива одну из аминокислот для того, чтобы заменить в существующем ДНК.
В задании же сказано, что нельзя дважды менять аминокислоту в одном и том же месте. То есть я могу кислоту "T" поменять на кислоту "Т", но не могу заменить кислоту на позиции 15 два раза.

[GxocT]

Ок. Исправлю.
Спасибо.

[Sachs]

То есть я могу кислоту "T" поменять на кислоту "Т"

да, из формулировки задания это не очевидно, но в комментариях к заданию автором было сказано, что ИЗМЕНИТЬ - значит именно изменить на другое значение, иначе слово изменить теряет свой смысл)

а вообще хороший код написал, очень аккуратно и читаемо

[GxocT]

Внёс изменения. Нуклеотиды проверяются на равенство. Если они совпадают, генерируется новый нуклеотид.

[Sachs]

@GxocT следующий шаг - сделать то же самое в одно действие, без лишнего перебора в while)

[GxocT]

Готово. Без While.

[Sachs]

arc4random_uniform(3) может выдать ноль, то есть элемент не изменится, что тоже не верно) должно быть случайное число из диапазона от 1 до count - 1

[GxocT]

arc4random_uniform(3) выбирает значение из укороченного массива ATCG

Например, в заданной позиции был нуклеотид "T".
Берём массив ATCG и исключаем из него T, получаем новый массив ACG.
Далее функция arc4random_uniform(3) выбирает индекс из значений 0, 1 или 2.
То есть выбирается нуклеотид A, C или G.

[Sachs]

да, просмотрел) но проще же взять элемент из того же массива со сдвигом [1, count - 1] от текущего, а не плодить дополнительные массивы для каждого варианта?

[HexletStudent13]

После того как получили self от родителя, необходимо проверить не вернули ли нам nil и после этого можно уже инициализировать.
self = [super init];
if(self){
//my init
}
return self;

[GxocT]

init исправил

[GxocT]

Sachs
Да понял о чём идёт речь, но оставил так как сделал, мне данный вариант кажется более легким для восприятия.

[HexletStudent13]

percentToMutate - это процент, поэтому надо посчитать количество ячеек, которые нужно заменить. А сейчас значение этой переменной используется как количество ячеек.

[GxocT]

В примере используется ДНК длиной в 100 знаков, соответственно процент это количество ячеек, зачем усложнять код в конкретной ситуации? Мы ведь не пишем универсальную программу.
Если руководствоваться принципами экстремального программирования подобные изменения вредны для кода.

[GxocT]

Внёс изменения в mutate для того, чтобы использовались проценты, а не количество клеток, также добавил проверку, чтобы проценты не были больше 100 и меньше 0.

[GxocT]

Внёс изменения в mutate для того, чтобы использовались проценты, а не количество клеток, также добавил проверку, чтобы проценты не были больше 100 и меньше 0.

[GxocT]

Внёс изменения в mutate для того, чтобы использовались проценты, а не количество клеток, также добавил проверку, чтобы проценты не были больше 100 и меньше 0.