Что заменяет "на"?

Ответ 1

with плохо, потому что внутри блока вы понятия не имеете, откуда происходят переменные. Является ли f1 равным mylib.f1 или window.f1?

Я бы предложил перейти с опцией b). Вы можете использовать c), если хотите, это может помочь с длинными именами объектов, но b) - это путь.

EDIT. Еще одно предложение - использовать опцию a), но затем удалите ссылки из окна, когда закончите. Кроме того, будьте осторожны, чтобы не перезаписывать любые существующие свойства в window. Вы можете бросить их в другой объект для сохранения, если хотите. Основываясь на @Alex answer:

var bkupWin = {};  // Object to store existing window properties

for(var key in myLib) {
    if(myLib.hasOwnProperty(key)){
        if(window.hasOwnProperty(key)){
          bkupWin[key] = window[key];  // Backup current property
        }
        window[key] = myLib[key];  // Add property to window
    }
}

f1(f2(f1(), f3()));

for(var key in myLib){
    if(myLib.hasOwnProperty(key)){
        delete window[key]; // Remove property from window
        if(bkupWin.hasOwnProperty(key)){
          window[key] = bkupWin[key];  // Restore old property
        }
    }
}

Ответ 2

Согласно Mozilla:

Использование с не рекомендуется и запрещено в ECMAScript 5 строго Режим. Рекомендуемая альтернатива - назначить объект, чей свойства, к которым вы хотите получить доступ к временной переменной. Источник: https://developer.mozilla.org/en/JavaScript/Reference/Statements/with

Две вещи, которые нужно убрать:

• Строгий режим - это единственное место, где это запрещено в ECMA5
• Рекомендуемая альтернатива - это ваш вариант (c)

Единственная причина, по которой это не "значимо", как вы говорите, это выбор переменной типа x - наименее описательная вещь.

 mylib.f1(
      mylib.f2(
         mylib.f1(), 
         mylib.f3()
      )
 );

Является более читаемым и на самом деле занимает больше места, чем with. Если вы действительно хотите называть вызовы методов, возвращайте эти методы this:

var mylib = {
  f1: function () {
     /* do stuff */
     return this;
  },
  f2: function () {
    /* do stuff */
    return this;
  }
}

// then you can chain calls, organize with indentation
mylib
 .f1()
 .f2();

Ответ 3

Ни один из примеров не является вполне читаемым, я бы предпочел бы что-то вроде этого:

var 
    param1 = mylib.f1(),
    param2 = mylib.f3(),
    result = mylib.f2(param1, param2);
mylib.f1(result);

Конечно, имена переменных должны быть переименованы во что-то, что описывает их контент. Это создает много кода, но его читаемость и, что еще важнее, легче отлаживать. Чтобы его минимизировать, используйте специальные программы, которые даже могут каким-то образом оптимизировать код.

Ответ 4

Если все они применяются к тому же объекту, что и ваш пример, вы можете написать методы обертки, такие как

function f4() {
    return this.f1(this.f2(this.f1(), this.f3()));
}

и добавьте его в mylib. Таким образом, по крайней мере, уродство будет содержаться в вашем объекте.

Ответ 5

Возможно, вы могли бы сделать несколько функций фасада в своей библиотеке, что уменьшит некоторые из параметров опции b или c. Итак, это...

mylib.f1(mylib.f2(mylib.f1(), mylib.f3()));

Стал бы...

mylib.f123_facade();

Кроме этого, я думаю, вам просто придется мириться с беспорядком.

Ответ 6

Если библиотека не слишком навязчивая (перезаписывает что-либо), вы можете экспортировать всю библиотеку в глобальную область:

for (var key in myLib) {
    if (myLib.hasOwnProperty(key)) {
        window[key] = myLib[key];
    }
}

EDIT: Также вы должны быть осторожны, если библиотека использует this.

EDIT2: вы можете заставить свою собственную область и подражать функции с:

function doWith(func, myLib) {
    for (var key in myLib) {
        if (myLib.hasOwnProperty(key)) {
            this[key] = myLib[key];
        }
    }

    func.apply(this)
}

И пример:

var lib = {
    init: function(param1) {
        console.log(param1);
    },
    f2: function() {
        return "hi";
    }
};
doWith(function(){init(f2());},lib)

Посмотрите, как это работает: http://jsfiddle.net/msm595/AenUc/1/

EDIT3: NVM, Rocket является правильным OTL.