Sign upLogin

silentbalanceyh/vertx-zero

View on GitHub
Star

Showing 9,050 of 9,050 total issues

Method hashCode has 36 lines of code (exceeds 25 allowed). Consider refactoring.
Open

    @Override
    public int hashCode() {
        final int prime = 31;
        int result = 1;
        result = prime * result + ((this.key == null) ? 0 : this.key.hashCode());
Severity: Minor
Found in vertx-pin/zero-jet/src/main/java/cn/vertxup/jet/domain/tables/pojos/IService.java - About 1 hr to fix

    Method toString has 36 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

        @Override
    public String toString() {
        StringBuilder sb = new StringBuilder("IService (");

        sb.append(key);
    Severity: Minor
    Found in vertx-pin/zero-jet/src/main/java/cn/vertxup/jet/domain/tables/pojos/IService.java - About 1 hr to fix

      Method importFile has 36 lines of code (exceeds 25 allowed). Consider refactoring.
      Open

          @Address(Addr.User.IMPORT)
    public Future<Envelop> importFile(final Envelop request) {
        /* Import data here for result */
        final String filename = Ux.getString(request);
      Severity: Minor
      Found in vertx-pin/zero-rbac/src/main/java/cn/vertxup/rbac/api/FileActor.java - About 1 hr to fix

        Method printReport has 36 lines of code (exceeds 25 allowed). Consider refactoring.
        Open

            private void printReport(final JsonArray normalized) {
        /*
         * 先过滤，只包含 INVALID, REMAIN，滤掉 MATCH
         * 1. 条件1：过滤掉 MATCH 的，匹配的不呈现
         * 2. 条件2：$button 字段滤掉
        Severity: Minor
        Found in vertx-pin/zero-vie/src/main/jib/io/mature/extension/uca/console/AdjustUiInstruction.java - About 1 hr to fix

          Method fromJson has 36 lines of code (exceeds 25 allowed). Consider refactoring.
          Open

                  @Override
        public default IPInTicket fromJson(io.vertx.core.json.JsonObject json) {
                setOrThrow(this::setKey,json::getString,"KEY","java.lang.String");
                setOrThrow(this::setSerial,json::getString,"SERIAL","java.lang.String");
                setOrThrow(this::setType,json::getString,"TYPE","java.lang.String");
          Severity: Minor
          Found in vertx-pin/zero-psi/src/main/java/cn/vertxup/psi/domain/tables/interfaces/IPInTicket.java - About 1 hr to fix

            Method forwardDebt has 36 lines of code (exceeds 25 allowed). Consider refactoring.
            Open

                private Future<JsonObject> forwardDebt(final List<FPaymentItem> payments, final Set<String> endKeys) {
        return this.fetchDebt(payments).compose(debts -> {
            final List<FDebt> qUpdate = new ArrayList<>();
            /*
             * 针对 PaymentItem 按结算总金额进行分组
            Severity: Minor
            Found in vertx-pin/zero-fm/src/main/java/cn/vertxup/fm/service/end/PayService.java - About 1 hr to fix

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

                  @Me
    @Address(HighWay.Report.TICKET_LIST)
    public Future<JsonObject> list(final JsonObject qr) {
        LOG.Queue.info(this.getClass(), "Qr Report Input: {0}", qr.encode());
        // Status Must be in following
              Severity: Major
              Found in vertx-pin/zero-wf/src/main/java/cn/vertxup/workflow/api/ReportActor.java and 1 other location - About 1 hr to fix
              vertx-pin/zero-wf/src/main/java/cn/vertxup/workflow/api/QueueActor.java on lines 110..122

              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 96.

              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

                  @Address(HighWay.Queue.TASK_QUEUE)
    public Future<JsonObject> fetchQueue(final JsonObject qr) {
        LOG.Queue.info(this.getClass(), "Qr Queue Input: {0}", qr.encode());
        // Status Must be in following
        // -- PENDING
              Severity: Major
              Found in vertx-pin/zero-wf/src/main/java/cn/vertxup/workflow/api/QueueActor.java and 1 other location - About 1 hr to fix
              vertx-pin/zero-wf/src/main/java/cn/vertxup/workflow/api/ReportActor.java on lines 73..86

              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 96.

              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

              Method typeWrap has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.
              Open

                  private static <T> Class<T> typeWrap(final Class<T> type) {
        if (type == null) {
            return null;
        } else if (type.isPrimitive()) {
            if (boolean.class == type) {
              Severity: Minor
              Found in vertx-gaia/vertx-ams/src/main/java/io/horizon/util/HInstance.java - About 1 hr 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 wrapAop has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.
              Open

                  @SuppressWarnings("all")
    <T> Function<T, Future<T>> wrapAop(
        final Before before, final Function<T, Future<T>> executor, final After after,
        final JsonObject configuration) {
        Objects.requireNonNull(executor);
              Severity: Minor
              Found in vertx-gaia/vertx-ams/src/main/jib/io/horizon/uca/aop/AspectJObject.java - About 1 hr 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 invokeAsync has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.
              Open

                  static <T> Future<T> invokeAsync(final Object instance,
                                     final Method method,
                                     final Object... args) {
        /*
         * Analyzing method returnType first
              Severity: Minor
              Found in vertx-gaia/vertx-co/src/main/java/io/vertx/up/util/Invoker.java - About 1 hr 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 transform has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.
              Open

                  private Frame transform(final Message<Object> msg, final Subscription subscription, final Object bodyData) {
        final String messageId = UUID.randomUUID().toString();

        final Frame frame = new Frame();
        frame.setCommand(Command.MESSAGE);
              Severity: Minor
              Found in vertx-ifx/zero-ifx-stomp/src/main/java/io/vertx/ext/stomp/impl/RemindDestination.java - About 1 hr 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 combineTodo has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.
              Open

                  private WTodo combineTodo(final WTodo todo, final JsonObject params) {
        if (Objects.isNull(todo)) {
            return null;
        } else {
            /*
              Severity: Minor
              Found in vertx-pin/zero-wf/src/main/java/cn/vertxup/workflow/service/TodoService.java - About 1 hr 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 combine has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.
              Open

                  private static void combine(final HRecord record, final String field, final JAmb amb, final RResult result) {
        /* Amb */
        final EmValue.Format format = result.format();
        if (EmValue.Format.JsonArray == format) {
            /*
              Severity: Minor
              Found in vertx-pin/zero-atom/src/main/modeler/io/modello/dynamic/modular/reference/RayResult.java - About 1 hr 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 connect has a Cognitive Complexity of 12 (exceeds 5 allowed). Consider refactoring.
              Open

                  static void connect(final Model model,
                        final String unique) {
        // 1. 读取 DataKey
        final DataKey key = DataKey.create(unique);
        // 2. 读取 Schema
              Severity: Minor
              Found in vertx-pin/zero-atom/src/main/modulat/io/vertx/mod/atom/modeling/builtin/ZERO.java - About 1 hr 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

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

                  static Set<String> vStringSet(final JsonArray array, final String field) {
        Set<String> set = new HashSet<>();
        if (Objects.nonNull(array)) {
            set = array.stream()
                .filter(item -> item instanceof JsonObject)
              Severity: Major
              Found in vertx-gaia/vertx-ams/src/main/java/io/horizon/util/TV.java and 1 other location - About 1 hr to fix
              vertx-gaia/vertx-ams/src/main/java/io/horizon/util/TV.java on lines 55..67

              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 95.

              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

                  @Override
    public ConcurrentMap<String, JsonObject> module() {
        if (this.modules.isEmpty()) {
            final Set<String> modules = this.moduleSet();
            modules.forEach(item -> {
              Severity: Major
              Found in vertx-gaia/vertx-macro/src/main/plugin/io/vertx/up/plugin/booting/AbstractBoot.java and 1 other location - About 1 hr to fix
              vertx-gaia/vertx-macro/src/main/plugin/io/vertx/up/plugin/booting/AbstractBoot.java on lines 87..100

              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 95.

              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 io.mature.extension.uca.concrete;

import io.vertx.core.Future;
import io.vertx.core.json.JsonArray;
import io.vertx.core.json.JsonObject;
              Severity: Major
              Found in vertx-pin/zero-vie/src/main/jib/io/mature/extension/uca/concrete/AgileEdit.java and 1 other location - About 1 hr to fix
              vertx-pin/zero-vie/src/main/jib/io/mature/extension/uca/concrete/AgileAdd.java on lines 1..27

              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 95.

              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 io.mature.extension.uca.concrete;

import io.vertx.core.Future;
import io.vertx.core.json.JsonArray;
import io.vertx.core.json.JsonObject;
              Severity: Major
              Found in vertx-pin/zero-vie/src/main/jib/io/mature/extension/uca/concrete/AgileAdd.java and 1 other location - About 1 hr to fix
              vertx-pin/zero-vie/src/main/jib/io/mature/extension/uca/concrete/AgileEdit.java on lines 1..27

              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 95.

              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

                  static Set<JsonArray> vArraySet(final JsonArray array, final String field) {
        Set<JsonArray> set = new HashSet<>();
        if (Objects.nonNull(array)) {
            set = array.stream()
                .filter(item -> item instanceof JsonObject)
              Severity: Major
              Found in vertx-gaia/vertx-ams/src/main/java/io/horizon/util/TV.java and 1 other location - About 1 hr to fix
              vertx-gaia/vertx-ams/src/main/java/io/horizon/util/TV.java on lines 29..41

              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 95.

              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

              Severity
              Category
              Status
              Source
              Language