Clean code #26

Sign in to view logs

Triggered via push March 29, 2024 03:53

release/v8

Status Success

Total duration 2m 14s

Artifacts –

qodana.yml

on: push

Annotations

35 warnings and 2 notices

AutoCloseable used without 'try'-with-resources: src/main/java/com/severalcircles/flames/data/FlamesDataManager.java#L73

'FileOutputStream' used without 'try'-with-resources statement

AutoCloseable used without 'try'-with-resources: src/main/java/com/severalcircles/flames/data/FlamesDataManager.java#L118

'FileOutputStream' used without 'try'-with-resources statement

AutoCloseable used without 'try'-with-resources: src/main/java/com/severalcircles/flames/data/FlamesDataManager.java#L170

'FileInputStream' used without 'try'-with-resources statement

AutoCloseable used without 'try'-with-resources: src/main/java/com/severalcircles/flames/Flames.java#L190

'FileWriter' used without 'try'-with-resources statement

Class initializer may be 'static': src/main/java/com/severalcircles/flames/data/user/UserConfig.java#L15

Class initializer may be 'static'

Statement with empty body: src/main/java/com/severalcircles/flames/frontend/conversations/SparkStartEmbed.java#L33

`if` statement has empty body

Mismatched read and write of array: src/main/java/com/severalcircles/flames/data/FlamesDataManager.java#L312

Contents of array `newChannelWords` are written to, but never read

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/frontend/thanks/ThanksCommand.java#L32

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/util/StringUtil.java#L74

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/Flames.java#L194

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/frontend/data/user/LocaleCommand.java#L45

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/frontend/thanks/ThanksEmbed.java#L55

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/frontend/conversations/ConversationEmbed.java#L57

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/Flames.java#L124

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/events/MessageEvent.java#L68

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/events/SelectMenuEvent.java#L33

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/events/SelectMenuEvent.java#L36

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/data/FlamesDataManager.java#L50

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/frontend/data/user/embed/GetAlongEmbed.java#L33

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/conversations/Conversation.java#L159

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/frontend/data/user/GetAlongCommand.java#L26

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/frontend/data/user/UserContextData.java#L33

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/exception/handle/ExceptionHandler.java#L21

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/frontend/thanks/ThanksContext.java#L35

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/Flames.java#L72

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/frontend/data/user/GetAlongCommand.java#L29

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/exception/FlamesHandlerEmbed.java#L64

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/events/MessageEvent.java#L90

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/frontend/data/user/HiCommand.java#L51

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/Flames.java#L181

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/frontend/data/user/HiCommand.java#L44

Call to `printStackTrace()` should probably be replaced with more robust logging

Call to 'printStackTrace()': src/main/java/com/severalcircles/flames/frontend/message/twohundred/UpdatedSettingsEmbed.java#L36

Call to `printStackTrace()` should probably be replaced with more robust logging

Unused import: src/main/java/com/severalcircles/flames/frontend/conversations/SparkCommand.java#L13

Unused import `import java.io.IOException;`

Unused import: src/main/java/com/severalcircles/flames/frontend/conversations/SparkCommand.java#L9

Unused import `import com.severalcircles.flames.exception.ConsentException;`

Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.

Duplicated code fragment: src/main/java/com/severalcircles/flames/data/FlamesDataManager.java#L62

Duplicated code

Duplicated code fragment: src/main/java/com/severalcircles/flames/data/FlamesDataManager.java#L184

Duplicated code