Это, вероятно, не сработает так, как вы хотите, потому что, если они нажмут на последний?
Но, во-первых, вы не говорите, что это что-то:
var id_prev;
var id_new;
Вы сказали, что они переменные, но не то, что они содержат. Вам нужно что-то вроде этого наверху:
if ($(this).index() > 0) {
var id_prev = $(this).index()-1;
}
Так что, если div не первый, установите предыдущий идентификатор на выбранный, минус 1 ...
Но я бы действительно предложил начать с двух и полностью их кодировать, а не увеличивать значения и переменные, а затем взять этот код и сжать его там, где это возможно.