1

День добрый. Сделал функцию для XOR двух массивов байтов. Когда проверил в C# Interactive, оказалось что не работает.

private byte[] xorBytes(byte[] first, byte[] second)
{
    if(first == null || second == null)
        return null;
    byte[] result = new byte[Math.Max(first.Length, second.Length)];
    for (int i = 0; i < result.Length; i++)
    {
        byte f;
        byte s;
        if (first.Length >= i)
        {
            f = 0;
            s = second[i];
        }
        else if (second.Length >= i)
        {
            f = first[i];
            s = 0;
        }
        else
        {
            f = first[i];
            s = second[i];
        }
        result[i] = (byte)(f^s);
    }
    return result;
}

Скормил ей {0,1,2} и {4,7,2,8}, должно было получится {4,6,0,8}, но получилось {4,7,2,8}. Кто разбирается, помогите пожалуйста.

5
  • 1
    Воспользуйтесь BitArray.Xor, это вроде то, что Вам нужно.
    – Aarnihauta
    12 мая 2022 в 11:26
  • 1
    У вас условия в if неправильные.
    – Alexey Ten
    12 мая 2022 в 11:37
  • 1
    @Aarnihauta когда это x ^ y отменили? :)
    – aepot
    12 мая 2022 в 12:06
  • @aepot Да, хотел об этом написать. Но тут ведь надо просто два массива xor-нуть, так чего придумывать
    – Aarnihauta
    12 мая 2022 в 12:13
  • @Aarnihauta проще, чем в ответе ниже, вы в целом не напишете.
    – aepot
    12 мая 2022 в 13:32

1 ответ 1

2

Условия в if-ах некорректны. Например, для i = 0 с указанными массивами: first.Length >= i - true (3 >= 0), в итоге вместо first[0] возьмётся 0. И так для всех итераций

Можно переписать условия так:

for (int i = 0; i < result.Length; i++)
{
    byte f = i < first.Length ? first[i] : (byte)0;
    byte s = i < second.Length ? second[i] : (byte)0;
    result[i] = (byte)(f ^ s);
}
3
  • 3
    Ну и всё же понятнее было бы i < first.Length писать, а когда first.Length > i то приходится думать, что это условие означает.
    – CrazyElf
    12 мая 2022 в 12:27
  • 1
    Вообще, я хотел оставить код похожим на код автора, но сейчас подумал что first.Length > i уж больно тяжело выглядит, да
    – Regent
    12 мая 2022 в 13:56
  • Да я понял мотивы, но ещё в коде автора сломал об это условие мозг. А когда мозг об код ломается, то лучше код таки менять )
    – CrazyElf
    12 мая 2022 в 14:13

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

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