iOSアプリでクラウドから取得した値を配列に格納できない

Question

前提・実現したいこと

objective-Cを使ったアプリを開発しています。
クラウドのデータストアに保存された値を検索し、検索された値を配列に保存し、カスタムセルに配置したラベルにテキストとして反映されたいと思います。
ちなみ使用しているクラウドはニフクラ mobile backendです。
ニフクラドキュメント　https://mbaas.nifcloud.com/doc/current/
実行してビルドは通るのですが、セルを表示する画面になるとアプリがクラッシュします。
(11/10プログラム修正しました)

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

クラッシュする際に表示されるエラーメッセージがこちらです。
-[__NSCFString objectAtIndexedSubscript:]: unrecognized selector sent to instance 0x114e12a20

該当のソースコード

//hファイルです。
#import <UIKit/UIKit.h>
#import "TransitionObj.h"
#import "CustomCell.h"

@interface SecandViewController : UIViewController<UITableViewDelegate,UITableViewDataSource>

//@property (weak, nonatomic) IBOutlet UILabel *l1v;
@property (strong, nonatomic) TransitionObj* transitionObj;
//@property (weak, nonatomic) IBOutlet UITextField *f2v;

// テーブルビュー
@property (weak, nonatomic) IBOutlet UITableView *tblView;


@end

//mファイルです。
#import <Foundation/Foundation.h>
#import "SecandViewController.h"
#import "TransitionObj.h"
#import "givewater.h"
#import "CustomCell.h"
#import "NCMB/NCMB.h"

@interface SecandViewController ()<UITableViewDataSource, UITableViewDelegate>
{
    
}

@end

// テーブル表示件数
//const NSInteger historyNumber = 100;

@implementation SecandViewController

#pragma mark Life Cycle

- (void)viewDidLoad {
    [super viewDidLoad];
    self.msgs = [[NSMutableArray alloc] init];
    self.amountwaters = [[NSMutableArray alloc] init];
    self.dates = [[NSMutableArray alloc] init];
    self.realTemperatures = [[NSMutableArray alloc] init];
    self.damiTemperatures = [[NSMutableArray alloc] init];
    self.array = [[NSMutableArray alloc] init];
    
    // DelegateとDataSourceはstoryboard(またはxib)のところで結び付けるのでもO.K.
    _tblView.delegate = self;
    _tblView.dataSource = self;
    
    // カスタムセルを使用
    UINib *nib = [UINib nibWithNibName:@"CustomCell" bundle:nil];
    [_tblView registerNib:nib forCellReuseIdentifier:@"Cell"];
    
    [self kaeruQuery];
    
    
}
-(void)kaeruQuery
{
    NCMBQuery *query = [NCMBQuery queryWithClassName:@"HAGE"];
    //    query.skip = 4;
    [query orderByDescending:@"createDate"];
    query.limit = 40;
    [query findObjectsInBackgroundWithBlock:^(NSArray *array, NSError *error)
     {
         if (!error) {
             for (NCMBObject *obj in array) {
                 {
                     if ([obj objectForKey:@"kaerumsg"]) {
                         NSMutableArray* hoge1;
                         hoge1 = [obj objectForKey:@"kaerumsg"];
                         [self.msgs addObject:hoge1];
                         NSLog(@"%@",self.msgs);
                     }
                     if ([obj objectForKey:@"water"]) {
                         NSMutableArray* hoge2;
                         hoge2 = [obj objectForKey:@"water"];
                         [self.amountwaters addObject:hoge2];
                         NSLog(@"%@",self.amountwaters);
                     }
                     if ([obj objectForKey:@"date"]) {
                         NSMutableArray* hoge3;
                         hoge3 = [obj objectForKey:@"date"];
                         [self.dates addObject:hoge3];
                         NSLog(@"%@",self.dates);
                     }
                     if ([obj objectForKey:@"hikakuleaf"]) {
                         NSMutableArray *hoge4;
                         hoge4 = [obj objectForKey:@"hikakuleaf"];
                         [self.realTemperatures addObject:hoge4];
                         NSLog(@"%@",self.realTemperatures);
                     }
                     if ([obj objectForKey:@"damileaf"]) {
                         NSMutableArray *hoge5;
                         hoge5 = [obj objectForKey:@"damileaf"];
                         [self.damiTemperatures addObject:hoge5];
                         NSLog(@"%@",self.realTemperatures);
                     }
                     
                 }
                 
             }
             [self.tblView reloadData];
         }
         
         
     }];
}


#pragma mark UITableView

// セクション数
- (NSInteger)numberOfSectionsInTableView:(UITableView *)tableView {
    return 1;
}

// セル数
- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section {
    
    return [self.array count];
    
}


/*
- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section {
    
    return [self.amountwaters count];
    
}
*/
// セルの内容を設定
- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {
    
    NSLog(@"cellForRowAtIndexPath indexPath:%ld", indexPath.row);
    
    // カスタムセルを取得
    CustomCell *cell = [tableView dequeueReusableCellWithIdentifier:@"Cell" forIndexPath:indexPath];
    
    cell.date.numberOfLines = 0;
    cell.amountwater.numberOfLines = 0;
    cell.damiTemperature.numberOfLines = 0;
    cell.realTemperature.numberOfLines = 0;
    cell.msg.numberOfLines = 0;
    
    
    cell.msg.text = [self.msgs objectAtIndex:indexPath.row];
    cell.amountwater.text = [self.amountwaters objectAtIndex:indexPath.row];
    cell.date.text = [self.dates objectAtIndex:indexPath.row];
    cell.realTemperature.text = [self.realTemperatures objectAtIndex:indexPath.row];
    cell.damiTemperature.text = [self.damiTemperatures objectAtIndex:indexPath.row];
    
    /*
     cell.msg.text = self.msgs [indexPath.row];
     cell.amountwater.text = self.amountwaters [indexPath.row];
     cell.date.text = self.dates [indexPath.row];
     cell.realTemperature.text = self.realTemperatures [indexPath.row];
     cell.damiTemperature.text = self.damiTemperatures [indexPath.row];*/
    
    return cell;
}

// セルの高さ設定
- (CGFloat)tableView:(UITableView*)tableView heightForRowAtIndexPath:(NSIndexPath *)indexPath {
    
    NSLog(@"heightForRowAtIndexPath indexPath:%ld", indexPath.row);
    
    //dequeueReusableCellWithIdentifier(セルを使用する)
    CustomCell *Cell = [self.tblView dequeueReusableCellWithIdentifier:@"Cell"];
    
    //とりあえず固定値に
    return 200;
}

/**
 * 文字列に必要な高さを計算する処理
 * @param text 文字列
 * @return CGFloat 高さ
 */

- (CGFloat)calcHeight:(NSString *)text {
    
    // 端末サイズ取得
    CGRect screen = [[UIScreen mainScreen] bounds];
    // 表示する幅 (40 + 8 * 3):画像の幅(40)とマージンの合計(8*3)
    CGFloat width = screen.size.width - (40 + 8 * 3);
    
    // 表示する最大幅と高さ
    CGSize maxSize = CGSizeMake(width, CGFLOAT_MAX);
    
    // 文字列のフォント種類、サイズなど属性を指定
    NSDictionary *attrDic = @{NSFontAttributeName:[UIFont systemFontOfSize:15.0]};
    
    CGRect rect = [text boundingRectWithSize:maxSize
                                     options:NSStringDrawingUsesLineFragmentOrigin
                                  attributes:attrDic
                                     context:nil];
    return rect.size.height;
}


// セル押下時の処理
- (void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath {
    
}

#pragma mark Memory

- (void)didReceiveMemoryWarning {
    [super didReceiveMemoryWarning];
    // Dispose of any resources that can be recreated.
}

@end

//DBの中身
|damileaf|date|hikakuleaf||kaerumsg||water||createDate|
| |11/15|| | | ||value ||2018-11-15T21:29:37.942+09:00|
| value| || value | value || ||2018-11-15T21:29:37.942+09:00|
| |11/15|| | | ||value ||2018-11-15T21:29:37.942+09:00|
| value| || value | value || ||2018-11-15T21:29:37.942+09:00|
//以下省略しています,valueには何かしらの値が入っています

試したこと

NSLogを用いてデータが検索できているかどうか調べましたが、mBassからの検索はできています。
おそらくデータを配列に入れてテキストに表示させるところで間違っていると思いますので、どなたかご指摘よろしくお願いします。

補足情報（FW/ツールのバージョンなど）

iOS11.4
Xcode10.1

Comments

[soo]

Firebaseを使うのがオススメです。

Answer 1

damileaf,hikakuleaf,kaerumsgとwatarの取得時期が違う為、画面を2つに分けて別個に処理することで解決しました。

Answer 2

    [query findObjectsInBackgroundWithBlock:^(NSArray *objects, NSError *error) {

            // ... （中略）

            // 検索に成功した場合の処理
            NSLog(@"検索に成功しました。");

            NSArray *hoge = objects;
            //型変換を行う
            NCMBObject *object = (NCMBObject*)hoge; // ←こんなことは出来ません
 

findObjectsInBackgroundWithBlock で取得した objects は
NCMBObject*が0個以上入っている配列(NSArray)です。
ですからキャストしちゃだめです。

Comments

[donarudo]

コメントありがとうございます。
プログラムの方を修正したので、ご覧いただけるとありがたいです。

[daisuke7]

クラッシュはどこで起きているのでしょうか？
デバッグ実行してクラッシュした時の場所など分かりませんか？

[donarudo]

- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath
の中のcell.msg.text = _msgs[indexPath.row];でクラッシュしています。

[daisuke7]

_msgs は配列ではなく NSString* （単一の文字列）なんじゃないでしょうか？

[donarudo]

プログラムを変更しました。
配列の定義なども間違ってるといけないので、割愛した箇所も載せました。
はい、setupDataクラスの中でNSLogで確認したのですが、クエリを送ってきちんと検索できてます。
- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPathの中でcell.msg.text = (NSString*)msgs;とするとテキストは表示されるのですが、配列の0番目の文字列しか表示できません。やはり、ご指摘を受けた通り、単一の文字列になってしまっていると思います。[indexPath.row]で検索した配列の中身を全て表示させたいのですが上手く行かないです。

[daisuke7]

msgsも配列（NSMutableArray）ですから、クエリ成功後のforループでは
[msgs addObject:[object objectForKey:@"kaerumsg"]];
しないとおかしいです。

[donarudo]

[msgs addObject:[object objectForKey:@"kaerumsg"]];としましたが、配列の中身が全て(null)になってしまいました。調べましたが、原因がよくわかりません。

[daisuke7]

msgs ちゃんと確保してますか？ msgs = [[NSMutableArray alloc] init]; みたいに

[donarudo]

@implementation SecandViewControllerの後に
- (NSMutableArray *)msgs{
if (!_msgs) {
_msgs = [[NSMutableArray alloc] init];
}
return _msgs;
}
入れましたが、同じく配列の中身が(null)のままです。初期化の方法がまずいのでしょうか?

[daisuke7]

msgsと _msgsがどういう扱いになっているのかよくわかりません。
もう一度今のソースコードを提示し直していただけますか？ *.h *.m 両方です。

[donarudo]

編集しました。分かりにくいところがあればご指摘願います。

[daisuke7]

@propertyで定義するプロパティについて理解出来てないと思います。

@property (strong, nonatomic)NSMutableArray *msgs;

としてください。ARC潰してないならretainではなくstrongです。

_msgs というアクセスの仕方は一切やめてください。そうではなく self.msgs と書きます。

また- (NSMutableArray *)msgs　は定義する必要はありません。

viewDidLoad中で

self.msgs = [[NSMutableArray alloc] init];

としてください。

[donarudo]

self.msgs = [[NSMutableArray alloc] init];としたら配列の値が取得できなくなりました。
現在のソースコードを提示しなおすのでご指摘願います。

[daisuke7]

numberOfRowsInSection　が返す値は self.msgs.count でなければなりません。
また、おそらくクエリが完了する前にテーブルの描画が始まります。その時点では self.msgs.count は0でないとクラッシュします。

[donarudo]

numberOfRowsInSection　が返す値をself.msgs.count にしてしまうとCustomCellが生成されなくなります。
さらに配列の中身が"\U6c34\U304c\U8db3\U308a\U306a\U3044\U3088\U3002"などとおかしくなります。

[daisuke7]

最初は0件なのでセルがないのは当然です。
まずUITableViewの理解からはじめた方が良いと思います。

[donarudo]

daisuke7さま
おかげさまでカスタムセルのテキストにDBから取得した値を表示することができました。
ありがとうございます。
しかし、2回に1回の割合でアプリがクラッシュします。
その際に表示されるメッセージがこちらです。
'NSRangeException', reason: '*** -[__NSArrayM objectAtIndex:]: index 0 beyond bounds for empty array'
色々調べましたがどこが間違っているのか分かりません。
よろしければご指摘願います。

[daisuke7]

[self setupData1] から[self setupData5] でmsgsなど5種類の配列それぞれにクエリ結果を読み込んでいますが、これらはどの順序で完了するかは分かりません。（非同期型のネットワークAPIを呼び出しているのでいつ戻ってくるかは分からない）
またテーブルの行数は[self.amountwaters count] で取得していますが、self.amountwatersと他の配列の個数が同数かどうかは分かりません。想定データでは同数だったとしても、APIを非同期で呼び出しているため、「amountwatersは5件だがそれ以外はすべて0件」という状態もありえます。
その状況でreloadData()してしまうと、amountwaters以外の配列でも5件分読み出そうとしてクラッシュします。

[donarudo]

daisuke7さま
ソースコードを訂正しました。
DBからカスタムセルへの値の表示をし、アプリもクラッシュすることなく動作します。
しかし、damileaf,hikakuleaf,kaerumsgとdate,waterのプログラムで取得するタイミングが違うため、セルに表示した時ずれていってしまいます。(damileaf,hikakuleaf,kaerumsgの方が取得時期が先)
daisuke7さまもおっしゃった通りテーブルの行数を[self.amountwaters count] で取得すると最新のwaterの値が表示できないので、書き換えたいと思いますが、書き方がよく分かりません。
ご指摘の方よろしくお願いします。

[daisuke7]

[self.tblView reloadData]; を for()ループの後にしたら挙動変わりませんか？
（いま { } が二重になっててわかりにくいですが、forループの中になっています）

ずっと不思議なのは NCMBObject の配列をそのまま使っちゃだめなのかな、ってことですが…

[donarudo]

ソースコード訂正しました
[self.tblView reloadData];をforループの後にしましたが、挙動は変わりませんでした。
NCMBObject の配列をそのまま使いましたが、カスタムセルが表示されなくなりました。

[daisuke7]

そうですか。これ以上は実際にデバッグしてみないと話が進まないかもしれませんね。お役に立てなくてごめんなさい。