関数の引数・返り値などについて

Question

import UIKit

class ViewController: UIViewController,UITableViewDataSource,UITableViewDelegate{

  @IBOutlet weak var mytableView: UITableView!
  var item = [String]()

  func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
    return item.count
  }

  func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
    let cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath)
    let todoLabel = item[indexPath.row]
    cell.textLabel?.text = todoLabel

    //ボタンについて
    let button = UIButton()
    button.backgroundColor = UIColor.blue
    button.setTitle(" 追 加 ", for: .normal)
    cell.contentView.addSubview(button)

    button.translatesAutoresizingMaskIntoConstraints = false
    cell.contentView.rightAnchor.constraint(equalTo: button.rightAnchor, constant: 12).isActive = true
    // 中央にする
    //button.centerYAnchor.constraint(equalTo: cell.centerYAnchor).isActive = true
    //丸みに対して
    button.layer.cornerRadius = 10
    button.layer.masksToBounds = true

    cell.contentView.heightAnchor.constraint(equalTo: button.heightAnchor, multiplier: 1).isActive = true

    //ラベルについて
    var label = UILabel()
    label.backgroundColor = UIColor.darkGray
    label.text = "120×10"
    //丸みに対して
    label.layer.cornerRadius = 5
    label.layer.masksToBounds = true
    cell.contentView.addSubview(label)

    label.translatesAutoresizingMaskIntoConstraints = false
    cell.contentView.leftAnchor.constraint(equalTo: label.leftAnchor, constant: -6).isActive = true
    cell.contentView.heightAnchor.constraint(equalTo: label.heightAnchor, multiplier: 1).isActive = true

    //アラート
    func addalert()-> String{
      let alert = UIAlertController(title: "タイトル", message: "メッセージ", preferredStyle: .alert)

      // OKボタンの設定
      let okAction = UIAlertAction(title: "OK", style: .default, handler: {
        (action:UIAlertAction!) -> Void in

        // OKを押した時入力されていたテキストを表示
        if let textFields = alert.textFields {

          // アラートに含まれるすべてのテキストフィールドを調べる
          for textField in textFields {

            self.item.insert(textField.text!, at: 0)
            self.label.text = textField.text!
            self.mytableView.insertRows(at: [IndexPath(row: 0, section: 0)],with: UITableViewRowAnimation.automatic)
            print(textField.text!)
          }
        }
      })
      alert.addAction(okAction)

      // キャンセルボタンの設定
      let cancelAction = UIAlertAction(title: "Cancel", style: .cancel, handler: nil)
      alert.addAction(cancelAction)

      // テキストフィールドを追加
      alert.addTextField(configurationHandler: {(textField: UITextField!) -> Void in
        textField.placeholder = "テキスト"
      })

      alert.view.setNeedsLayout() // シミュレータの種類によっては、これがないと警告が発生

      // アラートを画面に表示
      self.present(alert, animated: true, completion: nil)
    }
    return cell
  }

@IBAction func addlabel(_ sender: Any) {

     let key = addalert()//alertの値を渡したつもり

  }


  override func viewDidLoad() {
    super.viewDidLoad()

  }



  override func didReceiveMemoryWarning() {
    super.didReceiveMemoryWarning()
    // Dispose of any resources that can be recreated.
  }


}

やりたいこと

ボタンをタップした時、アラートを呼び出したい。

困っていること
cellForRowAtの中にアラートを書いてしまった為にアラートを呼び出せなくなってしまった。

試したこと
func addalert()-> String{を関数として@IBAction func addlabel(_ sender: Any) {に値を渡そうとしているが大学のC言語で勉強した程度しか関数への理解が乏しいのでエラーが取れずに困っています

** エラーメッセージ**
Use of unresolved identifier 'addalert'

イメージ

//labelインスタンス
    //label1
    let label = UILabel()
    label.backgroundColor = UIColor.darkGray
    //label2
    let label2 = UILabel()
    label2.backgroundColor = UIColor.darkGray
    //label3
    let label3 = UILabel()
    label3.backgroundColor = UIColor.darkGray
    //label4
    let label4 = UILabel()
    label4.backgroundColor = UIColor.darkGray
    //label5
    let label5 = UILabel()
    label5.backgroundColor = UIColor.darkGray
    

Comments

[退会済みユーザー]

エラーメッセージくらい載せよう

[退会済みユーザー]

すいません、追記しました。

Answer 1

itemに入力した値を格納しているので戻り値で設定せずにitemに格納した値を設定してあげれば良いのではないでしょうか？

import UIKit

class ViewController: UIViewController,UITableViewDataSource,UITableViewDelegate{
    
    @IBOutlet weak var mytableView: UITableView!
    var item = [String]()
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return item.count
    }
    
    //アラート
    func addalert() {

        ...
        
        // OKボタンの設定
        let okAction = UIAlertAction(title: "OK", style: .default, handler: {
            (action:UIAlertAction!) -> Void in
            
            // OKを押した時入力されていたテキストを表示
            if let textFields = alert.textFields {
                
                // アラートに含まれるすべてのテキストフィールドを調べる
                for textField in textFields {
                    
                    self.item.insert(textField.text!, at: 0)
                    self.mytableView.insertRows(at: [IndexPath(row: 0, section: 0)],with: UITableViewRowAnimation.automatic)
                    print(textField.text!)
                }
            }
        })

        ...
        
    }

    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        
        ...

        //ラベルについて
        let label = UILabel()
        label.backgroundColor = UIColor.darkGray

        // itemに格納した値を設定        
        if indexPath.row < item.count {
            label.text = item[indexPath.row]
        }
        ...        
    }
    
    @IBAction func addlabel(_ sender: Any) {
        addalert()
    }

    ...    
}

[追記]

import UIKit

class ViewController: UIViewController,UITableViewDataSource,UITableViewDelegate{
    
    @IBOutlet weak var mytableView: UITableView!
    var item = [String]()
    
    func tableView(_ tableView: UITableView, numberOfRowsInSection section: Int) -> Int {
        return item.count
    }
    
    //アラート
    func addalert() {
        let alert = UIAlertController(title: "タイトル", message: "メッセージ", preferredStyle: .alert)
        
        // OKボタンの設定
        let okAction = UIAlertAction(title: "OK", style: .default, handler: {
            (action:UIAlertAction!) -> Void in
            
            // OKを押した時入力されていたテキストを表示
            if let textFields = alert.textFields {
                
                // アラートに含まれるすべてのテキストフィールドを調べる
                for textField in textFields {
                    
                    self.item.insert(textField.text!, at: 0)
                    self.mytableView.insertRows(at: [IndexPath(row: 0, section: 0)],with: UITableViewRowAnimation.automatic)
                    print(textField.text!)
                }
            }
        })
        alert.addAction(okAction)
        
        // キャンセルボタンの設定
        let cancelAction = UIAlertAction(title: "Cancel", style: .cancel, handler: nil)
        alert.addAction(cancelAction)
        
        // テキストフィールドを追加
        alert.addTextField(configurationHandler: {(textField: UITextField!) -> Void in
            textField.placeholder = "テキスト"
        })
        
        alert.view.setNeedsLayout() // シミュレータの種類によっては、これがないと警告が発生
        
        // アラートを画面に表示
        self.present(alert, animated: true, completion: nil)
    }

    func tableView(_ tableView: UITableView, cellForRowAt indexPath: IndexPath) -> UITableViewCell {
        let cell = tableView.dequeueReusableCell(withIdentifier: "cell", for: indexPath)
        let todoLabel = item[indexPath.row]
        cell.textLabel?.text = todoLabel
        
        //ボタンについて
        let button = UIButton()
        button.backgroundColor = UIColor.blue
        button.setTitle(" 追 加 ", for: .normal)
        cell.contentView.addSubview(button)
        
        button.translatesAutoresizingMaskIntoConstraints = false
        cell.contentView.rightAnchor.constraint(equalTo: button.rightAnchor, constant: 12).isActive = true
        // 中央にする
        //button.centerYAnchor.constraint(equalTo: cell.centerYAnchor).isActive = true
        //丸みに対して
        button.layer.cornerRadius = 10
        button.layer.masksToBounds = true
        
        cell.contentView.heightAnchor.constraint(equalTo: button.heightAnchor, multiplier: 1).isActive = true
        
        //ラベルについて
        let label = UILabel()
        label.backgroundColor = UIColor.darkGray
        
        if indexPath.row < item.count {
            label.text = item[indexPath.row]
        }
        
        //丸みに対して
        label.layer.cornerRadius = 5
        label.layer.masksToBounds = true
        cell.contentView.addSubview(label)
        
        label.translatesAutoresizingMaskIntoConstraints = false
        cell.contentView.leftAnchor.constraint(equalTo: label.leftAnchor, constant: -6).isActive = true
        cell.contentView.heightAnchor.constraint(equalTo: label.heightAnchor, multiplier: 1).isActive = true
        
        return cell
    }
    
    @IBAction func addlabel(_ sender: Any) {
        addalert()
    }
    
    
    override func viewDidLoad() {
        super.viewDidLoad()
        mytableView.dataSource = self
        mytableView.register(UITableViewCell.self, forCellReuseIdentifier: "cell")
    }
}

Comments

[退会済みユーザー]

@newmtさん が提示して頂いた上記のコードですと、cellForRowAt内のlabelのインスタンスが呼び出せなくなるのでそうするとlabelをcellに挿入出来なくなってしまいます。なのでcellForRowAt内のalertをなんとか呼び出せないかをもしお分かりでしたら教えて頂けたら嬉しいです。

[newmt]

私がやりたいことを勘違いしているのかもしれませんが、アラートのOKボタンの処理の中で
self.mytableView.insertRows
とされているのでアラートで入力するごとに新しい行を追加するというものではないのでしょうか？

[退会済みユーザー]

すいませんでした。提示して頂いたコードを勘違いしていました。
なのでやりたい事としては出来ました。しかし、labelが挿入されると同時にitemのテキストも挿入されているので見た目的におかしな状態になっています。コードを見てみてもlabelとitemは紐ずけられているのでセルの中に入るのはlabelだけになると思うのですがなぜitemのテキストも挿入されるのでしょうか？

[newmt]

私の環境ですと起きませんが、追記したコードで実行しても起きますでしょうか？

[退会済みユーザー]

let todoLabel = item[indexPath.row]
cell.textLabel?.text = todoLabel
解決できました。
調べてみましたら上記の部分のせいで二重に挿入されていました。
それとなんですが5回まで同列内のセルに左詰でlabelを挿入させていきたいのですがその場合5つlabelのインスタンスを用意しなければならないんでしょうか？
私としてはfor文を使うと一個のインスタンスで出来るのではないかと思うのですが出来ますでしょうか？

[newmt]

＞5つlabelのインスタンスを用意しなければならないんでしょうか？
インスタンスが同じですと結局そのインスタンスの値を上書きしているだけになるので、１個のインスタンスではできないと思います。

[退会済みユーザー]

質問にコードを追記しました。このように五つlabelを用意した後、var itemも五つ用意し、解答して頂いたようにif文で条件分岐をすれば私のイメージすることが出来るのでしょうか？
alertについてもいじる必要があるのでしょうか？

Answer 2

cellForRowAtの中にアラートを書いてしまった為にアラートを呼び出せなくなってしまった。

cellForRowAtの外にアラートを書けば良いのでは？

@IBAction func addlabel(_ sender: Any) {

     let key = addalert()//alertの値を渡したつもり

}

//アラート
func addalert()-> String{
    ...
}

Comments

[退会済みユーザー]

外に書くとエラーは取れるのですがその場合だとアラートで追加したいものを追加出来なくなるので今回は関数を利用してなんとか結びつけたいと考えています

Answer 3

エラーメッセージでググろう

Comments

[退会済みユーザー]

宣言していない変数等を記入しているために出るエラーっていうのは分かるのですがどうすればいいか分からないので質問させて頂きました。