Javascript-форум (https://javascript.ru/forum/)
-   jQuery (https://javascript.ru/forum/jquery/)
-   -   Красивый код (https://javascript.ru/forum/jquery/57914-krasivyjj-kod.html)

corsik 26.08.2015 09:50

Красивый код
 
Всем доброго времени суток!
Подскажите, есть код
$.getJSON('../include/partners.json', function(data) {
		$('#rb4').on('click', function() {
			divval.html(" ");
			$.each(data, function(key, val) {
				for (var keys in val) {
					if (~val[keys].indexOf("Интернет")) {
						divval.append(
							"<div class='grid-3 shop-list__item'>" +
							" <h3 class='shop-list__ttl'>" + val["NAME"] + "</h3>" +
							"<div class='shop-list__adress'>" + val["ADDRESS"] + "</div>" +
							"<div class='shop-list__lnk'>" +
							"<i class='shop-list__icon icon-location-pin'></i>" +
							"<a href='#map_popup' rel='modal:open' class='lnk'>Показать на карте</a>" +
							"</div>" +
							"<div class='shop-list__phone'>" + val["PHONE"] + "</div>" +
							"</div>"

						);
					}
				}
			});
		});
	});

Все работает, но нужно повесить это действие на разные клики. Меняется только #rb4 и значение поиск indexOf. Просто скопировать один и тот же будет как то не красиво
С JS не давно дружу.

ksa 26.08.2015 13:38

Цитата:

Сообщение от corsik
Меняется только #rb4 и значение поиск indexOf.

Оформить в виде функции и вызывать уже ее...

function myGetJSON(Selector,FindText){
	$.getJSON('../include/partners.json', function(data) {
		$(Selector).on('click', function() {
			divval.html(" ");
			$.each(data, function(key, val) {
				for (var keys in val) {
					if (~val[keys].indexOf(FindText)) {
						divval.append(
							"<div class='grid-3 shop-list__item'>" +
							" <h3 class='shop-list__ttl'>" + val["NAME"] + "</h3>" +
							"<div class='shop-list__adress'>" + val["ADDRESS"] + "</div>" +
							"<div class='shop-list__lnk'>" +
							"<i class='shop-list__icon icon-location-pin'></i>" +
							"<a href='#map_popup' rel='modal:open' class='lnk'>Показать на карте</a>" +
							"</div>" +
							"<div class='shop-list__phone'>" + val["PHONE"] + "</div>" +
							"</div>"
						);
					}
				}
			});
		});
	});
};

corsik 26.08.2015 16:00

ksa,
Спасибо, чет сразу и не подумал =)
А вообще переписал весь кода полностью. =)

nerv_ 26.08.2015 16:23

corsik, в красивом коде минимум 10 уровней вложенности, у тебя только 7. Недобрал :D
http://govnokod.ru/18187


Часовой пояс GMT +3, время: 05:49.