画像をシャッフルして表示したら、同じ画像が出てきてしまいます。

Question

前提

ここに質問の内容を詳しく書いてください。
swiftで画像をボタンを押すと、ランダムに表示させるアプリを作ろうと思っています。
画像の配列を.shuffledでシャッフルするとなぜか同じ画像が出てきてしまいます。
さらに画像の欄がたまに空白になってしまいます。
もしよろしければよろしくお願いします。

実現したいこと

画像を重複なくランダムに表示する。

発生している問題・エラーメッセージ

画像の重複と、空白

該当のソースコード

swift
ソースコード
import UIKit

class ViewController: UIViewController {

@IBOutlet weak var imageView: UIImageView!

@IBAction func action(_ sender: Any) {
    
    dispImageNo += 1
    
    if dispImageNo < 5{
        displayImage()
    }else if dispImageNo > 5{
         performSegue(withIdentifier: "toranpu", sender: nil)

           }
 

}


   var dispImageNo = 0


func displayImage(){
    
   
    let imageNameArray = [
        "image01",
        "image02",
        "image03",
        "image04",
        "image05"
    ]
    
    
    
    
    
    let testtest = imageNameArray.shuffled()
    let name = testtest[dispImageNo]
    
    
   
    let image = UIImage(named: name)
    
  
    imageView.image = image}
   



override func viewDidLoad() {
    super.viewDidLoad()
    let image = UIImage(named: "image01")
    imageView.image = image
    

試したこと

removeについて調べましたが、有意な情報が見つかりませんでした。

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

初心者ですが、よろしくお願いします。

Comments

[hoshi-takanori]

表示のたびに配列をシャッフルしたら、かぶるでしょうね…。

[ip33845586]

回答ありがとうございます！表示のたびということは、どこをシャッフルすれば値が被らないのでしょうか？もしよろしければ教えていただきたいです。

Answer 1

次の内容でどうでしょうか？
申し訳ありませんが、動作確認はしておりません。

また、UIImage(named: name) は
メモリリークの可能性があるため
UIImage(contentsOfFile: imagePath) の使用を検討しても良いかもしれません。

//A → B → A の順番を許容するパターン

@IBOutlet weak var imageView: UIImageView!

@IBAction func action(_ sender: Any) {

    var targetNum = Int.random(in: 0..<5)
    while(targetNum == dispImageNo){
        var targetNum = Int.random(in: 0..<5)
    }
    
    dispImageNo = targetNum
    displayImage()
}


   var dispImageNo = 0

func displayImage(){
    
    let imageNameArray = [
        "image01",
        "image02",
        "image03",
        "image04",
        "image05"
    ]
    
    //let testtest = imageNameArray.shuffled() //コメントアウト
    //let name = testtest[dispImageNo] //コメントアウト
    let name = imageNameArray[dispImageNo] //追加
    let image = UIImage(named: name) //追加
    imageView.image = image
}

override func viewDidLoad() {
    super.viewDidLoad()
    //let image = UIImage(named: "image01") //コメントアウト
    //imageView.image = image //コメントアウト

    let imageNameArray = [
        "image01",
        "image02",
        "image03",
        "image04",
        "image05"
    ] //追加
    var dispImageNo = Int.random(in: 0..<5) //追加
    let name = imageNameArray[dispImageNo] //追加
    let image = UIImage(named: name) //追加
    imageView.image = image //追加
}

//A → B → A の順番を許容しないパターン

@IBOutlet weak var imageView: UIImageView!

@IBAction func action(_ sender: Any) {
    
    dispImageNo += 1
    if dispImageNo < 5{
        displayImage()
    }else if dispImageNo >= 5{ // 5の時変化しなくなるため、大なりイコールに修正
         performSegue(withIdentifier: "toranpu", sender: nil)
    }
}


   var dispImageNo = 0
   var testtest: [String] = []

func displayImage(){
    
   /* コメントアウト
    let imageNameArray = [
        "image01",
        "image02",
        "image03",
        "image04",
        "image05"
    ]
    */
    
    //let testtest = imageNameArray.shuffled() //コメントアウト
    let name = testtest[dispImageNo]
    let image = UIImage(named: name)
    imageView.image = image
}

override func viewDidLoad() {
    super.viewDidLoad()
    //let image = UIImage(named: "image01") //コメントアウト
    //imageView.image = image //コメントアウト

    let imageNameArray = [
        "image01",
        "image02",
        "image03",
        "image04",
        "image05"
    ] //追加

    testtest  = imageNameArray.shuffled() //追加
    let name = testtest[dispImageNo] //追加
    let image = UIImage(named: name) //追加
    imageView.image = image //追加
}