Доработать операторы сравнения

@Hopeco · Регистрация: 18.03.2014

Студворк — интернет-сервис помощи студентам

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
namespace BuilderScripts
{
    public partial class Form1
    {
        private static int nextId = 0;
 
        public struct Link
        {
            public readonly SOutput Output;
            public readonly SInput Input;
 
            public Link(SOutput Output, SInput Input)
            {
                this.Input = Input;
                this.Output = Output;
            }
        }
 
        public struct SInput
        {
            private readonly int ID;
            public readonly string name;
 
            public SInput(string name)
            {
                ID = nextId;
                this.name = name;
                nextId++;
            }
 
            public override int GetHashCode()
            {
                int hash = 0;
                for (int i = 0; i < name.Length; i++) hash += name[i];
                return hash << ID;
            }
            public static bool operator ==(SInput c1, SInput c2)
            {
                if (c1.ID == c2.ID && c1.name == c2.name)
                    return true;
                return false;
            }
            public static bool operator !=(SInput c1, SInput c2)
            {
                if (c1.name != c2.name)
                    return true;
                return false;
            }
        }
        public struct SOutput
        {
            private readonly int ID;
            public readonly string name;
 
            public SOutput(string name)
            {
                ID = nextId;
                this.name = name;
                nextId++;
            }
 
            public override int GetHashCode()
            {
                int hash = ID * name.Length;
                for (int i = 0; i < name.Length; i++) hash += name[i];
                return hash;
            }
            public static bool operator ==(SOutput c1, SOutput c2)
            {
                if (c1.ID == c2.ID && c1.name == c2.name)
                    return true;
                return false;
            }
            public static bool operator !=(SOutput c1, SOutput c2)
            {
                if (c1.name != c2.name)
                    return true;
                return false;
            }
        }
    }
}

@kolorotur · 18.10.2015, 10:12

Сообщение от Hopeco

C#
1
2
3
4
5
6
            public override int GetHashCode()
            {
                int hash = 0;
                for (int i = 0; i < name.Length; i++) hash += name[i];
                return hash << ID;
            }

Несбалансированный хэш — при значениях ID > 31 GetHashCode будет всегда возвращать 0, что не есть хорошо.

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
public override int GetHashCode()
{
    // FNV hash
    const int mult = 16777619;
 
    unchecked
    {
        int hash = (int)2166136261 * mult ^ ID;
        foreach (char c in name)
            hash = hash * mult ^ c;
    }
    return hash;
}

Сообщение от Hopeco

C#
1
2
3
4
5
6
7
8
9
10
11
12
            public static bool operator ==(SInput c1, SInput c2)
            {
                if (c1.ID == c2.ID && c1.name == c2.name)
                    return true;
                return false;
            }
            public static bool operator !=(SInput c1, SInput c2)
            {
                if (c1.name != c2.name)
                    return true;
                return false;
            }

Сравнение нестабильное: (a != b) не то же самое, что !(a == b):

C#
1
2
3
4
5
6
7
8
9
10
public static bool operator ==(SInput c1, SInput c2)
{
    if (c1.ID == c2.ID && c1.name == c2.name)
        return true;
    return false;
}
public static bool operator !=(SInput c1, SInput c2)
{
    return !(c1 == c2);
}

Если уж перегружаете операторы, то перегрузите и методы Equals и до кучи реализуйте интерфейс IEquatable:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
public struct SInput : IEquatable<SInput>
{
   ...
 
   public override bool Equals(object other)
   {
      return (other is SInput) && Equals((SInput)other);
   }
   public bool Equals(SInput that)
   {
      return this == that;
   }
}

@Hopeco · 18.10.2015, 12:03 **[ТС]**

kolorotur, вот ещё один вопрос попутно две структуры одинаковые можно ли как-то их использовать по отдельности как разные структуры, но при это не дублировать код?

@kolorotur · 18.10.2015, 12:07

Сообщение от Hopeco

две структуры одинаковые можно ли как-то их использовать по отдельности как разные структуры, но при это не дублировать код?

Нет, но сразу же возникают встречные вопросы:
1. Почему именно структуры, а не классы?
2. Если они одинаковые, то зачем их две штуки?

@Hopeco · 18.10.2015, 12:22 **[ТС]**

kolorotur,

Сообщение от kolorotur

1. Почему именно структуры, а не классы?

Методов нету так что смысла классов тут я не вижу или может я чего-то не понимаю и лучше использовать классы?

Сообщение от kolorotur

2. Если они одинаковые, то зачем их две штуки?

Тут целая история, но если вкратце то в одном классе есть входы и выходы и их надо различать.

Добавлено через 1 минуту
kolorotur, код после исправлений:

C#
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
104
105
106
107
108
109
110
111
112
113
114
115
116
117
118
119
using System;
 
namespace BuilderScripts
{
    public partial class Form1
    {
        private static int nextId = 0;
 
        public struct Link
        {
            public readonly SOutput Output;
            public readonly SInput Input;
 
            public Link(SOutput Output, SInput Input)
            {
                this.Input = Input;
                this.Output = Output;
            }
        }
 
        public struct SInput : IEquatable<SInput>
        {
            private readonly int ID;
            public readonly string name;
 
            public SInput(string name)
            {
                ID = nextId;
                this.name = name;
                nextId++;
            }
 
            public override int GetHashCode()
            {
                const int mult = 16777619;
 
                int hash;
                unchecked
                {
                    hash = (int)2166136261 * mult ^ ID;
                    foreach (char c in name)
                        hash = hash * mult ^ c;
                }
                return hash;
            }
 
            public static bool operator ==(SInput c1, SInput c2)
            {
                if (c1.ID == c2.ID && c1.name == c2.name)
                    return true;
                return false;
            }
            public static bool operator !=(SInput c1, SInput c2)
            {
                if (c1.ID != c2.ID && c1.name != c2.name)
                    return true;
                return false;
            }
 
            public override bool Equals(object other)
            {
                return (other is SInput) && Equals((SInput)other);
            }
            public bool Equals(SInput that)
            {
                return this == that;
            }
        }
 
        public struct SOutput : IEquatable<SOutput>
        {
            private readonly int ID;
            public readonly string name;
 
            public SOutput(string name)
            {
                ID = nextId;
                this.name = name;
                nextId++;
            }
 
            public override int GetHashCode()
            {
                const int mult = 16777619;
 
                int hash;
                unchecked
                {
                    hash = (int)2166136261 * mult ^ ID;
                    foreach (char c in name)
                        hash = hash * mult ^ c;
                }
                return hash;
            }
 
            public static bool operator ==(SOutput c1, SOutput c2)
            {
                if (c1.ID == c2.ID && c1.name == c2.name)
                    return true;
                return false;
            }
            public static bool operator !=(SOutput c1, SOutput c2)
            {
                if (c1.ID != c2.ID && c1.name != c2.name)
                    return true;
                return false;
            }
 
            public override bool Equals(object other)
            {
                return (other is SOutput) && Equals((SOutput)other);
            }
            public bool Equals(SOutput that)
            {
                return this == that;
            }
        }
    }
}

Добавлено через 2 минуты

Сообщение от Hopeco

сть входы и выходы и их надо различать.

хотя сейчас задумался это ведь бессмысленно

@kolorotur · 18.10.2015, 12:45

Сообщение от Hopeco

Методов нету так что смысла классов тут я не вижу или может я чего-то не понимаю и лучше использовать классы?

Второй вариант.
Структуру от класса отличает не наличие методов, а семантика копирования: по значению для структур и по ссылке для классов.

Сообщение от Hopeco

в одном классе есть входы и выходы и их надо различать.

Если входы и выходы ведут себя одинаково, то это одни и те же сущности

Сообщение от Hopeco

код после исправлений

Почему вы упорно переписываете с нуля оператор "!="?
Новый вариант опять не стабильный:

C#
1
2
3
4
5
var a = new SInput("Abc");
var b = new SInput("Abc");
 
bool areEqual = (a == b); // False
bool areNotEqual = (a != b); // False?!

@Hopeco · 18.10.2015, 12:52 **[ТС]**

Сообщение от kolorotur

Почему вы упорно переписываете с нуля оператор "!="?

Просто не знаю как сделать

Добавлено через 3 минуты

Сообщение от Hopeco

Просто не знаю как сделать

Вот

C#
1
2
3
4
public static bool operator !=(SOutput c1, SOutput c2)
            {
                return !(c1 == c2);
            }

@kolorotur · 18.10.2015, 12:52

Сообщение от Hopeco

не знаю как сделать

Я же вам выше привел исправленный вариант:

Сообщение от kolorotur

Сравнение нестабильное: (a != b) не то же самое, что !(a == b):

C#
1
2
3
4
5
6
7
8
9
10
public static bool operator ==(SInput c1, SInput c2)
{
   if (c1.ID == c2.ID && c1.name == c2.name)
     return true;
     return false;
}
public static bool operator !=(SInput c1, SInput c2)
{
    return !(c1 == c2);
}

@Hopeco · 18.10.2015, 12:54 **[ТС]**

kolorotur, уже понял )) Просто туплю очень много

Новые блоги и статьи Все статьи Все блоги /
Отчёт о спецтехнике находящейся в ремонте Maks 20.04.2026 Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .	Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)" Hrethgir 19.04.2026 Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .	Запрет удаления строк ТЧ документа при определенном условии Maks 19.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "Аккумуляторы", разработанного в конфигурации КА2. У данного документа есть ТЧ, в которой в зависимости от прав доступа. . .	Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .
Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.	Фиксация колонок в отчете СКД Maks 14.04.2026 Фиксация колонок в СКД отчета типа Таблица. Задача: зафиксировать три левых колонки в отчете. Процедура ПриКомпоновкеРезультата(ДокументРезультат, ДанныеРасшифровки, СтандартнаяОбработка) / / . . .

@Hopeco 16 / 16 / 8 Регистрация: 18.03.2014 Сообщений: 268
	18.10.2015, 12:03 [ТС]
	kolorotur, вот ещё один вопрос попутно две структуры одинаковые можно ли как-то их использовать по отдельности как разные структуры, но при это не дублировать код? 0

@Hopeco 16 / 16 / 8 Регистрация: 18.03.2014 Сообщений: 268
	18.10.2015, 12:54 [ТС]
	kolorotur, уже понял )) Просто туплю очень много 0

Доработать операторы сравнения

Решение