0

それで、うまく機能する小さなアプリを作成しましたが、いくつかのベストプラクティス/効率についてはわかりません. アプリには、テーブル ビューに読み込まれるコア データ ソースがあります。テキスト フィールドからアイテムをテーブル ビューに追加できます。問題は、TVViewController[_tvds init]のメソッドの行にあります。cellForRowAtIndexPathこの行が原因になると思います_tvdsテーブルを上下にスクロールするときにインスタンス変数を再インスタンス化する - これによりパフォーマンスが低下しますか? 私はまだ iOS と Objective C に慣れていないので、コードをリファクタリングする方法を教えてください。ビュー コントローラーとデータ ストアを (Core Data メソッドなどで) 分離する必要はないように思えるかもしれませんが、MVC 設計を実践しようとしていました。コードは次のとおりです (App Delegate と xib ファイルは省略しました - これらは標準的なものです)。アプリは機能しますが、私の初心者の心には最適ではないようです。

TVViewController.h

#import <UIKit/UIKit.h>
#import "TVDataSource.h"
#import "TVAppDelegate.h"
@interface TVViewController : UIViewController <UITableViewDataSource, UITableViewDelegate,UITextFieldDelegate>

@property (strong, nonatomic) IBOutlet UITableView *tableView;
@property (nonatomic, retain) TVDataSource *tvDS;
@property (weak, nonatomic) IBOutlet UITextField *addSomethingTextField;
- (IBAction)goButton:(id)sender;

@end

TVViewController.m

#import "TVViewController.h"

@interface TVViewController ()
@end
@implementation TVViewController

- (void)viewDidLoad
{
[super viewDidLoad];
_tvDS = [[TVDataSource alloc]init];
NSLog(@"%@",[_tvDS dataArray]);

NSLog(@"%@",[[_tvDS dataArray] valueForKey:@"name"]);


[_addSomethingTextField setDelegate:self];

}

- (void)didReceiveMemoryWarning
{
[super didReceiveMemoryWarning];
// Dispose of any resources that can be recreated.
}
- (NSInteger)numberOfSectionsInTableView:(UITableView *)tableView {
// Return the number of sections.
return 1;
}

- (NSInteger)tableView:(UITableView *)tableView numberOfRowsInSection:(NSInteger)section {
// Return the number of rows in the section.
// If you're serving data from an array, return the length of the array:

return [[_tvDS dataArray] count];
}

// Customize the appearance of table view cells.
- (UITableViewCell *)tableView:(UITableView *)tableView cellForRowAtIndexPath:(NSIndexPath *)indexPath {

static NSString *CellIdentifier = @"Cell";

UITableViewCell *cell = [tableView dequeueReusableCellWithIdentifier:CellIdentifier];
if (cell == nil) {
    cell = [[UITableViewCell alloc] initWithStyle:UITableViewCellStyleDefault reuseIdentifier:CellIdentifier];
            }

// Set the data for this cell:

cell.textLabel.text = @"dfjklsl";
NSInteger myInt = [indexPath row];
[_tvDS init];
NSObject *myObj = [[_tvDS dataArray] objectAtIndex:myInt];

[[cell textLabel]setText:[myObj valueForKey:@"name"]];

// NSArray *toBeDisplayed = [_tvDS getAnArray];


// [[cell textLabel]setText:[toBeDisplayed objectAtIndex:myInt]];

// set the accessory view:
cell.accessoryType =  UITableViewCellAccessoryNone;

return cell;
}
 - (IBAction)goButton:(id)sender {
 NSString *textFieldContents = [_addSomethingTextField text];
[[_tvDS dataArray] addObject:textFieldContents];
[_tvDS addItem:textFieldContents];
[_addSomethingTextField setText:@""];

[[self tableView] reloadData];
NSLog(@"%u",[[_tvDS dataArray] count]);



}
-(BOOL)textFieldShouldReturn:(UITextField *)textField
{
[self goButton:textField];
[_addSomethingTextField resignFirstResponder];
//NSLog(@"%@", [_addSomethingTextField text]);
return YES;
}
-(void)tableView:(UITableView *)tableView didSelectRowAtIndexPath:(NSIndexPath *)indexPath
{
NSUInteger row = [indexPath row];

NSLog(@"%@", [[_tvDS dataArray] objectAtIndex:row]);


}

@end

TVDataSource.h

#import <Foundation/Foundation.h>
#import "TVAppDelegate.h"
@interface TVDataSource : NSObject
@property (nonatomic, retain) NSMutableArray *dataArray;
@property (nonatomic, retain) NSManagedObjectContext *context;
@property (nonatomic, retain) NSFetchRequest *request;
@property (nonatomic, retain) NSError *error;
//@property (nonatomic, strong) NSArray *obj
//@property (nonatomic, weak) NSManagedObjectContext *context;
//-(void)saveInput;
-(void)addItem:(id)input;


@end

TVDataSource.m

#import "TVDataSource.h"

@implementation TVDataSource

- (id)init
{
    self = [super init];
    TVAppDelegate *appDelegate = [[UIApplication sharedApplication]delegate];
    NSManagedObjectContext *context = [appDelegate managedObjectContext];
    NSEntityDescription *entityDesc = [NSEntityDescription entityForName:@"Todo" inManagedObjectContext:context];
    NSFetchRequest *request = [[NSFetchRequest alloc]init];
    [request setEntity:entityDesc];
    // NSPredicate *pred = [NSPredicate predicateWithFormat:@"(name = %@)", [_name text]];

    NSManagedObject *matches = nil;
    NSError *error;
    NSArray *objects = [context executeFetchRequest:request error:&error];

    if ([objects count]==0) {
        NSLog(@"nothing there");
    } else {
        matches = objects[0];

        [self setDataArray:[objects mutableCopy]];
        NSLog(@"initialise %@",[self dataArray]);
        //NSLog(@"%@",[_dataArray valueForKey:@"name"]);

    }
    return self;

}
-(void)addItem:(id)input
{
    TVAppDelegate *appDelegate = [[UIApplication sharedApplication]delegate];

    NSManagedObjectContext *context = [appDelegate managedObjectContext];
    NSManagedObject *newTask;
    newTask = [NSEntityDescription insertNewObjectForEntityForName:@"Todo" inManagedObjectContext:context];
    [newTask setValue:input forKey:@"name"];

    if ([input isEqualToString:@""]) {
        NSLog(@"CONTENTS BLANK");

    } else {
        NSLog(@"Contents were %@", input);
        NSLog(@"goButton: was called");
        NSError *error;
        [context save:&error];
    }
}

@end

助けてくれてありがとう。シンプルなものになると思います。

編集：

私が試したこと/私が抱えている問題をより明確にする必要がありました。init を再度実行する必要があるのは、テキスト フィールドから何か新しいものを追加するときに Core Data ストアからデータを再入力できるようにするためです。2 番目を削除するinitと、新しいアイテムを追加して下にスクロールしようとすると、SIGABRT エラーが発生します。このエラーは、「名前キー」を持たない配列にテキスト フィールドの内容を追加しているためだと思います。これを修正する方法がわかりません。ビューコントローラーがユーザーに表示するためだけに作業配列を作成し、コアデータストアにこの更新を行うことを検討しました。これは良い解決策ですか、それとももっと良い方法がありますか?

4

1 に答える 1

2

メソッドで既にinitデータ ソースを ing してviewDidLoadいるため、セルを返す場合は必要ありません。

私はそれを完全に削除し、それによって引き起こされる問題があるかどうかを確認します. あなたのクラスが正しいなら、あなたは正しいです。それが呼び出されるたびに時間を無駄にしているだけです。

init を呼び出さずにデータ ソースに追加するわけではないため、新しいオブジェクトを追加する場合にのみ init を呼び出すことができます。または、さらに良いaddItemことに、次の行を追加して、再初期化する必要なく機能するようにメソッドを修正します。

[self.dataArray addObject:input];
于 2013-01-02T03:52:03.330 に答える