条件を付けてシャッフルさせる関数で時々Fatal errorが出る

Question

前提・実現したいこと

配列をシャッフルして先頭を取り除き、全て無くなればもう１度シャッフルする。
しかし、条件として前回の配列の末尾と新しい配列の先頭と二番目が同じものでなくなるまで繰り返すという関数を作っています。

問題ないときはうまくシャッフルするのですが、時々Fatal error: Index out of rangeが出ます。

whileの中に
sa4 = shuffledArray[4]
rs0 = reShuffledArray[0]
rs1 = reShuffledArray[1]
を書いているのは元々はエラーが出る代わりに while ループから抜け出せないということがあった為です。

このエラーを出さないためにはどのようにすればよいかご教示いただければと思います。

該当のソースコード

var strArray = ["Apple", "Google", "Facebook", "Instagram", "Twitter"]

func shuffle() {
    
    var shuffledArray = strArray.shuffled()
    var sa4 = shuffledArray[4]
    
    for _ in shuffledArray {
        print(shuffledArray)
        shuffledArray.remove(at: 0)
    }
    
    var reShuffledArray = strArray.shuffled()
    var rs0 = reShuffledArray[0]
    var rs1 = reShuffledArray[1]
    
    while rs0 == sa4 || rs1 == sa4 {
        _ = reShuffledArray.shuffled()
        
        sa4 = shuffledArray[4]
        rs0 = reShuffledArray[0]
        rs1 = reShuffledArray[1]
    }
    print(reShuffledArray)
}

shuffle()

補足情報（FW/ツールのバージョンなど）

x code 10.2.1 swift5.0.1 macOs Mojave 10.14.5

mode_edit編集
arrow_downward自己解決方法を記入する

ここにより詳細な情報を記載してください。

Comments

[TsukubaDepot]

次ご質問の際は、是非ソースコードのマークダウン機能をご利用ください。
ソースコードを```で囲むと綺麗に表示されます（範囲選択し、ボタンを押すのでも構いません）。

Answer 1

これはXY問題と思われますので僕が思いついたものを。

以下のstruct群は配列を与えるとその配列からランダムにかつ重複せずに要素を取り出し、全要素を取り出し終えるとまた初めから同じように無限に繰り返すものです。

struct RepeatingRandomIterator<T>: IteratorProtocol {
  private let base: [T]
  private var indexes: [Int] = []
  
  init(base: [T]) {
    self.base  = base
  }
  
  
  mutating func next() -> T? {
    
    if indexes.isEmpty {
      indexes = (0..<base.count).shuffled()
    }
    
    let index = indexes.removeFirst()
    
    return base[index]
  }
}

struct RepeatingRandomQueue<T>: Sequence {
  private let base: [T]
  
  init(_ base: [T]) {
    self.base  = base
  }
  
  func makeIterator() -> RepeatingRandomIterator<T> {
    .init(base: base)
  }
}

使い方としては回数が決まっている場合は

let rrq = RepeatingRandomQueue([0, 1, 2])

zip(0..<10, rrq).forEach { print($0.1) }

こんな感じですね。
無限に繰り返されるため、回数指定は必須です。

あるいはミュージックプレイヤーやルーレットのようなものを想定するならRepeatingRandomIteratorだけを使って

class Player {
  private var iter = RepeatingRandomIterator(base: [0, 1, 2])
  private var isStopped = true
  
  func play() {
    guard isStopped else { return }
    
    isStopped = false
    
    DispatchQueue(label: "Player")
      .async { [weak self] in
        while let m = self?.iter.next(), self!.isStopped {
          print(m)
        }
    }
  }
  
  func stop() {
    isStopped = true
  }
}

こんなクラスを作りますね。

Playgroundで試してみるのなら

let p = Player()
p.play()
sleep(2)
p.stop()
print("Stopped")

のような感じで。

Comments

[sssaqj]

ご忠告ありがとうございます。自分ではまだすぐに理解できない内容だったため新しい表現の仕方を覚えることができました。ただ、残念なことに希望した条件を達成することはできませんでした。

Answer 2

落ちる原因は存在しない要素にアクセスしているためです（Index out of range = 添字が範囲外にある）。
shuffledArrayはremove()で自身の要素を減らしているため、最終的には空配列になってしまいます。

また、shuffled()とshuffle()の扱いに混乱されている感じがします。
配列自身をシャッフルするのがshuffle()で、シャッフルした配列を返すのがshuffled()です。

なので、おそらくこのような感じに修正すればいいのではないでしょうか（これで動きますが、最終的に望まれている形なのかはわかりません）。

while rs0 == sa4 || rs1 == sa4 {
    // 配列自身をシャッフルする
    reShuffledArray.shuffle()

    // shuffledArray は既に空配列なので、4番目の要素は存在しない。
    //sa4 = shuffledArray[4]
    rs0 = reShuffledArray[0]
    rs1 = reShuffledArray[1]
}

Comments

[sssaqj]

ありがとうございました。ご指摘の通り変更することでエラーもなくなり、無限ループに陥ることもなく、希望した条件で作動するようになりました。