refactor(sonar): merge collapsible nested if statements (cpp:S1066)

Comments that preceded the inner if are kept above the merged statement.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>

world/EvLaunch.cpp

@@ -195,12 +195,10 @@ bool TEventLauncher::check_activation() {
     else {
         // jeśli nie cykliczny, to sprawdzić czas
         if( simulation::Time.data().wHour == iHour ) {
-            if( simulation::Time.data().wMinute == iMinute ) {
-                // zgodność czasu uruchomienia
-                if( UpdatedTime < 10 ) {
-                    UpdatedTime = 20; // czas do kolejnego wyzwolenia?
-                    bCond = true;
-                }
+            // zgodność czasu uruchomienia
+            if (simulation::Time.data().wMinute == iMinute && UpdatedTime < 10) {
+                UpdatedTime = 20; // czas do kolejnego wyzwolenia?
+                bCond = true;
             }
         }
         else {

world/Segment.cpp

@@ -502,13 +502,12 @@ bool TSegment::RenderLoft( gfx::vertex_array &Output, glm::dvec3 const &Origin,
                         glm::normalize( norm ),
                         glm::vec2 { ( jmm1 * ShapePoints[ j ].texture.x + m1 * ShapePoints[ j + iNumShapePoints ].texture.x ) / texturescale, tv1 } );
                 }
-                if( p ) // jeśli jest wskaźnik do tablicy
-                    if( *p )
-                        if( !j ) // to dla pierwszego punktu
-                        {
-                            **p = pt;
-                            ( *p )++;
-                        } // zapamiętanie brzegu jezdni
+                // jeśli jest wskaźnik do tablicy
+                if (p && *p && !j) // to dla pierwszego punktu
+                {
+                    **p = pt;
+                    ( *p )++;
+                } // zapamiętanie brzegu jezdni
                 // dla trapezu drugi koniec ma inne współrzędne
                 pt = parallel2 * ( jmm2 * ( ShapePoints[ j ].position.x - fOffsetX.first ) + m2 * ( ShapePoints[ j + iNumShapePoints ].position.x - fOffsetX.second ) ) + pos2;
                 pt.y += jmm2 * ShapePoints[ j ].position.y + m2 * ShapePoints[ j + iNumShapePoints ].position.y;
@@ -521,13 +520,12 @@ bool TSegment::RenderLoft( gfx::vertex_array &Output, glm::dvec3 const &Origin,
                         glm::normalize( norm ),
                         glm::vec2 { ( jmm2 * ShapePoints[ j ].texture.x + m2 * ShapePoints[ j + iNumShapePoints ].texture.x ) / texturescale, tv2 } );
                 }
-                if( p ) // jeśli jest wskaźnik do tablicy
-                    if( *p )
-                        if( !j ) // to dla pierwszego punktu
-                            if( i == iSegCount ) {
-                                **p = pt;
-                                ( *p )++;
-                            } // zapamiętanie brzegu jezdni
+                // jeśli jest wskaźnik do tablicy
+                // to dla pierwszego punktu
+                if (p && *p && !j && i == iSegCount) {
+                    **p = pt;
+                    ( *p )++;
+                } // zapamiętanie brzegu jezdni
             }
         }
         else {

world/Traction.cpp

@@ -472,9 +472,8 @@ void TTraction::PowerSet(TTractionPowerSource *ps)
 
 double TTraction::VoltageGet(double u, double i)
 { // pobranie napięcia na przęśle po podłączeniu do niego rezystancji (res) - na razie jest to prąd
-    if (!psSection)
-        if (!psPowered)
-            return NominalVoltage; // jak nie ma zasilacza, to napięcie podane w przęśle
+    if (!psSection && !psPowered)
+        return NominalVoltage; // jak nie ma zasilacza, to napięcie podane w przęśle
     // na początek można założyć, że wszystkie podstacje mają to samo napięcie i nie płynie prąd
     // pomiędzy nimi
     // dla danego przęsła mamy 3 źródła zasilania
@@ -740,29 +739,25 @@ traction_table::InitTraction() {
                     break;
                 }
             }
-            if( traction->hvNext[ 0 ] ) {
-                // jeśli został podłączony
-                if( traction->psSection != nullptr
-                 && matchingtraction->psSection != nullptr ) {
-                    // tylko przęsło z izolatorem może nie mieć zasilania, bo ma 2, trzeba sprawdzać sąsiednie
-                    if( traction->psSection != matchingtraction->psSection ) {
-                        // połączone odcinki mają różne zasilacze
-                        // to może być albo podłączenie podstacji lub kabiny sekcyjnej do sekcji, albo błąd
-                        if( true == traction->psSection->bSection
-                         && false == matchingtraction->psSection->bSection ) {
-                            //(tmp->psSection) jest podstacją, a (Traction->psSection) nazwą sekcji
-                            matchingtraction->PowerSet( traction->psSection ); // zastąpienie wskazaniem sekcji
-                        }
-                        else if( false == traction->psSection->bSection
-                              && true == matchingtraction->psSection->bSection ) {
-                            //(Traction->psSection) jest podstacją, a (tmp->psSection) nazwą sekcji
-                            traction->PowerSet( matchingtraction->psSection ); // zastąpienie wskazaniem sekcji
-                        }
-                        else {
-                            // jeśli obie to sekcje albo obie podstacje, to będzie błąd
-                            ErrorLog( "Bad scenario: faulty traction power connection at location " + to_string( traction->pPoint1 ) );
-                        }
-                    }
+            // jeśli został podłączony
+            // tylko przęsło z izolatorem może nie mieć zasilania, bo ma 2, trzeba sprawdzać sąsiednie
+            if (traction->hvNext[ 0 ] && traction->psSection != nullptr
+                 && matchingtraction->psSection != nullptr && traction->psSection != matchingtraction->psSection) {
+                // połączone odcinki mają różne zasilacze
+                // to może być albo podłączenie podstacji lub kabiny sekcyjnej do sekcji, albo błąd
+                if( true == traction->psSection->bSection
+                 && false == matchingtraction->psSection->bSection ) {
+                    //(tmp->psSection) jest podstacją, a (Traction->psSection) nazwą sekcji
+                    matchingtraction->PowerSet( traction->psSection ); // zastąpienie wskazaniem sekcji
+                }
+                else if( false == traction->psSection->bSection
+                      && true == matchingtraction->psSection->bSection ) {
+                    //(Traction->psSection) jest podstacją, a (tmp->psSection) nazwą sekcji
+                    traction->PowerSet( matchingtraction->psSection ); // zastąpienie wskazaniem sekcji
+                }
+                else {
+                    // jeśli obie to sekcje albo obie podstacje, to będzie błąd
+                    ErrorLog( "Bad scenario: faulty traction power connection at location " + to_string( traction->pPoint1 ) );
                 }
             }
         }
@@ -779,28 +774,24 @@ traction_table::InitTraction() {
                     break;
                 }
             }
-            if( traction->hvNext[ 1 ] ) {
-                // jeśli został podłączony
-                if( traction->psSection != nullptr
-                 && matchingtraction->psSection != nullptr ) {
-                    // tylko przęsło z izolatorem może nie mieć zasilania, bo ma 2, trzeba sprawdzać sąsiednie
-                    if( traction->psSection != matchingtraction->psSection ) {
-                        // to może być albo podłączenie podstacji lub kabiny sekcyjnej do sekcji, albo błąd
-                        if( true == traction->psSection->bSection
-                         && false == matchingtraction->psSection->bSection ) {
-                            //(tmp->psSection) jest podstacją, a (Traction->psSection) nazwą sekcji
-                            matchingtraction->PowerSet( traction->psSection ); // zastąpienie wskazaniem sekcji
-                        }
-                        else if( false == traction->psSection->bSection
-                              && true == matchingtraction->psSection->bSection ) {
-                            //(Traction->psSection) jest podstacją, a (tmp->psSection) nazwą sekcji
-                            traction->PowerSet( matchingtraction->psSection ); // zastąpienie wskazaniem sekcji
-                        }
-                        else {
-                            // jeśli obie to sekcje albo obie podstacje, to będzie błąd
-                            ErrorLog( "Bad scenario: faulty traction power connection at location " + to_string( traction->pPoint2 ) );
-                        }
-                    }
+            // jeśli został podłączony
+            // tylko przęsło z izolatorem może nie mieć zasilania, bo ma 2, trzeba sprawdzać sąsiednie
+            if (traction->hvNext[ 1 ] && traction->psSection != nullptr
+                 && matchingtraction->psSection != nullptr && traction->psSection != matchingtraction->psSection) {
+                // to może być albo podłączenie podstacji lub kabiny sekcyjnej do sekcji, albo błąd
+                if( true == traction->psSection->bSection
+                 && false == matchingtraction->psSection->bSection ) {
+                    //(tmp->psSection) jest podstacją, a (Traction->psSection) nazwą sekcji
+                    matchingtraction->PowerSet( traction->psSection ); // zastąpienie wskazaniem sekcji
+                }
+                else if( false == traction->psSection->bSection
+                      && true == matchingtraction->psSection->bSection ) {
+                    //(Traction->psSection) jest podstacją, a (tmp->psSection) nazwą sekcji
+                    traction->PowerSet( matchingtraction->psSection ); // zastąpienie wskazaniem sekcji
+                }
+                else {
+                    // jeśli obie to sekcje albo obie podstacje, to będzie błąd
+                    ErrorLog( "Bad scenario: faulty traction power connection at location " + to_string( traction->pPoint2 ) );
                 }
             }
         }

world/mtable.cpp

@@ -565,14 +565,11 @@ bool TTrainParameters::LoadTTfile(std::string scnpath, int iPlus, double vmax)
             fin.close();
         }
     }
-    if (ConversionError == 0)
-    {
-        if (TimeTable[1].StationName == Relation1) // jeśli nazwa pierwszego zgodna z relacją
-            if (TimeTable[1].Ah < 0) // a nie podany czas przyjazdu
-            { // to mamy zatrzymanie na pierwszym, a nie przelot
-                TimeTable[1].Ah = TimeTable[1].Dh;
-                TimeTable[1].Am = TimeTable[1].Dm;
-            }
+    // jeśli nazwa pierwszego zgodna z relacją
+    if (ConversionError == 0 && TimeTable[1].StationName == Relation1 && TimeTable[1].Ah < 0) // a nie podany czas przyjazdu
+    { // to mamy zatrzymanie na pierwszym, a nie przelot
+        TimeTable[1].Ah = TimeTable[1].Dh;
+        TimeTable[1].Am = TimeTable[1].Dm;
     }
     //
     load_sounds();
@@ -632,9 +629,9 @@ TTrainParameters::load_sounds() {
 bool TTrainParameters::DirectionChange() const
 // sprawdzenie, czy po zatrzymaniu wykonać kolejne komendy
 {
-    if (StationIndex > 0 && StationIndex < StationCount) // dla ostatniej stacji nie
-        if (contains( TimeTable[StationIndex].StationWare, '@') )
-            return true;
+    // dla ostatniej stacji nie
+    if (StationIndex > 0 && StationIndex < StationCount && contains( TimeTable[StationIndex].StationWare, '@'))
+        return true;
     return false;
 }