質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.47%
iPad

iPadは、Appleがデザインしたタブレット型コンピュータです。iPadアプリケーションは通常Xcode IDEのObjective-Cで書かれますが、iPadアプリケーションを組むためのほかのツールを使うことも可能です。

Xcode

Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

Swift

Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

Q&A

0回答

633閲覧

システムが途中で止まる理由を解決したい

kikizi

総合スコア5

iPad

iPadは、Appleがデザインしたタブレット型コンピュータです。iPadアプリケーションは通常Xcode IDEのObjective-Cで書かれますが、iPadアプリケーションを組むためのほかのツールを使うことも可能です。

Xcode

Xcodeはソフトウェア開発のための、Appleの統合開発環境です。Mac OSXに付随するかたちで配布されています。

Swift

Swiftは、アップルのiOSおよびOS Xのためのプログラミング言語で、Objective-CやObjective-C++と共存することが意図されています

0グッド

0クリップ

投稿2020/11/22 16:29

編集2020/11/23 02:28

前提・実現したいこと

現在私が開発しているシステムはボタンを押すことで表示している画像の座標をprintし、次の画像へと移り変わっていくものなのですが、121点の座標があるのに必ず119点で以下のエラーが発生してアプリが止まります。
何故121点全て表示されないのでしょうか。
試したことに書いてある内容通り121に行く前にエラーが出ます。
調べても全く該当するものが見つからなくXcodeの経験の浅い私にはどうしようも無かったので質問させていただきました。ご教授いただけると幸いです。
補足:ボタンのコードでcounterで場合わけしているのは自分のシステムにはスタートボタンがなく最初の1点目を表示するのに同じボタンを使用しているためです。

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

エラーメッセージ Thread 1: Fatal error: Index out of range

該当のソースコード

import UIKit import AVFoundation class ViewController: UIViewController , AVAudioPlayerDelegate { var counter = 0 var i = 1 var myImageView1: UIImageView! var myImageView2: UIImageView! override func viewDidLoad() { super.viewDidLoad() myImageView1 = UIImageView(frame: CGRect(x:-50,y:-50,width:5.195*2.2567,height:5.195*2.2567)) myImageView2 = UIImageView(frame: CGRect(x:-50,y:-50,width:5.195*2.2567,height:5.195*2.2567)) // 表示する画像を設定する. let myImage1 = UIImage (named:"black@2x.png") let myImage2 = UIImage (named:"blue@2x.png") // 画像をUIImageViewに設定する. myImageView1.image = myImage1 myImageView2.image = myImage2 // UIImageViewをViewに追加する self.view.addSubview(myImageView1) self.view.addSubview(myImageView2) let position0 = CGPoint(x:300, y:300) myImageView1.layer.position = position0 } let position1 = CGPoint(x:581.266, y:245.323) let position2 = CGPoint(x:608.538, y:245.323) let position3 = CGPoint(x:635.810, y:245.323) ~ let position120 = CGPoint(x:826.713, y:518.043) let position121 = CGPoint(x:853.985, y:518.043) lazy var positions2 = [position1,position2,~~~,position120,position121].shuffled() @IBAction func No(_ sender: Any) { if counter == 0{ myImageView2.layer.position = positions2[i] counter = counter + 1 }else if counter >= 1{ print(i,positions2[i].x,"\t",positions2[i].y) i = i + 1 myImageView2.layer.position = positions2[I]//エラー分が出た箇所 counter = counter + 1 } } }

試したこと

for in , while のループ関数をボタンの中に入れてみたが、ボタンを一回押すと勝手にループする上、点の数も結局足りなかった。
終了条件を121に設定してもそもそもその値に到達しない。

終了条件を設定してもその値に到達する前に同様エラーが出ます。

点の数を増減した感じ、-2個の点が表示されませんでした。

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

miyabi_takatsuk

2020/11/22 19:50 編集

Thread 1: Fatal error: Index out of range はだいたいは、配列や辞書変数から、無効なインデックスで値を取り出そうとしている時のエラーです。 i変数は定義してますか? 定義している構文がないので、このままでは回答しようがありません。
YT0014

2020/11/23 00:12

無限ループなんですが、終了条件がないのが正しいのでしょうか? その場合、点がいくつであろうと、ご提示のエラーが出ることは確定しているのですが。
kikizi

2020/11/23 02:24

iの定義が書いてある部分の記載を忘れていました。申し訳ないです。 また終了条件を書いてもその値に到達する前にエラーが起きます。 Fatal error: Index out of rangeで調べてvar correctAnswer = Int(repeating: 0/*初期値*/, count: 10/*必要な要素数*/)  このような構文も試してみましたが、Extra argument 'i' in callとエラーが発生しこちらを調べたのですが、よくわからず自力では解決できませんでした。 何卒ご助力願います。
errolizer

2020/11/23 14:33

i が 1からスタートしていることがまず異常です。 さらに i をインクリメントした「後」に配列の添字として使用しているため、out of range になるのは自明です。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

まだ回答がついていません

会員登録して回答してみよう

アカウントをお持ちの方は

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.47%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問