Как-то не назначая класс с Ruby

Question

Во время выполнения мой код часто сталкивается с неопределенной ошибкой метода для метода mate, Насколько я могу понять, Person каким-то образом проскальзывает сквозь трещины, когда-то по ходу выполнения кода, и удается не иметь allele назначен на это.

Код (отказ от ответственности, не самый лучший формат):

class Allele
  attr_accessor :c1, :c2

  def initialize(c1, c2)
    @c1 = c1
    @c2 = c2
  end

 #formats it to be readable
 def to_s
  c1.to_s + c2.to_s
 end

 #given Allele a
 def combine(a)
  pick = rand(4)
  case pick
   when 0
    Allele.new(c1,a.c1)
   when 1
    Allele.new(c1,a.c2)
   when 2
    Allele.new(c2,a.c1)
   when 3
    Allele.new(c2,a.c2)
  end
 end
end

class Person
 attr_accessor :allele, :male

 def initialize(allele,male)
    @allele = allele
  @male= male
  end

 #def self.num_people
  #@@num_people
 #end

 def to_s
  "Allele:" + allele.to_s + " | Male:" + male.to_s
 end

 def male
  @male
 end

 def allele
  @allele
 end

 def mate(p)
  if rand(2) == 0
   Person.new(allele.combine(p.allele),true)
  else
   Person.new(allele.combine(p.allele),false)
  end
 end
end

male_array = Array.new
female_array = Array.new
male_child_array = Array.new
female_child_array = Array.new

# EVENLY POPULATE THE ARRAY WITH 5 THAT PHENOTYPICALLY MANIFEST TRAIT, 5 THAT DON'T
# AND 5 GIRLS, 5 GUYS
pheno_dist = rand(5)
#make guys with phenotype
pheno_dist.times { male_array << Person.new(Allele.new(1,rand(2)),true) }
#guys w/o
(5-pheno_dist).times { male_array << Person.new(Allele.new(0,0),true) }
#girls w/ pheno
(5-pheno_dist).times { female_array << Person.new(Allele.new(1,rand(2)),false) }
#girls w/o
pheno_dist.times { female_array << Person.new(Allele.new(0,0),false) }

puts male_array
puts female_array
puts "----------------------"

4.times do
 #mates male with females, adding children to children arrays. deletes partners as it iterates
 male_array.each do
  male_id = rand(male_array.length) #random selection function. adjust as needed
  female_id = rand(female_array.length)
  rand(8).times do
   child = male_array[male_id].mate(female_array[female_id])
   if child.male
    male_child_array << child
   else
    female_child_array << child
   end
  end
  male_array.delete_at(male_id)
  female_array.delete_at(female_id)
 end

 #makes males male children, females female children, resets child arrays
 male_array = male_child_array
 female_array = female_child_array
 male_child_array = []
 female_child_array = []

 puts male_array
 puts female_array
 puts "----------------------"
end

Что сразу выглядит не так?

Answer 1

Как говорит egosys, вам не следует удалять из массива, по которому вы выполняете итерацию.

Другая проблема в вашем цикле, который запускает "4.times do". Иногда женский массив пуст, поэтому возвращает размер 0; rand(0) - это случайное число с плавающей запятой>= 0 и < 1. Использование этого в качестве индекса массива для пустого female_array возвращает nil, который затем передается mate.

Но есть нечто большее, чем это неправильно. Вы перебираете male_array, используя каждый, но затем выбираете мужчину наугад. Это позволяет некоторым мужчинам спариваться более одного раза; других нет совсем. Точно так же, некоторые женщины достигают спаривания и размножаются более одного раза в каждой итерации, другие - совсем нет. Это твое намерение?

Давайте сначала рассмотрим вопрос о сохранении всех мужчин и женщин в одном массиве. Это упростит вещи. Однако, поскольку вам иногда нужно найти всех мужчин, а иногда и всех женщин, мы разработаем для этого следующие методы:

def males(population)
  population.find_all do |person|
    person.male?
  end
end

def females(population)
  population.find_all do |person|
    person.female?
  end
end

Было бы более биологически точным, если мужчины и женщины должны быть в паре случайным образом, но никто не может спариваться более одного раза. Это довольно просто:

def random_pairs(males, females)
  males.shuffle[0...females.size].zip(females.shuffle)
end

Тогда воспроизводство населения становится, просто

def make_children(male, female)
  # return an array of children
end

def reproduce(population)
  children = []
  random_pairs(males(population), females(population)).each do |male, female|
    children << make_children(male, female)
  end
  children
end

Имея такие функции, выполнить 4 цикла воспроизведения будет так просто:

people = # ... generate your initial list of people of all sexe.
4.times do
  people = reproduce(people)
end

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

Больше можно сделать в ОО-стиле, например, сделать Population объектом первого класса и переместить в него функции "мужчины", "женщины", "random_pairs" и "размножаться". Я оставлю это как упражнение для читателя.

Answer 2

Удаление из массива, с которым вы итерируете, имеет неопределенное поведение. Обычно совет состоит в том, чтобы использовать Array#delete_if, но я не уверен, как бы вы использовали его в этом случае.