0

Подскажите в чем проблема. Если строки одинаковые то "1", а если нет то undefined. А мне надо чтобы true или false.

function slowStringComparison(givenSignature, computedSignature, cb) {
if (!givenSignature || !computedSignature || givenSignature.length !== computedSignature.length) {
    return cb(false);
}
var n = computedSignature.length;
var signaturesMatch = true;

for (var i = 0; i < n; i++) {
     signaturesMatch &= (computedSignature.charAt(i) == givenSignature.charAt(i));
}
return cb(null, signaturesMatch);}
7
  • 2
    суть побитовых выражений в том, что они возвращают число. Если нужна булевское значение используй логическое && вместо побитового
    – Grundy
    24 ноя 2016 в 7:20
  • 2
    на счет slowStringComparison - ты погорячился. в твоем цикле все то же самое происходит, единственное, только проверки нет на null. Ну и яваскрипт уже давно позволяет обращаться к символу строки по индексу computedSignature[i] вместо вызова charAt
    – Grundy
    24 ноя 2016 в 7:22
  • Спасибо, сделал так signaturesMatch = computedSignature[i] == givenSignature[i];
    – Pantera
    24 ноя 2016 в 7:44
  • 1
    плохо - так каждый раз будет перезаписываться значение и в итоге будет только результат последнего сравнения. То есть если все кроме последнего символа разные - signaturesMatch после цикла все равно покажет true
    – Grundy
    24 ноя 2016 в 9:30
  • 1
    проверить значение signaturesMatch - если false - делать break циклу, так как дальше нет смысла проверять. Либо использовать логическое &&: signaturesMatch = signaturesMatch && computedSignature[i] == givenSignature[i];
    – Grundy
    24 ноя 2016 в 13:23

1 ответ 1

2

В node.js есть намного более простой способ сравнить две строки побитово. Для этого нужно использовать класс Buffer, и его метод Buffer.equals:

function slowStringComparison(givenSignature, computedSignature, cb) {
    var givenBuff = Buffer.from(givenSignature),
        computedBuff = Buffer.from(computedSignature);

    cb(givenBuff.equals(computedBuff));
}

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.