изменить стиль кодирования, чтобы избежать вложенных блоков if

Проблема с вашим кодом на самом деле не уровень вложенности ifs.

Хотя технически, else if добавляет один уровень вложенности, идиома достаточно распространена, так что она будет восприниматься как выбор случая. Ключевым моментом здесь является использование правильного форматирования. Я рекомендую, чтобы, если некоторые из ваших пунктов нуждались в фигурных скобках, вы использовали их для всех. В противном случае поток управления намного сложнее визуально понять. Это также может помочь, если вы примете стиль кодирования, где вы используете больше отступов, чем два символа на уровень.

Другая проблема с рассматриваемым кодом заключается в том, что он слишком сложен, независимо от того, какую конструкцию потока управления вы используете. Хотя простой случай выбора, как

if (std::strcmp("alpha", text) == 0) {
return get_alpha();
} else if (std::strcmp("beta", text) == 0) {
return get_beta();
} else if (std::strcmp("gamma", text) == 0) {
return get_gamma();
} else if ( … ) {
…
} else {
throw std::invalid_argument {};
}

легко проследить, даже если количество дел становится большим (это в основном switch кроме строки, за исключением того, что C ++ не может сделать это изначально), ваша функция содержит слишком много сложной логики. Попробуйте рефакторинг и разбейте его на несколько функций.

Например, начать с внешнего if,

if (auto face = detect_face(video_frame, draw_frames_flag_m)) {
handle_face(face);
} else {
helper::gl::display_cv_mat(video_frame);
if (photo_capture_flag_m) {
photo_capture_flag_m = false;
}
if (capture_done_flag_m) {
capture_done_flag_m = false;
}
}

Этот код больше не трудно следовать.

Теперь вы можете обратить свое внимание на извлеченный handle_face функция. После того, как отформатирован правильно …

void handle_face(face_t * face) {
helper::gl::display_cv_mat(video_frame);  // display face
face_out_of_range_msg_flag_m = false;
if (!photo_capture_flag_m) {
// start capturing
photo_capture_flag_m = true;
capture_counter_m = glfwGetTime() + 5 + WAIT_TIME_BETWEEN_PHOTOS;
} else if (glfwGetTime() < (capture_counter_m - photos_wait_time4)) {
draw_frames_flag_m = true;
render_text("Face detected; look at the camera and stand still.", 10, window_height_m - 76);
} else if (glfwGetTime() <= (capture_counter_m - photos_wait_time3)) {
render_text("3", window_width_m / 2, window_height_m / 2, 200);
} else if (glfwGetTime() <= (capture_counter_m - photos_wait_time2)) {
render_text("2", window_width_m / 2, window_height_m / 2, 200);
} else if (glfwGetTime() <= (capture_counter_m - photos_wait_time1)) {
render_text("1", window_width_m / 2, window_height_m / 2, 200);
} else if (glfwGetTime() <= (capture_counter_m - photos_wait_time_almost_done)) {
render_text("done", window_width_m / 2, window_height_m / 2, 200);
draw_frames_flag_m = false;
// Don't draw frames the next few times so that the photo can be taken
} else if (glfwGetTime() <= (capture_counter_m - photos_wait_time_done)) {
render_text("done", window_width_m / 2, window_height_m / 2, 200);
load_and_save_portait(video_frame, *face);
} else if (glfwGetTime() < capture_counter_m - 1) {
render_text("processing photo...", (window_width_m / 2) - 50, window_height_m / 2);
} else {  // reset
photo_capture_flag_m = false;
capture_done_flag_m = false;
}
}

… это не выглядит тот устрашающий больше.

Конечно, есть еще возможности для совершенствования. Например, вы повторяете

render_text("…", window_width_m / 2, window_height_m / 2, 200);

немного. Поскольку единственное, что меняется, это первый аргумент, почему бы не иметь вспомогательную функцию?

void render_text_center(const char * text) {
render_text(text, window_width_m / 2, window_height_m / 2, 200);
}

И различные

glfwGetTime() <= (capture_counter_m - photos_wait_timeX)

тесты могут быть включены в функцию

bool before_wait_time(const wait_time_t wt) {
return glfwGetTime() <= (capture_counter_m - wt);
}

Теперь мы получаем это.

void handle_face(face_t * face) {
helper::gl::display_cv_mat(video_frame);  // display face
face_out_of_range_msg_flag_m = false;
if (!photo_capture_flag_m) {
// start capturing
photo_capture_flag_m = true;
capture_counter_m = glfwGetTime() + 5 + WAIT_TIME_BETWEEN_PHOTOS;
} else if (before_wait_time(photos_wait_time4)) {
draw_frames_flag_m = true;
render_text_top_left("Face detected; look at the camera and stand still.");
} else if (before_wait_time(photos_wait_time3)) {
render_text_center("3");
} else if (before_wait_time(photos_wait_time2)) {
render_text_center("2");
} else if (before_wait_time(photos_wait_time1)) {
render_text_center("1");
} else if (before_wait_time(photos_wait_time_almost_done)) {
render_text_center("done");
draw_frames_flag_m = false;
// Don't draw frames the next few times so that the photo can be taken
} else if (before_wait_time(photos_wait_time_done)) {
render_text_center("done");
load_and_save_portait(video_frame, *face);
} else if (before_wait_time(capture_counter_m)) {
render_text_center_right("processing photo...");
} else {  // reset
photo_capture_flag_m = false;
capture_done_flag_m = false;
}
}

Хотя это не уменьшило количество строк кода или количество ifs, более простой код имеет меньше отвлекающих факторов и его легче анализировать для человеческого глаза.

Теперь мы можем увидеть причину сложного кода. То, что у нас есть, это в основном цикл, записанный в виде if … else if … else каскад. Я не знаю достаточно об остальной логике вашей программы, чтобы точно сказать, как вы будете выполнять рефакторинг этого, но мне кажется, что вы могли бы удалить все эти photos_wait_timeX переменные и соответствующие ifs — которых, честно говоря, здесь только три — и вместо этого вычислить интервал.

int get_current_step() {
// I'm not sure whether your logic actually works that way…
return (glfwGetTime() - capture_counter_m) / WAIT_TIME_BETWEEN_PHOTOS;
}

void handle_face(face_t * face) {
helper::gl::display_cv_mat(video_frame);  // display face
face_out_of_range_msg_flag_m = false;
if (!photo_capture_flag_m) {
// start capturing
photo_capture_flag_m = true;
capture_counter_m = glfwGetTime() + 5 + WAIT_TIME_BETWEEN_PHOTOS;
} else if (before_wait_time(photos_wait_time4)) {
draw_frames_flag_m = true;
render_text_top_left("Face detected; look at the camera and stand still.");
} else if (before_wait_time(photos_wait_time1)) {
const auto step = get_step();
render_text_center(std::to_string(step).c_str());
} else if (before_wait_time(photos_wait_time_almost_done)) {
render_text_center("done");
draw_frames_flag_m = false;
// Don't draw frames the next few times so that the photo can be taken
} else if (before_wait_time(photos_wait_time_done)) {
render_text_center("done");
load_and_save_portait(video_frame, *face);
} else if (before_wait_time(capture_counter_m)) {
render_text_center_right("processing photo...");
} else {  // reset
photo_capture_flag_m = false;
capture_done_flag_m = false;
}
}

Это все еще не красивый код, но я бы посчитал это приемлемым.

Чтобы расширить ответ @ πάντα ῥεῖ, вы также можете использовать старый добрый goto, Нормальные предостережения goto все еще применяется, хотя.

if (!firstCondition) {
goto done;
}

// Do something.

if (!secondCondition) {
goto done;
}

// Do something.

done:
// End here.

Один из наиболее часто используемых методов, чтобы избежать глубоко вложенных if/else if каскады это использовать do {} while(false); а также break перед проверкой следующего условия.

Вместо того, чтобы иметь такой код

if(firstCondition) {
// Do something
}
else {
if(secondCondition) {
// Do something
}
}

Ты можешь написать

do {
if(!firstCondition) {
break;
}

// Do something

if(!secondCondition) {
break;
}

// Do something

// ...
} while(false);