superfluous-else-cycle (S253)

Fix is sometimes available.

What it does

Checks for else statements with a cycle statement in the preceeding if block

Why is this bad?

The else statement is not needed as the cycle statement will always continue onto the next iteration of the loop. Removing the else will reduce nesting and make the code more readable.

Example

integer function foo(a, b):
  integer, intent(in) :: a, b
  integer :: i
  do i = 1, a
    if (b > i) then
      cycle
    else
      foo = b
    end if
  end do
end function foo

Use instead:

integer function foo(a, b):
  integer, intent(in) :: a, b
  integer :: i
  do i = 1, a
    if (b > i) then
      cycle
    end if
    foo = b
  end do
end function foo