yasshi2525/RushHour

Avoid deeply nested control flow statements.

Open

                    if rn != to {
                        var connected bool
                        for _, e := range rn.OutEdges {
                            if e.ToNode == to {
                                connected = true

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

Open

const useStyles = makeStyles((theme: Theme) =>
  createStyles({
    error: {
      color: theme.palette.error.main
    }

• Read up
  Read up

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

Open

const defaultValues: {
  pid: number;
  cid: number;
  mul: number;
} = {

• Read up
  Read up

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

Open

const reDefaultValues: {
  from: number;
  to: number;
  eid: number;
} = {

• Read up
  Read up

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

Open

  const onTouchMove = useCallback(
    (e: React.TouchEvent) => {
      if (pressed) {
        setNow(touch(e, model));
      }

• Read up
  Read up

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

Open

const rnDefaultValues: {
  pid: number;
  cid: number;
  mul: number;
} = {

• Read up
  Read up

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

Open

  const onMouseMove = useCallback(
    (e: React.MouseEvent) => {
      if (pressed) {
        setNow(mouse(e, model));
      }

• Read up
  Read up

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

Open

const useStyles = makeStyles((theme: Theme) =>
  createStyles({
    error: {
      color: theme.palette.error.main
    }

• Read up
  Read up

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

Open

  const onTouchMove = useCallback(
    (e: React.TouchEvent) => {
      if (pressed) {
        setNow(gravity(e, model));
      }

• Read up
  Read up

Method Auther.GetGitHubOAuthInfo has 6 return statements (exceeds 4 allowed).

Open

func (a *Auther) GetGitHubOAuthInfo(resState string, code string) (*OAuthInfo, error) {
    if resState != a.state {
        return nil, fmt.Errorf("invalid state")
    }
    ctx := context.Background()

Method Chunk.Has has 6 return statements (exceeds 4 allowed).

Open

func (ch *Chunk) Has(raw Entity) bool {
    id := reflect.ValueOf(raw.B().ID)
    switch obj := raw.(type) {
    case Localable:
        fieldName := obj.B().T.String()

Function ConnectRailNode has 6 return statements (exceeds 4 allowed).

Open

func ConnectRailNode(o *entities.Player, from *entities.RailNode, to *entities.RailNode, scale int) (*entities.DelegateRailEdge, error) {
    if err := CheckAuth(o, from); err != nil {
        return nil, err
    }
    if err := CheckAuth(o, to); err != nil {

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

Open

  setupAfterCallback() {
    super.setupAfterCallback();
    this.addAfterCallback(() => this.container.removeChild(this.graphics));
  }

• Read up
  Read up

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

Open

  setupAfterCallback() {
    super.setupAfterCallback();
    this.addAfterCallback(() => this.container.removeChild(this.sprite));
  }

• Read up
  Read up

Function WarnLongExec has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

func WarnLongExec(start time.Time, lock time.Time, max time.Duration, title string, verbose ...bool) {

Function PasswordSignUp has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

func PasswordSignUp(loginid string, name string, password string, hue int, lv entities.PlayerType) (*entities.Player, error) {

Function CreatePlayer has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

func CreatePlayer(loginid string, displayname string, password string, hue int, lv entities.PlayerType) (*entities.Player, error) {

Function ExtendRailNode has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

func ExtendRailNode(o *entities.Player, from *entities.RailNode,
    x float64, y float64, scale int) (*entities.DelegateRailNode, *entities.DelegateRailEdge, error) {

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

Open

      this.current.x =
        this.current.x * ratio + this.destination.x * (1 - ratio);

• Read up
  Read up

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

Open

      this.current.y =
        this.current.y * ratio + this.destination.y * (1 - ratio);

• Read up
  Read up