cellのテキスト値をランダムでpush通知する

Question

こんにちは。

現在覚えたての知識で、テーブルに並んだリストの値をランダムでプッシュ通知するアプリを作っているのですが、現状だと同じcellばかりプッシュされてしまうのでどこで値を変えればいいか、というところで詰まっています。

関連性のあるクラスファイルのみ以下記載します
.
.
.
テーブルビュー

import UIKit

var list = ["東京", "千葉", "埼玉"]
var notiList = list

var randomItem = Int(arc4random_uniform(UInt32(list.count)))


class ListViewController: UIViewController, UITableViewDelegate, UITableViewDataSource {
    
    // Outlets
    @IBOutlet weak var listTableView: UITableView!
    @IBOutlet weak var scrollView: UIScrollView!
    
    // variables
    var listScrollView = UIScrollView()
    
    
    
    
    
    //  delegate method to assign the number of cell
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        
        return list.count
    }
    

    
    //  dataSource method to set the value to cell
    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        
        let cell = UITableViewCell(style: UITableViewCell.CellStyle.default, reuseIdentifier: "cell")
        cell.textLabel?.text = list[indexPath.row]
        
        return cell
    }
    
    
    
    // have to add this function because it has 2 buttons
    func tableView(_ tableView: UITableView, canEditRowAt indexPath: IndexPath) -> Bool {
        return true
    }
    
    
    
    // swipe actions edit/delete
    func tableView(_ tableView: UITableView, editActionsForRowAt indexPath: IndexPath) -> [UITableViewRowAction]? {
        
        // delete action
        let deleteButton = UITableViewRowAction(style: .destructive, title: "Delete") { (rowAction, indexPath) in
            list.remove(at: indexPath.row)
            self.listTableView.reloadData()
            
        }
        deleteButton.backgroundColor = UIColor.red
        
        return [deleteButton]
    }
    
    
    
    //  display change to setting vc
    @IBAction func editButton(_ sender: Any) {
        
        let storyboard: UIStoryboard = self.storyboard!
        let settingVC = storyboard.instantiateViewController(withIdentifier: "SettingVC")
        self.present(settingVC, animated: true, completion: nil)
        
    }


    override func viewDidLoad() {
        super.viewDidLoad()
        LocalPushManager.shared.requestAuthorization()

    }


}

.
.
.
notification操作

import Foundation
import UserNotifications

class LocalPushManager: NSObject {
    
    // instance
    static var shared = LocalPushManager()
    let center = UNUserNotificationCenter.current()
    
    
    // this is the authorisation to access the notification center
    func requestAuthorization(){
        center.requestAuthorization(options: [.alert, .sound]) { (granted, error) in
            if error == nil{
                print("parmission granted")
            }
        }
    }
    
    
    
    func sendLocalPush(in time: TimeInterval){
        
        let content = UNMutableNotificationContent()
        content.body = NSString.localizedUserNotificationString(forKey: list[randomItem], arguments: nil)
        
        let trigger = UNTimeIntervalNotificationTrigger(timeInterval: time, repeats: true)
        let request = UNNotificationRequest(identifier: "Timer", content: content, trigger: trigger)
        
        center.add(request) { (error) in
            if error == nil{
                print("schedule push succeed")
            }
        }
        
    }
    
}

.
.
.

タイマー設定

import Foundation
import UIKit

class SettingViewController: UIViewController {
    
    @IBOutlet weak var timeField: UITextField!
    
    
    
    @IBAction func setButton(_ sender: Any) {
        
        guard let text = timeField.text else {return}
        guard let time = TimeInterval(text) else {return}
        LocalPushManager.shared.sendLocalPush(in: time)
        
        self.dismiss(animated: true, completion: nil)
        
    }
    
    
}

.
.
#アイデア？
ランダムに要素を取得する　randItem がグローバルなので　notificationContent の　forKey に変数をこのまま与えてしまうと、グローバルで作成されたランダム数がずっと入り続けてしまうのが原因なのはわかりました。
なので、cellの情報が詰まったlistをプッシュのディスプレイ用に複製（var notiList = list）して、
以下のようなのを用いて使ったら削除して、別のを取得してそれを代入し直し、というプロセスとどこかに入れたらいいのでは？と思いついたのですが、、、
この　func sendLocalPushは一度呼ばれるだけなので、ここで毎回発火させるのは難しい。。。？

notiList.remove(at: randomItem)
randomItem = Int(arc4random_uniform(UInt32(notiList.count)))

content.body = NSString.localizedUserNotificationString(forKey: notiList[randomItem], arguments: nil)

ほかに何か解決策があればよろしくお願いいたします。

Comments

[t_obara]

プッシュ通知をするタイミングはいつなんでしょう？

[mogiruri]

タイマー設定のところにテキストフィールドがあるので、ユーザに何分おきかを入力してもらったタイミングです。

[fuzzball]

randomItemがグローバル変数なのが原因なのであれば、プッシュ通知するタイミングで乱数を生成すればいいだけでは？

[mogiruri]

コメントありがとうございます。上記だとリクエストするのが sendLocaPush関数ですが、これが呼ばれるのは時間がセットされた時に限り呼ばれるので、その時に作られた乱数が入り続けてしまうのかなぁというところで繰り返し発生させるポイントが見つけられずにいます。具体的にはどこに差し込むのがいいのでしょうか？

Answer 1

受信時にすればいいんじゃ？
送信時に内容を決めておく必要なくないですか？

Comments

[mogiruri]

コメントありがとうございます。didReceiveRemoteNotificationのことでしょうか？調べてみて存在をしりました。受信時に動くメソッドを導入できるが探しているところです。

Answer 2

Push通知を送信する際の動作を記述している部分でランダム操作を行えばいいと思います。
（Push通知を送信するたびにランダムな数値を生成できる。）

この場合だとsendLocalPush(in time: TimeInterval)になるのでしょうか。

func sendLocalPush(in time: TimeInterval){
    var randomItem = Int(arc4random_uniform(UInt32(list.count)))
}

Comments

[mogiruri]

コメントありがとうございます。実装してみましたが、同じ乱数が格納されてしまいます。この関数は時間をセットする時だけ、一度だけ発火するからではないのでしょうか？