UIButtonのsetImageを使うと別のUIButton位置が初期化される

Question

前提・実現したいこと

Swift初心者です。
別個の画像がアサインされた複数のUIButtonが、
ランダムに配置されている状態を初期状態として、
①右方向へ同速度で移動させ
②画面から消えたらランダムに選択した画像をアサインし
③左側から再度登場し
④また右方向へ移動
という事を各UIButtonが独立に繰り返している、
という実装をしたいと考えています。

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

②のランダムに選択した画像をアサインする際にsetImageを用いると、
②の条件を満たしたUIButtonの画像がランダム選択されたものになるまでは良いですが、
各UIButtonの位置が初期状態となってしまいます。
前述の通り各UIの動作が独立してほしいため、
一つのUIButtonが②の条件を満たしたからと言って、
他のUIButtonの位置を勝手に変更して欲しくありません。
この各UIButtonの位置が初期状態となってしまう現象を解消できる方法があれば
ご教授いただければと思います。

該当のソースコード

class FruitContentsViewController: UIViewController {
    @IBOutlet var moveFruitCollection: [UIButton]!
    override func viewDidLoad() {
        super.viewDidLoad()
        let xbound = 100.0
        let s      = CGPoint(x: -xbound, y: 0)
        let e      = CGPoint(x:  xbound, y: 0)
        let i      = CGPoint(x:  5.0,    y: 0)
        // animation
        for moveFruit in self.moveFruitCollection {
            Timer.scheduledTimer(withTimeInterval: 0.01, repeats: true) { (timer) in
                moveFruit.center.x += i.x
                if moveFruit.center.x >= (self.view.bounds.size.width+e.x) {
                    moveFruit.center.x  = s.x
                    self.changeImage( moveFruit: moveFruit)
                }
            }
        }
    }
    func changeImage( moveFruit: UIButton!){
        let image_list      = ["f01”,”f02”,”f03”,”f04”,”f05”,”f06"]
        let index           = (Int)(arc4random_uniform(6))
        let fruit_uiimage   = UIImage(named:image_list[index])!
        moveFruit.setImage(fruit_uiimage, for: .normal)
    }
}

試したこと

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

Xcode10.1を用いています。

Answer 1

コードを見たところ、正しく動作しそうな気がするので、そのままコピペしてビルド/実行して見たところ、

let image_list      = ["f01”,”f02”,”f03”,”f04”,”f05”,”f06"]

このコードの中に書かれてある ”が、一部全角の "になっているため、ファイル名の配列が正しく定義されておらず、実行時にエラーが発生しました。

そこだけ修正して実行したところ、意図通りの動作をしており、別のUIButton位置が勝手に初期化されるようなことは発生せず、各ボタンとも画像を切り替えながら左端から右端へのスクロールを繰り返していました。

問題が起きているコードは、本当に掲載されているコードと同じですか?
同じはずであれば、ここに書かれていないこと(Storyboardの接続等)を見直してみるべきだと思います。
また、自分で新しいプロジェクトを作成し、質問に掲載したコードをコピペして、意図通りの動作をするかどうか確認してみると良いと思います。
それで意図通り動作するものが作れたら、問題のプロジェクトと何が違うか比較して問題点を検証できると思います。

Comments

[math0point]

ご回答ありがとうございます。またお礼が遅れまして申し訳ございません。

クォーテンションの誤記、失礼しました。
ただ、コードは正しいとのこと、安心しました。

デバッグを進めた、意図通りに動く事を確認しました。
全UIButtonに"Add missing constraints"で制約を付けていたのですが、
これをclearする事で位置が初期状態に戻るという意図しない現象が解消しました。

制約は機種間のボタン表示の差（大きさや位置のズレ）を解消するために必要、
と覚えていたのですが（これが勘違いかもしれませんが）、
上記のようにその制約が意図しない動作に起因していたようです。
ただ機種間のボタン表示の差は無くしたく、意図通りに動作との両立をさせるために、
もしご知見があればご教示頂けないでしょうか？

[TakeOne]

質問に掲載したコードは `moveFruit.center`を設定することでボタンを位置決めして移動しているのに、AutoLayoutも使っていたら、せっかくcenter指定で移動させたボタンがAutoLayoutの設定通り初期状態に戻ってしまうことになると思います。

移動処理をcenter指定で位置決めするなら、AutoLayoutは使わず、初期位置や初期サイズは画面サイズを元に適切に計算したものを最初にframe指定で設定しておくべきと思います。

AutoLayoutを使いたいなら、center指定で移動させるのではなく、AutoLayoutの制約値をコードで変更することでボタンの位置決めを変更して移動させるべきだと思います。

[math0point]

ご丁寧にご回答いただき、ありがとうございました。
大変参考になりました。