prometheus/rules
beorn7 5b53aa1108 style: Replace else if cascades with switch
Wiser coders than myself have come to the conclusion that a `switch`
statement is almost always superior to a statement that includes any
`else if`.

The exceptions that I have found in our codebase are just these two:

* The `if else` is followed by an additional statement before the next
  condition (separated by a `;`).
* The whole thing is within a `for` loop and `break` statements are
  used. In this case, using `switch` would require tagging the `for`
  loop, which probably tips the balance.

Why are `switch` statements more readable?

For one, fewer curly braces. But more importantly, the conditions all
have the same alignment, so the whole thing follows the natural flow
of going down a list of conditions. With `else if`, in contrast, all
conditions but the first are "hidden" behind `} else if `, harder to
spot and (for no good reason) presented differently from the first
condition.

I'm sure the aforemention wise coders can list even more reasons.

In any case, I like it so much that I have found myself recommending
it in code reviews. I would like to make it a habit in our code base,
without making it a hard requirement that we would test on the CI. But
for that, there has to be a role model, so this commit eliminates all
`if else` occurrences, unless it is autogenerated code or fits one of
the exceptions above.

Signed-off-by: beorn7 <beorn@grafana.com>
2023-04-19 17:22:31 +02:00
..
fixtures fixes yamllint errors 2021-06-12 12:47:47 +02:00
alerting.go promql: Separate Point into FPoint and HPoint 2023-04-13 19:25:16 +02:00
alerting_test.go style: Replace else if cascades with switch 2023-04-19 17:22:31 +02:00
manager.go lint: Adjust to the lint warnings raised by current versions of golint-ci 2023-04-19 17:10:10 +02:00
manager_test.go style: Replace else if cascades with switch 2023-04-19 17:22:31 +02:00
origin.go Panic if rule type was not expected 2023-01-20 10:27:50 +02:00
origin_test.go lint: Adjust to the lint warnings raised by current versions of golint-ci 2023-04-19 17:10:10 +02:00
recording.go labels: simplify call to get Labels from Builder 2023-03-22 17:05:20 +00:00
recording_test.go style: Replace else if cascades with switch 2023-04-19 17:22:31 +02:00