René's URL Explorer Experiment

{"@context":"https://schema.org","@type":"DiscussionForumPosting","headline":"Suggestion: Sonar S2129 Constructors should not be used to instantiate \"String\", \"BigInteger\", \"BigDecimal\" and primitive-wrapper classes","articleBody":"\r\nWhile analyzing your project using our Automatic bug fixing software, i noticed that sonar rule S2129 was sometimes violated. If you wish, I can offer you a correction in the form of a PR.\r\n\r\nhttps://sonarcloud.io/organizations/default/rules?open=java%3AS2129\u0026q=S2129\r\n\"Calling constructors for String, BigInteger, BigDecimal and the objects used to wrap primitives is less efficient and less clear than relying on autoboxing or valueOf.\r\nConsider simplifying when possible for more efficient and cleaner code.\"\r\n\r\nBelow are examples of the changes made by our software to correct this rule.\r\n\r\n    diff --git a/biojava-alignment/src/main/java/org/biojava/nbio/alignment/io/StockholmStructure.java b/biojava-alignment/src/main/java/org/biojava/nbio/alignment/io /StockholmStructure.java\r\n    index 97261433b..22783b43c 100644\r\n    --- a/biojava-alignment/src/main/java/org/biojava/nbio/alignment/io/StockholmStructure.java\r\n    +++ b/biojava-alignment/src/main/java/org/biojava/nbio/alignment/io/StockholmStructure.java\r\n    @@ -256.9 +256.8 @@ public class StockholmStructure {\r\n                         }\r\n                         String[] seqDetails = splitSeqName(sequencename);\r\n                         seq.setDescription(seqDetails[0]);\r\n    - seq.setBioBegin((seqDetails[1] == null || seqDetails[1].trim().equals(\"\") ? null: new Integer(\r\n    - seqDetails[1])));\r\n    - seq.setBioEnd((seqDetails[2] == null || seqDetails[2].trim().equals(\"\") ? null: new Integer(seqDetails[2])));\r\n    + seq.setBioBegin((seqDetails[1] == null || seqDetails[1].trim().equals(\"\") ? null : Integer.valueOf(seqDetails[1])));\r\n    + seq.setBioEnd((seqDetails[2] == null || seqDetails[2].trim().equals(\"\") ? null: Integer.valueOf(seqDetails[2])));\r\n\r\n                         seqs.add(seq);\r\n                 }\r\n\r\n","author":{"url":"https://github.com/jlerbsc","@type":"Person","name":"jlerbsc"},"datePublished":"2023-06-25T19:19:48.000Z","interactionStatistic":{"@type":"InteractionCounter","interactionType":"https://schema.org/CommentAction","userInteractionCount":2},"url":"https://github.com/1066/biojava/issues/1066"}