Skip to content

Update NetSdrMessageHelper.cs#18

Open
Sane4ka126 wants to merge 1 commit into
masterfrom
Sane4ka126-patch-18
Open

Update NetSdrMessageHelper.cs#18
Sane4ka126 wants to merge 1 commit into
masterfrom
Sane4ka126-patch-18

Conversation

@Sane4ka126
Copy link
Copy Markdown
Owner

@Sane4ka126 Sane4ka126 commented Oct 19, 2025

image image БАГ 1: Архітектурне порушення - залежність від Networking using NetSdrClientApp.Networking; БАГ 2: Використання Networking всередині Messages public static void TestArchitectureViolation() { var tcpClient = new TcpClientWrapper("localhost", 5000); var udpClient = new UdpClientWrapper(6000); } БАГ 3: Орфографічна помилка — msgEnumarable var msgEnumarable = msg as IEnumerable; БАГ 4: Виняток без опису throw new ArgumentOutOfRangeException(); БАГ 5: Неоптимальне використання Count() у циклі while (bodyEnumerable.Count() >= sampleSize) БАГ 6: ArgumentException без paramName throw new ArgumentException("Message length exceeds allowed value"); БАГ 7: Зайвий .ToArray() var num = BitConverter.ToUInt16(header.ToArray()); БАГ 8: Відсутня XML-документація public static byte[] CreateEmptyMessage()БАГ 9: Magic number без пояснення return size > 0 && size < 8191; БАГ 10: Повернення null для non-nullable типу if (string.IsNullOrEmpty(hexString)) return null; image image image image image image скрін невдалого прогону (до фіксу)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant