Modification suggested by codeFactor

Author: Nicola-Fonzi

SU2_PY/FSI_tools/FSIInterface.py

@@ -2189,10 +2189,10 @@ def MapModes(self, FSI_config, FluidSolver, SolidSolver):
 
               if myid == self.rootProcess:
                   AllFiles = os.listdir()
-                  for FileNumber in range(len(AllFiles)):
-                      if SurfaceFileName in AllFiles[FileNumber]:
-                          file = AllFiles[FileNumber].split(".")[0]
-                          extension = AllFiles[FileNumber].split(".")[1]
+                  for FileNumber,FileName in enumerate(AllFiles):
+                      if SurfaceFileName in FileName:
+                          file = FileName.split(".")[0]
+                          extension = FileName.split(".")[1]
                           os.rename(file+"."+extension,"Mode{}.".format(mode)+extension)
 
           self.MPIPrint('\n*************************')

SU2_PY/SU2_Nastran/pysu2_nastran.py

@@ -77,7 +77,7 @@ def GetDispl(self,time):
     if self.typeOfMotion == "BLENDED_STEP":
       if (time < 0.0) or (time > self.timeStop):
         return 0.0
-      elif time < self.tmax:
+      if time < self.tmax:
         return self.amplitude/2.0*(1.0-cos(self.omega0*time*self.vinf/self.lref))
       return self.amplitude
 
@@ -93,7 +93,7 @@ def GetVel(self,time):
     if self.typeOfMotion == "BLENDED_STEP":
       if (time < 0.0) or (time > self.timeStop):
         return 0.0
-      elif time < self.tmax:
+      if time < self.tmax:
         return self.amplitude/2.0*sin(self.omega0*time*self.vinf/self.lref)*(self.omega0*self.vinf/self.lref)
       return 0.0
 
@@ -108,7 +108,7 @@ def GetAcc(self,time):
     if self.typeOfMotion == "BLENDED_STEP":
       if (time < 0.0) or (time > self.timeStop):
         return 0.0
-      elif time < self.tmax:
+      if time < self.tmax:
         return self.amplitude/2.0*cos(self.omega0*time*self.vinf/self.lref)*(self.omega0*self.vinf/self.lref)**2
       return 0.0
 
@@ -511,8 +511,7 @@ def __checkBlankField(self, string):
 
     if string == '        ':
       return int(0)
-    else:
-      return int(string)
+    return int(string)
 
 
   def __setStructuralMatrices(self):
@@ -808,7 +807,7 @@ def __setRestart(self):
           self.__computeInterfacePosVel(False)
           nSet = True
           break
-          
+
     if (not nM1Set) or (not nSet):
       raise Exception('The restart iteration was not found in the structural history')