Pen^2 (./3) :
Concernant le premier extrait, j'aurais tendance à :

– privilégier la clause la plus courte avant le else pour la lisibilité.

Mais les 2 branches ont quasiment la même longueur, et en particulier le même nombre de lignes, donc je ne vois pas en quoi ça améliorerait la lisibilité.

Et puis personnellement, je trouve plus logique de mettre le code le plus long avant le else. C'est une préférence intuitive (et probablement explicable par la limite quand le code court tend vers le code vide, où passer de

if (b) {
  code_long();
  code_long();
} else {
  code_court();
}

à

if (b) {
  code_long();
  code_long();
}

est moins brutal que de passer de

if (!b) {
  code_court();
} else {
  code_long();
  code_long();
}

à

if (b) {
  code_long();
  code_long();
}

avec inversion de la condition) plutôt qu'une préférence pour simplifier la lecture.

Et je ne suis pas d'accord avec tes autres points non plus. Par exemple, j'évite les variables intermédiaires inutiles qui n'apportent rien, ni du point de vue efficacité (élimination de sous-expressions communes), ni en représentant une quantité bien définie qui mérite une variable dédiée (indice: si tu dois passer du temps à chercher un nom à ta variable, c'est une variable artificielle qui n'apporte rien à la lisibilité).