From 29571c0466e4ecbe26029dad88963743dc0a4603 Mon Sep 17 00:00:00 2001 From: Jagdish Prajapati Date: Wed, 9 Apr 2025 16:25:13 +0530 Subject: [PATCH 1/5] first push for leap issue --- analyzer-comments/java/leap/avoid_conditional_logic.md | 3 --- analyzer-comments/java/leap/avoid_if_statements.md | 3 +++ analyzer-comments/java/leap/avoid_redundant_ternary.md | 3 +++ 3 files changed, 6 insertions(+), 3 deletions(-) delete mode 100644 analyzer-comments/java/leap/avoid_conditional_logic.md create mode 100644 analyzer-comments/java/leap/avoid_if_statements.md create mode 100644 analyzer-comments/java/leap/avoid_redundant_ternary.md diff --git a/analyzer-comments/java/leap/avoid_conditional_logic.md b/analyzer-comments/java/leap/avoid_conditional_logic.md deleted file mode 100644 index c827be6e6..000000000 --- a/analyzer-comments/java/leap/avoid_conditional_logic.md +++ /dev/null @@ -1,3 +0,0 @@ -# avoid conditional logic - -Consider converting the `if`-statement(s) and/or ternary expressions into a single expression using the `&&` and `||` operators. diff --git a/analyzer-comments/java/leap/avoid_if_statements.md b/analyzer-comments/java/leap/avoid_if_statements.md new file mode 100644 index 000000000..ac89e96a2 --- /dev/null +++ b/analyzer-comments/java/leap/avoid_if_statements.md @@ -0,0 +1,3 @@ +# avoid `if` statements + +Consider converting the if statement(s) into a single expression using logical operators (`&&`, `||`) or ternary operators. diff --git a/analyzer-comments/java/leap/avoid_redundant_ternary.md b/analyzer-comments/java/leap/avoid_redundant_ternary.md new file mode 100644 index 000000000..ec2951b78 --- /dev/null +++ b/analyzer-comments/java/leap/avoid_redundant_ternary.md @@ -0,0 +1,3 @@ +# avoid redundant ternary + +Try returning the conditions directly instead of returning boolean literals (true and flase). From 6bb674ce5247be1b262c98ed6685a9c01c0b6d79 Mon Sep 17 00:00:00 2001 From: Jagdish Prajapati Date: Sun, 13 Apr 2025 19:56:54 +0530 Subject: [PATCH 2/5] Typos and formatting fixes --- analyzer-comments/java/leap/avoid_if_statements.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/analyzer-comments/java/leap/avoid_if_statements.md b/analyzer-comments/java/leap/avoid_if_statements.md index ac89e96a2..405ccf56b 100644 --- a/analyzer-comments/java/leap/avoid_if_statements.md +++ b/analyzer-comments/java/leap/avoid_if_statements.md @@ -1,3 +1,3 @@ # avoid `if` statements -Consider converting the if statement(s) into a single expression using logical operators (`&&`, `||`) or ternary operators. +Consider converting the `if` statement(s) into a single expression using logical operators (`&&`, `||`) or ternary operators. From 5761bc1556e0008a2930834f75d9821aa906ea1a Mon Sep 17 00:00:00 2001 From: Jagdish Prajapati Date: Sun, 13 Apr 2025 20:58:32 +0530 Subject: [PATCH 3/5] Typos and formatting fixes --- analyzer-comments/java/leap/avoid_redundant_ternary.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/analyzer-comments/java/leap/avoid_redundant_ternary.md b/analyzer-comments/java/leap/avoid_redundant_ternary.md index ec2951b78..6abf881c9 100644 --- a/analyzer-comments/java/leap/avoid_redundant_ternary.md +++ b/analyzer-comments/java/leap/avoid_redundant_ternary.md @@ -1,3 +1,3 @@ # avoid redundant ternary -Try returning the conditions directly instead of returning boolean literals (true and flase). +Try returning the conditions directly instead of returning boolean literals (`true` and `false`). From 90bf39277603babe3bffb29a5108fb03dce2e34d Mon Sep 17 00:00:00 2001 From: Jagdish Prajapati Date: Sat, 26 Apr 2025 11:43:10 +0530 Subject: [PATCH 4/5] Adding avoid_multiple_ternary.md --- analyzer-comments/java/leap/avoid_multiple_ternary.md | 3 +++ analyzer-comments/java/leap/avoid_redundant_ternary.md | 3 --- 2 files changed, 3 insertions(+), 3 deletions(-) create mode 100644 analyzer-comments/java/leap/avoid_multiple_ternary.md delete mode 100644 analyzer-comments/java/leap/avoid_redundant_ternary.md diff --git a/analyzer-comments/java/leap/avoid_multiple_ternary.md b/analyzer-comments/java/leap/avoid_multiple_ternary.md new file mode 100644 index 000000000..ba827e5a4 --- /dev/null +++ b/analyzer-comments/java/leap/avoid_multiple_ternary.md @@ -0,0 +1,3 @@ +# avoid multiple ternary + +Consider using a single ternary operator for a more efficient solution. diff --git a/analyzer-comments/java/leap/avoid_redundant_ternary.md b/analyzer-comments/java/leap/avoid_redundant_ternary.md deleted file mode 100644 index 6abf881c9..000000000 --- a/analyzer-comments/java/leap/avoid_redundant_ternary.md +++ /dev/null @@ -1,3 +0,0 @@ -# avoid redundant ternary - -Try returning the conditions directly instead of returning boolean literals (`true` and `false`). From 11cb5214bc7482ca6c9369e6a8e526a541a44d80 Mon Sep 17 00:00:00 2001 From: jagdish-15 Date: Sun, 27 Apr 2025 20:58:50 +0530 Subject: [PATCH 5/5] Update analyzer-comments/java/leap/avoid_multiple_ternary.md Co-authored-by: Kah Goh --- analyzer-comments/java/leap/avoid_multiple_ternary.md | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/analyzer-comments/java/leap/avoid_multiple_ternary.md b/analyzer-comments/java/leap/avoid_multiple_ternary.md index ba827e5a4..a02c9e786 100644 --- a/analyzer-comments/java/leap/avoid_multiple_ternary.md +++ b/analyzer-comments/java/leap/avoid_multiple_ternary.md @@ -1,3 +1,3 @@ # avoid multiple ternary -Consider using a single ternary operator for a more efficient solution. +Consider using a single ternary operator for a simpler solution.