Skip to content

Commit 722d3d3

Browse files
committed
made properties final
1 parent c060a01 commit 722d3d3

File tree

6 files changed

+9
-10
lines changed

6 files changed

+9
-10
lines changed

src/main/java/org/utplsql/cli/CommandProvider.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,17 +8,15 @@
88

99
public class CommandProvider {
1010

11-
private Map<String, ICommand> commands;
12-
private JCommander jCommander;
11+
private final Map<String, ICommand> commands = new HashMap<>();
12+
private final JCommander jCommander;
1313

1414
public CommandProvider( JCommander jCommander ) {
1515
this.jCommander = jCommander;
1616
init();
1717
}
1818

1919
private void init() {
20-
commands = new HashMap<>();
21-
2220
addCommand(new RunCommand());
2321
addCommand(new VersionInfoCommand());
2422
addCommand(new ReportersCommand());

src/main/java/org/utplsql/cli/ConnectionInfo.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,7 @@
55
public class ConnectionInfo {
66

77
public static final String COMMANDLINE_PARAM_DESCRIPTION = "<user>/<password>@//<host>[:<port>]/<service> OR <user>/<password>@<TNSName> OR <user>/<password>@<host>:<port>:<SID>";
8-
private String connectionInfo;
8+
private final String connectionInfo;
99

1010
public ConnectionInfo(String connectionInfo) {
1111
this.connectionInfo = connectionInfo;

src/main/java/org/utplsql/cli/HelpCommand.java

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,11 +7,12 @@
77
*/
88
public class HelpCommand implements ICommand {
99

10-
private String errorMessage;
11-
private JCommander jCommander;
10+
private final String errorMessage;
11+
private final JCommander jCommander;
1212

1313
public HelpCommand(JCommander jCommander) {
1414
this.jCommander = jCommander;
15+
this.errorMessage = null;
1516
}
1617

1718
public HelpCommand( JCommander jCommander, String errorMessage ) {

src/main/java/org/utplsql/cli/ReporterManager.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -118,5 +118,5 @@ public List<ReporterOptions> getReporterOptionsList() {
118118
return reporterOptionsList;
119119
}
120120

121-
public int getNumberOfReporters() { return reporterOptionsList.size(); };
121+
public int getNumberOfReporters() { return reporterOptionsList.size(); }
122122
}

src/main/java/org/utplsql/cli/datasource/TestedDataSourceProvider.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ interface ConnectStringPossibility {
2323

2424
private static final Logger logger = LoggerFactory.getLogger(TestedDataSourceProvider.class);
2525
private final ConnectionConfig config;
26-
private List<ConnectStringPossibility> possibilities = new ArrayList<>();
26+
private final List<ConnectStringPossibility> possibilities = new ArrayList<>();
2727

2828
public TestedDataSourceProvider(ConnectionConfig config) {
2929
this.config = config;

src/main/java/org/utplsql/cli/log/StringBlockFormatter.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@
33
public class StringBlockFormatter {
44

55
private String headline;
6-
private StringBuilder content = new StringBuilder();
6+
private final StringBuilder content = new StringBuilder();
77

88
public StringBlockFormatter() {}
99

0 commit comments

Comments
 (0)