?

Log in

No account? Create an account
записи друзья календарь о пользователе Homepage назад назад вперёд вперёд
не могу молчать - Hook-nosed
sema
не могу молчать
Project manager (как он себе думает) из американского офиса прислал спецификацию на метод. В которой есть следующие магические слова:

LoginResult lr = r.loginResult
// 0 – ok, 1 – invalid username/pwd, 2 – newer version available, 3 – version no longer supported
// Note that LoginResult == 2 is not an error but a warning and users can continue to work.


Убить нельзя кастрировать.

музыка : Buddy Guy - Cut You Loose
тэги : ,
Comments
dma From: dma Date: August 18th, 2006 10:17 am (UTC) (ссылка)
Даа. У чувака есть Стиль.
From: ex_irishterr534 Date: August 18th, 2006 10:58 am (UTC) (ссылка)
не очень приятно, но не смертельно imho
не любит чувак с битовыми операциями работать ну и пох.
sema From: sema Date: August 18th, 2006 11:07 am (UTC) (ссылка)
при чём тут битовые операции?
0 - хорошо, 1 - ошибка, 2 - хорошо, но есть ньюансы, 3 - опять ошибка
это по-твоему нормально?
причём это, я забыл указать, в публичном API!
т.е. людям, которым придётся это использовать мало того, что нужно будет проверять, что loginResult равен или 0 или 2, чтоб убедиться что всё хорошо, так ищё и будет полное ощущение, что может легко появится код равный 4, который так же будет означать, что всё хорошо, но есть другие ньюансы
From: ex_irishterr534 Date: August 18th, 2006 11:17 am (UTC) (ссылка)
несмертельно, или я просто уже привык к подробным пожеланиям.
а битовые операции понятно к чему -- подмешать тип варнинга или ошибки
elk From: elk Date: August 18th, 2006 12:17 pm (UTC) (ссылка)
Падумаешь! Тебе не все равно - это же процессор будет проверять, ему пофиг, что подряд, что неподряд. Засунь в enum и забудь про конкретные циферки.
Проблема кода в том, что если уж для типа возврата придуман специлаьный тип, то не надо подмешивать туда константы от другого типа (int).

(Deleted comment)
sema From: sema Date: August 18th, 2006 11:40 am (UTC) (ссылка)
Ну, тип это уже детали. На самом деле там int конечно.
Язык исполнения вообще-то C#, но на самом деле это webservice, так что не важно..
avva From: avva Date: August 18th, 2006 05:42 pm (UTC) (ссылка)
Хаха (бьется головой об стенку).
читать[7] | ответить