Visitar URL original
modify body expr · googleapis/sdk-platform-java@84ed106 · GitHub
Skip to content
This repository was archived by the owner on Oct 29, 2026. It is now read-only.

Commit 84ed106

Browse files
committed
modify body expr
1 parent 749b0d2 commit 84ed106

1 file changed

Lines changed: 5 additions & 11 deletions

File tree

‎src/main/java/com/google/api/generator/gapic/composer/MethodSampleCodeHelperComposer.java‎

Lines changed: 5 additions & 11 deletions
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,6 @@
2626
import com.google.api.generator.gapic.model.MethodArgument;
2727
import com.google.api.generator.gapic.model.ResourceName;
2828
import com.google.api.generator.gapic.utils.JavaStyle;
29-
import java.util.ArrayList;
3029
import java.util.List;
3130
import java.util.Map;
3231
import java.util.stream.Collectors;
@@ -42,7 +41,7 @@ public static TryCatchStatement composeUnaryRpcMethodSampleCode(
4241
VariableExpr clientVarExpr = createVariableExpr(getClientName(clientType), clientType);
4342
// Assign each method arguments with its default value.
4443
Map<String, VariableExpr> methodArgVarExprMap = createMethodArgumentsVariableExprs(arguments);
45-
List<Expr> methodArgumentsAssignmentExpr =
44+
List<Expr> bodyExpr =
4645
assignMethodArgumentsWithDefaultValues(arguments, methodArgVarExprMap, resourceNames);
4746
List<Expr> methodVarExprs =
4847
arguments.stream()
@@ -52,28 +51,23 @@ public static TryCatchStatement composeUnaryRpcMethodSampleCode(
5251
// e.g. if return void, echoClient.echo(..); or,
5352
// e.g. if return other type, EchoResponse response = echoClient.echo(...);
5453
boolean returnsVoid = isProtoEmptyType(method.outputType());
55-
Expr responseExpr = null;
5654
if (returnsVoid) {
57-
responseExpr =
55+
bodyExpr.add(
5856
MethodInvocationExpr.builder()
5957
.setExprReferenceExpr(clientVarExpr)
6058
.setMethodName(JavaStyle.toLowerCamelCase(method.name()))
6159
.setArguments(methodVarExprs)
6260
.setReturnType(clientType)
63-
.build();
61+
.build());
6462
} else {
65-
responseExpr =
63+
bodyExpr.add(
6664
createAssignExprForVariableWithClientMethod(
6765
createVariableExpr(RESPONSE, method.outputType()),
6866
clientVarExpr,
6967
JavaStyle.toLowerCamelCase(method.name()),
70-
methodVarExprs);
68+
methodVarExprs));
7169
}
7270

73-
List<Expr> bodyExpr = new ArrayList<>();
74-
bodyExpr.addAll(methodArgumentsAssignmentExpr);
75-
bodyExpr.add(responseExpr);
76-
7771
return TryCatchStatement.builder()
7872
.setTryResourceExpr(assignClientVariableWithCreateMethodExpr(clientVarExpr))
7973
.setTryBody(

0 commit comments

Comments
 (0)