Style has 34 methods (exceeds 20 allowed). Consider refactoring.

Open

public class Style implements Cloneable, SendableEntityCreatorNoIndex {
    /** The Constant PROPERTY_NAME for Name of Style */
    public static final String PROPERTY_NAME = "name";
    private String name;

File Style.java has 295 lines of code (exceeds 250 allowed). Consider refactoring.

Open

package de.uniks.networkparser;

/*
NetworkParser
The MIT License

Method setValue has 51 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    @Override
    public boolean setValue(Object entity, String attribute, Object value, String type) {
        if (entity instanceof Style == false) {
            return false;
        }

Method getValue has a Cognitive Complexity of 15 (exceeds 5 allowed). Consider refactoring.

Open

    @Override
    public Object getValue(Object entity, String attrName) {
        String attribute;
        if (attrName == null || entity instanceof Style == false) {
            return false;

• Read up
  Read up

Method getValue has 45 lines of code (exceeds 25 allowed). Consider refactoring.

Open

    @Override
    public Object getValue(Object entity, String attrName) {
        String attribute;
        if (attrName == null || entity instanceof Style == false) {
            return false;

Method setValue has a Cognitive Complexity of 14 (exceeds 5 allowed). Consider refactoring.

Open

    @Override
    public boolean setValue(Object entity, String attribute, Object value, String type) {
        if (entity instanceof Style == false) {
            return false;
        }

• Read up
  Read up

Method setBorder has a Cognitive Complexity of 10 (exceeds 5 allowed). Consider refactoring.

Open

    public void setBorder(GUIPosition position, String width, String color) {
        GUILine border = this.borders.get(position);
        if (width != null) {
            if (border == null) {
                this.borders.put(position, new GUILine().withColor(color).withWidth(width));

• Read up
  Read up

Avoid too many return statements within this method.

Open

            return style.getForground();

Avoid too many return statements within this method.

Open

            return true;

Avoid too many return statements within this method.

Open

            return true;

Avoid too many return statements within this method.

Open

            return true;

Avoid too many return statements within this method.

Open

            return true;

Avoid too many return statements within this method.

Open

            return true;

Avoid too many return statements within this method.

Open

            return style.getFontSize();

Avoid too many return statements within this method.

Open

        return null;

Avoid too many return statements within this method.

Open

        return false;

Avoid too many return statements within this method.

Open

            return style.getBackground();

Avoid too many return statements within this method.

Open

            return true;

Avoid too many return statements within this method.

Open

            return style.isUnderline();

Avoid too many return statements within this method.

Open

            return style.getWidth();

Avoid too many return statements within this method.

Open

            return true;

Avoid too many return statements within this method.

Open

            return style.getHeight();

Avoid too many return statements within this method.

Open

            return style.getName();

Avoid too many return statements within this method.

Open

            return true;

Avoid too many return statements within this method.

Open

            return style.getAlignment();

These nested if statements could be combined

Open

            if (border.isCustomLine() == false) {
                this.borders.remove(position);
                this.propertyChange(PROPERTY_BORDER, null, this.borders);
            }

• Read up
  Read up
• Exclude checks
  • Disable engine
  • Disable check

void bar() {
 if (x) { // original implementation
 if (y) {
 // do stuff
 }
 }
}

void bar() {
 if (x && y) { // optimized implementation
 // do stuff
 }
}