Присвоение значений списку глобальных переменных в JavaScript

Эй, прямо сейчас я использую jQuery, и у меня есть некоторые глобальные переменные, чтобы держать немного предварительно загруженного материала ajax (предварительно загруженный, чтобы страницы стали приятными и быстрыми):


$.get("content.py?pageName=viewer", function(data)
    {viewer = data;});
$.get("content.py?pageName=artists", function(data)
    {artists = data;});
$.get("content.py?pageName=instores", function(data)
    {instores = data;});
$.get("content.py?pageName=specs", function(data)
    {specs = data;});
$.get("content.py?pageName=about", function(data)
    {about = data;});

Как вы можете видеть, у нас есть огромное нарушение принципа DRY, но ... Я действительно не вижу способа исправить это ... любые идеи?

может быть, массив?

javascript,jquery,dry,

1

Ответов: 6


5 принят

Вам это не нужно eval()или Function()для этого. Массив, как вы подозревали, сделает работу красиво:

(function() // keep outer scope clean
{
   // pages to load. Each name is used both for the request and the name
   // of the property to store the result in (so keep them valid identifiers
   // unless you want to use window['my funky page'] to retrieve them)
   var pages = ['viewer', 'artists', 'instores', 'specs', 'about'];

   for (var i=0; i<pages.length; ++i)
   {
      // "this" refers to the outer scope; likely the window object. 
      // And will result in page contents being stored in global variables 
      // with the same names as the pages being loaded. We use the with({})
      // construct to create a local scope for each callback with the
      // appropriate context and page name.
      with ({context: this, pageName: pages[i]})
         $.get("content.py?pageName=" + pageName, function(data)
            {context[pageName] = data;});
   }

})(); // close scope, execute anonymous function

// at this point, viewer, artists, etc. are populated with page contents 
// (assuming all requests completed successfully)

6

Используя каждый метод jQuery для итерации по массиву имен страниц, а затем для установки глобальной переменной (в области окна):

jQuery.each(
    ["viewer", "artists", "instores", "specs", "about"],
    function (page) {
        $.get("content.py?pageName=" + page,
            new Function("window[" + page + "] = arguments[0]"));
    }
);

Обновление: на самом деле вам даже не нужна «новая функция»:

jQuery.each(
    ["viewer", "artists", "instores", "specs", "about"],
    function (page) {
        $.get("content.py?pageName=" + page, function () { window[page] = arguments[0]; });
    }
);

2

Вы можете избежать eval, используя новую функцию:

var names = ['viewer', 'artists', 'instores', 'specs', 'about'];
for (var i = 0; i < names.length; i++)
   $.get("content.py?pageName=" + names[i], new Function('data', names[i] + ' = data;'));

Это не намного лучше, хотя tbh


0

Вы можете вызывать только один раз эту страницу и возвращать объект json вместо текста

{
viewer:'me',
artists:'you',
instores:'instores',
specs:'specs',
about:'about'
}

и eval, что с тех пор, как вы вызываете N раз ваш сервер, это замедляет все, вы должны пересмотреть свою логику!

PS. как я пишу, я видел ответ RoBorg, вы видите, при использовании новой функции вы используете eval под капотом, поэтому, если вы хотите использовать его, заходите на него (в некотором браузере тоже быстрее)


0

Это не использует eval, хотя это немного более многословие.

function get_content(name){
   $.get("content.py?pageName=" + name, function(data){ window[name] = data;});
}

var names = ['viewer', 'artists', 'instores', 'specs', 'about'];
for (var i = 0; i < names.length; i++)
    get_content(names[i]);

Но один из респондентов сделал хороший момент, вы, вероятно, должны попытаться объединить все эти запросы в один, иначе ваш сервер будет поражен 6 раз для динамического контента по каждому запросу страницы.

JavaScript, JQuery, сухой,
Похожие вопросы