r4fterman/pdf.forms

View on GitHub
src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java

Summary

Maintainability
F
3 days
Test Coverage
F
43%

File SizeAndPositionPanel.java has 470 lines of code (exceeds 250 allowed). Consider refactoring.
Open

package org.pdf.forms.gui.properties.layout;

import static java.util.stream.Collectors.toUnmodifiableList;
import static org.jdesktop.layout.GroupLayout.BASELINE;
import static org.jdesktop.layout.GroupLayout.DEFAULT_SIZE;

    Method initializePanel has 109 lines of code (exceeds 25 allowed). Consider refactoring.
    Open

        private void initializePanel() {
            setBorder(BorderFactory.createTitledBorder("Size & Position"));
    
            final JLabel xLabel = new JLabel("X:");
            final JLabel widthLabel = new JLabel("Width:");

      SizeAndPositionPanel has 34 methods (exceeds 20 allowed). Consider refactoring.
      Open

      public class SizeAndPositionPanel extends JPanel {
      
          private static final String[] ANCHORS = {
                  "Top Left",
                  "Top Middle",

        Method createParallelGroup has 5 arguments (exceeds 4 allowed). Consider refactoring.
        Open

                    final JLabel labelForTextFieldA,
                    final JTextField textFieldA,
                    final JLabel labelForTextFieldB,
                    final JTextField textFieldB,
                    final GroupLayout groupLayout) {

          Method createSequentialGroup has 5 arguments (exceeds 4 allowed). Consider refactoring.
          Open

                      final JLabel labelForTextFieldA,
                      final JTextField textFieldA,
                      final JLabel labelForTextFieldB,
                      final JTextField textFieldB,
                      final GroupLayout groupLayout) {

            Remove this useless assignment to local variable "value".
            Open

                    value /= exponential;

            A dead store happens when a local variable is assigned a value that is not read by any subsequent instruction. Calculating or retrieving a value only to then overwrite it or throw it away, could indicate a serious error in the code. Even if it's not an error, it is at best a waste of resources. Therefore all calculated values should be used.

            Noncompliant Code Example

            i = a + b; // Noncompliant; calculation result not used before value is overwritten
            i = compute();
            

            Compliant Solution

            i = a + b;
            i += compute();
            

            Exceptions

            This rule ignores initializations to -1, 0, 1, null, true, false and "".

            See

            Define a constant instead of duplicating this literal "mixed" 15 times.
            Open

                    if (!xBox.getText().equals("mixed")) {

            Duplicated string literals make the process of refactoring error-prone, since you must be sure to update all occurrences.

            On the other hand, constants can be referenced from many places, but only need to be updated in a single place.

            Noncompliant Code Example

            With the default threshold of 3:

            public void run() {
              prepare("action1");                              // Noncompliant - "action1" is duplicated 3 times
              execute("action1");
              release("action1");
            }
            
            @SuppressWarning("all")                            // Compliant - annotations are excluded
            private void method1() { /* ... */ }
            @SuppressWarning("all")
            private void method2() { /* ... */ }
            
            public String method3(String a) {
              System.out.println("'" + a + "'");               // Compliant - literal "'" has less than 5 characters and is excluded
              return "";                                       // Compliant - literal "" has less than 5 characters and is excluded
            }
            

            Compliant Solution

            private static final String ACTION_1 = "action1";  // Compliant
            
            public void run() {
              prepare(ACTION_1);                               // Compliant
              execute(ACTION_1);
              release(ACTION_1);
            }
            

            Exceptions

            To prevent generating some false-positives, literals having less than 5 characters are excluded.

            This block of commented-out lines of code should be removed.
            Open

                    //xExpandToFitBox.setText(EXPAND_TO_FIT);

            Programmers should not comment out code as it bloats programs and reduces readability.

            Unused code should be deleted and can be retrieved from source control history if required.

            This block of commented-out lines of code should be removed.
            Open

                    //yExpandToFitBox.setText(EXPAND_TO_FIT);

            Programmers should not comment out code as it bloats programs and reduces readability.

            Unused code should be deleted and can be retrieved from source control history if required.

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

                    if (!yBox.getText().equals("mixed")) {
                        final String yText = yBox.getText().replace("cm", "");
                        final double customY = asDouble(yText);
                        yBox.setText(customY + " cm");
            
            
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 299..306

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

            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

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

                    if (!xBox.getText().equals("mixed")) {
                        final String xText = xBox.getText().replace("cm", "");
                        final double customX = asDouble(xText);
                        xBox.setText(customX + " cm");
            
            
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 308..315

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

            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

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

                    if (!heightBox.getText().equals("mixed")) {
                        final String heightText = heightBox.getText().replace("cm", "");
                        final double customHeight = asDouble(heightText);
                        heightBox.setText(customHeight + " cm");
            
            
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 323..330

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

            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

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

                    if (!widthBox.getText().equals("mixed")) {
                        final String widthText = widthBox.getText().replace("cm", "");
                        final double customWidth = asDouble(widthText);
                        widthBox.setText(customWidth + " cm");
            
            
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 332..339

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

            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

                private void setXCoordinate(final String xCordToUse) {
                    if (xCordToUse == null) {
                        return;
                    }
            
            
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 380..392

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

            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

                private void setYCoordinate(final String yCordToUse) {
                    if (yCordToUse == null) {
                        return;
                    }
            
            
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 366..378

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

            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

                private void setWidth(final String widthToUse) {
                    if (widthToUse == null) {
                        return;
                    }
            
            
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 408..420

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

            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

                private void setHeight(final String heightToUse) {
                    if (heightToUse == null) {
                        return;
                    }
            
            
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 394..406

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

            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 4 locations. Consider refactoring.
            Open

                private JToggleButton createRotate90DegreeButton() {
                    final JToggleButton rotateButton = new JToggleButton(new ImageIcon(getClass()
                            .getResource("/org/pdf/forms/res/Anchor Rotation 90.png")));
                    rotateButton.setEnabled(false);
                    rotateButton.setName("90");
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 204..211
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 213..220
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 231..238

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

            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 4 locations. Consider refactoring.
            Open

                private JToggleButton createRotate270DegreeButton() {
                    final JToggleButton rotateButton = new JToggleButton(new ImageIcon(getClass()
                            .getResource("/org/pdf/forms/res/Anchor Rotation 270.png")));
                    rotateButton.setEnabled(false);
                    rotateButton.setName("270");
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 213..220
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 222..229
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 231..238

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

            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 4 locations. Consider refactoring.
            Open

                private JToggleButton createRotate0DegreeButton() {
                    final JToggleButton rotateButton = new JToggleButton(new ImageIcon(getClass()
                            .getResource("/org/pdf/forms/res/Anchor Rotation 0.png")));
                    rotateButton.setEnabled(false);
                    rotateButton.setName("0");
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 204..211
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 213..220
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 222..229

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

            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 4 locations. Consider refactoring.
            Open

                private JToggleButton createRotate180DegreeButton() {
                    final JToggleButton rotateButton = new JToggleButton(new ImageIcon(getClass()
                            .getResource("/org/pdf/forms/res/Anchor Rotation 180.png")));
                    rotateButton.setEnabled(false);
                    rotateButton.setName("180");
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 204..211
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 222..229
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 231..238

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

            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

                        if (yExpandState != TriStateCheckBox.State.DONT_CARE) {
                            final String value = String.valueOf(yExpandState == TriStateCheckBox.State.SELECTED);
                            sizeAndPosition.setYExpandToFit(value);
                        }
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 552..555

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

            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

                        if (xExpandState != TriStateCheckBox.State.DONT_CARE) {
                            final String value = String.valueOf(xExpandState == TriStateCheckBox.State.SELECTED);
                            sizeAndPosition.setXExpandToFit(value);
                        }
            src/main/java/org/pdf/forms/gui/properties/layout/SizeAndPositionPanel.java on lines 557..560

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

            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

            Extra separation in import group before 'java.awt.*'
            Open

            import java.awt.*;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Wrong lexicographical order for 'java.util.Set' import. Should be before 'org.jdesktop.layout.LayoutStyle.RELATED'.
            Open

            import java.util.Set;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Member name 'xBox' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
            Open

                private JTextField xBox;

            Line is longer than 100 characters (found 115).
            Open

                            .map(widget -> widget.getWidgetModel().getProperties().getLayout().getSizeAndPosition().getHeight()

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Using the '.*' form of import should be avoided - javax.swing.*.
            Open

            import javax.swing.*;

            Checks that there are no import statements that use the * notation.

            Rationale: Importing all classes from a package or staticmembers from a class leads to tight coupling between packagesor classes and might lead to problems when a new version of alibrary introduces name clashes.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Parameter name 'xCordToUse' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
            Open

                private void setXCoordinate(final String xCordToUse) {

            Checks that method parameter names conform to a specified pattern.By using accessModifiers property it is possibleto specify different formats for methods at different visibility levels.

            To validate catch parameters please useCatchParameterName.

            To validate lambda parameters please useLambdaParameterName.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Line is longer than 100 characters (found 103).
            Open

                        final SizeAndPosition sizeAndPosition = widget.getWidgetModel().getProperties().getLayout()

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Line is longer than 100 characters (found 101).
            Open

                            final String value = String.valueOf(xExpandState == TriStateCheckBox.State.SELECTED);

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Wrong lexicographical order for 'java.awt.*' import. Should be before 'org.jdesktop.layout.LayoutStyle.RELATED'.
            Open

            import java.awt.*;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Wrong lexicographical order for 'java.awt.event.MouseEvent' import. Should be before 'org.jdesktop.layout.LayoutStyle.RELATED'.
            Open

            import java.awt.event.MouseEvent;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Member name 'xExpandToFitBox' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
            Open

                private TriStateCheckBox xExpandToFitBox;

            Member name 'yExpandToFitBox' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
            Open

                private TriStateCheckBox yExpandToFitBox;

            Line is longer than 100 characters (found 109).
            Open

                                                    .add(anchorLocationBox, PREFERRED_SIZE, DEFAULT_SIZE, PREFERRED_SIZE)

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Parameter name 'e' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
            Open

                private void updateRotation(final ActionEvent e) {

            Checks that method parameter names conform to a specified pattern.By using accessModifiers property it is possibleto specify different formats for methods at different visibility levels.

            To validate catch parameters please useCatchParameterName.

            To validate lambda parameters please useLambdaParameterName.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Line is longer than 100 characters (found 106).
            Open

                    widgets.forEach(widget -> widget.getWidgetModel().getProperties().getLayout().getSizeAndPosition()

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Extra separation in import group before 'javax.swing.*'
            Open

            import javax.swing.*;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Member name 'yBox' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
            Open

                private JTextField yBox;

            Line is longer than 100 characters (found 110).
            Open

                            .map(widget -> widget.getWidgetModel().getProperties().getLayout().getSizeAndPosition().getY()

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Line is longer than 100 characters (found 117).
            Open

                            .map(widget -> widget.getWidgetModel().getProperties().getLayout().getSizeAndPosition().getRotation()

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Wrong lexicographical order for 'java.awt.event.FocusEvent' import. Should be before 'org.jdesktop.layout.LayoutStyle.RELATED'.
            Open

            import java.awt.event.FocusEvent;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Using the '.*' form of import should be avoided - java.awt.*.
            Open

            import java.awt.*;

            Checks that there are no import statements that use the * notation.

            Rationale: Importing all classes from a package or staticmembers from a class leads to tight coupling between packagesor classes and might lead to problems when a new version of alibrary introduces name clashes.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Wrong lexicographical order for 'javax.swing.*' import. Should be before 'org.jdesktop.layout.LayoutStyle.RELATED'.
            Open

            import javax.swing.*;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Extra separation in import group before 'org.jdesktop.layout.GroupLayout'
            Open

            import org.jdesktop.layout.GroupLayout;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Line is longer than 100 characters (found 117).
            Open

                                                    .add(createSequentialGroup(xLabel, xBox, widthLabel, widthBox, groupLayout)))

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Line is longer than 100 characters (found 101).
            Open

                                            .add(anchorLocationBox, PREFERRED_SIZE, DEFAULT_SIZE, PREFERRED_SIZE)

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Line is longer than 100 characters (found 116).
            Open

                                            .add(createParallelGroup(widthLabel, widthBox, heightLabel, heightBox, groupLayout))

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Wrong lexicographical order for 'java.awt.event.ActionEvent' import. Should be before 'org.jdesktop.layout.LayoutStyle.RELATED'.
            Open

            import java.awt.event.ActionEvent;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Wrong lexicographical order for 'com.google.common.primitives.Doubles' import. Should be before 'org.slf4j.LoggerFactory'.
            Open

            import com.google.common.primitives.Doubles;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Wrong lexicographical order for 'java.util.Optional' import. Should be before 'org.jdesktop.layout.LayoutStyle.RELATED'.
            Open

            import java.util.Optional;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Line is longer than 100 characters (found 118).
            Open

                                                    .add(createSequentialGroup(yLabel, yBox, heightLabel, heightBox, groupLayout))

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Line is longer than 100 characters (found 106).
            Open

                    widgets.forEach(widget -> widget.getWidgetModel().getProperties().getLayout().getSizeAndPosition()

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Extra separation in import group before 'com.google.common.primitives.Doubles'
            Open

            import com.google.common.primitives.Doubles;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Line is longer than 100 characters (found 101).
            Open

                            final String value = String.valueOf(yExpandState == TriStateCheckBox.State.SELECTED);

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Wrong lexicographical order for 'com.google.common.primitives.Ints' import. Should be before 'org.slf4j.LoggerFactory'.
            Open

            import com.google.common.primitives.Ints;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Line is longer than 100 characters (found 114).
            Open

                            .map(widget -> widget.getWidgetModel().getProperties().getLayout().getSizeAndPosition().getWidth()

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Wrong lexicographical order for 'java.util.Collections' import. Should be before 'org.jdesktop.layout.LayoutStyle.RELATED'.
            Open

            import java.util.Collections;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Line is longer than 100 characters (found 110).
            Open

                            .map(widget -> widget.getWidgetModel().getProperties().getLayout().getSizeAndPosition().getX()

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Wrong lexicographical order for 'org.jdesktop.layout.GroupLayout' import. Should be before 'org.jdesktop.layout.LayoutStyle.RELATED'.
            Open

            import org.jdesktop.layout.GroupLayout;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Parameter name 'e' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
            Open

                private void updateAnchor(final ActionEvent e) {

            Checks that method parameter names conform to a specified pattern.By using accessModifiers property it is possibleto specify different formats for methods at different visibility levels.

            To validate catch parameters please useCatchParameterName.

            To validate lambda parameters please useLambdaParameterName.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Line is longer than 100 characters (found 115).
            Open

                            .map(widget -> widget.getWidgetModel().getProperties().getLayout().getSizeAndPosition().getAnchor()

            Checks for long lines.

            Rationale: Long lines are hard to read in printouts or if developershave limited screen space for the source code, e.g. if the IDEdisplays additional information like project tree, class hierarchy,etc.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Wrong lexicographical order for 'java.awt.event.FocusAdapter' import. Should be before 'org.jdesktop.layout.LayoutStyle.RELATED'.
            Open

            import java.awt.event.FocusAdapter;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Wrong lexicographical order for 'java.util.List' import. Should be before 'org.jdesktop.layout.LayoutStyle.RELATED'.
            Open

            import java.util.List;

            Checks that the groups of import declarations appear in the order specifiedby the user. If there is an import but its group is not specified in theconfiguration such an import should be placed at the end of the import list.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            Parameter name 'yCordToUse' must match pattern '^[a-z][a-z0-9][a-zA-Z0-9]*$'.
            Open

                private void setYCoordinate(final String yCordToUse) {

            Checks that method parameter names conform to a specified pattern.By using accessModifiers property it is possibleto specify different formats for methods at different visibility levels.

            To validate catch parameters please useCatchParameterName.

            To validate lambda parameters please useLambdaParameterName.

            This documentation is written and maintained by the Checkstyle community and is covered under the same license as the Checkstyle project.

            There are no issues that match your filters.

            Category
            Status