面试

原代码

typedef enum{
UserSex_Man,
UserSex_Woman
}UserSex;

@interface UserModel :NSObject

@property(nonatomic, strong) NSString *name;
@property (assign, nonatomic) int age;
@property (nonatomic, assign) UserSex sex;

- (id)initUserModelWithUserName:(NSString *)name withAge:(NSUInteger)age;

-(void)doLogIn;

@end

枚举处有如下几个槽点

不应该用 typedef enum 这种 C 形式定义枚举；既然做 iOS，就用 Apple 提供的东西。
性别规范书面的命名是 Gender/Male/Female，而非 Sex/Man/Woman/Boy/Girl 之类。
这一点可能是我多虑了：Male 和 Female 应该是互斥的，结构上用 Union 比较合适。但仍使用 enum 的原因两个：一是常用且易读；二是现实世界情况复杂，不排除极端情况。
要有前缀，且前缀以三个字母为宜。前缀个数其实是有争议的，目前绝大多数的代码，包括流行的开源库都是两个字母前缀，究其原因可能是两个字母更好记，而且苹果也不会刚刚好使用到和我们代码相同两个字母，so……实际情况和要求还是出入的。

类的地方有如下几个槽点

UserModel 的命名方式不太好。Model 是设计模型中的概念，不应该将其命名到具体的代码对象中，而且类型必须要有前缀，防止命名冲突，此处命名前缀取 BDK（Bai Du Knows）。

:

和

NSObject

之间要有空格。
属性定义的地方 property / 括号 / 类型 / 变量名 之间应该要有空格，此外符号 `

是和变量名紧贴。如

@property
(nonatomic, copy) NSString* name;` 就是一个不太好的命名习惯。
括号内的修饰词按需按一定顺序排列，两个修饰词间的逗号后面应当有空格。具体的顺序没有硬性规定，我的习惯是 nonatomic 放前面，因为多数的属性修饰词都是 nonatomic，而其他修饰词却不一定一样，把 nonatomic 放前面看起来比较整齐。
NSString 类型变量用 copy 修饰比较安全。
属性的类最好对齐。这点不知道算不算槽点，但对齐更好看（看下面的代码 NSString *userName; 和下面两行对齐）。
属性 age 是非负整数，所以使用 int 不太好，而且最好不要使用 C 的类型，应该用 NSUInteger。
方法本身的间隔类似的，参照下系统方法的命名，具体就不多说了，看我最终的代码间隔即可。
doLogIn 槽点太多：

两个动词表示一个登录的意思，累赘不合理；

login 本身就是一个动词，如果非要拆开，应该是 logOn/logOff，而非 logIn；

登录接口竟然没有参数。我使用了 NSDictionary 可以将所有登录需要的信息全放在里面，便于扩展和修改。

登录方法没有返回值，无法了解登录情况，所以如果有个回调会更好。

初始化方法的槽点也挺多：

方法名中的 Model 和之前提到的问题一致；

方法有返回值，所以方法名的开头应当是返回内容的名词形式，而非动词；

根据方法名的意思推测，返回值应该是用户信息。既如此，应该直接用 BDKUserInfo 作为返回值类型。但也有可能这么做有其他用意，所以为了尊重原代码的意思，我还是将返回值设为 instancetype，返回一个关联类型。

更正过的代码

前缀 BDK 取 Bai Du Knows
为什么类的属性和方法不加前缀？因为它们编译后不在顶级符号表。而且主流开源库的属性和方法命名时，也没有对类的属性和方法加前缀，已经约定俗成。从可读性的角度来说，加前缀是一种无奈之举，降低了代码的可读性。

typedef NS_ENUM(NSInteger, BDKGender)
{
BDKMale,
BDKFemale
};

@interface BDKUserInfo : NSObject

@property (nonatomic, copy)   NSString *userName;
@property (nonatomic, assign) NSUInteger userAge;
@property (nonatomic, assign) Gender userGender;

/**
*
*
*  @param info         登录所需信息
*  @param successBlock 登录成功 block
*  @param failedBlock  登录失败 block
*/
- (void)loginWithInfo:(NSDictionary *)info
successBlock:(void(^)(NSDictionary *resultInfo))successBlock
failed:(void(^)(NSError *error))failedBlock;

/**
*  初始化用户信息
*
*  @param userName 用户名
*  @param userAge  用户年龄
*
*  @return 用户信息
*/
- (instancetype)userInfoInitialedByUserName:(NSString *)userName
userAge:(NSUInteger)userAge;

@end