Conversation
bzaar
left a comment
bzaar
left a comment
There was a problem hiding this comment.
В остальном все нормально, но пока не будем мерджить, пока не будет принят основной PR.
|
|
||
| using (var client = _newClient()) | ||
| { | ||
| client.AddParam("n", n.ToString()); |
There was a problem hiding this comment.
Я бы добавил: ToString(CultureInfo.InvariantCulture). Иначе все может работать, пока на хостинге добрый дядя не поменяет региональные настройки.
| } | ||
| } | ||
|
|
||
| public SummaPropisResult SpellSum(decimal n, string currency, |
There was a problem hiding this comment.
Переименуй SpellSum в GetPropis, пожалуйста.
|
@bzaar Не подскажете, почему не может создать ответ из Propis-а? Разные варианты атрибутов попробовал поставить внутри класса, но все равно не может сформировать. |
|
Не очень понял. Что именно ты делаешь и что получаешь? |
Не в том PR-e написал вопрос. |
| client.AddParam("nbsp", nbsp); | ||
| client.AddParam("delim", delim); | ||
|
|
||
| return client.GetObject<Propis>("/russian/propis"); |
There was a problem hiding this comment.
А обработка ошибок? Что если сервис вернет ошибку 400? Нужно преобразовать ее в известное исключение.
| public class Propis | ||
| { | ||
| [DataMember] | ||
| public string propis1 { get; set; } |
There was a problem hiding this comment.
А зачем он? Вроде в других классах такого не было.
| public string propis1 { get; set; } | ||
|
|
||
| [DataMember] | ||
| public string propis2 { get; set; } |
There was a problem hiding this comment.
В C# свойства принято называть с большой буквы: Propis2.
|
Добавил изменения по замечаниям. |
2 participants
https://github.com/morpher-ru/morpher/issues/603