看了眼 tinyfool 经常提的日历控件

先贴上源码地址：Here 代码是是 08 年写的，这点还是非常服的，这个时候我还在上高中。 08 年到现在 OC 在一些特性上改进很多，其中使用的CFGregorianDate相关 api 在 iOS 8.0 之后也不建议使用了，所以这里我只说我确认不是好的实践的部分。

先来看下头文件:

// // CalendarView.h // ZhangBen // // Created by tinyfool on 08-10-26. // Copyright 2008 __MyCompanyName__. All rights reserved. // #import <UIKit/UIKit.h> @protocol CalendarViewDelegate; @interface TdCalendarView : UIView { CFGregorianDate currentMonthDate; CFGregorianDate currentSelectDate; CFAbsoluteTime currentTime; UIImageView* viewImageView; id<CalendarViewDelegate> calendarViewDelegate; int *monthFlagArray; } @property CFGregorianDate currentMonthDate; @property CFGregorianDate currentSelectDate; @property CFAbsoluteTime currentTime; @property (nonatomic, retain) UIImageView* viewImageView; @property (nonatomic, assign) id<CalendarViewDelegate> calendarViewDelegate; -(int)getDayCountOfaMonth:(CFGregorianDate)date; -(int)getMonthWeekday:(CFGregorianDate)date; -(int)getDayFlag:(int)day; -(void)setDayFlag:(int)day flag:(int)flag; -(void)clearAllDayFlag; @end @protocol CalendarViewDelegate<NSObject> @optional - (void) selectDateChanged:(CFGregorianDate) selectDate; - (void) monthChanged:(CFGregorianDate) currentMonth viewLeftTop:(CGPoint)viewLeftTop height:(float)height; - (void) beforeMonthChange:(TdCalendarView*) calendarView willto:(CFGregorianDate) currentMonth; @end 

代码格式我就不说了，空格写的很随意，大小写对于我这种强迫症来说也很受不了。最重要的是我认为稍微参考下苹果 framework 中函数的命名方式，就不会写出协议中这么扯淡的命名。 interface 中的命名大多数都还不错，但是-(void)setDayFlag:(int)day flag:(int)flag;这个是什么鬼，我认为- (void)setFlag:(int)flag forDay:(int)day;更好。

下面来看下源文件：

源文件中定义了全局的几个变量：

const float headHeight=60; const float itemHeight=35; const float prevNextButtOnSize=20; const float prevNextButtOnSpaceWidth=15; const float prevNextButtOnSpaceHeight=12; const float titleFOntSize=30; const int weekFOntSize=12; 

这几个变量主要是来控制试图的大小、字体的大小，只有在这个文件内部试哟哦难过，没有用static修饰，不是很明白，可能 tinyfool 记忆里好，不会再工程中其他地方使用相同名字的变量。

学 C 语言的时候，大概都做过这么一个练习：获取某年某月的天数。来看看 tinyfool 的实现：**

-(int)getDayCountOfaMonth:(CFGregorianDate)date{ switch (date.month) { case 1: case 3: case 5: case 7: case 8: case 10: case 12: return 31; case 2: if((date.year % 4==0 && date.year % 100!=0) || date.year % 400==0) return 29; else return 28; case 4: case 6: case 9: case 11: return 30; default: return 31; } } 

interesting ，自行体会一下。

实现中有很多类似的实践（个人认为这不是好的实践）：

- (void)movePrevMonth{ if(currentMonthDate.month>1) currentMonthDate.month-=1; else { currentMonthDate.mOnth=12; currentMonthDate.year-=1; } [self movePrevNext:0]; } 

再来看下这个类的析构函数：

- (void)dealloc { [super dealloc]; free(monthFlagArray); } 

Excuse me. 这里虽然不太可能出现崩溃的情况，但不是应该先释放子类持有的资源？

最后，总结一些 tinyfool 的变量命名风格：title_Month、weekfont、tabHeight、s_width。

我观察到的是，如果有人在微博上反驳他什么，他通常的回应是毫无道理（有道理的时候也有，少），其他的不想评价，怕被喷。