index.js ➔ buildCharts - Code Metrics - Inspection of "Boom, self updating trail information and display" - abkfenris/sugarloaf - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 3eec9f...47bd99 )

by Alex

created 2016-12-05 12:35 UTC

index.js ➔ buildCharts B

↳ Parent: sugarloaf/static/js/index.js

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	98
Ratio	100 %

Importance

Changes	1
Bugs	0	Features	0

Metric	Value
cc	1
c	1
b	0
f	0
dl	98
loc	98
rs	8.3352
nc	1
nop	0

7 Functions

Rating	Name	Duplication	Size	Complexity
A	index.js ➔ ... ➔ sugarloaf.groomedDim.reduceCount	3	3	1
A	index.js ➔ ... ➔ sugarloaf.difficultyDim.reduceCount	3	3	1
A	index.js ➔ ... ➔ sugarloaf.difficultyChart.ordering	3	3	1
A	index.js ➔ ... ➔ sugarloaf.openDim.reduceCount	3	3	1
A	index.js ➔ ... ➔ sugarloaf.ndx.dimension	7	7	2
A	index.js ➔ ... ➔ sugarloaf.areaDim.reduceCount	3	3	1
A	index.js ➔ ... ➔ sugarloaf.snowmakingDim.reduceCount	3	3	1

How to fix Long Method

var filename_status = '/api/current',
    WIDTH = 300,
    HEIGHT = 200,
    MARGINS = {top: 10, left: 20, right: 20, bottom: 20},
    sugarloaf = {};

sugarloaf.difficulty_order = {
    'beginner': 1,
    'intermediate': 2,
    'black': 3,
    'double-black': 4,
    'terrain-park': 5
}

function buildCharts() {

    sugarloaf.ndx = crossfilter(sugarloaf.data.trails);

    sugarloaf.openDim = sugarloaf.ndx.dimension(function(d) {
        if (d.open) {
            return 'Open';
        } else {
            return 'Closed';
        };
    });
    sugarloaf.openGroup = sugarloaf.openDim.group().reduceCount(function(d) {
        return d.open;
    });
    sugarloaf.openChart = dc.rowChart('#chart-row-open');

    sugarloaf.openChart
        .width(WIDTH)
        .height(HEIGHT/2)
        .margins(MARGINS)
        .dimension(sugarloaf.openDim)
        .group(sugarloaf.openGroup)
        .elasticX(true);


    sugarloaf.groomedDim = sugarloaf.ndx.dimension(function(d) {
        if (d.groomed) {
            return 'Groomed';
        } else {
            return 'Ungroomed';
        }
    });
    sugarloaf.groomedGroup = sugarloaf.groomedDim.group().reduceCount(function(d) {
        return d.groomed;
    })
    sugarloaf.groomedChart = dc.rowChart('#chart-row-groomed');

    sugarloaf.groomedChart
        .width(WIDTH)
        .height(HEIGHT/2)
        .margins(MARGINS)
        .dimension(sugarloaf.groomedDim)
        .group(sugarloaf.groomedGroup)
        .elasticX(true);
    

    sugarloaf.snowmakingDim = sugarloaf.ndx.dimension(function(d) {
        if (d.snowmaking) {
            return 'Snowmaking in progress';
        } else {
            return 'Not snowmaking';
        }
    });
    sugarloaf.snowmakingGroup = sugarloaf.snowmakingDim.group().reduceCount(function(d) {
        return d.snowmaking;
    });
    sugarloaf.snowmakingChart = dc.rowChart('#chart-row-snowmaking');

    sugarloaf.snowmakingChart
        .width(WIDTH)
        .height(HEIGHT/2)
        .margins(MARGINS)
        .dimension(sugarloaf.snowmakingDim)
        .group(sugarloaf.snowmakingGroup)
        .elasticX(true);

    sugarloaf.difficultyDim = sugarloaf.ndx.dimension(function(d) {
        return d.difficulty;
    });
    sugarloaf.difficultyGroup = sugarloaf.difficultyDim.group().reduceCount(function(d) {
        return d.difficulty;
    });
    sugarloaf.difficultyChart = dc.rowChart('#chart-row-difficulty');

    sugarloaf.difficultyChart
        .width(WIDTH)
        .height(HEIGHT)
        .margins(MARGINS)
        .dimension(sugarloaf.difficultyDim)
        .group(sugarloaf.difficultyGroup)
        .ordering(function(d) {
            return sugarloaf.difficulty_order[d.key];
        })
        .elasticX(true);
    
    
    sugarloaf.areaDim = sugarloaf.ndx.dimension(function(d) {
        return d.area;
    });
    sugarloaf.areaGroup = sugarloaf.areaDim.group().reduceCount(function(d) {
        return d.area;
    });
    sugarloaf.areaChart = dc.rowChart('#chart-row-area');

    sugarloaf.areaChart
        .width(WIDTH)
        .height(HEIGHT * 2)
        .margins(MARGINS)
        .dimension(sugarloaf.areaDim)
        .group(sugarloaf.areaGroup)
        .elasticX(true);

    dc.renderAll();
}

d3.json(filename_status, function(data) {

    sugarloaf.data = data;

    buildCharts();
});

1		var filename_status = '/api/current',
2		WIDTH = 300,
3		HEIGHT = 200,
4		MARGINS = {top: 10, left: 20, right: 20, bottom: 20},
5		sugarloaf = {};
6
7		sugarloaf.difficulty_order = {
8		'beginner': 1,
9		'intermediate': 2,
10		'black': 3,
11		'double-black': 4,
12		'terrain-park': 5
13		}
14
15	View Code Duplication	function buildCharts() {
		0 ignored issues – show Duplication introduced 2016-12-05 12:37 UTC by Report Bug Copy Issue Report This code seems to be duplicated in your project. Loading history...
16		sugarloaf.ndx = crossfilter(sugarloaf.data.trails);
17
18		sugarloaf.openDim = sugarloaf.ndx.dimension(function(d) {
19		if (d.open) {
20		return 'Open';
21		} else {
22		return 'Closed';
23		};
24		});
25		sugarloaf.openGroup = sugarloaf.openDim.group().reduceCount(function(d) {
26		return d.open;
27		});
28		sugarloaf.openChart = dc.rowChart('#chart-row-open');
		0 ignored issues – show Bug introduced 2016-12-05 12:37 UTC by Report Bug Copy Issue Report The variable `dc` seems to be never declared. If this is a global, consider adding a `/** global: dc */` 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...
29		sugarloaf.openChart
30		.width(WIDTH)
31		.height(HEIGHT/2)
32		.margins(MARGINS)
33		.dimension(sugarloaf.openDim)
34		.group(sugarloaf.openGroup)
35		.elasticX(true);
36
37
38		sugarloaf.groomedDim = sugarloaf.ndx.dimension(function(d) {
39		if (d.groomed) {
40		return 'Groomed';
41		} else {
42		return 'Ungroomed';
43		}
44		});
45		sugarloaf.groomedGroup = sugarloaf.groomedDim.group().reduceCount(function(d) {
46		return d.groomed;
47		})
48		sugarloaf.groomedChart = dc.rowChart('#chart-row-groomed');
		0 ignored issues – show Bug introduced 2016-12-05 12:37 UTC by Report Bug Copy Issue Report The variable `dc` seems to be never declared. If this is a global, consider adding a `/** global: dc */` 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...
49		sugarloaf.groomedChart
50		.width(WIDTH)
51		.height(HEIGHT/2)
52		.margins(MARGINS)
53		.dimension(sugarloaf.groomedDim)
54		.group(sugarloaf.groomedGroup)
55		.elasticX(true);
56
57
58		sugarloaf.snowmakingDim = sugarloaf.ndx.dimension(function(d) {
59		if (d.snowmaking) {
60		return 'Snowmaking in progress';
61		} else {
62		return 'Not snowmaking';
63		}
64		});
65		sugarloaf.snowmakingGroup = sugarloaf.snowmakingDim.group().reduceCount(function(d) {
66		return d.snowmaking;
67		});
68		sugarloaf.snowmakingChart = dc.rowChart('#chart-row-snowmaking');
		0 ignored issues – show Bug introduced 2016-12-05 12:37 UTC by Report Bug Copy Issue Report The variable `dc` seems to be never declared. If this is a global, consider adding a `/** global: dc */` 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...
69		sugarloaf.snowmakingChart
70		.width(WIDTH)
71		.height(HEIGHT/2)
72		.margins(MARGINS)
73		.dimension(sugarloaf.snowmakingDim)
74		.group(sugarloaf.snowmakingGroup)
75		.elasticX(true);
76
77		sugarloaf.difficultyDim = sugarloaf.ndx.dimension(function(d) {
78		return d.difficulty;
79		});
80		sugarloaf.difficultyGroup = sugarloaf.difficultyDim.group().reduceCount(function(d) {
81		return d.difficulty;
82		});
83		sugarloaf.difficultyChart = dc.rowChart('#chart-row-difficulty');
		0 ignored issues – show Bug introduced 2016-12-05 12:37 UTC by Report Bug Copy Issue Report The variable `dc` seems to be never declared. If this is a global, consider adding a `/** global: dc */` 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...
84		sugarloaf.difficultyChart
85		.width(WIDTH)
86		.height(HEIGHT)
87		.margins(MARGINS)
88		.dimension(sugarloaf.difficultyDim)
89		.group(sugarloaf.difficultyGroup)
90		.ordering(function(d) {
91		return sugarloaf.difficulty_order[d.key];
92		})
93		.elasticX(true);
94
95
96		sugarloaf.areaDim = sugarloaf.ndx.dimension(function(d) {
97		return d.area;
98		});
99		sugarloaf.areaGroup = sugarloaf.areaDim.group().reduceCount(function(d) {
100		return d.area;
101		});
102		sugarloaf.areaChart = dc.rowChart('#chart-row-area');
		0 ignored issues – show Bug introduced 2016-12-05 12:37 UTC by Report Bug Copy Issue Report The variable `dc` seems to be never declared. If this is a global, consider adding a `/** global: dc */` 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...
103		sugarloaf.areaChart
104		.width(WIDTH)
105		.height(HEIGHT * 2)
106		.margins(MARGINS)
107		.dimension(sugarloaf.areaDim)
108		.group(sugarloaf.areaGroup)
109		.elasticX(true);
110
111		dc.renderAll();
112		}
113
114		d3.json(filename_status, function(data) {
		0 ignored issues – show Bug introduced 2016-12-05 12:37 UTC by Report Bug Copy Issue Report The variable `d3` seems to be never declared. If this is a global, consider adding a `/** global: d3 */` 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...
115		sugarloaf.data = data;
116
117		buildCharts();
118		});

abkfenris / sugarloaf

GitHub Access Token became invalid

Push — master ( 3eec9f...47bd99 )

index.js ➔ buildCharts B

Complexity

Size

Duplication

Importance

7 Functions

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like