stringFuncs.splitWords - Code Metrics - nightfury7/universalbasics.js - Measure and Improve Code Quality continuously with Scrutinizer

stringFuncs.splitWords F
last analyzed 2018-11-20 12:26 UTC

↳ Parent: src/ub.strings.words.js

Complexity

Conditions	18
Paths	11

Size

Total Lines	67
Code Lines	35

Duplication

Lines	0
Ratio	0 %

Importance

Changes	6
Bugs	5	Features	0

Metric	Value
cc	18
eloc	35
nc	11
nop	1
dl	0
loc	67
rs	1.2
c	6
b	5
f	0

How to fix Long Method Complexity

/** global: UB */

UB.wordSeperators = [" ", ".", ",", ";", "[", "]", "{", "}", "(", ")"];



var arrayFuncs = {

	joinWords: function(){
		var words = this;
		return words.join(" ");
	},

none:null
};

// register funcs
UB.registerFuncs(Array.prototype, arrayFuncs);



var stringFuncs = {
	
	/** advanced: splits by any operator or number char, space & newline.
	 * simple: splits by newline & space.
	 * always returns an array of non-blank, trimmed words. */
	splitWords: function(advanced = true){
		var text = this;

		if (advanced){
			
			var results = [];
			
			// find start of word
			for (var c = 0, cl = text.length;c<cl;c++){
				var cc = text.charAt(c);
				var start = c;
				
				if (cc.isLetter()) {
					
					// find end of word
					for (var e = c + 1;e<cl;e++){
						var ec = text.charAt(e);
						var isLastChar = (e == (cl - 1));
						
						if ((!ec.isLetter() && ec != ' ') || isLastChar){
							
							// skip if "letter [dash] letter"
							if (!isLastChar && ec == '-' && text.charAt(e + 1).isLetter()){
								continue;
							}
							
							// add word if something found
							var end = isLastChar?e:e-1;
							if (end > start) {
								
								var term = text.range(start, end).trim();
								if (term.exists()){
									
									if (splitBySpaceAlso) {

										term.splitWords().addArray(results);
									} else {
										results.push(term);
									}
								}
							}
							
							// continue at ending
							c = e;
							break;
						}
					}
				}
			}
			return results;

		}
		
		// SPLIT BY NEWLINE & SPACES
		results = [];
		var lines = text.splitLines(true, true);
		for (var s = 0, sl = lines.length; s < sl; s++) {
			var line = lines[s];
			results = line.split(" ");
			for (var w = 0, wl = results.length; w < wl; w++) {
				var word = results[w];
				if (word.length > 0) {
					results.push(word);
				}
			}
		}
		return results;
	},
	
	/** Count words in a string */
	wordCount: function(){
		var text = this;
		return text.match(new RegExp("\\b\\w+\\b", "g")).length;
	},
	none:null
};

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

1			/** global: UB */
2
3			UB.wordSeperators = [" ", ".", ",", ";", "[", "]", "{", "}", "(", ")"];
4
5
6
7			var arrayFuncs = {
8
9			joinWords: function(){
10			var words = this;
11			return words.join(" ");
12			},
13
14			none:null
15			};
16
17			// register funcs
18			UB.registerFuncs(Array.prototype, arrayFuncs);
19
20
21
22			var stringFuncs = {
23
24			/** advanced: splits by any operator or number char, space & newline.
25			* simple: splits by newline & space.
26			* always returns an array of non-blank, trimmed words. */
27			splitWords: function(advanced = true){
28			var text = this;
29
30			if (advanced){
31
32			var results = [];
33
34			// find start of word
35			for (var c = 0, cl = text.length;c<cl;c++){
36			var cc = text.charAt(c);
37			var start = c;
38
39			if (cc.isLetter()) {
40
41			// find end of word
42			for (var e = c + 1;e<cl;e++){
43			var ec = text.charAt(e);
44			var isLastChar = (e == (cl - 1));
45
46			if ((!ec.isLetter() && ec != ' ') \|\| isLastChar){
47
48			// skip if "letter [dash] letter"
49			if (!isLastChar && ec == '-' && text.charAt(e + 1).isLetter()){
50			continue;
51			}
52
53			// add word if something found
54			var end = isLastChar?e:e-1;
55			if (end > start) {
56
57			var term = text.range(start, end).trim();
58			if (term.exists()){
59
60			if (splitBySpaceAlso) {
			0 ignored issues – show Best Practice introduced 2018-07-15 11:48 UTC by Report Bug Copy Issue Report If you intend to check if the variable `splitBySpaceAlso` is declared in the current environment, consider using `typeof splitBySpaceAlso === "undefined"` instead. This is safe if the variable is not actually declared. Loading history...
61			term.splitWords().addArray(results);
62			} else {
63			results.push(term);
64			}
65			}
66			}
67
68			// continue at ending
69			c = e;
70			break;
71			}
72			}
73			}
74			}
75			return results;
76
77			}
78
79			// SPLIT BY NEWLINE & SPACES
80			results = [];
81			var lines = text.splitLines(true, true);
82			for (var s = 0, sl = lines.length; s < sl; s++) {
83			var line = lines[s];
84			results = line.split(" ");
85			for (var w = 0, wl = results.length; w < wl; w++) {
86			var word = results[w];
87			if (word.length > 0) {
88			results.push(word);
89			}
90			}
91			}
92			return results;
93			},
94
95			/** Count words in a string */
96			wordCount: function(){
97			var text = this;
98			return text.match(new RegExp("\\b\\w+\\b", "g")).length;
99			},
100			none:null
101			};
102
103			// register funcs
104			UB.registerFuncs(String.prototype, stringFuncs);

nightfury7 / universalbasics.js

stringFuncs.splitWords F last analyzed 2018-11-20 12:26 UTC

Complexity

Size

Duplication

Importance

How to fix Long Method Complexity

Long Method

Complexity

Duplication Side-by-Side

Filter issues like

stringFuncs.splitWords F
last analyzed 2018-11-20 12:26 UTC