3

У меня имеется класс Packet, от которого наследуются другие Packet'ы.

public abstract class Packet
{
    public static byte Id;

    public virtual void Write(ref RawPacket packet)
    {
        packet.WriteByte(Id);
    }
}

Как сделать так чтобы для каждого наследника этого класса был свой static Id, чтобы сэкономить пару байт? Пытался выносить Id в Static Generic классы, но тогда метод Write не видит Id.

Например у меня имеется PacketA, PacketB, которые наследуются от класса Packet. И ко мне по сети приходит массив байт, первый элемент которого указывает на тип пакета, то есть Id. И с помощью Id, используя Dictionary<byte, Type> я могу обратится к экземпляру класса PacketA или B и узнать каким способом считывать этот массив. А экземпляры пакетов PacketA, PacketB очень часто создаются, потому Id в экземплярах одних и тех же PacketA или PacketB будут расходовать лишний байт памяти на каждое Id.

10
  • 1
    почему важно делать это статикой? Commented 25 нояб. 2023 в 16:00
  • 1
    У вас не должно быть статика там вовсе. "Пакет" - это один конкретный объект, как допустим, сотрудник, у которого есть свое имя, должность, зарплата, сделав что-то статичным, вы делаете это общим, то есть условно делая у сотрудника зарплату статичной - у всех сотрудников будет одна и таж зарплата. Поэтому статика в данном случае, это странное решение, яб сказал даже вредное.
    – EvgeniyZ
    Commented 25 нояб. 2023 в 16:02
  • 1
    Статика к ООП не имеет никакого отношения. Начните с рассказа о том, что вы вообще хотите сделать. На поставленный вопрос ответ "никак".
    – aepot
    Commented 25 нояб. 2023 в 16:18
  • 1
    @Diyabya всё равное не вижу причин для использования static. Вы кажется пытаетесь экономить на спичках Commented 25 нояб. 2023 в 16:29
  • 2
    Во-первых не 1 байт, а 4 байта, вы забыли про выравнивание. Во-вторых если хотите реальной производительности и минимального расхода памяти, используйте структуры, а не классы и никакого наследования, чтобы избежать виртуализации. Если заголовок пакета фиксированной длины, его можно вообще напрямую читать в участок памяти, который использовать как структуру данных без затрат на копирование данных. Но такие штуки надо разбирать и писать имея реальную задачу, а не абстрактную.
    – aepot
    Commented 25 нояб. 2023 в 16:45

3 ответа 3

3

Например, можно применить CRTP:

public abstract class Packet<TSelf> where TSelf : Packet<TSelf>
{
    public static byte Id;

    public virtual void Write(ref RawPacket packet)
    {
        packet.WriteByte(Id);
    }
}

Использовать нужно так:

public class Ipv4Packet : Packet<Ipv4Packet>
{
    // ...
}

Заметьте, что с CRTP различные пакеты не имеют общего предка, так что если он реально нужен, вам понадобится унаследовать Packet<TSelf> от нешаблонного Packet:

public abstract class Packet
{
    // общая функциональность тут, возможно как abstract
}

public abstract class Packet<TSelf> : Packet where TSelf : Packet<TSelf>
{
    // ...
9
  • С таким подходом virtual можно убрать. Супер!
    – rotabor
    Commented 26 нояб. 2023 в 20:56
  • @rotabor Спасибо!
    – VladD
    Commented 26 нояб. 2023 в 21:57
  • Аааааа, а я только на неприятность наткнулся, и она как раз решается последним добавлением.
    – rotabor
    Commented 26 нояб. 2023 в 22:09
  • @rotabor вот и хорошо 8-)
    – VladD
    Commented 26 нояб. 2023 в 22:11
  • К сожалению, инициализировать статическую переменную в контексте класса-наследника можно только в нём самом, но не в базовом классе.
    – rotabor
    Commented 26 нояб. 2023 в 23:03
3

Вам нужно объявлять Id в каждом наследнике:

public class PacketA : Packet
{
    public static byte Id;

    public override void Write(ref RawPacket packet)
    {
        packet.WriteByte(Id);
    }
}

Скорее всего, Id как-то связан с классом, так что он может быть константой.

Проверил CRTP by VladD:

using System;
static class Programm {
    public abstract class Packet<TSelf> where TSelf : Packet<TSelf> {
        public static byte Id;
        public void Write() { Console.WriteLine(Id); }
    }
    public class Ipv4Packet : Packet<Ipv4Packet> {
        static Ipv4Packet() { Id = 1; }
    }
    public class Ipv6Packet : Packet<Ipv6Packet> {
        static Ipv6Packet() { Id = 2; }
    }
    static void Main() {
        var a1 = new Ipv4Packet(); a1.Write();
        var a2 = new Ipv6Packet(); a2.Write();
        a1.Write();
        a2.Write();
    }
}

-->

1
2
1
2

Можно взять на вооружение!

2

Можно вместо статики прописать в базовом классе абстрактное свойство:

public abstract class Packet
{
    public abstract byte Id
    {
        get;
    }

    public virtual void Write (ref RawPacket packet)
    {
        packet.WriteByte(Id);
    }
}

И имплементировать это свойство в производных классах:

public class PacketA : Packet
{
    public override byte Id => 1;
}

public class PacketB : Packet
{
    public override byte Id => 2;
}

Ваш ответ

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

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