Code smell for several similar groups of constants

In programming, if you defined several similar groups of constants like these, and you have several switch statements in your code to do the mapping:

typedef NS_ENUM(NSUInteger, AssetCompressionQuality) {
    AssetCompressionQualitySmall,
    AssetCompressionQualityMedium,
    AssetCompressionQualityLarge,
    AssetCompressionQualityOriginal
};

// These constants are used for the image/video file size calculation
extern float const kAssetSizeCalculationRatioSmall;
extern float const kAssetSizeCalculationRatioMedium;
extern float const kAssetSizeCalculationRatioLarge;
extern float const kAssetSizeCalculationRatioOriginal;

// These constants are used for the image compression parameters
extern float const kJPEGRepresentationSmall;
extern float const kJPEGRepresentationMedium;
extern float const kJPEGRepresentationLarge;
extern float const kJPEGRepresentationOriginal;

switch (self.currentQuality) {
    case AssetCompressionQualitySmall:
        size = size * kAssetSizeCalculationRatioSmall;
        break;
    case AssetCompressionQualityMedium:
        size = size * kAssetSizeCalculationRatioMedium;
        break;
    case AssetCompressionQualityLarge:
        size = size * kAssetSizeCalculationRatioLarge;
        break;
    default:
        size = size * kAssetSizeCalculationRatioOriginal;
        break;
}

You should stop doing this because they look similar and they will have a mapping relationship. The correct way will be creating a new helper class and have the enum defined there and have multiple static mapping methods there. Like this:

typedef NS_ENUM(NSUInteger, AssetCompressionQuality) {
    AssetCompressionQualitySmall,
    AssetCompressionQualityMedium,
    AssetCompressionQualityLarge,
    AssetCompressionQualityOriginal
};

+ (float)assetSizeCalculationRatio:(AssetCompressionQuality)currentQuality {
        switch (currentQuality) {
            case AssetCompressionQualitySmall:
                return kAssetSizeCalculationRatioSmall;
            case AssetCompressionQualityMedium:
                return kAssetSizeCalculationRatioMedium;
            case AssetCompressionQualityLarge:
                return kAssetSizeCalculationRatioLarge;
            default:
                return kAssetSizeCalculationRatioOriginal;
        }
}

In this way, you will have a couple of benefits:

  1. The code is better organized and stayed in one file. So it is easy to maintain if there're any changes or any new types added.
  2. No need to repeat the switch statement in multiple places.
  3. Easy to write unit tests and reduce the typos possibility in multiple places.
最后編輯于
?著作權歸作者所有,轉載或內容合作請聯(lián)系作者
  • 序言:七十年代末,一起剝皮案震驚了整個濱河市借笙,隨后出現(xiàn)的幾起案子距潘,更是在濱河造成了極大的恐慌,老刑警劉巖圆恤,帶你破解...
    沈念sama閱讀 218,386評論 6 506
  • 序言:濱河連續(xù)發(fā)生了三起死亡事件,死亡現(xiàn)場離奇詭異腔稀,居然都是意外死亡盆昙,警方通過查閱死者的電腦和手機,發(fā)現(xiàn)死者居然都...
    沈念sama閱讀 93,142評論 3 394
  • 文/潘曉璐 我一進店門焊虏,熙熙樓的掌柜王于貴愁眉苦臉地迎上來淡喜,“玉大人,你說我怎么就攤上這事诵闭×锻牛” “怎么了?”我有些...
    開封第一講書人閱讀 164,704評論 0 353
  • 文/不壞的土叔 我叫張陵疏尿,是天一觀的道長瘟芝。 經(jīng)常有香客問我,道長褥琐,這世上最難降的妖魔是什么锌俱? 我笑而不...
    開封第一講書人閱讀 58,702評論 1 294
  • 正文 為了忘掉前任,我火速辦了婚禮敌呈,結果婚禮上嚼鹉,老公的妹妹穿的比我還像新娘。我一直安慰自己驱富,他們只是感情好锚赤,可當我...
    茶點故事閱讀 67,716評論 6 392
  • 文/花漫 我一把揭開白布。 她就那樣靜靜地躺著褐鸥,像睡著了一般线脚。 火紅的嫁衣襯著肌膚如雪。 梳的紋絲不亂的頭發(fā)上,一...
    開封第一講書人閱讀 51,573評論 1 305
  • 那天浑侥,我揣著相機與錄音姊舵,去河邊找鬼。 笑死寓落,一個胖子當著我的面吹牛括丁,可吹牛的內容都是我干的。 我是一名探鬼主播伶选,決...
    沈念sama閱讀 40,314評論 3 418
  • 文/蒼蘭香墨 我猛地睜開眼史飞,長吁一口氣:“原來是場噩夢啊……” “哼!你這毒婦竟也來了仰税?” 一聲冷哼從身側響起构资,我...
    開封第一講書人閱讀 39,230評論 0 276
  • 序言:老撾萬榮一對情侶失蹤,失蹤者是張志新(化名)和其女友劉穎陨簇,沒想到半個月后吐绵,有當?shù)厝嗽跇淞掷锇l(fā)現(xiàn)了一具尸體,經(jīng)...
    沈念sama閱讀 45,680評論 1 314
  • 正文 獨居荒郊野嶺守林人離奇死亡河绽,尸身上長有42處帶血的膿包…… 初始之章·張勛 以下內容為張勛視角 年9月15日...
    茶點故事閱讀 37,873評論 3 336
  • 正文 我和宋清朗相戀三年己单,在試婚紗的時候發(fā)現(xiàn)自己被綠了。 大學時的朋友給我發(fā)了我未婚夫和他白月光在一起吃飯的照片耙饰。...
    茶點故事閱讀 39,991評論 1 348
  • 序言:一個原本活蹦亂跳的男人離奇死亡纹笼,死狀恐怖,靈堂內的尸體忽然破棺而出榔幸,到底是詐尸還是另有隱情,我是刑警寧澤矮嫉,帶...
    沈念sama閱讀 35,706評論 5 346
  • 正文 年R本政府宣布削咆,位于F島的核電站,受9級特大地震影響蠢笋,放射性物質發(fā)生泄漏拨齐。R本人自食惡果不足惜,卻給世界環(huán)境...
    茶點故事閱讀 41,329評論 3 330
  • 文/蒙蒙 一昨寞、第九天 我趴在偏房一處隱蔽的房頂上張望瞻惋。 院中可真熱鬧,春花似錦援岩、人聲如沸歼狼。這莊子的主人今日做“春日...
    開封第一講書人閱讀 31,910評論 0 22
  • 文/蒼蘭香墨 我抬頭看了看天上的太陽羽峰。三九已至,卻和暖如春,著一層夾襖步出監(jiān)牢的瞬間梅屉,已是汗流浹背值纱。 一陣腳步聲響...
    開封第一講書人閱讀 33,038評論 1 270
  • 我被黑心中介騙來泰國打工, 沒想到剛下飛機就差點兒被人妖公主榨干…… 1. 我叫王不留坯汤,地道東北人虐唠。 一個月前我還...
    沈念sama閱讀 48,158評論 3 370
  • 正文 我出身青樓,卻偏偏與公主長得像惰聂,于是被迫代替她去往敵國和親疆偿。 傳聞我的和親對象是個殘疾皇子,可洞房花燭夜當晚...
    茶點故事閱讀 44,941評論 2 355

推薦閱讀更多精彩內容