gopheracademy/gcon

File highcharts-3d.src.js has 1146 lines of code (exceeds 250 allowed). Consider refactoring.

Open

// ==ClosureCompiler==
// @compilation_level SIMPLE_OPTIMIZATIONS

/**
 * @license Highcharts JS v4.1.9 (2015-10-07)

Function arc3d has 109 lines of code (exceeds 25 allowed). Consider refactoring.

Open

Highcharts.SVGRenderer.prototype.arc3d = function (shapeArgs) {

    shapeArgs.alpha *= deg2rad;
    shapeArgs.beta *= deg2rad;
    var result = this.g(),

Function arc3dPath has 98 lines of code (exceeds 25 allowed). Consider refactoring.

Open

Highcharts.SVGRenderer.prototype.arc3dPath = function (shapeArgs) {
    var cx = shapeArgs.x, // x coordinate of the center
        cy = shapeArgs.y, // y coordinate of the center
        start = shapeArgs.start, // start angle
        end = shapeArgs.end - 0.00001, // end angle

Function cuboid has 56 lines of code (exceeds 25 allowed). Consider refactoring.

Open

Highcharts.SVGRenderer.prototype.cuboid = function (shapeArgs) {

    var result = this.g(),
    paths = this.cuboidPath(shapeArgs);

Function perspective has 52 lines of code (exceeds 25 allowed). Consider refactoring.

Open

function perspective(points, chart, insidePlotArea) {
    var options3d = chart.options.chart.options3d,
        inverted = false,
        origin;

Function animate has 45 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    result.animate = function (args, duration, complete) {
        if (defined(args.end) || defined(args.start)) {
            this._shapeArgs = this.shapeArgs;

            Highcharts.SVGElement.prototype.animate.call(this, {

Function cuboidPath has 40 lines of code (exceeds 25 allowed). Consider refactoring.

Open

Highcharts.SVGRenderer.prototype.cuboidPath = function (shapeArgs) {
    var x = shapeArgs.x,
        y = shapeArgs.y,
        z = shapeArgs.z,
        h = shapeArgs.height,

Consider simplifying this complex logical expression.

Open

    if (this.is3d()) {
        var inverted = this.inverted,
            clipBox = this.clipBox,
            margin = this.margin,
            x = inverted ? 'y' : 'x',

Function curveTo has 8 arguments (exceeds 4 allowed). Consider refactoring.

Open

function curveTo(cx, cy, rx, ry, start, end, dx, dy) {

Avoid deeply nested control flow statements.

Open

                        if (point.graphic) {
                            point.graphic.animate(point.shapeArgs, series.options.animation);                    
                        }

Avoid deeply nested control flow statements.

Open

                        if (!reversed) {
                            if (point.stackY) {
                                point.shapeArgs.y = point.plotY + yAxis.translate(point.stackY);
                            } else {
                                point.shapeArgs.y = point.plotY + (point.negative ? -point.height : point.height);

Consider simplifying this complex logical expression.

Open

    if (args[0].chart.options3d && args[0].chart.options3d.enabled) {
        // Normalize alpha and beta to (-360, 360) range
        args[0].chart.options3d.alpha = (args[0].chart.options3d.alpha || 0) % 360;
        args[0].chart.options3d.beta = (args[0].chart.options3d.beta || 0) % 360;

Identical blocks of code found in 2 locations. Consider refactoring.

Open

(function (Highcharts) {
    /**
    Shorthands for often used function
*/ 
/**

• Read up
  Read up