adobe/brackets

File main.js has 445 lines of code (exceeds 250 allowed). Consider refactoring.

Open

/*
 * Copyright (c) 2013 - present Adobe Systems Incorporated. All rights reserved.
 *
 * Permission is hereby granted, free of charge, to any person obtaining a
 * copy of this software and associated documentation files (the "Software"),

Function _getUrlList has 89 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    UrlCodeHints.prototype._getUrlList = function (query) {
        var directory,
            doc,
            docDir,
            queryDir = "",

Function getHints has 89 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    UrlCodeHints.prototype.getHints = function (key) {
        var mode = this.editor.getModeForSelection(),
            cursor = this.editor.getCursorPos(),
            filter = "",
            hints = [],

Function insertCssHint has 62 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    UrlCodeHints.prototype.insertCssHint = function (completion) {
        var cursor = this.editor.getCursorPos(),
            start  = { line: cursor.line, ch: cursor.ch },
            end    = { line: cursor.line, ch: cursor.ch };

Function insertHtmlHint has 44 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    UrlCodeHints.prototype.insertHtmlHint = function (completion) {
        var cursor = this.editor.getCursorPos(),
            start = {line: -1, ch: -1},
            end = {line: -1, ch: -1},
            tagInfo = HTMLUtils.getTagInfo(this.editor, cursor),

Function hasHtmlHints has 32 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    UrlCodeHints.prototype.hasHtmlHints = function (editor, implicitChar) {
        var tagInfo,
            query,
            tokenType;

Avoid deeply nested control flow statements.

Open

                            if (self.cachedHints.deferred &&
                                    self.cachedHints.deferred.state() === "pending") {
                                self.cachedHints.deferred.reject();
                                self.cachedHints.deferred = null;
                            }

Avoid deeply nested control flow statements.

Open

                        if (hints[i].toLowerCase().indexOf(query) === 0) {
                            foundPrefix = true;
                            break;
                        }

Avoid deeply nested control flow statements.

Open

                            if (currentDeferred && currentDeferred.state() === "pending") {
                                currentDeferred.reject();
                            }

Avoid too many return statements within this function.

Open

            return deferred;

Avoid too many return statements within this function.

Open

        return null;

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

Open

                result = $.map(asyncHints, function (item) {
                    if (item.toLowerCase().indexOf(lowerCaseFilter) === 0) {
                        return item;
                    }
                }).sort(sortFunc);

• Read up
  Read up

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

Open

            result = $.map(hints, function (item) {
                if (item.toLowerCase().indexOf(lowerCaseFilter) === 0) {
                    return item;
                }
            }).sort(sortFunc);

• Read up
  Read up