Skip to content

Commit bf04ee7

Browse files
committed
Modified issue142 code format
1 parent d8f43d2 commit bf04ee7

File tree

7 files changed

+114
-115
lines changed

7 files changed

+114
-115
lines changed
Lines changed: 25 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,17 @@
11
/**
2-
* Copyright 2016-2019 the original author or authors.
3-
*
4-
* Licensed under the Apache License, Version 2.0 (the "License");
5-
* you may not use this file except in compliance with the License.
6-
* You may obtain a copy of the License at
7-
*
8-
* http://www.apache.org/licenses/LICENSE-2.0
9-
*
10-
* Unless required by applicable law or agreed to in writing, software
11-
* distributed under the License is distributed on an "AS IS" BASIS,
12-
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13-
* See the License for the specific language governing permissions and
14-
* limitations under the License.
2+
* Copyright 2016-2019 the original author or authors.
3+
* <p>
4+
* Licensed under the Apache License, Version 2.0 (the "License");
5+
* you may not use this file except in compliance with the License.
6+
* You may obtain a copy of the License at
7+
* <p>
8+
* http://www.apache.org/licenses/LICENSE-2.0
9+
* <p>
10+
* Unless required by applicable law or agreed to in writing, software
11+
* distributed under the License is distributed on an "AS IS" BASIS,
12+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
* See the License for the specific language governing permissions and
14+
* limitations under the License.
1515
*/
1616
package org.mybatis.dynamic.sql.select.render;
1717

@@ -30,12 +30,12 @@ public class FetchFirstPagingModelRenderer {
3030
private AtomicInteger sequence;
3131

3232
public FetchFirstPagingModelRenderer(RenderingStrategy renderingStrategy,
33-
PagingModel pagingModel,AtomicInteger sequence) {
33+
PagingModel pagingModel, AtomicInteger sequence) {
3434
this.renderingStrategy = renderingStrategy;
3535
this.pagingModel = pagingModel;
3636
this.sequence = sequence;
3737
}
38-
38+
3939
public Optional<FragmentAndParameters> render() {
4040
return pagingModel.offset()
4141
.map(this::renderWithOffset)
@@ -47,34 +47,34 @@ private Optional<FragmentAndParameters> renderWithOffset(Long offset) {
4747
.map(ffr -> renderOffsetAndFetchFirstRows(offset, ffr))
4848
.orElseGet(() -> renderOffsetOnly(offset));
4949
}
50-
50+
5151
private Optional<FragmentAndParameters> renderFetchFirstRowsOnly() {
5252
return pagingModel.fetchFirstRows().flatMap(this::renderFetchFirstRowsOnly);
5353
}
54-
54+
5555
private Optional<FragmentAndParameters> renderFetchFirstRowsOnly(Long fetchFirstRows) {
5656
String mapKey = formatParameterMapKey(FETCH_FIRST_ROWS_PARAMETER);
5757
return FragmentAndParameters
5858
.withFragment("fetch first " + renderPlaceholder(mapKey) //$NON-NLS-1$
59-
+ " rows only") //$NON-NLS-1$
59+
+ " rows only") //$NON-NLS-1$
6060
.withParameter(mapKey, fetchFirstRows)
6161
.buildOptional();
6262
}
63-
63+
6464
private Optional<FragmentAndParameters> renderOffsetOnly(Long offset) {
6565
String mapKey = formatParameterMapKey(OFFSET_PARAMETER);
6666
return FragmentAndParameters.withFragment("offset " + renderPlaceholder(mapKey) //$NON-NLS-1$
67-
+ " rows") //$NON-NLS-1$
67+
+ " rows") //$NON-NLS-1$
6868
.withParameter(mapKey, offset)
6969
.buildOptional();
7070
}
71-
71+
7272
private Optional<FragmentAndParameters> renderOffsetAndFetchFirstRows(Long offset, Long fetchFirstRows) {
7373
String mapKey1 = formatParameterMapKey(OFFSET_PARAMETER);
7474
String mapKey2 = formatParameterMapKey(FETCH_FIRST_ROWS_PARAMETER);
7575
return FragmentAndParameters.withFragment("offset " + renderPlaceholder(mapKey1) //$NON-NLS-1$
76-
+ " rows fetch first " + renderPlaceholder(mapKey2) //$NON-NLS-1$
77-
+ " rows only") //$NON-NLS-1$
76+
+ " rows fetch first " + renderPlaceholder(mapKey2) //$NON-NLS-1$
77+
+ " rows only") //$NON-NLS-1$
7878
.withParameter(mapKey1, offset)
7979
.withParameter(mapKey2, fetchFirstRows)
8080
.buildOptional();
@@ -83,9 +83,9 @@ private Optional<FragmentAndParameters> renderOffsetAndFetchFirstRows(Long offse
8383
private String formatParameterMapKey(String parameterMapKey) {
8484
return parameterMapKey + sequence.getAndIncrement(); //$NON-NLS-1$
8585
}
86-
86+
8787
private String renderPlaceholder(String parameterName) {
8888
return renderingStrategy.getFormattedJdbcPlaceholder(RenderingStrategy.DEFAULT_PARAMETER_PREFIX,
89-
parameterName);
89+
parameterName);
9090
}
9191
}
Lines changed: 19 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,17 @@
11
/**
2-
* Copyright 2016-2019 the original author or authors.
3-
*
4-
* Licensed under the Apache License, Version 2.0 (the "License");
5-
* you may not use this file except in compliance with the License.
6-
* You may obtain a copy of the License at
7-
*
8-
* http://www.apache.org/licenses/LICENSE-2.0
9-
*
10-
* Unless required by applicable law or agreed to in writing, software
11-
* distributed under the License is distributed on an "AS IS" BASIS,
12-
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13-
* See the License for the specific language governing permissions and
14-
* limitations under the License.
2+
* Copyright 2016-2019 the original author or authors.
3+
* <p>
4+
* Licensed under the Apache License, Version 2.0 (the "License");
5+
* you may not use this file except in compliance with the License.
6+
* You may obtain a copy of the License at
7+
* <p>
8+
* http://www.apache.org/licenses/LICENSE-2.0
9+
* <p>
10+
* Unless required by applicable law or agreed to in writing, software
11+
* distributed under the License is distributed on an "AS IS" BASIS,
12+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
* See the License for the specific language governing permissions and
14+
* limitations under the License.
1515
*/
1616
package org.mybatis.dynamic.sql.select.render;
1717

@@ -31,13 +31,13 @@ public class LimitAndOffsetPagingModelRenderer {
3131
private AtomicInteger sequence;
3232

3333
public LimitAndOffsetPagingModelRenderer(RenderingStrategy renderingStrategy,
34-
Long limit, PagingModel pagingModel,AtomicInteger sequence) {
34+
Long limit, PagingModel pagingModel, AtomicInteger sequence) {
3535
this.renderingStrategy = renderingStrategy;
3636
this.limit = limit;
3737
this.pagingModel = pagingModel;
3838
this.sequence = sequence;
3939
}
40-
40+
4141
public Optional<FragmentAndParameters> render() {
4242
return pagingModel.offset().map(this::renderLimitAndOffset)
4343
.orElseGet(this::renderLimitOnly);
@@ -49,12 +49,12 @@ private Optional<FragmentAndParameters> renderLimitOnly() {
4949
.withParameter(mapKey, limit)
5050
.buildOptional();
5151
}
52-
52+
5353
private Optional<FragmentAndParameters> renderLimitAndOffset(Long offset) {
5454
String mapKey1 = formatParameterMapKey(LIMIT_PARAMETER);
5555
String mapKey2 = formatParameterMapKey(OFFSET_PARAMETER);
5656
return FragmentAndParameters.withFragment("limit " + renderPlaceholder(mapKey1) //$NON-NLS-1$
57-
+ " offset " + renderPlaceholder(mapKey2)) //$NON-NLS-1$
57+
+ " offset " + renderPlaceholder(mapKey2)) //$NON-NLS-1$
5858
.withParameter(mapKey1, limit)
5959
.withParameter(mapKey2, offset)
6060
.buildOptional();
@@ -63,9 +63,9 @@ private Optional<FragmentAndParameters> renderLimitAndOffset(Long offset) {
6363
private String formatParameterMapKey(String parameterMapKey) {
6464
return parameterMapKey + sequence.getAndIncrement(); //$NON-NLS-1$
6565
}
66-
66+
6767
private String renderPlaceholder(String parameterName) {
6868
return renderingStrategy.getFormattedJdbcPlaceholder(RenderingStrategy.DEFAULT_PARAMETER_PREFIX,
69-
parameterName);
69+
parameterName);
7070
}
7171
}

src/main/java/org/mybatis/dynamic/sql/select/render/PagingModelRenderer.java

Lines changed: 21 additions & 25 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,17 @@
11
/**
2-
* Copyright 2016-2019 the original author or authors.
3-
*
4-
* Licensed under the Apache License, Version 2.0 (the "License");
5-
* you may not use this file except in compliance with the License.
6-
* You may obtain a copy of the License at
7-
*
8-
* http://www.apache.org/licenses/LICENSE-2.0
9-
*
10-
* Unless required by applicable law or agreed to in writing, software
11-
* distributed under the License is distributed on an "AS IS" BASIS,
12-
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13-
* See the License for the specific language governing permissions and
14-
* limitations under the License.
2+
* Copyright 2016-2019 the original author or authors.
3+
* <p>
4+
* Licensed under the Apache License, Version 2.0 (the "License");
5+
* you may not use this file except in compliance with the License.
6+
* You may obtain a copy of the License at
7+
* <p>
8+
* http://www.apache.org/licenses/LICENSE-2.0
9+
* <p>
10+
* Unless required by applicable law or agreed to in writing, software
11+
* distributed under the License is distributed on an "AS IS" BASIS,
12+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
* See the License for the specific language governing permissions and
14+
* limitations under the License.
1515
*/
1616
package org.mybatis.dynamic.sql.select.render;
1717

@@ -31,28 +31,28 @@ public class PagingModelRenderer {
3131
private PagingModelRenderer(Builder builder) {
3232
renderingStrategy = Objects.requireNonNull(builder.renderingStrategy);
3333
pagingModel = Objects.requireNonNull(builder.pagingModel);
34-
sequence = builder.sequence().orElseGet(() -> new AtomicInteger(1));
34+
sequence = Objects.requireNonNull(builder.sequence);
3535
}
36-
36+
3737
public Optional<FragmentAndParameters> render() {
3838
return pagingModel.limit().map(this::limitAndOffsetRender)
3939
.orElseGet(this::fetchFirstRender);
4040
}
41-
41+
4242
private Optional<FragmentAndParameters> limitAndOffsetRender(Long limit) {
4343
return new LimitAndOffsetPagingModelRenderer(renderingStrategy, limit,
44-
pagingModel,sequence).render();
44+
pagingModel, sequence).render();
4545
}
46-
46+
4747
private Optional<FragmentAndParameters> fetchFirstRender() {
48-
return new FetchFirstPagingModelRenderer(renderingStrategy, pagingModel,sequence).render();
48+
return new FetchFirstPagingModelRenderer(renderingStrategy, pagingModel, sequence).render();
4949
}
50-
50+
5151
public static class Builder {
5252
private RenderingStrategy renderingStrategy;
5353
private PagingModel pagingModel;
5454
private AtomicInteger sequence;
55-
55+
5656
public Builder withRenderingStrategy(RenderingStrategy renderingStrategy) {
5757
this.renderingStrategy = renderingStrategy;
5858
return this;
@@ -68,10 +68,6 @@ public Builder withSequence(AtomicInteger sequence) {
6868
return this;
6969
}
7070

71-
private Optional<AtomicInteger> sequence() {
72-
return Optional.ofNullable(sequence);
73-
}
74-
7571
public PagingModelRenderer build() {
7672
return new PagingModelRenderer(this);
7773
}

src/main/java/org/mybatis/dynamic/sql/select/render/SelectRenderer.java

Lines changed: 28 additions & 28 deletions
Original file line numberDiff line numberDiff line change
@@ -1,17 +1,17 @@
11
/**
2-
* Copyright 2016-2019 the original author or authors.
3-
*
4-
* Licensed under the Apache License, Version 2.0 (the "License");
5-
* you may not use this file except in compliance with the License.
6-
* You may obtain a copy of the License at
7-
*
8-
* http://www.apache.org/licenses/LICENSE-2.0
9-
*
10-
* Unless required by applicable law or agreed to in writing, software
11-
* distributed under the License is distributed on an "AS IS" BASIS,
12-
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13-
* See the License for the specific language governing permissions and
14-
* limitations under the License.
2+
* Copyright 2016-2019 the original author or authors.
3+
* <p>
4+
* Licensed under the Apache License, Version 2.0 (the "License");
5+
* you may not use this file except in compliance with the License.
6+
* You may obtain a copy of the License at
7+
* <p>
8+
* http://www.apache.org/licenses/LICENSE-2.0
9+
* <p>
10+
* Unless required by applicable law or agreed to in writing, software
11+
* distributed under the License is distributed on an "AS IS" BASIS,
12+
* WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
13+
* See the License for the specific language governing permissions and
14+
* limitations under the License.
1515
*/
1616
package org.mybatis.dynamic.sql.select.render;
1717

@@ -34,22 +34,22 @@ public class SelectRenderer {
3434
private SelectModel selectModel;
3535
private RenderingStrategy renderingStrategy;
3636
private AtomicInteger sequence;
37-
37+
3838
private SelectRenderer(Builder builder) {
3939
selectModel = Objects.requireNonNull(builder.selectModel);
4040
renderingStrategy = Objects.requireNonNull(builder.renderingStrategy);
4141
sequence = builder.sequence().orElseGet(() -> new AtomicInteger(1));
4242
}
43-
43+
4444
public SelectStatementProvider render() {
4545
FragmentCollector fragmentCollector = selectModel
4646
.mapQueryExpressions(this::renderQueryExpression)
4747
.collect(FragmentCollector.collect());
4848
renderOrderBy(fragmentCollector);
4949
renderPagingModel(fragmentCollector);
50-
50+
5151
String selectStatement = fragmentCollector.fragments().collect(Collectors.joining(" ")); //$NON-NLS-1$
52-
52+
5353
return DefaultSelectStatementProvider.withSelectStatement(selectStatement)
5454
.withParameters(fragmentCollector.parameters())
5555
.build();
@@ -66,26 +66,26 @@ private FragmentAndParameters renderQueryExpression(QueryExpressionModel queryEx
6666
private void renderOrderBy(FragmentCollector fragmentCollector) {
6767
selectModel.orderByModel().ifPresent(om -> renderOrderBy(fragmentCollector, om));
6868
}
69-
69+
7070
private void renderOrderBy(FragmentCollector fragmentCollector, OrderByModel orderByModel) {
7171
String phrase = orderByModel.mapColumns(this::calculateOrderByPhrase)
7272
.collect(CustomCollectors.joining(", ", "order by ", "")); //$NON-NLS-1$ //$NON-NLS-2$ //$NON-NLS-3$
7373
fragmentCollector.add(FragmentAndParameters.withFragment(phrase).build());
7474
}
75-
75+
7676
private String calculateOrderByPhrase(SortSpecification column) {
7777
String phrase = column.aliasOrName();
7878
if (column.isDescending()) {
7979
phrase = phrase + " DESC"; //$NON-NLS-1$
8080
}
8181
return phrase;
8282
}
83-
83+
8484
private void renderPagingModel(FragmentCollector fragmentCollector) {
8585
selectModel.pagingModel().flatMap(this::renderPagingModel)
86-
.ifPresent(fragmentCollector::add);
86+
.ifPresent(fragmentCollector::add);
8787
}
88-
88+
8989
private Optional<FragmentAndParameters> renderPagingModel(PagingModel pagingModel) {
9090
return new PagingModelRenderer.Builder()
9191
.withPagingModel(pagingModel)
@@ -98,31 +98,31 @@ private Optional<FragmentAndParameters> renderPagingModel(PagingModel pagingMode
9898
public static Builder withSelectModel(SelectModel selectModel) {
9999
return new Builder().withSelectModel(selectModel);
100100
}
101-
101+
102102
public static class Builder {
103103
private SelectModel selectModel;
104104
private RenderingStrategy renderingStrategy;
105105
private AtomicInteger sequence;
106-
106+
107107
public Builder withSelectModel(SelectModel selectModel) {
108108
this.selectModel = selectModel;
109109
return this;
110110
}
111-
111+
112112
public Builder withRenderingStrategy(RenderingStrategy renderingStrategy) {
113113
this.renderingStrategy = renderingStrategy;
114114
return this;
115115
}
116-
116+
117117
public Builder withSequence(AtomicInteger sequence) {
118118
this.sequence = sequence;
119119
return this;
120120
}
121-
121+
122122
private Optional<AtomicInteger> sequence() {
123123
return Optional.ofNullable(sequence);
124124
}
125-
125+
126126
public SelectRenderer build() {
127127
return new SelectRenderer(this);
128128
}

0 commit comments

Comments
 (0)