Reactive-Extensions/RxJS

File compiler.ts has 966 lines of code (exceeds 250 allowed). Consider refactoring.

Open

class Expression {
    nodeType: ExpressionType;
 
    constructor(nodeType: ExpressionType) {
        this.nodeType = nodeType;

Expression has 36 functions (exceeds 20 allowed). Consider refactoring.

Open

class Expression {
    nodeType: ExpressionType;
 
    constructor(nodeType: ExpressionType) {
        this.nodeType = nodeType;

Function visitBinary has 64 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    visitBinary(node: BinaryExpression): Expression {
        this.visit(node.left);
        this.visit(node.right);
 
        var r = this._stack.pop();

Function visitBinary has 58 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    visitBinary(node: BinaryExpression): any {
        var i = "";
 
        switch (node.nodeType) {
            case ExpressionType.Add:

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

Open

        switch (node.nodeType) {
            case ExpressionType.Add:
                i = "+";
                break;
            case ExpressionType.Subtract:

• Read up
  Read up

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

Open

        switch (node.nodeType) {
            case ExpressionType.Add:
                i = "+";
                break;
            case ExpressionType.Subtract:

• Read up
  Read up

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

Open

class IndexExpression extends Expression {
    _expression: Expression;
    _args: Expression[];
 
    constructor(expression: Expression, args: Expression[]) {

• Read up
  Read up

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

Open

class InvocationExpression extends Expression {
    _expression: Expression;
    _args: Expression[];
 
    constructor(expression: Expression, args: Expression[]) {

• Read up
  Read up

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

Open

    visitMany<T extends Expression>(nodes: T[]): T[] {
        var res = new Array<T>(nodes.length);
 
        for (var i = 0; i < nodes.length; i++) {
            var oldNode = nodes[i];

• Read up
  Read up

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

Open

    visitMany<E extends Expression>(nodes: E[]): T[] {
        var res = new Array<T>(nodes.length);
 
        for (var i = 0; i < nodes.length; i++) {
            var oldNode = nodes[i];

• Read up
  Read up

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

Open

    visitLambda<T extends Function>(node: LambdaExpression<T>): Expression {
        this._stack.push(node.parameters);
 
        this.visit(node.body);
 
 

• Read up
  Read up

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

Open

    visitLambda<T extends Function>(node: LambdaExpression<T>): Expression {
        this._stack.push(node.parameters);
 
        this.visit(node.body);
 
 

• Read up
  Read up

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

Open

        switch (node.nodeType) {
            case ExpressionType.Negate:
                i = "-";
                break;
            case ExpressionType.UnaryPlus:

• Read up
  Read up

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

Open

    visitLambda<T extends Function>(node: LambdaExpression<T>): string {
        var body = this.visit(node.body);
        var children = this.visitMany(node.parameters);
        children.unshift(body);
        return "Lambda(" + children.join(", ") + ")";

• Read up
  Read up

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

Open

        switch (node.nodeType) {
            case ExpressionType.Negate:
                i = "-";
                break;
            case ExpressionType.UnaryPlus:

• Read up
  Read up

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

Open

    visitInvoke(node: InvocationExpression): string {
        var expression = this.visit(node.expression);
        var children = this.visitMany(node.args);
        children.unshift(expression);
        return "Invoke(" + children.join(", ") + ")";

• Read up
  Read up

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

Open

        for (var i = this._stack.length - 1; i >= 0; i--) {
            if (this._stack[i].indexOf(node) >= 0) {
                found = true;
                break;
            }

• Read up
  Read up

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

Open

        for (var i = this._stack.length - 1; i >= 0; i--) {
            if (this._stack[i].indexOf(node) >= 0) {
                found = true;
                break;
            }

• Read up
  Read up

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

Open

    visitCall(node: FunctionCallExpression): Expression {
        return node.update(this.visit(node.obj), this.visitMany(node.args));
    }

• Read up
  Read up

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

Open

    visitInvoke(node: InvocationExpression): Expression {
        return node.update(this.visit(node.expression), this.visitMany(node.args));
    }

• Read up
  Read up

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

Open

    visitBinary(node: BinaryExpression): Expression {
        return node.update(this.visit(node.left), this.visit(node.right));
    }

• Read up
  Read up

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

Open

    visitIndex(node: IndexExpression): Expression {
        return node.update(this.visit(node.obj), this.visitMany(node.args));
    }

• Read up
  Read up

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

Open

        for (var i = 0; i < n; i++) {
            args[n - i - 1] = this._stack.pop();
        }

• Read up
  Read up

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

Open

        for (var i = 0; i < n; i++) {
            args[n - i - 1] = this._stack.pop();
        }

• Read up
  Read up

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

Open

        for (var i = 0; i < n; i++) {
            args[n - i - 1] = this._stack.pop();
        }

• Read up
  Read up

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

Open

        for (var i = 0; i < n; i++) {
            args[n - i - 1] = this._stack.pop();
        }

• Read up
  Read up

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

Open

        for (var i = 0; i < n; i++) {
            args[n - i - 1] = this._stack.pop();
        }

• Read up
  Read up

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

Open

        if (node.obj !== null) {
            this.visit(node.obj);
            res = this._stack.pop() + ".";
        }

• Read up
  Read up

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

Open

        if (node.obj !== null) {
            this.visit(node.obj);
            res = this._stack.pop() + ".";
        }

• Read up
  Read up

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

Open

    constructor(test: Expression, ifTrue: Expression, ifFalse: Expression) {
        super(ExpressionType.Condition);
        this._test = test;
        this._ifTrue = ifTrue;
        this._ifFalse = ifFalse;

• Read up
  Read up

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

Open

    constructor(expression: Expression, methodName: string, args: Expression[]) {
        super(ExpressionType.Call);
        this._expression = expression;
        this._method = methodName;
        this._args = args;

• Read up
  Read up

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

Open

    visit(node: Expression): Expression {
        if (node === null) {
            return null;
        }
        return node.accept(this);

• Read up
  Read up

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

Open

    visit(node: Expression): T {
        if (node === null) {
            return null;
        }
        return node.acceptGeneric(this);

• Read up
  Read up