Рекомендации по присвоению имен для метода - checkSignature или isSignatureValid или что-то еще? - PullRequest
0 голосов
/ 03 ноября 2010

У меня есть метод, который проверяет правильность подписи запроса.
Метод возвращает логическое значение:
_ true: допустимая подпись
_ false: подпись недействительна

Я не уверен насчет лучшего названия для него (возможно, потому что я не являюсь носителем английского языка).
Я не уверен среди:
_ checkSignature
_ isSignatureValid

КакойКак вы думаете, лучше и почему, или, может быть, у вас есть лучшее предложение.

Спасибо,
Дан

Ответы [ 2 ]

3 голосов
/ 03 ноября 2010

isSignatureValid (...).Он говорит вам, что метод собирается вернуть, и это хорошо.Кроме того, он не дает обещаний, так или иначе, что касается кэширования (checkSignature подразумевает для меня, что вы сделаете всю математику, чтобы проверить подпись, когда я ее вызываю, и вам, возможно, не потребуется повторять всю эту работу).

1 голос
/ 04 ноября 2010

Я предпочитаю

hasValidSignature()

, поскольку isSignatureValid() семантически не имеет смысла, поскольку запрос не является подписью, он имеет подпись. Я полагаю, вы собираетесь использовать это в выражении if, поэтому разве это не имеет смысла?

if (request.hasValidSignature()) {
    ...
}

Кроме того, если вы хотите проверить, действительно ли request сам , то это будет более уместно

if (request.isValid()) {
    ...
}
...