コンパイルは通るのに動作せずアプリが落ちてしまいます、、

Question

###前提・実現したいこと
textviewに加速度値のデータを少数３桁までだけ表示させたいのですが、うまくいきません
コンパイルは通るしクリーンしてもエラーが消えずどこをどう直していいかわかりません

###発生している問題・エラーメッセージ
acceleration CMAcceleration
self Mytest5.ViewController 0x000000010350d640
UIKit.UIViewController UIViewController
motionManager CMMotionManager 0x000005a10248d4cd
dataX [Double]
dataY [Double]
dataZ [Double]
accelX UILabel! 0x000005a10248d4cd
accelY UILabel! 0x000005a10248d4cd
accelZ UILabel! 0x000005a10248d4cd
TextView UITextView! 0x000005a10248d4cd

###該当のソースコード

import UIKit
import CoreMotion

class ViewController: UIViewController {

    let motionManager = CMMotionManager()
    var dataX:[Double] = []
    var dataY:[Double] = []
    var dataZ:[Double] = []
    
    @IBOutlet weak var accelX: UILabel!
    @IBOutlet weak var accelY: UILabel!
    @IBOutlet weak var accelZ: UILabel!
    @IBOutlet weak var TextView: UITextView!
    
    override func viewDidLoad() {
        super.viewDidLoad()
        if motionManager.isAccelerometerAvailable {
            motionManager.accelerometerUpdateInterval = 0.2
            
            motionManager.startAccelerometerUpdates(
                to: OperationQueue.current!,
                withHandler: {(accelData: CMAccelerometerData?, errorOC: Error?) in
                    self.outputAccelData(acceleration: accelData!.acceleration)
            })
            
        }
    }
    func outputAccelData(acceleration: CMAcceleration){
        accelX.text = String(format: "x = %.4f", acceleration.x)
        accelY.text = String(format: "y = %.4f", acceleration.y)
        accelZ.text = String(format: "z = %.4f", acceleration.z)
        
        dataX.append(acceleration.x)
        dataY.append(acceleration.y)
        dataZ.append(acceleration.z)
        
        TextView.text = String(format: "%.3f %.3f %.3f \n",dataX,dataY,dataZ)
        
        if ((acceleration.x >= 1.0)||(acceleration.x <= -1.0)) {
            accelX.textColor = UIColor.red }
        else if((acceleration.x <= 1.0)||(acceleration.x >= -1.0)) {
            accelX.textColor = UIColor.black }
        if((acceleration.y >= 1.0)||(acceleration.y <= -1.0)) {
            accelY.textColor = UIColor.red }
        else if((acceleration.y <= 1.0)||(acceleration.y >= -1.0)) {
            accelY.textColor = UIColor.black }
        if((acceleration.z >= 1.0)||(acceleration.z <= -1.0)) {
            accelZ.textColor = UIColor.red }
        else if((acceleration.z <= 1.0)||(acceleration.z >= -1.0)) {
            accelZ.textColor = UIColor.black }
        
    }
    func stopAccelerometer(){
        if (motionManager.isAccelerometerActive) {
            motionManager.stopAccelerometerUpdates()
        }
    }
    override func didReceiveMemoryWarning() {
        super.didReceiveMemoryWarning()
    }

}

###試したこと
ビルド前にクリーンしてみた

###補足情報(言語/FW/ツール等のバージョンなど)

Answer 1

dataX.append(acceleration.x)
ではデータが配列の要素として追加されどんどん要素が増加していってしまいます

dataX.append(acceleration.x)以下をコメントアウトさせてみてください
それでエラーにならなければ、もう一度なにをしたいのか考え直して
修正してみてはどうでしょう

Comments

[SKMT]

回答ありがとうございます。要素を追加していってそのデータとして表示させたいんですけどそういう場合はどうしたらいいですか？最終的には配列に格納したデータをcsvなどにしてパソコンに移動してエクセルなどでグラフ化したいと考えてます

[aja]

グラフにするにしても横軸、多分時間はどの程度で間隔がどれくらいが適当なのか仕様を検討しましょう。間隔が１分であれば要素は１分毎にするか、平均を出すとか、いずれにしろ加速度値の生データでは瞬間変動が大きいのでフィルターをかけてから取り込んだ方がいいでしょう。もっともその瞬間変動をとらえたい場合もありますから、仕様、目的に合ったように手をかける必要があります

[SKMT]

コメントありがとうございます。返信遅れて申し訳ありません。フィルターかけていきたいと思います。細かな動作などの検知がしたいので時間感覚はこのままでローパスとハイパス両方とりたいのですが、オススメのサイトや書籍などあれば教えて欲しいです。現状swift4の知識がなさすぎて困ってます。。

[aja]

その前に、元々の質問のアプリが落ちる原因ははっきりしたのでしょうか
それを確認しないでフィルターの実装をしても時間の浪費になりかねません
「dataX.append(acceleration.x)以下をコメントアウト」してどうなったのでしょう

＞ススメのサイトや書籍など

これは別にSwiftの事ではありません、概念的な事や、ソフトウェアでどう実現するかというのはネットで勉強できます。
概念が分かったうえでSwiftでのEMAフィルターについては
こちらにあります（Swift3）
https://i-app-tec.com/ios/accelerometer.html

[SKMT]

以下４行をコメントアウトしたらDevelopment cannot be enabled while your device is locked.と表示されます。iPhoneのロックは外れてるのですが、そのまま動きません。
サイトありがとうございます。精進します、、、

[aja]

プライオリティとして思うに
エラー原因を明確にして修正する
dataX:[Double] = []の使い方を検討。
このやり方が問題と前のスレッドから言っていますが、無限に配列の要素を作り出すこと
最終的にどうするのがいいか考える、あるいはテストして最適なものを探す
全てのデータをここで取り込む必要があるのか、限定してもいいのでは、
合わせて、CSVに書き込むことも並行して検討

その後で、データとしてフィルターをかましたほうがいいかどうかの検討
ではないかと思います

[SKMT]

わかりました。ありがとうございます。ajaさんの助言を踏まえた上でもうちょっと頑張ってみます。

[SKMT]

例えば配列の個数を1000コまでにしようと思ったらどのようにしたらいいですか？

[aja]

試してからその結果をもって質問するのがTeratail的なのですが、ただそうすると当初の質問から外れてしまいますね。

当初の質問から外れていくのはあまりよくないので、このスレッドの答え（あなたの疑問は続くとして）が出ているのであればクローズして新しく質問したほうがいいでしょう。コードも書けますし。

1000個がいいのか悪いのかは別にして非同期という方法があるのを理解しておいたほうがいいでしょう
https://www.google.com/search?q=%E9%9D%9E%E5%90%8C%E6%9C%9F+swift&oq=%E9%9D%9E%E5%90%8C%E6%9C%9F+swift&aqs=chrome..69i57j0l5.19980j1j8&sourceid=chrome&ie=UTF-8

[SKMT]

配列で調べてみたり参考書を読んでいるのですが、空の配列の数の指定が分からないです。　そうですね、ここは解決したことにして次に行ってみます
非同期について調べてみたのですがGCDのAPIとかが出てきて難しそうでしたしあとこれが自分のプログラムにどう生かせるのか分からないです。

[aja]

非同期とは何かということを理解すると必要性がわかります

[SKMT]

配列に加速度を格納する処理とその配列をcsvに書き込む処理を非同期的に行うとかそういうことですか？むしろその後グラフ化することまで考えたら同期させた方がいい気がしますけど、、、　理解が足りず申し訳ありません

[aja]

リアルタイムで加速度をグラフで表示させることとcsvに書き込むことを一連にする必要はないですね、グラフはそのままダイレクトに表示させてもいいでしょうけれど、その記録は非同期で別に取ればいいでしょう。いずれにしろ実際にやってみればいいとは思います。加速度計測＞CSV記録＞グラフ表示をそのまま同期でできる「ケース」もあります。結局何がゴールなのかどういう結果を目的にしているかによります。

[SKMT]

グラフの表示に外部ライブラリを使おうと思ったのですが難しそうなので、データをcsvで出力してそれをエクセルでグラフ化しようと思います。非同期だと配列に入る順番と表示とかの順番がグチャグチャになって正しい時間ー加速度のデータが取れなそうな気もしますけどどうなんですかね

[SKMT]

あ、とりあえず意見を参考にやってみてぶち当たったらまた質問させてください。いろいろな助言ありがとうございます