【发布时间】:2016-03-23 21:41:22
【问题描述】:
我的问题:我该怎么做?
所以,直到本周,我在大约 6 年的时间里没有接触过任何 .Net。有很多我忘记了,甚至更多是我不知道的,虽然我喜欢 async/await 关键字的想法,但我在为客户端的 API 实现实现以下要求时遇到了一个小问题:
-
ServerAPI类对每个 API 方法都有一个方法,采用适当的输入参数(例如,Login方法接受id和password,进行 API 调用并将结果返回给调用者)。 - 我想抽象出 JSON,以便我的 API 方法返回您正在获取的实际对象(例如,上面的
Login方法返回带有您的身份验证令牌、uid 等的User对象) - 某些 API 方法在成功或没有有意义的内容时返回 204(在我的用例中没有意义,也许我只关心成功/失败),对于这些我想返回布尔值(
true= 成功)或状态码。 - 我想保留 async/await(或等效的)设计,因为到目前为止它似乎确实运行良好。
- 对于某些方法,我可能只需要返回
HttpResponseMessage对象并让调用者处理它。
到目前为止,这大致是我所拥有的,我不确定如何使其符合上述要求,或者我是否做得正确。任何指导都表示赞赏(但是,燃烧不是)。
// 200 (+User JSON) = success, otherwise APIError JSON
internal async Task<User> Login (string id, string password)
{
LoginPayload payload = new LoginPayload() { LoginId = id, Password = password};
var request = NewRequest(HttpMethod.Post, "login");
JsonPayload<LoginPayload>(payload, ref request);
return await Execute<Account>(request, false);
}
// 204: success, anything else failure
internal async Task<Boolean> LogOut ()
{
return await Execute<Boolean>(NewRequest(HttpMethod.Delete, "login"), true);
}
internal async Task<HttpResponseMessage> GetRawResponse ()
{
return await Execute<HttpResponseMessage>(NewRequest(HttpMethod.Get, "raw/something"), true);
}
internal async Task<Int32> GetMeStatusCode ()
{
return await Execute<Int32>(NewRequest(HttpMethod.Get, "some/intstatus"), true);
}
private async Task<RESULT> Execute<RESULT>(HttpRequestMessage request, bool authenticate)
{
if (authenticate)
AuthenticateRequest(ref request); // add auth token to request
var tcs = new TaskCompletionSource<RESULT>();
var response = await client.SendAsync(request);
// TODO: If the RESULT is just HTTPResponseMessage, the rest is unnecessary
if (response.IsSuccessStatusCode)
{
try
{
// TryParse needs to handle Boolean differently than other types
RESULT result = await TryParse<RESULT>(response);
tcs.SetResult(result);
}
catch (Exception e)
{
tcs.SetException(e);
}
}
else
{
try
{
APIError error = await TryParse<APIError>(response);
tcs.SetException(new APIException(error));
}
catch (Exception e)
{
tcs.SetException(new APIException("Unknown error"));
}
}
return tcs.Task.Result;
}
这是APIError JSON 结构(它是状态代码 + 自定义错误代码)。
{
"status": 404,
"code":216,
"msg":"User not found"
}
我更愿意继续使用System.Net,但这主要是因为我不想切换所有代码。如果我想要通过其他方式更容易完成,那么显然值得额外的工作。
谢谢。
【问题讨论】:
-
我的问题:我该怎么做?我想说代码审查是针对您考虑的代码完成不是你不知道如何完成的代码。
-
试试看
[JsonProperty("myPropertyName")],它可能对你有帮助。 -
那么,你的意思是你已经完成了整个 UWP 项目并且测试它运行良好,但是想知道这些代码是否满足上述要求,以及是否有人知道更好的方法来实现这些要求?
-
@Sunteen 我让它部分工作:登录有效,因为它返回一个对象,但注销总是失败,因为它试图解析 204 (NoContent) 的内容并且那里什么都没有(我想要这将返回
true/false)。所以我需要帮助返回Int32或HttpResponseMessage,具体取决于类型。我总是可以有 3 种不同的execute方法并在必要时相应地调用它们,但似乎我在这里遗漏了一点点。朝着正确的方向推进。 -
根据您的想法,似乎
if (RESULT IS HttpResponseMessage)if(RESULT IS Boolen)if(!(RESULT is ValueType))这样的代码是满足您要求的唯一方法。但我建议你不要使用上面的代码。不同的功能有不同的执行方法,因为您认为这是一种更好的方法。代码看起来有点多并不代表设计不好,相反,像上面的代码一样,把执行逻辑放在一起,会增加耦合度。例如,如果你只想改变Logout的逻辑,你应该更新唯一的execute方法,不小心影响Login功能。
标签: c# win-universal-app system.net