Skip to content

Commit 6af6819

Browse files
author
Quim
committed
code smell fixes
1 parent 0aeb07c commit 6af6819

File tree

5 files changed

+15
-42
lines changed

5 files changed

+15
-42
lines changed

src/main/java/com/essi/Dependency/Application.java

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -11,27 +11,17 @@
1111
import com.essi.Dependency.Service.DependencyService;
1212
import com.essi.Dependency.Service.StorageProperties;
1313

14-
import springfox.documentation.builders.ApiInfoBuilder;
15-
import springfox.documentation.builders.PathSelectors;
16-
import springfox.documentation.builders.RequestHandlerSelectors;
17-
import springfox.documentation.service.ApiInfo;
18-
import springfox.documentation.service.Contact;
19-
import springfox.documentation.spi.DocumentationType;
20-
import springfox.documentation.spring.web.plugins.Docket;
21-
2214
@SpringBootApplication
23-
// @PropertySource({"classpath:application.properties"})
2415
@EnableConfigurationProperties(StorageProperties.class)
2516
public class Application {
26-
public static void main(String[] args) throws IOException {
17+
public static void main(String[] args) {
2718
SpringApplication.run(Application.class, args);
2819
}
2920

3021
@Bean
3122
CommandLineRunner init(DependencyService storageService) {
3223
return (args) -> {
3324
storageService.deleteAll();
34-
//storageService.init();
3525
};
3626
}
3727

src/main/java/com/essi/Dependency/Components/Clause.java

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -15,11 +15,11 @@ public class Clause {
1515
private String subsect;
1616
private String parag;
1717
private String subparg;
18-
private String clause;
18+
private String clauseString;
1919
private int id;
2020

2121
public Clause(String doc, String vol, String part, String sect, String subsect, String parag, String subparg,
22-
String clause, int id) {
22+
String clauseString, int id) {
2323
super();
2424
this.doc = doc;
2525
this.vol = vol;
@@ -28,13 +28,13 @@ public Clause(String doc, String vol, String part, String sect, String subsect,
2828
this.subsect = subsect;
2929
this.parag = parag;
3030
this.subparg = subparg;
31-
this.clause = clause;
31+
this.clauseString = clauseString;
3232
this.id = id;
3333
}
3434

3535
public Clause(ArrayList<String> components) {
3636
this.id = Integer.parseInt(components.get(0));
37-
this.clause = components.get(1);
37+
this.clauseString = components.get(1);
3838
this.doc = components.get(2);
3939
this.vol = components.get(3);
4040
this.part = components.get(4);
@@ -80,27 +80,27 @@ public String getSubparg() {
8080
}
8181

8282
@ApiModelProperty(value = "Text")
83-
public String getClause() {
84-
return clause;
83+
public String getClauseString() {
84+
return clauseString;
8585
}
8686

8787
@ApiModelProperty(value = "Identification number")
8888
public int getId() {
8989
return id;
9090
}
9191

92-
public void setClause(String clause) {
93-
this.clause = clause;
92+
public void setClauseString(String clauseString) {
93+
this.clauseString = clauseString;
9494
}
9595

9696
public String printMe() {
97-
return this.clause;
97+
return this.clauseString;
9898
}
9999

100100
@Override
101101
public String toString() {
102102
return "Clause [id=" + id + ",\n doc=" + doc + ",\n vol=" + vol + ",\n part=" + part + ",\n sect=" + sect
103-
+ ",\n subsect=" + subsect + ",\n parag=" + parag + ",\n subparg=" + subparg + ",\n clause=" + clause
103+
+ ",\n subsect=" + subsect + ",\n parag=" + parag + ",\n subparg=" + subparg + ",\n clauseString=" + clauseString
104104
+ "]";
105105
}
106106

src/main/java/com/essi/Dependency/Components/ExternalDependency.java

Lines changed: 1 addition & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -9,23 +9,16 @@ public class ExternalDependency {
99
public ExternalDependency(String externalDependency, /* int id, */ Object requirmenet) {
1010
super();
1111
this.externalDependency = externalDependency;
12-
// this.id = id;
1312
this.requirement = requirmenet;
1413
}
1514

1615
public String getExternalDependency() {
1716
return externalDependency;
1817
}
1918

20-
// public void setExternalDependency(String extarnelDependency) {
21-
// this.extarnelDependency = extarnelDependency;
22-
// }
2319
public int referenceId() {
2420
return ((Clause) requirement).getId();
2521
}
26-
// public void setId(int id) {
27-
// this.id = id;
28-
// }
2922

3023
public Object getRequirement() {
3124
return requirement;
@@ -37,7 +30,7 @@ public void setRequirement(Object requirement) {
3730

3831
@Override
3932
public String toString() {
40-
return ((Clause) requirement).getId() + ", External-Dependency=" + ((Clause) requirement).getClause();
33+
return ((Clause) requirement).getId() + ", External-Dependency=" + ((Clause) requirement).getClauseString();
4134
}
4235

4336
}

src/main/java/com/essi/Dependency/Functionalities/CallableTask.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,8 @@ public class CallableTask implements Callable<ArrayList<Object>> {
4343
public ArrayList<Object> call() {
4444
if (expression instanceof Clause) {
4545

46-
((Clause) expression).setClause(((Clause) expression).getClause().toLowerCase().replaceAll(",", " ,"));
47-
Matcher matcher = pattern.matcher(((Clause) expression).getClause());
46+
((Clause) expression).setClauseString(((Clause) expression).getClauseString().toLowerCase().replaceAll(",", " ,"));
47+
Matcher matcher = pattern.matcher(((Clause) expression).getClauseString());
4848

4949
dep = grammar.resolvingCrossReference(((Clause) expression), matcher, expressionList/* , outputWriter */);
5050

src/main/java/com/essi/Dependency/Functionalities/JSONHandler.java

Lines changed: 1 addition & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -1,23 +1,13 @@
11
package com.essi.Dependency.Functionalities;
22

3-
import java.awt.List;
4-
import java.io.File;
5-
import java.nio.file.Files;
63
import java.io.IOException;
7-
import java.nio.file.Paths;
84
import java.util.ArrayList;
9-
import java.util.HashMap;
10-
import java.util.Iterator;
115

126
import com.essi.Dependency.Components.Clause;
137
import com.essi.Dependency.Components.Dependency;
14-
import com.fasterxml.jackson.core.JsonParser;
158
import com.fasterxml.jackson.core.JsonProcessingException;
16-
import com.fasterxml.jackson.core.JsonToken;
17-
import com.fasterxml.jackson.core.util.DefaultPrettyPrinter;
189
import com.fasterxml.jackson.databind.JsonNode;
1910
import com.fasterxml.jackson.databind.ObjectMapper;
20-
import com.fasterxml.jackson.databind.ObjectWriter;
2111
import com.fasterxml.jackson.databind.node.ArrayNode;
2212
import com.fasterxml.jackson.databind.node.ObjectNode;
2313

@@ -250,7 +240,7 @@ public ObjectNode storeRequirements(ObjectNode objectNode, ArrayList<Object> cla
250240
ObjectNode objN = objectMapper.createObjectNode();
251241
objN.put("id", Integer.toString(((Clause) clause).getId()));
252242
objN.put("name", Integer.toString(((Clause) clause).getId()));
253-
objN.put("text", ((Clause) clause).getClause());
243+
objN.put("text", ((Clause) clause).getClauseString());
254244
objN.put("sect", ((Clause) clause).getSect());
255245
objN.put("subsect", ((Clause) clause).getSubsect());
256246
objN.put("parag", ((Clause) clause).getParag());

0 commit comments

Comments
 (0)