Sign upLogin

weexteam/weex_devtools_android

View on GitHub
Star

Showing 2,519 of 2,519 total issues

File FastXmlSerializer.java has 331 lines of code (exceeds 250 allowed). Consider refactoring.
Open

/*
 * Copyright (C) 2006 The Android Open Source Project
 *
 * Licensed under the Apache License, Version 2.0 (the "License");
 * you may not use this file except in compliance with the License.
Severity: Minor
Found in appframework/src/main/java/com/alibaba/weex/commons/util/FastXmlSerializer.java - About 3 hrs to fix

    Method setImage has 98 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

        @Override
    public void setImage(final String url, final ImageView view,
                         WXImageQuality quality, WXImageStrategy strategy) {

        WXSDKManager.getInstance().postOnUiThread(new Runnable() {
    Severity: Major
    Found in appframework/src/main/java/com/alibaba/weex/commons/adapter/FrescoImageAdapter.java - About 3 hrs to fix

      AbsWeexActivity has 31 methods (exceeds 20 allowed). Consider refactoring.
      Open

      public abstract class AbsWeexActivity extends AppCompatActivity implements IWXRenderListener {
  private static final String TAG = "AbsWeexActivity";
  protected BroadcastReceiver mBroadcastReceiver;
  protected ViewGroup mContainer;
  protected WXSDKInstance mInstance;
      Severity: Minor
      Found in appframework/src/main/java/com/alibaba/weex/commons/AbsWeexActivity.java - About 3 hrs to fix

        DOM has 31 methods (exceeds 20 allowed). Consider refactoring.
        Open

        public class DOM implements ChromeDevtoolsDomain {
  private static boolean sNativeMode = true;
  private final ObjectMapper mObjectMapper;
  private final Document mDocument;
  private final Map<String, List<Integer>> mSearchResults;
        Severity: Minor
        Found in inspector/src/main/java/com/taobao/weex/devtools/inspector/protocol/module/DOM.java - About 3 hrs to fix

          Method writeValueXml has a Cognitive Complexity of 26 (exceeds 5 allowed). Consider refactoring.
          Open

              private static final void writeValueXml(Object v, String name, XmlSerializer out,
                                            WriteMapCallback callback)  throws XmlPullParserException, java.io.IOException {
        String typeStr;
        if (v == null) {
            out.startTag(null, "null");
          Severity: Minor
          Found in appframework/src/main/java/com/alibaba/weex/commons/util/XmlUtils.java - About 3 hrs to fix

          Cognitive Complexity

          Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

          A method's cognitive complexity is based on a few simple rules:

          • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
          • Code is considered more complex for each "break in the linear flow of the code"
          • Code is considered more complex when "flow breaking structures are nested"

          Further reading

          Function 0 has 90 lines of code (exceeds 25 allowed). Consider refactoring.
          Open

          /***/ function(module, exports, __webpack_require__) {

    ;__weex_define__("@weex-component/6be669a40db0f3641576862213ed29fa", [], function(__weex_require__, __weex_exports__, __weex_module__){
    __webpack_require__(18);
          Severity: Major
          Found in playground/app/src/main/assets/showcase/new-fashion/banners.js - About 3 hrs to fix

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

            /***/ function(module, exports, __webpack_require__) {

    ;__weex_define__("@weex-component/bc3fc70627d1feaf83ef4ae9faa6c91e", [], function(__weex_require__, __weex_exports__, __weex_module__){
    __webpack_require__(18);
    __webpack_require__(19);
            Severity: Major
            Found in playground/app/src/main/assets/showcase/new-fashion/fashion.js - About 3 hrs to fix

              Method readThisValueXml has 87 lines of code (exceeds 25 allowed). Consider refactoring.
              Open

                  private static final Object readThisValueXml(XmlPullParser parser, String[] name,
                                                 ReadMapCallback callback)  throws XmlPullParserException, java.io.IOException {
        final String valueName = parser.getAttributeValue(null, "name");
        final String tagName = parser.getName();
              Severity: Major
              Found in appframework/src/main/java/com/alibaba/weex/commons/util/XmlUtils.java - About 3 hrs to fix

                Function 0 has 87 lines of code (exceeds 25 allowed). Consider refactoring.
                Open

                /***/ function(module, exports, __webpack_require__) {

    ;__weex_define__("@weex-component/82549c19c22bf0499e7fb5d202c2b2c1", [], function(__weex_require__, __weex_exports__, __weex_module__){
    __webpack_require__(19);
                Severity: Major
                Found in playground/app/src/main/assets/showcase/new-fashion/brand.js - About 3 hrs to fix

                  Function 19 has 87 lines of code (exceeds 25 allowed). Consider refactoring.
                  Open

                  /***/ function(module, exports, __webpack_require__) {

    ;__weex_define__("@weex-component/banners", [], function(__weex_require__, __weex_exports__, __weex_module__){
    __webpack_require__(18);
                  Severity: Major
                  Found in playground/app/src/main/assets/showcase/new-fashion/brand.js - About 3 hrs to fix

                    Function 19 has 87 lines of code (exceeds 25 allowed). Consider refactoring.
                    Open

                    /***/ function(module, exports, __webpack_require__) {

    ;__weex_define__("@weex-component/banners", [], function(__weex_require__, __weex_exports__, __weex_module__){
    __webpack_require__(18);
                    Severity: Major
                    Found in playground/app/src/main/assets/showcase/new-fashion/fashion.js - About 3 hrs to fix

                      Function 19 has 87 lines of code (exceeds 25 allowed). Consider refactoring.
                      Open

                      /***/ function(module, exports, __webpack_require__) {

    ;__weex_define__("@weex-component/banners", [], function(__weex_require__, __weex_exports__, __weex_module__){
    __webpack_require__(18);
                      Severity: Major
                      Found in playground/app/src/main/assets/showcase/new-fashion/resource.js - About 3 hrs to fix

                        Function 19 has 87 lines of code (exceeds 25 allowed). Consider refactoring.
                        Open

                        /***/ function(module, exports, __webpack_require__) {

    ;__weex_define__("@weex-component/banners", [], function(__weex_require__, __weex_exports__, __weex_module__){
    __webpack_require__(18);
                        Severity: Major
                        Found in playground/app/src/main/assets/showcase/new-fashion/headlines.js - About 3 hrs to fix

                          File Page.java has 311 lines of code (exceeds 250 allowed). Consider refactoring.
                          Open

                          /*
 * Copyright (c) 2014-present, Facebook, Inc.
 * All rights reserved.
 *
 * This source code is licensed under the BSD-style license found in the
                          Severity: Minor
                          Found in inspector/src/main/java/com/taobao/weex/devtools/inspector/protocol/module/Page.java - About 3 hrs to fix

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

                            package com.alibaba.weex.uitest.TC_AG;
import com.alibaba.weex.WXPageActivity;
import com.alibaba.weex.util.TestFlow;
import java.util.HashMap;
import org.junit.Before;
                            Severity: Major
                            Found in playground/app/src/androidTest/java/com/alibaba/weex/uitest/TC_AG/AG_Input_Input_Type.java and 1 other location - About 3 hrs to fix
                            playground/app/src/androidTest/java/com/alibaba/weex/uitest/TC_AG/AG_Text_Text_Text_Overflow.java on lines 1..57

                            Duplicated Code

                            Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                            Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                            When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                            Tuning

                            This issue has a mass of 216.

                            We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                            The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                            If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                            See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                            Refactorings

                            Further Reading

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

                            package com.alibaba.weex.uitest.TC_AG;
import com.alibaba.weex.WXPageActivity;
import com.alibaba.weex.util.TestFlow;
import java.util.HashMap;
import org.junit.Before;
                            Severity: Major
                            Found in playground/app/src/androidTest/java/com/alibaba/weex/uitest/TC_AG/AG_Text_Text_Text_Overflow.java and 1 other location - About 3 hrs to fix
                            playground/app/src/androidTest/java/com/alibaba/weex/uitest/TC_AG/AG_Input_Input_Type.java on lines 1..57

                            Duplicated Code

                            Duplicated code can lead to software that is hard to understand and difficult to change. The Don't Repeat Yourself (DRY) principle states:

                            Every piece of knowledge must have a single, unambiguous, authoritative representation within a system.

                            When you violate DRY, bugs and maintenance problems are sure to follow. Duplicated code has a tendency to both continue to replicate and also to diverge (leaving bugs as two similar implementations differ in subtle ways).

                            Tuning

                            This issue has a mass of 216.

                            We set useful threshold defaults for the languages we support but you may want to adjust these settings based on your project guidelines.

                            The threshold configuration represents the minimum mass a code block must have to be analyzed for duplication. The lower the threshold, the more fine-grained the comparison.

                            If the engine is too easily reporting duplication, try raising the threshold. If you suspect that the engine isn't catching enough duplication, try lowering the threshold. The best setting tends to differ from language to language.

                            See codeclimate-duplication's documentation for more information about tuning the mass threshold in your .codeclimate.yml.

                            Refactorings

                            Further Reading

                            File WeexUiTestCaseTcElementOpt.java has 305 lines of code (exceeds 250 allowed). Consider refactoring.
                            Open

                            package com.alibaba.weex.uitest;

import android.app.Application;
import android.app.Instrumentation;
import android.content.Intent;
                            Severity: Minor
                            Found in playground/app/src/androidTest/java/com/alibaba/weex/uitest/WeexUiTestCaseTcElementOpt.java - About 3 hrs to fix

                              File CSS.java has 304 lines of code (exceeds 250 allowed). Consider refactoring.
                              Open

                              /*
 * Copyright (c) 2014-present, Facebook, Inc.
 * All rights reserved.
 *
 * This source code is licensed under the BSD-style license found in the
                              Severity: Minor
                              Found in inspector/src/main/java/com/taobao/weex/devtools/inspector/protocol/module/CSS.java - About 3 hrs to fix

                                Method readThisLongArrayXml has a Cognitive Complexity of 23 (exceeds 5 allowed). Consider refactoring.
                                Open

                                    public static final long[] readThisLongArrayXml(XmlPullParser parser,
                                                    String endTag, String[] name)
        throws XmlPullParserException, java.io.IOException {

        int num;
                                Severity: Minor
                                Found in appframework/src/main/java/com/alibaba/weex/commons/util/XmlUtils.java - About 3 hrs to fix

                                Cognitive Complexity

                                Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                                A method's cognitive complexity is based on a few simple rules:

                                • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                                • Code is considered more complex for each "break in the linear flow of the code"
                                • Code is considered more complex when "flow breaking structures are nested"

                                Further reading

                                Method readThisIntArrayXml has a Cognitive Complexity of 23 (exceeds 5 allowed). Consider refactoring.
                                Open

                                    public static final int[] readThisIntArrayXml(XmlPullParser parser,
                                                  String endTag, String[] name)
        throws XmlPullParserException, java.io.IOException {

        int num;
                                Severity: Minor
                                Found in appframework/src/main/java/com/alibaba/weex/commons/util/XmlUtils.java - About 3 hrs to fix

                                Cognitive Complexity

                                Cognitive Complexity is a measure of how difficult a unit of code is to intuitively understand. Unlike Cyclomatic Complexity, which determines how difficult your code will be to test, Cognitive Complexity tells you how difficult your code will be to read and comprehend.

                                A method's cognitive complexity is based on a few simple rules:

                                • Code is not considered more complex when it uses shorthand that the language provides for collapsing multiple statements into one
                                • Code is considered more complex for each "break in the linear flow of the code"
                                • Code is considered more complex when "flow breaking structures are nested"

                                Further reading

                                Severity
                                Category
                                Status
                                Source
                                Language