objective-Cについてご相談があります。

質問

xcode6でMasterDetailApplicationを試しております。
Master画面のセルをクリックしてDetail画面に遷移した際、
標準でDetail画面の中央に貼られているLabelに、クリックした
セル内の文字列が表示される様にしたいのです。

しかし、一度目のクリックでは表示されず、二度目のクリックで初めて
一度目にクリックしたセルの文字列がLabelに表示されてしまいます。
つまり見た目上は、Labelの表示がクリックより一回分遅れるのです。
※添付画像を参考にして下さい

以下がその部分のコードです。
-----------------------------------------------------------------------
// セグエの設定
- (void)prepareForSegue:(UIStoryboardSegue *)segue sender:(id)sender {

[self tableView];

// セグエを「showDetail」と名付ける
if ([[segue identifier] isEqualToString:@"showDetail"]) {

// _cellMojiはid型のインスタンス変数。
[[segue destinationViewController] setDetailItem:_cellMoji];
NSLog(@"１番:%@",_cellMoji);

}
}

// セル内の文字列を取得するメソッド
- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath
{
// セル内の文字列を取得する
UITableViewCell *cell = [self.tableView cellForRowAtIndexPath:indexPath];

// 文字列データをNSString型に変換する
_cellMoji = cell.textLabel.text;
NSLog(@"２番:%@",_cellMoji);
}
----------------------------------------------------------------------
以上、どなたか解決策をご教示頂けないでしょうか。
何卒宜しくお願い致します。

回答No.2

harawo · ベストアンサー

メソッドの実行順と、インスタンス変数への代入の順が逆になっているのは、別回答者のご指摘のとおりですが……
インスタンス変数「_cellMoji」をわざわざ作って、代入と値の参照を別メソッドで行う理由、必然性そのものが、はたしてあるんでしょうか？
インスタンス変数を作るなら、テーブルビューのインスタンス変数、ないしプロパティを作ったほうが有益です。
以下、テーブルビューのプロパティ「tableView」を作った上で、メソッドを修正してみました。これで「- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath」メソッドは不用です。

// セグエの設定 // 設定じゃあありません。セグエを実行したときに呼ばれるメソッドです。
- (void)prepareForSegue:(UIStoryboardSegue *)segue sender:(id)sender {

[self tableView]; // これ、なにやってんの？そしてなにに使ってるの？

// セグエを「showDetail」と名付ける // 名付けてなんかいませんよ。実行したセグエのIdentifierが「showDetail」であるかどうかを確認している。
if ([[segue identifier] isEqualToString:@"showDetail"]) {

// _cellMojiはid型のインスタンス変数。 // id型じゃなくて、NSString *型と、決めてしまったほうが、あとあとめんどうにならないと思うが？
// [[segue destinationViewController] setDetailItem:_cellMoji]; // これを変更
// タップしたセルを取得。
NSIndexPath *path = [self.tableView indexPathForSelectedRow];
UITableViewCell *cell = [self.tableView cellForRowAtIndexPath: path];
// 遷移先のView Controllerのメソッドを実行
[segue.destinationViewController setDetailItem: cell.textLabel.text;
}
}

お礼

ご回答ありがとうございます！本当に助かりました！
やっと中身が本当に理解できました。私の考えでは、

(1)セルをタップする
(2)最初のメソッドの先頭にある[self tableview]（下の方に書いたメソッド）
　が実行され、セルの中身が取得される
(3)if文の中で、取得されたセルの中身が格納された_cellMojiが
　destinationViewControllerに転送される

・・・という感じで考えておりました。

回答No.3

harawo

No.2回答に対するお礼＞ やっと中身が本当に理解できました。

いや、「本当」の理解はできていないと思いますよ。そういうおごりを持たず、謙虚に勉学にいそしんでください。
こういうことをいわれるのは、あなたにとって心外だとはわかっていて、あえていわせてもらいますが、あなたはプログラミングをなめてかかっています。いまのうちに考えを改めないと、目標を達成することはかないませんよ。

お礼

ご回答ありがとうございます。
すみません、なめてかかっているとは、どのあたりについてのこと
なのでしょうか？もっと自力でしっかり調べるべきであるということ
でしょうか？

回答No.1

Lchan0211b

質問に添付されたログの画像を見ればわかると思うんですが、
didSelectRowAtIndexPathよりもprepareForSegueの方が先に
呼ばれているので、prepareForSegueでdestinationViewControllerに
セルの文字列を渡す時に_cellMojiにはまだ何も入ってないか
前回格納した値が入ったままなので、Detail画面には前回の値が
表示されます。

以下の参考URLのように、segueの線をUITableViewCellから接続するのではなく、
UIViewControllerから接続し、didSelectRowAtIndexPathの中から
performSegueWithIdentifierを呼び出してプログラムでsegueを起動すれば、
自分が意図した順序で呼び出され、正しく表示されるようになります。
(参考)
https://gist.github.com/weed/3323922

お礼

ご回答ありがとうございます！