nightfury7 / universalbasics.js

src/ub.strings.findAndRemove.js

/** global: UB */

var stringFuncs = {
	
	removeAllCI: function(find, replace){
		var s = this.toString();
		return s.removeAll(find, replace, false, false);
	},
	removeAll: function(find, wholeWords = false, caseSensitive = true){
		var s = this.toString();
		
		if (find.isArray()){
			for (var r = 0, rl = find.length; r < rl; r++) {
				s = s.removeAll(find[r], wholeWords, caseSensitive);
			}
			return s;
		}else{

else is not necessary here since all if branches return, consider removing it to reduce nesting and make code more readable.

			return s.replaceAll(find, "", wholeWords, caseSensitive);
		}
	},
	removeBetween: function(findStart, findEnd, keepEnds = true, startAt = 0, times = 0){
		var s = this.toString();
		return s.replaceBetween(findStart, findEnd, "", keepEnds, startAt, times);
	},
	removeOnce: function(find, wholeWords = false, caseSensitive = true){
		var s = this.toString();
		return s.replaceOnce(find, "", wholeWords, caseSensitive);
	},
	
	removePrefix: function(prefix, caseSensitive = true){
		var str = this.toString();
		
		// remove all the given prefixes
		if (prefix.isArray()){
			for (var r = 0, rl = prefixes.length; r < rl; r++) {

The variable prefixes seems to be never declared. If this is a global, consider adding a /** global: prefixes */ comment.

				var prefix = prefixes[r];
				str = str.removePrefix(prefix, caseSensitive);
			}
		}else{

			// some quick checks
			if (prefix && prefix.length > 0 && str.length >= prefix.length && str.charCodeAt(0) == prefix.charCodeAt(0)) {
				
				// extract prefix
				var p = str.substr(0, prefix.length);
				
				// compare case insensitively if wanted
				if (!caseSensitive) {
					p = p.toLowerCase();
					prefix = prefix.toLowerCase();
				}
				
				// compare
				if (p == prefix) {
					
					// remove prefix
					return str.substring(prefix.length);
				}
			}
		}
		return str;
	},
	removePostfix: function(postfix, caseSensitive = true){
		var str = this.toString();
		
		// remove all the given postfixes
		if (postfix.isArray()){
			for (var r = 0, rl = postfixes.length; r < rl; r++) {

The variable postfixes seems to be never declared. If this is a global, consider adding a /** global: postfixes */ comment.

				var postfix = postfixes[r];
				str = str.removePostfix(postfix, caseSensitive);
			}
		}else{

			// some quick checks
			if (postfix && postfix.length > 0 && str.length >= postfix.length && str.charCodeAt(str.length - 1) == postfix.charCodeAt(postfix.length - 1)) {
				
				// extract postfix
				var p = str.substr(str.length - postfix.length);
				
				// compare case insensitively if wanted
				if (!caseSensitive) {
					p = p.toLowerCase();
					postfix = postfix.toLowerCase();
				}
				
				// compare
				if (p == postfix) {
					
					// remove postfix
					return str.substr(0, str.length - p.length);
				}
				
			}
		}
		return str;
	},
	
	removeEnding: function(chars){
		var str = this.toString();
		if (chars >= str.length) {
			return "";
		}
		return str.substring(0, str.length - chars);
	},
	removeStarting: function(chars){
		var str = this.toString();
		if (chars >= str.length) {
			return "";
		}
		return str.substring(chars, str.length);
	},
	none:null
};

// register funcs
UB.registerFuncs(String.prototype, stringFuncs);