Je n'ai pas trouvé de rubrique vraiment adaptée, c'est encore une question à la con

J'ai parcouru récemment un code, qui était présenté dans ce genre là :
void DlgEditCountry::requestValidation() { QString CurrentName = this->EditedCountry; // The button OK is enabled: ui->ButtonOK->setEnabled( // - if the validator says that the entry is valid DocIndex::instance()->validateNewCountryName(CurrentName) // - or if the edited name has not changed || (CurrentName == this->EditedCountry) ); }Perso, j'ai l'habitude d'écrire sous cette forme :
void DlgEditCountry::requestValidation() { QString CurrentName = this->EditedCountry; // The button OK is enabled: // - if the validator says that the entry is valid // - or if the edited name has not changed ui->ButtonOK->setEnabled(DocIndex::instance()->validateNewCountryName(CurrentName) || (CurrentName == this->EditedCountry)); }Je ne vous cache pas que la première forme me plait beaucoup.
Avantage, les commentaires et donc la pensée sont mis en exergue, le code n'est qu'un accessoire
Inconvénient, la structure naturelle du code est du langage sont cassés, avec les risques de bugs que ça peut provoquer.
Question que je me pose : est-ce que la première forme est au final avantageuse, et notamment facile à maintenir, modifier, corriger, comprendre longtemps après etc ?
Merci de vos retours d'expérience
