Skip to content

Commit d4548dc

Browse files
committed
Sonar
1 parent 7e26ed5 commit d4548dc

File tree

5 files changed

+18
-13
lines changed

5 files changed

+18
-13
lines changed

annotations/src/main/java/com/javadiscord/jdi/core/processor/validator/EventListenerValidator.java

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -416,10 +416,11 @@ private boolean validateMethodAnnotations(Method method) {
416416
for (Map.Entry<Class<? extends Annotation>, String[]> entry : EXPECTED_PARAM_TYPES_MAP
417417
.entrySet()) {
418418
Class<? extends Annotation> annotationClass = entry.getKey();
419-
if (method.isAnnotationPresent(annotationClass)) {
420-
if (!validateMethodParameters(method, entry.getValue())) {
421-
return false;
422-
}
419+
if (
420+
method.isAnnotationPresent(annotationClass)
421+
&& !validateMethodParameters(method, entry.getValue())
422+
) {
423+
return false;
423424
}
424425
}
425426
return true;

api/src/main/java/com/javadiscord/jdi/core/api/builders/command/CallbackMessage.java

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -35,8 +35,6 @@ public class CallbackMessage {
3535
@JsonProperty("poll")
3636
private Poll poll;
3737

38-
public CallbackMessage() {}
39-
4038
public boolean isTts() {
4139
return tts;
4240
}

api/src/main/java/com/javadiscord/jdi/internal/api/application_commands/CreateCommandRequest.java

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -4,6 +4,7 @@
44
import java.util.List;
55
import java.util.Map;
66
import java.util.Optional;
7+
import java.util.concurrent.atomic.AtomicReference;
78

89
import com.javadiscord.jdi.core.api.builders.command.CommandOption;
910
import com.javadiscord.jdi.core.api.builders.command.CommandOptionType;
@@ -23,11 +24,12 @@ public record CreateCommandRequest(
2324
@Override
2425
public DiscordRequestBuilder create() {
2526

26-
String path = "/applications/%s/commands".formatted(applicationId);
27+
AtomicReference<String> path =
28+
new AtomicReference<>("/applications/%s/commands".formatted(applicationId));
2729

28-
if (global.isPresent() && global.get()) {
29-
path = "/applications/%s/guilds/%s/commands".formatted(applicationId, guildId);
30-
}
30+
global.ifPresent(
31+
val -> path.set("/applications/%s/guilds/%s/commands".formatted(applicationId, guildId))
32+
);
3133

3234
Map<String, Object> body = new HashMap<>();
3335
body.put("name", name);
@@ -41,6 +43,6 @@ public DiscordRequestBuilder create() {
4143
return new DiscordRequestBuilder()
4244
.post()
4345
.body(body)
44-
.path(path);
46+
.path(path.get());
4547
}
4648
}

core/src/main/java/com/javadiscord/jdi/core/Constants.java

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,4 +23,8 @@ public class Constants {
2323
\\___/|____/___| Version 1.0
2424
""";
2525

26+
private Constants() {
27+
throw new UnsupportedOperationException("Utility class");
28+
}
29+
2630
}

example/lj-discord-bot/src/main/java/com/javadiscord/bot/utils/CurseWords.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,15 +5,15 @@
55

66
public class CurseWords {
77
public static boolean containsCurseWord(String text) {
8-
for (String s : curseWords) {
8+
for (String s : CURSE_WORDS) {
99
if (text.toLowerCase().contains(s.toLowerCase())) {
1010
return true;
1111
}
1212
}
1313
return false;
1414
}
1515

16-
private static final List<String> curseWords =
16+
private static final List<String> CURSE_WORDS =
1717
Arrays.asList(
1818
"fuck",
1919
"shit",

0 commit comments

Comments
 (0)