fixed initialization of track classes and basic 3d types

2026-07-19 03:09:18 +02:00 · 2017-01-27 00:50:54 +01:00
parent 647e1f9000
commit 71cd60b543
6 changed files with 97 additions and 144 deletions
--- a/ResourceManager.h
+++ b/ResourceManager.h
@@ -26,13 +26,13 @@ class Resource
    }

  protected:
-    void SetLastUsage(double lastUsage)
+    void SetLastUsage(double const lastUsage)
    {
        _lastUsage = lastUsage;
    }

  private:
-    double _lastUsage;
+      double _lastUsage = 0.0;
 };

 class ResourceManager
--- a/Segment.cpp
+++ b/Segment.cpp
@@ -26,15 +26,11 @@ std::string Where(vector3 p)
    return std::to_string(p.x) + " " + std::to_string(p.y) + " " + std::to_string(p.z);
 };

-TSegment::TSegment(TTrack *owner)
+TSegment::TSegment(TTrack *owner) :
+    pOwner( owner )
 {
-    Point1 = CPointOut = CPointIn = Point2 = vector3(0.0f, 0.0f, 0.0f);
-    fLength = 0;
-    fRoll1 = 0;
-    fRoll2 = 0;
-    fTsBuffer = NULL;
-    fStep = 0;
-    pOwner = owner;
+    fAngle[ 0 ] = 0.0;
+    fAngle[ 1 ] = 0.0;
 };

 TSegment::~TSegment()
--- a/Segment.h
+++ b/Segment.h
@@ -52,18 +52,20 @@ class TSegment
 { // aproksymacja toru (zwrotnica ma dwa takie, jeden z nich jest aktywny)
  private:
    vector3 Point1, CPointOut, CPointIn, Point2;
-    double fRoll1, fRoll2; // przechyłka na końcach
-    double fLength; // długość policzona
-    double *fTsBuffer; // wartości parametru krzywej dla równych odcinków
-    double fStep;
-    int iSegCount; // ilość odcinków do rysowania krzywej
-    double fDirection; // Ra: kąt prostego w planie; dla łuku kąt od Point1
-    double fStoop; // Ra: kąt wzniesienia; dla łuku od Point1
+    double fRoll1 = 0.0,
+           fRoll2 = 0.0; // przechyłka na końcach
+    double fLength = 0.0; // długość policzona
+    double *fTsBuffer = nullptr; // wartości parametru krzywej dla równych odcinków
+    double fStep = 0.0;
+    int iSegCount = 0; // ilość odcinków do rysowania krzywej
+    double fDirection = 0.0; // Ra: kąt prostego w planie; dla łuku kąt od Point1
+    double fStoop = 0.0; // Ra: kąt wzniesienia; dla łuku od Point1
    vector3 vA, vB, vC; // współczynniki wielomianów trzeciego stopnia vD==Point1
    // TSegment *pPrev; //odcinek od strony punktu 1 - w segmencie, żeby nie skakać na zwrotnicach
    // TSegment *pNext; //odcinek od strony punktu 2
-    TTrack *pOwner; // wskaźnik na właściciela
+    TTrack *pOwner = nullptr; // wskaźnik na właściciela
    double fAngle[2]; // kąty zakończenia drogi na przejazdach
+
    vector3 GetFirstDerivative(double fTime);
    double RombergIntegral(double fA, double fB);
    double GetTFromS(double s);
@@ -72,7 +74,7 @@ class TSegment
    // TSegment *segNeightbour[2]; //sąsiednie odcinki - musi być przeniesione z Track
    // int iNeightbour[2]; //do którego końca doczepiony
  public:
-    bool bCurve;
+    bool bCurve = false;
    // int iShape; //Ra: flagi kształtu dadzą więcej możliwości optymalizacji
    // (0-Bezier,1-prosty,2/3-łuk w lewo/prawo,6/7-przejściowa w lewo/prawo)
    TSegment(TTrack *owner);
--- a/Track.cpp
+++ b/Track.cpp
@@ -45,20 +45,14 @@ const int iEnds3[13] = {3, 0, 2, 1, 2, 0, -1,
                        1, 0, 2, 0, 3, 1}; // numer sąsiedniego toru na końcu segmentu "-1"
 TIsolated *TIsolated::pRoot = NULL;

-TSwitchExtension::TSwitchExtension(TTrack *owner, int what)
+TSwitchExtension::TSwitchExtension(TTrack *owner, int const what)
 { // na początku wszystko puste
-    CurrentIndex = 0;
-    pNexts[0] = NULL; // wskaźniki do kolejnych odcinków ruchu
-    pNexts[1] = NULL;
-    pPrevs[0] = NULL;
-    pPrevs[1] = NULL;
-    fOffsetSpeed = 0.1; // prędkość liniowa iglic
-    fOffsetDelay = 0.05; // dodatkowy ruch drugiej iglicy po zablokowaniu pierwszej na opornicy
+    pNexts[0] = nullptr; // wskaźniki do kolejnych odcinków ruchu
+    pNexts[1] = nullptr;
+    pPrevs[0] = nullptr;
+    pPrevs[1] = nullptr;
    fOffset1 = fOffset = fDesiredOffset = -fOffsetDelay; // położenie zasadnicze
    fOffset2 = 0.0; // w zasadniczym wewnętrzna iglica dolega
-    pOwner = NULL;
-    pNextAnim = NULL;
-    bMovement = false; // nie potrzeba przeliczać fOffset1
    Segments[0] = std::make_shared<TSegment>(owner); // z punktu 1 do 2
    Segments[1] = std::make_shared<TSegment>(owner); // z punktu 3 do 4 (1=3 dla zwrotnic; odwrócony dla skrzyżowań, ewentualnie 1=4)
    Segments[2] = (what >= 3) ?
@@ -73,9 +67,6 @@ TSwitchExtension::TSwitchExtension(TTrack *owner, int what)
    Segments[5] = (what >= 6) ?
                      std::make_shared<TSegment>(owner) :
                      nullptr; // z punktu 3 do 2              2       2            1       1
-    evPlus = evMinus = NULL;
-    fVelocity = -1.0; // maksymalne ograniczenie prędkości (ustawianej eventem)
-    vTrans = vector3(0, 0, 0); // docelowa translacja przesuwnicy
 }
 TSwitchExtension::~TSwitchExtension()
 { // nie ma nic do usuwania
@@ -153,56 +144,15 @@ void TIsolated::Modify(int i, TDynamicObject *o)
    }
 };

-TTrack::TTrack(TGroundNode *g)
-{ // tworzenie nowego odcinka ruchu
-    trNext = trPrev = NULL; // sąsiednie
-    Segment = NULL; // dane odcinka
-    SwitchExtension = NULL; // dodatkowe parametry zwrotnicy i obrotnicy
-    TextureID1 = 0; // tekstura szyny
-    fTexLength = 4.0; // powtarzanie tekstury
-    TextureID2 = 0; // tekstura podsypki albo drugiego toru zwrotnicy
-    fTexHeight1 = 0.6; // nowy profil podsypki ;)
-    fTexWidth = 0.9;
-    fTexSlope = 0.9;
-    eType = tt_Normal; // domyślnie zwykły
-    iCategoryFlag = 1; // 1-tor, 2-droga, 4-rzeka, 8-samolot?
-    fTrackWidth = 1.435; // rozstaw toru, szerokość nawierzchni
-    fFriction = 0.15; // współczynnik tarcia
-    fSoundDistance = -1;
-    iQualityFlag = 20;
-    iDamageFlag = 0;
-    eEnvironment = e_flat;
-    bVisible = true;
-    iEvents = 0; // Ra: flaga informująca o obecności eventów
-    evEvent0 = NULL;
-    evEvent1 = NULL;
-    evEvent2 = NULL;
-    evEventall0 = NULL;
-    evEventall1 = NULL;
-    evEventall2 = NULL;
-    fVelocity = -1; // ograniczenie prędkości
-    fTrackLength = 100.0;
-    fRadius = 0; // promień wybranego toru zwrotnicy
-    fRadiusTable[0] = 0; // dwa promienie nawet dla prostego
-    fRadiusTable[1] = 0;
-    iNumDynamics = 0;
-    ScannedFlag = false;
-    DisplayListID = 0;
-    iTrapezoid = 0; // parametry kształtu: 0-standard, 1-przechyłka, 2-trapez, 3-oba
-    hvOverhead = NULL; // drut zasilający, najbliższy Point1 toru
-    fTexRatio1 =
-        1.0; // proporcja boków tekstury nawierzchni (żeby zaoszczędzić na rozmiarach tekstur...)
-    fTexRatio2 =
-        1.0; // proporcja boków tekstury chodnika (żeby zaoszczędzić na rozmiarach tekstur...)
-    iPrevDirection = 0; // domyślnie wirtualne odcinki dołączamy stroną od Point1
-    iNextDirection = 0;
-    pIsolated = NULL;
-    pMyNode = g; // Ra: proteza, żeby tor znał swoją nazwę TODO: odziedziczyć TTrack z TGroundNode
-    iAction = 0; // normalnie może być pomijany podczas skanowania
-    fOverhead = -1.0; // można normalnie pobierać prąd (0 dla jazdy bezprądowej po danym odcinku
-    nFouling[0] = NULL; // ukres albo kozioł od strony Point1
-    nFouling[1] = NULL; // ukres albo kozioł od strony Point2
-    trColides = NULL; // tor kolizyjny, na którym trzeba sprawdzać pojazdy pod kątem zderzenia
+// tworzenie nowego odcinka ruchu
+TTrack::TTrack(TGroundNode *g) :
+    pMyNode( g ) // Ra: proteza, żeby tor znał swoją nazwę TODO: odziedziczyć TTrack z TGroundNode
+{
+    ::SecureZeroMemory( Dynamics, sizeof( Dynamics ) );
+    fRadiusTable[ 0 ] = 0.0;
+    fRadiusTable[ 1 ] = 0.0;
+    nFouling[ 0 ] = nullptr;
+    nFouling[ 1 ] = nullptr;
 }

 TTrack::~TTrack()
--- a/Track.h
+++ b/Track.h
@@ -46,25 +46,26 @@ class TTraction;
 class TSwitchExtension
 { // dodatkowe dane do toru, który jest zwrotnicą
  public:
-    TSwitchExtension(TTrack *owner, int what);
+    TSwitchExtension(TTrack *owner, int const what);
    ~TSwitchExtension();
-    std::shared_ptr<TSegment> Segments[6]; // dwa tory od punktu 1, pozosta³e dwa od 2? Ra 140101: 6 po³¹czeñ dla
-    // skrzy¿owañ
+    std::shared_ptr<TSegment> Segments[6]; // dwa tory od punktu 1, pozosta³e dwa od 2? Ra 140101: 6 po³¹czeñ dla skrzy¿owañ
    // TTrack *trNear[4]; //tory do³¹czone do punktów 1, 2, 3 i 4
    // dotychczasowe [2]+[2] wskaŸniki zamieniæ na nowe [4]
    TTrack *pNexts[2]; // tory do³¹czone do punktów 2 i 4
    TTrack *pPrevs[2]; // tory do³¹czone do punktów 1 i 3
    int iNextDirection[2]; // to te¿ z [2]+[2] przerobiæ na [4]
    int iPrevDirection[2];
-    int CurrentIndex; // dla zwrotnicy
-    double fOffset, fDesiredOffset; // aktualne i docelowe położenie napędu iglic
-    double fOffsetSpeed; // prędkość liniowa ruchu iglic
-    double fOffsetDelay; // opóźnienie ruchu drugiej iglicy względem pierwszej
+    int CurrentIndex = 0; // dla zwrotnicy
+    double fOffset = 0.0,
+           fDesiredOffset = 0.0; // aktualne i docelowe położenie napędu iglic
+    double fOffsetSpeed = 0.1; // prędkość liniowa ruchu iglic
+    double fOffsetDelay = 0.05; // opóźnienie ruchu drugiej iglicy względem pierwszej // dodatkowy ruch drugiej iglicy po zablokowaniu pierwszej na opornicy
    union
    {
        struct
        { // zmienne potrzebne tylko dla zwrotnicy
-            double fOffset1, fOffset2; // przesunięcia iglic - 0=na wprost
+            double fOffset1,
+                   fOffset2; // przesunięcia iglic - 0=na wprost
            bool RightSwitch; // czy zwrotnica w prawo
        };
        struct
@@ -81,12 +82,14 @@ class TSwitchExtension
            int iRoads; // ile dróg się spotyka?
        };
    };
-    bool bMovement; // czy w trakcie animacji
-    int iLeftVBO, iRightVBO; // indeksy iglic w VBO
-    TSubRect *pOwner; // sektor, któremu trzeba zgłosić animację
-    TTrack *pNextAnim; // następny tor do animowania
-    TEvent *evPlus, *evMinus; // zdarzenia sygnalizacji rozprucia
-    float fVelocity; // maksymalne ograniczenie prędkości (ustawianej eventem)
+    bool bMovement = false; // czy w trakcie animacji
+    int iLeftVBO = 0,
+        iRightVBO = 0; // indeksy iglic w VBO
+    TSubRect *pOwner = nullptr; // sektor, któremu trzeba zgłosić animację
+    TTrack *pNextAnim = nullptr; // następny tor do animowania
+    TEvent *evPlus = nullptr,
+           *evMinus = nullptr; // zdarzenia sygnalizacji rozprucia
+    float fVelocity = -1.0; // maksymalne ograniczenie prędkości (ustawianej eventem)
    vector3 vTrans; // docelowa translacja przesuwnicy
  private:
 };
@@ -128,64 +131,63 @@ class TTrack : public Resource
  private:
 	std::shared_ptr<TSwitchExtension> SwitchExtension; // dodatkowe dane do toru, który jest zwrotnicą
    std::shared_ptr<TSegment> Segment;
-    TTrack *trNext; // odcinek od strony punktu 2 - to powinno być w segmencie
-    TTrack *trPrev; // odcinek od strony punktu 1
+    TTrack *trNext = nullptr; // odcinek od strony punktu 2 - to powinno być w segmencie
+    TTrack *trPrev = nullptr; // odcinek od strony punktu 1
    // McZapkie-070402: dodalem zmienne opisujace rozmiary tekstur
-    GLuint TextureID1; // tekstura szyn albo nawierzchni
-    GLuint TextureID2; // tekstura automatycznej podsypki albo pobocza
-    float fTexLength; // długość powtarzania tekstury w metrach
-    float fTexRatio1; // proporcja rozmiarów tekstury dla nawierzchni drogi
-    float fTexRatio2; // proporcja rozmiarów tekstury dla chodnika
-    float fTexHeight1; // wysokość brzegu względem trajektorii
-    float fTexWidth; // szerokość boku
-    float fTexSlope;
-    double fRadiusTable[2]; // dwa promienie, drugi dla zwrotnicy
-    int iTrapezoid; // 0-standard, 1-przechyłka, 2-trapez, 3-oba
-    GLuint DisplayListID;
-    TIsolated *pIsolated; // obwód izolowany obsługujący zajęcia/zwolnienia grupy torów
+    GLuint TextureID1 = 0; // tekstura szyn albo nawierzchni
+    GLuint TextureID2 = 0; // tekstura automatycznej podsypki albo pobocza
+    float fTexLength = 4.0; // długość powtarzania tekstury w metrach
+    float fTexRatio1 = 1.0; // proporcja boków tekstury nawierzchni (żeby zaoszczędzić na rozmiarach tekstur...)
+    float fTexRatio2 = 1.0; // proporcja boków tekstury chodnika (żeby zaoszczędzić na rozmiarach tekstur...)
+    float fTexHeight1 = 0.6; // wysokość brzegu względem trajektorii
+    float fTexWidth = 0.9; // szerokość boku
+    float fTexSlope = 0.9;
+    double fRadiusTable[ 2 ]; // dwa promienie, drugi dla zwrotnicy
+    int iTrapezoid = 0; // 0-standard, 1-przechyłka, 2-trapez, 3-oba
+    GLuint DisplayListID = 0;
+    TIsolated *pIsolated = nullptr; // obwód izolowany obsługujący zajęcia/zwolnienia grupy torów
    TGroundNode *
-        pMyNode; // Ra: proteza, żeby tor znał swoją nazwę TODO: odziedziczyć TTrack z TGroundNode
+        pMyNode = nullptr; // Ra: proteza, żeby tor znał swoją nazwę TODO: odziedziczyć TTrack z TGroundNode
  public:
-    int iNumDynamics;
+    int iNumDynamics = 0;
    TDynamicObject *Dynamics[iMaxNumDynamics];
-    int iEvents; // Ra: flaga informująca o obecności eventów
-    TEvent *evEventall0; // McZapkie-140302: wyzwalany gdy pojazd stoi
-    TEvent *evEventall1;
-    TEvent *evEventall2;
-    TEvent *evEvent0; // McZapkie-280503: wyzwalany tylko gdy headdriver
-    TEvent *evEvent1;
-    TEvent *evEvent2;
+    int iEvents = 0; // Ra: flaga informująca o obecności eventów
+    TEvent *evEventall0 = nullptr; // McZapkie-140302: wyzwalany gdy pojazd stoi
+    TEvent *evEventall1 = nullptr;
+    TEvent *evEventall2 = nullptr;
+    TEvent *evEvent0 = nullptr; // McZapkie-280503: wyzwalany tylko gdy headdriver
+    TEvent *evEvent1 = nullptr;
+    TEvent *evEvent2 = nullptr;
    std::string asEventall0Name; // nazwy eventów
 	std::string asEventall1Name;
 	std::string asEventall2Name;
 	std::string asEvent0Name;
 	std::string asEvent1Name;
 	std::string asEvent2Name;
-    int iNextDirection; // 0:Point1, 1:Point2, 3:do odchylonego na zwrotnicy
-    int iPrevDirection;
-    TTrackType eType;
-    int iCategoryFlag; // 0x100 - usuwanie pojazów
-    float fTrackWidth; // szerokość w punkcie 1
-    float fTrackWidth2; // szerokość w punkcie 2 (głównie drogi i rzeki)
-    float fFriction; // współczynnik tarcia
-    float fSoundDistance;
-    int iQualityFlag;
-    int iDamageFlag;
-    TEnvironmentType eEnvironment; // dźwięk i oświetlenie
-    bool bVisible; // czy rysowany
-    int iAction; // czy modyfikowany eventami (specjalna obsługa przy skanowaniu)
-    float fOverhead; // informacja o stanie sieci: 0-jazda bezprądowa, >0-z opuszczonym i
-    // ograniczeniem prędkości
+    int iNextDirection = 0; // 0:Point1, 1:Point2, 3:do odchylonego na zwrotnicy
+    int iPrevDirection = 0; // domyślnie wirtualne odcinki dołączamy stroną od Point1
+    TTrackType eType = tt_Normal; // domyślnie zwykły
+    int iCategoryFlag = 1; // 0x100 - usuwanie pojazów // 1-tor, 2-droga, 4-rzeka, 8-samolot?
+    float fTrackWidth = 1.435; // szerokość w punkcie 1 // rozstaw toru, szerokość nawierzchni
+    float fTrackWidth2 = 0.0; // szerokość w punkcie 2 (głównie drogi i rzeki)
+    float fFriction = 0.15; // współczynnik tarcia
+    float fSoundDistance = -1.0;
+    int iQualityFlag = 20;
+    int iDamageFlag = 0;
+    TEnvironmentType eEnvironment = e_flat; // dźwięk i oświetlenie
+    bool bVisible = true; // czy rysowany
+    int iAction = 0; // czy modyfikowany eventami (specjalna obsługa przy skanowaniu)
+    float fOverhead = -1.0; // można normalnie pobierać prąd (0 dla jazdy bezprądowej po danym odcinku, >0-z opuszczonym i ograniczeniem prędkości)
  private:
-    double fVelocity; // prędkość dla AI (powyżej rośnie prawdopowobieństwo wykolejenia)
+    double fVelocity = -1.0; // ograniczenie prędkości // prędkość dla AI (powyżej rośnie prawdopowobieństwo wykolejenia)
  public:
    // McZapkie-100502:
-    double fTrackLength; // długość z wpisu, nigdzie nie używana
-    double fRadius; // promień, dla zwrotnicy kopiowany z tabeli
-    bool ScannedFlag; // McZapkie: do zaznaczania kolorem torów skanowanych przez AI
-    TTraction *hvOverhead; // drut zasilający do szybkiego znalezienia (nie używany)
-    TGroundNode *nFouling[2]; // współrzędne ukresu albo oporu kozła
-    TTrack *trColides; // tor kolizyjny, na którym trzeba sprawdzać pojazdy pod kątem zderzenia
+    double fTrackLength = 100.0; // długość z wpisu, nigdzie nie używana
+    double fRadius = 0.0; // promień, dla zwrotnicy kopiowany z tabeli
+    bool ScannedFlag = false; // McZapkie: do zaznaczania kolorem torów skanowanych przez AI
+    TTraction *hvOverhead = nullptr; // drut zasilający do szybkiego znalezienia (nie używany)
+    TGroundNode *nFouling[ 2 ]; // współrzędne ukresu albo oporu kozła
+    TTrack *trColides = nullptr; // tor kolizyjny, na którym trzeba sprawdzać pojazdy pod kątem zderzenia

    TTrack(TGroundNode *g);
    ~TTrack();
--- a/dumb3d.h
+++ b/dumb3d.h
@@ -42,7 +42,8 @@ inline scalar_t SQRT_FUNCTION(scalar_t x)
 class vector2
 {
  public:
-    vector2(void)
+    vector2(void) :
+        x(0.0), y(0.0)
    {
    }
    vector2(scalar_t a, scalar_t b)
@@ -61,7 +62,8 @@ class vector2
 class vector3
 {
  public:
-    vector3(void)
+    vector3(void) :
+        x(0.0), y(0.0), z(0.0)
    {
    }
    vector3(scalar_t a, scalar_t b, scalar_t c)
@@ -129,6 +131,7 @@ class matrix4x4
  public:
    matrix4x4(void)
    {
+        ::SecureZeroMemory( e, sizeof( e ) );
    }

    // When defining matrices in C arrays, it is easiest to define them with