Skip to content

Commit 8528182

Browse files
authored
Fixed sonar warnings, updated readme.
1 parent 25b126d commit 8528182

File tree

7 files changed

+2455
-2451
lines changed

7 files changed

+2455
-2451
lines changed

README.md

Lines changed: 2436 additions & 2436 deletions
Large diffs are not rendered by default.

src/main/java/g0301_0400/s0313_super_ugly_number/Solution.java

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,6 +2,7 @@
22

33
// #Medium #Array #Dynamic_Programming #Math #2022_07_10_Time_16_ms_(96.57%)_Space_42.3_MB_(64.43%)
44

5+
@SuppressWarnings("java:S3012")
56
public class Solution {
67
public int nthSuperUglyNumber(int n, int[] primes) {
78
long[] primes1 = new long[primes.length];

src/main/java/g0401_0500/s0488_zuma_game/Solution.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -3,14 +3,15 @@
33
// #Hard #String #Dynamic_Programming #Breadth_First_Search #Memoization
44
// #2022_07_13_Time_1989_ms_(32.79%)_Space_56.5_MB_(81.97%)
55

6+
import java.util.ArrayDeque;
7+
import java.util.Deque;
68
import java.util.HashMap;
7-
import java.util.Stack;
89

910
public class Solution {
1011
private HashMap<String, Integer> map = new HashMap<>();
1112

1213
private String removeConsecutiveThreeOrMoreBalls(String board) {
13-
Stack<Character> st = new Stack<>();
14+
Deque<Character> st = new ArrayDeque<>();
1415
int n = board.length();
1516
for (int i = 0; i < n; i++) {
1617
char ch = board.charAt(i);

src/main/java/g0801_0900/s0828_count_unique_characters_of_all_substrings_of_a_given_string/Solution.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@ public int uniqueLetterString(String s) {
2222
prevLastCharIdx[s.charAt(0) - 'A'] = 0;
2323
for (int i = 1; i < len; i++) {
2424
char ch = s.charAt(i);
25-
int chIdx = (int) (ch - 'A');
25+
int chIdx = ch - 'A';
2626
int lastSeenIdx = lastCharIdx[chIdx];
2727
int prevLastIdx = prevLastCharIdx[chIdx];
2828
dp[i] = dp[i - 1] + 1;

src/main/java/g0901_1000/s0914_x_of_a_kind_in_a_deck_of_cards/Solution.java

Lines changed: 7 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -4,20 +4,21 @@
44
// #2022_07_14_Time_11_ms_(51.18%)_Space_52.7_MB_(22.45%)
55

66
import java.util.HashMap;
7+
import java.util.Map;
78

89
public class Solution {
910
public boolean hasGroupsSizeX(int[] deck) {
1011
HashMap<Integer, Integer> hmap = new HashMap<>();
11-
for (int i = 0; i < deck.length; i++) {
12-
if (hmap.containsKey(deck[i])) {
13-
hmap.put(deck[i], hmap.get(deck[i]) + 1);
12+
for (int j : deck) {
13+
if (hmap.containsKey(j)) {
14+
hmap.put(j, hmap.get(j) + 1);
1415
} else {
15-
hmap.put(deck[i], 1);
16+
hmap.put(j, 1);
1617
}
1718
}
1819
int x = hmap.get(deck[0]);
19-
for (Integer i : hmap.keySet()) {
20-
x = gcd(x, hmap.get(i));
20+
for (Map.Entry<Integer, Integer> entry : hmap.entrySet()) {
21+
x = gcd(x, entry.getValue());
2122
}
2223
return x >= 2;
2324
}

src/main/java/g1101_1200/s1110_delete_nodes_and_return_forest/Solution.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -55,7 +55,7 @@ public List<TreeNode> delNodes(TreeNode root, int[] localToDelete) {
5555
}
5656
nodes.add(root);
5757
List<TreeNode> forests = new ArrayList<>();
58-
while (nodes.size() > 0) {
58+
while (!nodes.isEmpty()) {
5959
TreeNode node = nodes.poll();
6060
node = deleteAndSplit(node);
6161
if (node != null) {

src/main/java/g1201_1300/s1260_shift_2d_grid/Solution.java

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -3,28 +3,29 @@
33
// #Easy #Array #Matrix #Simulation #2022_07_10_Time_6_ms_(89.10%)_Space_54.8_MB_(44.55%)
44

55
import java.util.ArrayList;
6+
import java.util.Collections;
67
import java.util.List;
78

89
public class Solution {
910
public List<List<Integer>> shiftGrid(int[][] grid, int k) {
1011
if (grid == null) {
11-
return null;
12+
return Collections.emptyList();
1213
}
1314
int[] flat = new int[grid.length * grid[0].length];
1415
int index = 0;
15-
for (int i = 0; i < grid.length; ++i) {
16+
for (int[] ints : grid) {
1617
for (int j = 0; j < grid[0].length; ++j) {
17-
flat[index++] = grid[i][j];
18+
flat[index++] = ints[j];
1819
}
1920
}
2021
int mode = k % flat.length;
2122
int readingIndex = flat.length - mode;
2223
if (readingIndex == flat.length) {
2324
readingIndex = 0;
2425
}
25-
List<List<Integer>> result = new ArrayList<List<Integer>>();
26+
List<List<Integer>> result = new ArrayList<>();
2627
for (int i = 0; i < grid.length; ++i) {
27-
List<Integer> eachRow = new ArrayList<Integer>();
28+
List<Integer> eachRow = new ArrayList<>();
2829
for (int j = 0; j < grid[0].length; ++j) {
2930
eachRow.add(flat[readingIndex++]);
3031
if (readingIndex == flat.length) {

0 commit comments

Comments
 (0)