app/experiments/mongo-fetch.js - Code Metrics - ashokdey/todo-rest - Measure and Improve Code Quality continuously with Scrutinizer

app/experiments/mongo-fetch.js A
last analyzed 2017-06-14 04:20 UTC

↳ Parent: Project

Complexity

Total Complexity	11
Complexity/F	1.57

Size

Lines of Code	40
Function Count	7

Duplication

Duplicated Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
cc	0
nc	1
dl	0
loc	40
rs	10
c	0
b	0
f	0
wmc	11
mnd	1
bc	11
fnc	7
bpm	1.5713
cpm	1.5713
noi	11

1 Function

Rating	Name	Duplication	Size	Complexity
B	mongo-fetch.js ➔ ???	0	37	2

// import MongoClient from the mongodb package
const {MongoClient} = require('mongodb');

// connect to mongodb server
MongoClient.connect('mongodb://localhost:27017/JokesApp', (err, db) => {
    if (err) {
        return console.log('Failed to connect to mongoDB server : ' + err);

    }

    console.log('Connected to MongoDB Server');

    //get the count of jokes stored
    db.collection('Jokes').count().then((count) => {
        console.log(`Jokes count = ${count}\n`);

    }, (err) => {
        if (err) {
function isBig(a) {
    if (a > 5000) {
        return "yes";
    }
}

console.log(isBig(5001)); //returns yes
console.log(isBig(42)); //returns undefined
            return console.log('error getting count', err);

        }
    });

    // fetch all the jokes/documets with 10 likes
    db.collection('Jokes').find({likes : 10}).toArray().then((docs) => {
        console.log(JSON.stringify(docs, undefined, 2));

    }, (err)=> {
        if (err) {
function isBig(a) {
    if (a > 5000) {
        return "yes";
    }
}

console.log(isBig(5001)); //returns yes
console.log(isBig(42)); //returns undefined
            return console.log('Fetch error : ', err);

        }
    });

    //fetch all the users from the db
    db.collection('Users').find().toArray().then((docs) => {
        // print all the users 
        console.log(JSON.stringify(docs, undefined, 2));

    }, (err) => {
        if(err) {
function isBig(a) {
    if (a > 5000) {
        return "yes";
    }
}

console.log(isBig(5001)); //returns yes
console.log(isBig(42)); //returns undefined
            return console.log('Error Fetching users : ', err);

        }
    });


    //db.close();
});

1			// import MongoClient from the mongodb package
2			const {MongoClient} = require('mongodb');
3
4			// connect to mongodb server
5			MongoClient.connect('mongodb://localhost:27017/JokesApp', (err, db) => {
6			if (err) {
7			return console.log('Failed to connect to mongoDB server : ' + err);
			0 ignored issues – show Debugging Code introduced 2017-06-11 18:41 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...
8			}
9
10			console.log('Connected to MongoDB Server');
11
12			//get the count of jokes stored
13			db.collection('Jokes').count().then((count) => {
14			console.log(`Jokes count = ${count}\n`);
			0 ignored issues – show Debugging Code introduced 2017-06-11 18:41 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...
15			}, (err) => {
16			if (err) {
			0 ignored issues – show Complexity Best Practice introduced 2017-06-11 18:41 UTC by Report Bug Copy Issue Report There is no return statement if `err` 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...
17			return console.log('error getting count', err);
			0 ignored issues – show Debugging Code introduced 2017-06-11 18:41 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...
18			}
19			});
20
21			// fetch all the jokes/documets with 10 likes
22			db.collection('Jokes').find({likes : 10}).toArray().then((docs) => {
23			console.log(JSON.stringify(docs, undefined, 2));
			0 ignored issues – show Debugging Code introduced 2017-06-11 18:41 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...
24			}, (err)=> {
25			if (err) {
			0 ignored issues – show Complexity Best Practice introduced 2017-06-11 18:41 UTC by Report Bug Copy Issue Report There is no return statement if `err` 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...
26			return console.log('Fetch error : ', err);
			0 ignored issues – show Debugging Code introduced 2017-06-11 18:41 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...
27			}
28			});
29
30			//fetch all the users from the db
31			db.collection('Users').find().toArray().then((docs) => {
32			// print all the users
33			console.log(JSON.stringify(docs, undefined, 2));
			0 ignored issues – show Debugging Code introduced 2017-06-11 18:41 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...
34			}, (err) => {
35			if(err) {
			0 ignored issues – show Complexity Best Practice introduced 2017-06-11 18:41 UTC by Report Bug Copy Issue Report There is no return statement if `err` 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...
36			return console.log('Error Fetching users : ', err);
			0 ignored issues – show Debugging Code introduced 2017-06-11 18:41 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...
37			}
38			});
			0 ignored issues – show Best Practice introduced 2017-06-11 18:41 UTC by Report Bug Copy Issue Report There is no `return` statement in this branch, but you do return something in other branches. Did you maybe miss it? If you do not want to return anything, consider adding `return undefined;` explicitly. Loading history...
39
40			//db.close();
41			});

ashokdey / todo-rest

app/experiments/mongo-fetch.js A last analyzed 2017-06-14 04:20 UTC

Complexity

Size

Duplication

Importance

1 Function

Duplication Side-by-Side

Filter issues like

app/experiments/mongo-fetch.js A
last analyzed 2017-06-14 04:20 UTC