Skip to content

曾祥意-iOS-Task1-反馈5 #22

@zekunyan

Description

@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类型的,为啥要用字符串?

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type
    No fields configured for issues without a type.

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions