Показать сообщение отдельно
  #5 (permalink)  
Старый 11.01.2013, 14:05
Профессор
Отправить личное сообщение для DjDiablo Посмотреть профиль Найти все сообщения от DjDiablo
 
Регистрация: 04.02.2011
Сообщений: 1,815

Цитата:
Похоже, уменьшить его в разы можно только потеряв часть функциональности
да прям
если уменьшить его в разы можно только потеряв часть функциональности, то это значит что вы достигли совершенства в коде. Вы себе скажем так прямо льстите

1) куча дублирующегося кода
функцию Podschet() можно записать так (я невдумывался, так что рассматривайте это как образец а не готовый код)
function Podschet(){
		val text="";

		document.getElementById('freedaysr').style.display='none';
		document.getElementById('strelka_akcia').style.display='none';
		
		if (valsrok < 60){
		    document.getElementById('skidkar').style.display='block';
		} else {				
		    if (valsrok > 59 & valsrok <120) text='Плюс неделя размещения в подарок!';
		    else if (valsrok > 119 & valsrok <150) text ='Плюс 2 недели размещения в подарок!';
		    else if (valsrok > 149 & valsrok <180) text='Плюс 3 недели размещения в подарок!';
		    else if (valsrok > 179 & valsrok <365) text='Плюс месяц размещения в подарок!';
		    else if (valsrok > 364) text ='Плюс 2 месяца размещения в подарок!';

		    document.getElementById('strelka_akcia').style.display='block';
		    document.getElementById('freedaysr').style.display='block';
		    document.getElementById('freedaysr').value=text;
		    document.getElementById('srok').value='Срок: ' + valsrok + ' дней';
  		    document.getElementById('skidkar').style.display='none';
		}
		if ((kolvoIstr+ kolvoVisokovsk + kolvoKlin) > 0 & valsrok > 0){
			 MinDat();
		}
		summ=(kolvoIstr + kolvoVisokovsk + kolvoKlin) * prf * valsrok;
		document.getElementById('summar').value='Итого: ' + Math.round(summ) + ' руб.';
		
	}


2я) ещё очень грубая ошибка в том что у вас все функции глобальны
правильно делать хотя бы так
(function(){
   //здесь код вашего скрипта.
})()

если это jquery плагин то тут записывается немного иначе.
jQuery(function($){
   //здесь код вашего скрипта.
})


3) можно заранее найти все нужные элементы.
(function(){
   var srok=document.getElementById('srok');

   function какаято функция( ){
      //бла бла 
      srok.value='Срок: ' + valsrok + ' дней';
      //бла бла бла
   }
})()


4) циклы вида for (var i = 20; i <= 21; i++) не есть красиво как минимум, можно подготовить массив с чекбоксами заранее. или Использовать query.
пример ("div").css("стиль")-установит css для всех div и неважно сколько их 1 ,50 или 100500. Установит всем div, всего одной строчкой, без цикла.


ещё уйма другого улучшайзинга возможно,
я замучаюсь перечислять

тем более я попрежнему непонимаю толи в jQuery стиле писать толии нативно, если везде юзать jQuery то получится и читабильнее и короче. А у вас ппрежнему фарш ищ=з двух подходов, в одном случаете вы используете jQuery, а в другом где бы jQuery был полезен вы его игноририруете. перепишите на jQuery уж тогда чтоли.
__________________
Лучше калымить в гандурасе чем гандурасить на колыме

Последний раз редактировалось DjDiablo, 11.01.2013 в 14:57.
Ответить с цитированием