Как можно сократить код JS?

Question

[Андрей]

Помогите сократить код PHP и JS. Нужно из базы взять значения переменных и в JS вставить значения.
30 строк с данными.

PHP:

$db =&JFactory::getDBO();
$inn_1 = 'SELECT * FROM `#__virtuemart_countries` WHERE virtuemart_country_id ="1"'; 
$kpp_1 = 'SELECT * FROM `#__virtuemart_countries` WHERE virtuemart_country_id ="1"';
$db->setQuery($inn_1); 
$result = $db->loadObjectList(); 
foreach($result as $row){ 
$s1_i = $row->country_3_code;
$s1_k = $row->country_2_code; };

$inn_2 = 'SELECT * FROM `#__virtuemart_countries` WHERE virtuemart_country_id ="2"'; 
$kpp_2 = 'SELECT * FROM `#__virtuemart_countries` WHERE virtuemart_country_id ="2"';
$db->setQuery($inn_2); 
$result = $db->loadObjectList(); 
foreach($result as $row){ 
	$s2_i = $row->country_3_code; 
	$s2_k = $row->country_2_code; 
};
...........

JS:

jQuery(function($) {
		$('body').on('change', '#virtuemart_country_id', function() {
			$ssnull = '';
			$s1_i = '<?php echo $s1_i ?>'; $s1_k = '<?php echo $s1_k ?>';
			$s2_i = '<?php echo $s2_i ?>'; $s2_k = '<?php echo $s2_k ?>';
			....................
			if($(this).val() == 0) {$('#inn_field').val($ssnull); $('#kpp_field').val($ssnull);}
			if($(this).val() == 1) {$('#inn_field').val($s1_i); $('#kpp_field').val($s1_k);}
			if($(this).val() == 2) {$('#inn_field').val($s2_i); $('#kpp_field').val($s2_k);}
			.....................
		});
});

Answer 1

[AlexRas]

Так должно работать, могут быть ошибки, проверить не могу.

// PHP
$db =&JFactory::getDBO();

$query = 'SELECT * FROM `#__virtuemart_countries` WHERE virtuemart_country_id >= "1" AND virtuemart_country_id <= "30"'; 
$db->setQuery($query); 
$result = $db->loadObjectList(); 

$si = [];
$sk = [];
foreach($result as $row) { 
    $si[$row->virtuemart_country_id] = $row->country_3_code;
    $sk[$row->virtuemart_country_id] = $row->country_3_code;
};


// JS
jQuery(function($) {
    $('body').on('change', '#virtuemart_country_id', function() {
        var ssnull = '',
            s = {
                i: <?php echo json_encode($si) ?>,
                k: <?php echo json_encode($sk) ?>,
            },
            selector = {
                i: $('#inn_field'),
                k: $('#kpp_field'),
            }
            val = $(this).val();

        if (val == 0) {
            selector.i.val($ssnull); 
            selector.k.val($ssnull);

            return;
        }

        $.each(s, function(key, data) {
            $.each(data, function(index, value) {
                if(val == index + 1) {
                    selector[key].val(value);
                }
            });
        });
    });
});

Comments

[Андрей]

PHP отработал, в JS данные внеслись. Ошибок в инспекторе нет. Но в переменные $('#inn_field') $('#kpp_field') значения не передались.

[AlexRas]

Андрей, в консоли браузера есть ошибки?

[AlexRas]

Попробуйте такой код:

jQuery(function($) {
    $('body').on('change', '#virtuemart_country_id', function() {
        var ssnull = '',
            s = {
                i: <?php echo json_encode($si) ?>,
                k: <?php echo json_encode($sk) ?>,
            },
            selector = {
                i: $('#inn_field'),
                k: $('#kpp_field'),
            }
            val = $(this).val();

        if (val == 0) {
            selector.i.val($ssnull); 
            selector.k.val($ssnull);

            return;
        }

        selector.i.val(s.i[val]); 
        selector.k.val(s.k[val]); 
    });
});

[Андрей]

AlexRas, красота. работает! Куда ништячёк скинуть?

Answer 2

[Bharata]

Избавляешься от jQuery и сокращается твой код на целых 100 kb.!

document.getElementById('virtuemart_country_id').onchange = function(e) 
{
...
}

Answer 3

[Иван Иванов]

jQuery(function($) {
$('body').on('change', '#virtuemart_country_id', function() {
    $ssnull = '';
    for (var j = 1; j < 30; j++) {
         window['$s'+j+'_i'] =  '<?php echo $s'+j+'_i ?>';  window['$s'+j+'_k'] = '<?php echo $s'+j+'_k ?>';  
     }

     if($(this).val() == 0) {$('#inn_field').val($ssnull); $('#kpp_field').val($ssnull);}

     for (var x = 1; x < 30; x++) {
        if($(this).val() == x) {$('#inn_field').val( window['$s'+x+'_i']); $('#kpp_field').val( window['$s'+j+'_k']);}
     }   
}
  )});

Вроде бы так, но с условием if мог накосячить...

Comments

[AlexRas]

php отрабатывает раньше js)

[Иван Иванов]

AlexRas, я писал сокращение для js, а про php только щас увидел как страницу обновил. Пока я пытался что-то родить пост отредактировали ) Но скорее всего я не правильно написал ((((

[AlexRas]

Иван Иванов, <?php echo $s'+j+'_i ?> это не будет работать, php выдаст ошибку.

[Иван Иванов]

AlexRas, да не работает =(((((((((((((( тогда как у него вот это отрабатывает?

$s1_i = '<?php echo $s1_i ?>';

[AlexRas]

Иван Иванов, Просто пытаешься переменную из js вставить в php, а это невозможно в данном случае. А тут $s1_i = '<?php echo $s1_i ?>'; такого не происходит.