Skip to content

Commit 58cada4

Browse files
committed
Немного причесал код
1 parent 80eb31e commit 58cada4

File tree

2 files changed

+35
-19
lines changed

2 files changed

+35
-19
lines changed

src/main/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/IncorrectLineBreakDiagnostic.java

Lines changed: 33 additions & 17 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticSeverity;
2727
import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticTag;
2828
import com.github._1c_syntax.bsl.languageserver.diagnostics.metadata.DiagnosticType;
29+
import com.github._1c_syntax.bsl.languageserver.utils.DiagnosticHelper;
2930
import com.github._1c_syntax.utils.CaseInsensitivePattern;
3031

3132
import java.util.HashSet;
@@ -68,8 +69,7 @@ public class IncorrectLineBreakDiagnostic extends AbstractDiagnostic {
6869
type = String.class,
6970
defaultValue = "" + DEFAULT_LIST_FOR_CHECK_START
7071
)
71-
private String listOfIncorrectFirstSymbol = DEFAULT_LIST_FOR_CHECK_START;
72-
private Pattern INCORRECT_START_LINE_PATTERN = getPatternSearch("^\\s*(:?", listOfIncorrectFirstSymbol, ")");
72+
private Pattern listOfIncorrectFirstSymbol = createPatternIncorrectStartLine(DEFAULT_LIST_FOR_CHECK_START);
7373

7474
@DiagnosticParameter(
7575
type = Boolean.class,
@@ -81,22 +81,18 @@ public class IncorrectLineBreakDiagnostic extends AbstractDiagnostic {
8181
type = String.class,
8282
defaultValue = "" + DEFAULT_LIST_FOR_CHECK_END
8383
)
84-
private String listOfIncorrectLastSymbol = DEFAULT_LIST_FOR_CHECK_END;
85-
private Pattern INCORRECT_END_LINE_PATTERN = getPatternSearch("\\s+(:?", listOfIncorrectLastSymbol, ")\\s*(?://.*)?$");
84+
private Pattern listOfIncorrectLastSymbol = createPatternIncorrectEndLine(DEFAULT_LIST_FOR_CHECK_END);
8685

8786
@Override
8887
public void configure(Map<String, Object> configuration) {
89-
90-
super.configure(configuration);
91-
92-
listOfIncorrectFirstSymbol = (String) configuration.getOrDefault("listOfIncorrectFirstSymbol", DEFAULT_LIST_FOR_CHECK_START);
93-
listOfIncorrectLastSymbol = (String) configuration.getOrDefault("listOfIncorrectLastSymbol", DEFAULT_LIST_FOR_CHECK_END);
94-
INCORRECT_START_LINE_PATTERN = getPatternSearch("^\\s*(:?", listOfIncorrectFirstSymbol, ")");
95-
INCORRECT_END_LINE_PATTERN = getPatternSearch("\\s+(:?", listOfIncorrectLastSymbol, ")\\s*(?://.*)?$");
96-
}
97-
98-
private static Pattern getPatternSearch(String StartPattern, String SearchSymbols, String EndPattern) {
99-
return CaseInsensitivePattern.compile(StartPattern + SearchSymbols + EndPattern);
88+
DiagnosticHelper.configureDiagnostic(this, configuration, "checkFirstSymbol", "checkLastSymbol");
89+
90+
listOfIncorrectFirstSymbol = createPatternIncorrectStartLine(
91+
(String) configuration.getOrDefault("listOfIncorrectFirstSymbol", DEFAULT_LIST_FOR_CHECK_START)
92+
);
93+
listOfIncorrectLastSymbol = createPatternIncorrectEndLine(
94+
(String) configuration.getOrDefault("listOfIncorrectLastSymbol", DEFAULT_LIST_FOR_CHECK_END)
95+
);
10096
}
10197

10298
@Override
@@ -105,10 +101,10 @@ protected void check() {
105101
findQueryFirstLines();
106102

107103
if (checkFirstSymbol){
108-
checkContent(INCORRECT_START_LINE_PATTERN);
104+
checkContent(listOfIncorrectFirstSymbol);
109105
}
110106
if (checkLastSymbol) {
111-
checkContent(INCORRECT_END_LINE_PATTERN);
107+
checkContent(listOfIncorrectLastSymbol);
112108
}
113109
}
114110

@@ -130,4 +126,24 @@ private void checkContent(Pattern pattern) {
130126
}
131127
}
132128
}
129+
130+
private static Pattern getPatternSearch(String startPattern, String searchSymbols, String endPattern) {
131+
return CaseInsensitivePattern.compile(startPattern + searchSymbols + endPattern);
132+
}
133+
134+
private static Pattern createPatternIncorrectStartLine(String listOfSymbols) {
135+
return getPatternSearch(
136+
"^\\s*(:?",
137+
listOfSymbols,
138+
")"
139+
);
140+
}
141+
142+
private static Pattern createPatternIncorrectEndLine(String listOfSymbols) {
143+
return getPatternSearch(
144+
"\\s+(:?",
145+
listOfSymbols,
146+
")\\s*(?://.*)?$"
147+
);
148+
}
133149
}

src/test/java/com/github/_1c_syntax/bsl/languageserver/diagnostics/IncorrectLineBreakDiagnosticTest.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -73,11 +73,11 @@ void testDisableDiagnostics() {
7373

7474
List<Diagnostic> diagnostics = getDiagnostics();
7575

76-
assertThat(diagnostics).hasSize(0);
76+
assertThat(diagnostics).isEmpty();
7777
}
7878

7979
@Test
80-
void testNixelCodestyle() {
80+
void testTrailingBraceCodestyle() {
8181

8282
Map<String, Object> configuration = diagnosticInstance.getInfo().getDefaultConfiguration();
8383
configuration.put("checkFirstSymbol", true);

0 commit comments

Comments
 (0)