Problem z dodaniem metody do klasy Packet (G2O v.0.1.0)

[Patrix]

Witam, ostatnio myślałem nad dodaniem metody w squirrelu do klasy Packet, która automatycznie określałaby z jakiej metody skorzystać do zapisania Int'a na jak najmniejszej ilości bitów. Niby wszystko fajnie, pięknie, lecz przy próbie wywołania metody następuje crash, poniżej załączam screen z błędem:



Tu fragment kodu testowego, po którym crashuje:

Squirrel Script

packet <- Packet()
packet.writeInt(1)

Squirrel Script

function Packet::writeInt(value)
{
    if (typeof(value) == "integer")
    {
 
        if (value >= 0)
        {
            if (value >= 0 && value <= 255)
            {
                Packet.writeUInt8(value)
            }
            else if (value >= 256 && value <= 65535)
            {
                Packet.writeUInt16(value)
            }
            else if (value >= 65536)
            {
                Packet.writeUInt32(value)
            }
        }
        else
        {
            if (value < 0 && value >= -128)
            {
                Packet.writeInt8(value)
            }
            else if (value <= -129 && value >= -32768)
            {
                Packet.writeInt16(value)
            }
            else if (value <= -32769)
            {
                Packet.writeInt32(value)
            }
        }
        
    }
}

Metoda Packet.writeInt się wywołuje, lecz po wywołaniu metody Packet.writeUint8 następuje crash. Ktoś ma jakiś pomysł dlaczego?
Przy okazji, czy dodawania takiej metody to w ogóle dobry pomysł?

[KimiorV]

A skąd będziesz wiedział jakiego reada użyć po drugiej stronie?

[Patrix]

Właśnie nad tym myślałem i tu jest problem xD. Teoretycznie mógłym użyć metod writeString i readString, ale nie jest to zbytnio dobry pomysł. Problem udało mi się przed chwilą rozwiązać zamieniając w if'ach słówko Packet na this.

[Bimbol]

No nic dziwnego, że crashowało. Chciałeś wpisać wartości do szablonu klasy? Złamałeś tutaj zasadę działania obiektowości.
No tak jak mówi Kimior, nie wiesz ile bitów będziesz musiał odczytać. Aby ominąć ten problem i tak musisz wpisywać dodatkowy bajt, który określi ile danych wpisałeś.
Czytaj 1 bajt - jakie dane, następne faktyczne dane. Przy czym kiedy przesyłasz małe wartości np. zakres 0-100 to tracisz 1 bajt, którego by tam nie musiało być.
Generalnie nie wiem po co tutaj takie uproszczenia.

[Patrix]

Rozumiem. Zastanawiałem się czy miałoby to w ogóle jakiś sens, ale z tego co widzę, to niezbyt opłacalne jest dodawanie takiej metody. Temat można zamknąć.