Как правильно сравнивать record-ы с byte[] внутри?

Question

[Виктор П.]

Привет,
В c# 10 появились типы record, которые еще могут быть и struct. Их удобно использовать потому, что внутри уже встроен компаратор и не надо дописывать своего. Но я не до конца понимаю, как это работает, с примитивными стрингами всё норм, а вот, например, если передать byte[], то сравнивать не хочет, объясните, пожаулйста

private record struct TypeEncoding(byte[] contentType, byte[] contentEncoding);
...

var k1 = new TypeEncoding(new byte[]{1,2,3}, new byte[]{4,5});
var k2 = new TypeEncoding(new byte[]{1,2,3}, new byte[]{4,5});

var kk = k1 == k2; // false, хотя я ожидаю true

И что надо сделать, чтобы сравнивало по значению?

Answer 1

[Василий Банников]

если передать byte[], то сравнивать не хочет, объясните, пожаулйста

Потому что компилятор генерирует вот такое:

EqualityComparer<byte[]>.Default.Equals(<x>k__BackingField, other.<x>k__BackingField);

Если тебе нужно сравнение по содержимому массивов - тебе придётся самостоятельно определить метод с вот такой сигнатурой:

public virtual bool Equals(TypeEncoding other) {
  // код сравнения this с other
}

А также тебе придётся переопределить GetHashCode, чтобы он на одинаковых массивах отдавал одинаковое число.

Но тут есть нюанс - GetHashCode не должен зависеть от изменяемых типов/полей. Тоесть тебе по хорошему нельзя будет менять содержимое этих массивов, а то можешь получить неприятные спецэффекты, если hashcode поменяется.

Comments

[Виктор П.]

Ага, убрал struct так как всё равно нужно это дописывать, вот такое получилось в итоге

private record DeserializerKey(byte[] contentType, byte[] contentEncoding)
    {
        public virtual bool Equals(DeserializerKey? other)
        {
            if (other == null) return false;
            return this.contentType.SequenceEqual(other.contentType) &&
                   this.contentEncoding.SequenceEqual(other.contentEncoding);
        }

        public override int GetHashCode()
        {
            return HashCode.Combine(new BigInteger(contentType), new BigInteger(contentEncoding));
        }
    };

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

[Василий Банников]

Виктор П., ну никто не запрещает тебе сделать свою обёртку над байтовым массивом или Memory и обозвать её ReadOnlyBytes, и уже её использовать в своих рекордах.

[Станислав Макаров]

Василий Банников Соглашусь, автору тут нужно что-то вроде байтовых string-ов. Чтобы сравнивалось по значению, как это делают строки. Тут в целом абстракция шарповского массива не очень походит.

Answer 2

[Аристарх Загородников]

Я соглашусь с Василием, но дополню что можно завернуть byte[] в какой-то тип, который содержит только одно значение (известно в разных языках как newtype или domain), и будет реализовывать `IEquatable<>`.

Comments

[Виктор П.]

Дописал ответ комментом к ответу Василия, действительно, там нужно реализовывать и Equals и даже в методе GetHashCode пришлось кастовать на другой тип, иначе тоже не работало. Как по мне, неудобно использовать, типа всё равно постоянно кастит под капотом в другие типы