include/include-html.js - Code Metrics - UnitApi/js - Measure and Improve Code Quality continuously with Scrutinizer

include/include-html.js A
last analyzed 2020-02-10 08:27 UTC

↳ Parent: Project

Complexity

Total Complexity	10
Complexity/F	2.5

Size

Lines of Code	57
Function Count	4

Duplication

Duplicated Lines	57
Ratio	100 %

Importance

Changes

Metric	Value
wmc	10
eloc	25
mnd	6
bc	6
fnc	4
dl	57
loc	57
rs	10
bpm	1.5
cpm	2.5
noi	8
c	0
b	0
f	0

How to fix Duplicated Code

function includeHtml(url, target, error, success) {

    var xhttp;

    var el = new E(target);

    var elmnt = el.first();

    if (typeof success !== 'function') {
        success = function () {
            console.log('includeHtml success', "included");

        }
    }

    if (typeof error !== 'function') {
        error = function () {
            console.log('includeHtml error', "Page not found.");

        }
    }
    console.log('includeHtml url', url);


    if (url) {
function isBig(a) {
    if (a > 5000) {
        return "yes";
    }
}

console.log(isBig(5001)); //returns yes
console.log(isBig(42)); //returns undefined
        /* Make an HTTP request using the attribute value as the url name: */
        xhttp = new XMLHttpRequest();
        xhttp.onreadystatechange = function () {
            console.log('includeHtml el_id', target);


            if (this.readyState == 4) {
                if (this.status == 200) {
                    // console.log('elmnt', elmnt);
                    // console.log('responseText', this.responseText);
                    // elmnt.innerHTML = this.responseText;
                    // elmnt.appendChild(this.responseText);
                    // elmnt.insertAdjacentHTML('beforeend', this.responseText);
                    // var e = document.createElement('div');
                    // e.innerHTML = this.responseText;
                    // while(e.firstChild) {
                    // elmnt.appendChild(e);
                    // }

                    // elmnt.insertAdjacentHTML('afterend', this.responseText);
                    elmnt.insertAdjacentHTML('beforeend', this.responseText);

                    success(this);

                }
                if (this.status == 404) {
                    elmnt.innerHTML = "includeHtml Page not found.";
                    error(this);

                }
                /* Remove the attribute, and call this function once more: */
                // includeHtml(url, success, error);
            }
        }
        xhttp.open("GET", url, true);
        xhttp.send();
        /* Exit the function: */
        return this;
    }
}


1	View Code Duplication	function includeHtml(url, target, error, success) {
		0 ignored issues – show Duplication introduced 2019-10-09 13:01 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
2		var xhttp;
3
4		var el = new E(target);
		0 ignored issues – show Bug introduced 2019-10-09 13:01 UTC by Report Bug Copy Issue Report The variable `E` seems to be never declared. If this is a global, consider adding a `/** global: E */` comment. This checks looks for references to variables that have not been declared. This is most likey a typographical error or a variable has been renamed. To learn more about declaring variables in Javascript, see the MDN. Loading history...
5		var elmnt = el.first();
6
7		if (typeof success !== 'function') {
8		success = function () {
9		console.log('includeHtml success', "included");
		0 ignored issues – show Debugging Code introduced 2019-10-09 13:01 UTC by Report Bug Copy Issue Report `console.log` looks like debug code. Are you sure you do not want to remove it? Loading history...
10		}
11		}
12
13		if (typeof error !== 'function') {
14		error = function () {
15		console.log('includeHtml error', "Page not found.");
		0 ignored issues – show Debugging Code introduced 2019-10-09 13:01 UTC by Report Bug Copy Issue Report `console.log` looks like debug code. Are you sure you do not want to remove it? Loading history...
16		}
17		}
18		console.log('includeHtml url', url);
		0 ignored issues – show Debugging Code introduced 2019-10-09 13:01 UTC by Report Bug Copy Issue Report `console.log` looks like debug code. Are you sure you do not want to remove it? Loading history...
19
20		if (url) {
		0 ignored issues – show Complexity Best Practice introduced 2019-10-09 13:01 UTC by Report Bug Copy Issue Report There is no return statement if `url` is `false`. Are you sure this is correct? If so, consider adding `return;` explicitly. This check looks for functions where a `return` statement is found in some execution paths, but not in all. Consider this little piece of code function isBig(a) { if (a > 5000) { return "yes"; } } console.log(isBig(5001)); //returns yes console.log(isBig(42)); //returns undefined The function `isBig` will only return a specific value when its parameter is bigger than 5000. In any other case, it will implicitly return `undefined`. This behaviour may not be what you had intended. In any case, you can add a `return undefined` to the other execution path to make the return value explicit. Loading history...
21		/* Make an HTTP request using the attribute value as the url name: */
22		xhttp = new XMLHttpRequest();
23		xhttp.onreadystatechange = function () {
24		console.log('includeHtml el_id', target);
		0 ignored issues – show Debugging Code introduced 2019-10-09 13:01 UTC by Report Bug Copy Issue Report `console.log` looks like debug code. Are you sure you do not want to remove it? Loading history...
25
26		if (this.readyState == 4) {
27		if (this.status == 200) {
28		// console.log('elmnt', elmnt);
29		// console.log('responseText', this.responseText);
30		// elmnt.innerHTML = this.responseText;
31		// elmnt.appendChild(this.responseText);
32		// elmnt.insertAdjacentHTML('beforeend', this.responseText);
33		// var e = document.createElement('div');
34		// e.innerHTML = this.responseText;
35		// while(e.firstChild) {
36		// elmnt.appendChild(e);
37		// }
38
39		// elmnt.insertAdjacentHTML('afterend', this.responseText);
40		elmnt.insertAdjacentHTML('beforeend', this.responseText);
41
42		success(this);
		0 ignored issues – show Bug introduced 2019-10-09 13:01 UTC by Report Bug Copy Issue Report The call to `success` seems to have too many arguments starting with `this`. Loading history...
43		}
44		if (this.status == 404) {
45		elmnt.innerHTML = "includeHtml Page not found.";
46		error(this);
		0 ignored issues – show Bug introduced 2019-10-09 13:01 UTC by Report Bug Copy Issue Report The call to `error` seems to have too many arguments starting with `this`. Loading history...
47		}
48		/* Remove the attribute, and call this function once more: */
49		// includeHtml(url, success, error);
50		}
51		}
52		xhttp.open("GET", url, true);
53		xhttp.send();
54		/* Exit the function: */
55		return this;
56		}
57		}
58

UnitApi / js

include/include-html.js A last analyzed 2020-02-10 08:27 UTC

Complexity

Size

Duplication

Importance

How to fix Duplicated Code

Duplicated Code

Duplication Side-by-Side

Filter issues like

include/include-html.js A
last analyzed 2020-02-10 08:27 UTC