曾祥意-iOS-Task1-反馈5

[zekunyan]

• 数据库还是没有回调！如何知道数据操作执行完了没有？应该确保执行成功了才执行下一步。
• SNDBService里面的dbPath、queue没有用的都删掉，不应该有无用的东西
• getAllDataWithBlockcompletion应该是getAllDataWithComplete，不用明显的Block注明，除非要区分，命名应该是先是形容词再是名词
• SNItem的idNum为什么类型是字符串？为什么都是字符串？这个要改。isFavor、date都不应该是字符串。
• SNItemViewController里面的dbPath也删掉，代码里面无用的都删掉，以后多人合作怎么办？
• 能直接下划线访问的变量全部用下划线，需要调用setter、getter采用self.xxx
• #pragma mark 是用来区分代码逻辑段的，不是为了给函数注释
• searchbar的bar首字母大写，还有其它的命名错误一并改
• 代码空行、组织不够清晰，如下

// 点击 cancel 时去掉 mask ,reloadData
    searchBar.text = @"";
    [searchBar setShowsCancelButton:NO animated:YES];
    self.tableView.allowsSelection = YES;
    self.tableView.scrollEnabled = YES;
    [searchBar resignFirstResponder];
    self.mask.alpha = 0;
    self.isFiltered = NO;
    [self.tableView reloadData];
    self.navigationController.toolbarHidden = NO;

这种应该按照self、searchBar、tableView的分开写，而不是一行这个一行另一个，并用空行分开，其他地方也是如此

• UITextView *contentField命名错误
• setDateFormat只用调用一次
• else应该在前一个if的大括号之后
• SNDetailViewController里面的isFavo明明是BOOL类型的，为啥要用字符串？