eu07/maszyna
17
0
Fork 0
mirror of https://github.com/MaSzyna-EU07/maszyna.git synced 2026-11-04 00:42:05 +01:00
Code Issues 156 Wiki Activity

refactor(sonar): add braces around nested if-else (cpp:S5261)

Browse Source
This commit is contained in:
jerrrrycho
2026-10-10 16:25:40 +02:00
parent e7827cfe05
commit a510996dc7
3 changed files with 12 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
McZapkie/Mover.cpp
View File

@@ -4544,6 +4544,7 @@ void TMoverParameters::UpdatePipePressure(double dt)
} }
if (dpMainValve < 0) // && (PipePressureVal > 0.01) //50 if (dpMainValve < 0) // && (PipePressureVal > 0.01) //50
{
if (Compressor > ScndPipePress) if (Compressor > ScndPipePress)
{ {
CompressedVolume = CompressedVolume + dpMainValve / 1500.0; CompressedVolume = CompressedVolume + dpMainValve / 1500.0;
@@ -4551,6 +4552,7 @@ void TMoverParameters::UpdatePipePressure(double dt)
} }
else else
Pipe2->Flow(dpMainValve); Pipe2->Flow(dpMainValve);
}
} }
// ulepszony hamulec bezp. // ulepszony hamulec bezp.
@@ -4625,10 +4627,12 @@ void TMoverParameters::UpdatePipePressure(double dt)
//(Hamulec as TLSt).SetLBP(LocBrakePress); //(Hamulec as TLSt).SetLBP(LocBrakePress);
Hamulec->SetLBP(LocBrakePress); Hamulec->SetLBP(LocBrakePress);
if (BrakeValve == TBrakeValve::EStED) if (BrakeValve == TBrakeValve::EStED)
{
if (MBPM < 2) if (MBPM < 2)
Hamulec->PLC(MaxBrakePress[LoadFlag]); Hamulec->PLC(MaxBrakePress[LoadFlag]);
else else
Hamulec->PLC(TotalMass - Mred); Hamulec->PLC(TotalMass - Mred);
}
break; break;
} }

6
McZapkie/hamulce.cpp
View File

@@ -833,12 +833,14 @@ void TESt::CheckReleaser(double const dt)
// odluzniacz // odluzniacz
if ((BrakeStatus & b_rls) == b_rls) if ((BrakeStatus & b_rls) == b_rls)
{
if (CVP - VVP < 0) if (CVP - VVP < 0)
BrakeStatus &= ~b_rls; BrakeStatus &= ~b_rls;
else else
{ {
CntrlRes->Flow(+PF(CVP, 0, 0.1) * dt); CntrlRes->Flow(+PF(CVP, 0, 0.1) * dt);
} }
}
} }
/// <summary> /// <summary>
@@ -2468,10 +2470,12 @@ void TKE::CheckReleaser(double const dt)
// odluzniacz // odluzniacz
if (true == ((BrakeStatus & b_rls) == b_rls)) if (true == ((BrakeStatus & b_rls) == b_rls))
{
if (CVP - VVP < 0) if (CVP - VVP < 0)
BrakeStatus &= ~b_rls; BrakeStatus &= ~b_rls;
else else
CntrlRes->Flow(+PF(CVP, 0, 0.1) * dt); CntrlRes->Flow(+PF(CVP, 0, 0.1) * dt);
}
} }
/// <summary> /// <summary>
@@ -3821,6 +3825,7 @@ double TM394::GetPF(double i_bcp, double PP, double HP, double dt, double ep)
if (BCP == 0) if (BCP == 0)
LimPP = LimPP + RedAdj; LimPP = LimPP + RedAdj;
if (BCP != 2) if (BCP != 2)
{
if (CP < LimPP) if (CP < LimPP)
CP = CP + 4 * std::min(abs(LimPP - CP), 0.05) * PR(CP, LimPP) * dt; // zbiornik sterujacy CP = CP + 4 * std::min(abs(LimPP - CP), 0.05) * PR(CP, LimPP) * dt; // zbiornik sterujacy
// cp:=cp+6*(2+int(bcp<0))*std::min(abs(Limpp-cp),0.05)*PR(cp,Limpp)*dt //zbiornik // cp:=cp+6*(2+int(bcp<0))*std::min(abs(Limpp-cp),0.05)*PR(cp,Limpp)*dt //zbiornik
@@ -3829,6 +3834,7 @@ double TM394::GetPF(double i_bcp, double PP, double HP, double dt, double ep)
CP = CP - 0.2 * dt / 100; CP = CP - 0.2 * dt / 100;
else else
CP = CP + 4 * (1 + int(BCP != 3) + int(BCP > 4)) * std::min(abs(LimPP - CP), 0.05) * PR(CP, LimPP) * dt; // zbiornik sterujacy CP = CP + 4 * (1 + int(BCP != 3) + int(BCP > 4)) * std::min(abs(LimPP - CP), 0.05) * PR(CP, LimPP) * dt; // zbiornik sterujacy
}
LimPP = CP; LimPP = CP;
dpPipe = std::min(HP, LimPP); dpPipe = std::min(HP, LimPP);

2
vehicle/Train.cpp
View File

@@ -10721,6 +10721,7 @@ void TTrain::DynamicSet(TDynamicObject *d)
mvControlled = DynamicObject->FindPowered()->MoverParameters; mvControlled = DynamicObject->FindPowered()->MoverParameters;
mvSecond = nullptr; // gdyby się nic nie znalazło mvSecond = nullptr; // gdyby się nic nie znalazło
if (mvOccupied->Power > 1.0) // dwuczłonowe lub ukrotnienia, żeby nie szukać każdorazowo if (mvOccupied->Power > 1.0) // dwuczłonowe lub ukrotnienia, żeby nie szukać każdorazowo
{
if (mvOccupied->Couplers[1].Connected ? mvOccupied->Couplers[1].AllowedFlag & coupling::control : false) if (mvOccupied->Couplers[1].Connected ? mvOccupied->Couplers[1].AllowedFlag & coupling::control : false)
{ // gdy jest człon od sprzęgu 1, a sprzęg łączony { // gdy jest człon od sprzęgu 1, a sprzęg łączony
// warsztatowo (powiedzmy) // warsztatowo (powiedzmy)
@@ -10733,6 +10734,7 @@ void TTrain::DynamicSet(TDynamicObject *d)
if (mvOccupied->Couplers[0].Connected->Power > 1.0) // ale ten drugi ma moc if (mvOccupied->Couplers[0].Connected->Power > 1.0) // ale ten drugi ma moc
mvSecond = mvOccupied->Couplers[0].Connected; // wskaźnik na drugiego mvSecond = mvOccupied->Couplers[0].Connected; // wskaźnik na drugiego
} }
}
// cache nearest unit equipped with pantographs // cache nearest unit equipped with pantographs
{ {
auto *lookup{DynamicObject->FindPantographCarrier()}; auto *lookup{DynamicObject->FindPantographCarrier()};