COM_SECT_ITEMS = SECT_GATESn[i].getElementsByTagName("Com_Sect_Items");

			var arKalitka = new Array();

			for (var j = 0; j < COM_SECT_ITEMS.length; j++){
				CONDITION_ID = COM_SECT_ITEMS[j].getAttribute("CONDITION_ID");
				if(CONDITION_ID == 31 || CONDITION_ID == 206 || CONDITION_ID == 512){
					arKalitka[j] = "К, ";
				}
			}

			if(arKalitka.indexOf("К, ") != -1){
				alert("Калитка имеется");
				KALITKA = "К, ";
			}
			else{
				alert("Калитка НЕ имеется");
				KALITKA = "";
			}

если имеется калитка, то пишу в массив
после цикла проверяю полученный массив на наличие калитки
если имеется хотя бы одна калитка, то присваиваю KALITKA значение и вывожу, иначе делаю пустой

так нормально будет? :)
или можно было проще реализовать?

Если тебе интересен только сам факт наличия "калитки" в массиве - достаточно найти одну "калитку" и прекратить дальнейший поиск совсем. ;)

Т.е. нечто такое...

KALITKA = "";
for (var j = 0; j < COM_SECT_ITEMS.length; j++){
	CONDITION_ID = COM_SECT_ITEMS[j].getAttribute("CONDITION_ID");
	if({31:1,206:1,512:1}[CONDITION_ID]){
		KALITKA = "К, ";
		break;
	};
};
alert(KALITKA);

P.S.
Какие у тебя страшные идентификаторы у переменных...

for (var j = 0; j < COM_SECT_ITEMS.length; j++){
				CONDITION_ID = COM_SECT_ITEMS[j].getAttribute("CONDITION_ID");
				if(CONDITION_ID == 31 || CONDITION_ID == 206 || CONDITION_ID == 512){
					KALITKA = "К, ";
					break;
				}
				else{
					KALITKA = "";
				}
			}

понятно, сделано
спасибо :)

Зачем обрабатывать else? :D См. мой пример выше...

да я уже понял :) тороплюсь просто

что не так?)

Длинные... Капслоком...

разрабатываю на Битрикс
а у них переменные в основном так и записываются

вот привычка и выработалась :)

Так то системные переменные..