From 65b80598627592e3c557c976695b6c42ec2fe779 Mon Sep 17 00:00:00 2001 From: coderaiser Date: Tue, 11 Mar 2014 06:05:11 -0400 Subject: [PATCH] refactor(util) checkExtension --- lib/util.js | 41 ++++++++++++++++++++--------------------- 1 file changed, 20 insertions(+), 21 deletions(-) diff --git a/lib/util.js b/lib/util.js index be55cbec..14ba8658 100644 --- a/lib/util.js +++ b/lib/util.js @@ -98,33 +98,32 @@ /** * Функция ищет в имени файла расширение * и если находит возвращает true - * @param pName - получает имя файла - * @param pExt - расширение + * @param name - получает имя файла + * @param ext - расширение */ - this.checkExtension = function(pName, pExt) { - var i, lExtNum, lExtSub, - lRet = false, - lExtLength = pExt && pExt.length, - lLength = pName && pName.length; /* длина имени*/ + this.checkExtension = function(name, ext) { + var i, extNum, extSub, ret, + extLength = ext && ext.length, + length = name && name.length; - /* если длина имени больше длинны расширения - имеет смысл продолжать - */ - if (Util.isString(pExt) && lLength > lExtLength) { - lExtNum = pName.lastIndexOf(pExt), /* последнее вхождение расширения*/ - lExtSub = lLength - lExtNum; /* длина расширения*/ - - /* если pExt - расширение pName */ - lRet = lExtSub === lExtLength; - - } else if (Util.isObject(pExt) && lExtLength) - for(i = 0; i < pName.length; i++) { - lRet = Util.checkExtension(pName, pExt[i]); + /* если длина имени больше длинны расширения - имеет смысл продолжать */ + if (Util.isObject(ext) && extLength) + for (i = 0; i < length; i++) { + ret = Util.checkExtension(name, ext[i]); - if (lRet) + if (ret) break; } + else if (length > extLength) { + extNum = name.lastIndexOf(ext), /* последнее вхождение расширения*/ + extSub = length - extNum; /* длина расширения*/ + + /* если pExt - расширение pName */ + ret = extSub === extLength; - return lRet; + } + + return ret; };