attogram/8queens

A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute logic with side effects, but should not do both. The first symbol is defined on line 105 and the first side effect is on line 3.

Open

<?php

• Exclude checks
  • Disable engine
  • Disable check

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  if (cfg.hasOwnProperty('snapSpeed') !== true ||
      validAnimationSpeed(cfg.snapSpeed) !== true) {
    cfg.snapSpeed = 25;
  }

• Read up
  Read up

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

Open

  if (cfg.sparePieces === true) {
    html += '<div class="' + CSS.sparePieces + ' ' +
      CSS.sparePiecesTop + '"></div>';
  }

• Read up
  Read up

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

Open

  if (validSquare(location) === true) {
    $('#' + SQUARE_ELS_IDS[location]).addClass(CSS.highlight2);
  }

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  if (cfg.hasOwnProperty('snapbackSpeed') !== true ||
      validAnimationSpeed(cfg.snapbackSpeed) !== true) {
    cfg.snapbackSpeed = 50;
  }

• Read up
  Read up

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

Open

  if (validSquare(DRAGGED_PIECE_LOCATION) === true) {
    $('#' + SQUARE_ELS_IDS[DRAGGED_PIECE_LOCATION])
      .removeClass(CSS.highlight2);
  }

• Read up
  Read up

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

Open

  if (cfg.sparePieces === true) {
    html += '<div class="' + CSS.sparePieces + ' ' +
      CSS.sparePiecesBottom + '"></div>';
  }

• Read up
  Read up

Similar blocks of code found in 3 locations. Consider refactoring.

Open

  if (cfg.hasOwnProperty('moveSpeed') !== true ||
      validAnimationSpeed(cfg.moveSpeed) !== true) {
    cfg.moveSpeed = 200;
  }

• Read up
  Read up

Avoid too many return statements within this method.

Open

            return 'd';

Avoid too many return statements within this function.

Open

  return true;

Avoid too many return statements within this method.

Open

            return 4;

Avoid too many return statements within this function.

Open

    return false;

Avoid too many return statements within this method.

Open

            return 'f';

Avoid too many return statements within this method.

Open

            return 8;

Avoid too many return statements within this method.

Open

            return 'g';

Avoid too many return statements within this method.

Open

            return 'h';

Avoid too many return statements within this method.

Open

            return 6;

Avoid too many return statements within this method.

Open

            return 'e';

Avoid too many return statements within this method.

Open

            return 5;

Avoid too many return statements within this method.

Open

            return 7;