Как передать переменную в вызванную функцию?

Question

[ivandao]

const f1 = function(data) {
   var a = data.modifyMethod()   
   f2 ()

}

const f2 = function() {
  console.log(a)
  return a
}

Функция f2 не видит переменную a .
Я понимаю, что связано что-то с замыканиями, но никак не могу понять - что нужно сделать, чтобы f2 видело a, не передавая ее в качестве параметра функции?
Вроде как доступ к областям видимости родительских функций (тех кто вызвал данную функцию) должен ведь быть.

(функцию f1 экспортирую в качестве модуля)

Answer 1

[longclaps]

Ух, поклонники глобальных переменных фестиваль констант устроили. Зачем, спрашивается?

function foo() {
    bar.a = 42;
}

function bar() {
    console.log(bar.a)
}

foo();
bar();

Comments

[ivandao]

функцию f1 экспортирую в качестве модуля

[ivandao]

longclaps,

поклонники глобальных переменных фестиваль констант устроили

Я думал разница только в синтаксисе :3

[longclaps]

ivandao, в чем твоя проблема?

const f1 = function (data) {
    bar.a = 42;
};

function bar() {
    console.log(bar.a)
}

f1();
bar();

[ivandao]

longclaps, bar.a = 42
Хитро, а это вообще нормальная практика? Они у меня так рекурсивно вызываются, надеюсь никаких конфликтов не будет.
У меня просто много таких переменных, которые нужно передать.

[longclaps]

ivandao, а чего тут ненормального? bar точно в своей области видимости (а как бы ты без этого мог устроить рекурсию?), да еще с читаемостью сильно лучше.

[ivandao]

longclaps, сейчас затестирую данную методу

[longclaps]

ivandao, если сильно много можно так:

const f1 = function (data) {
    Object.assign(bar, {a: 42, b: 'lalala'});
};

[ivandao]

longclaps,

а чего тут ненормального

Просто выглядит как-то неакадемично что ли.
Что-то среднее между багом и фичей.
Просто функция не предназначения для хранения чего-то, функция это лямбда, которая обрабатывает, модифицирует и т.п.

[longclaps]

ivandao, не знаю, я не академик, ты вроде тоже ) Всё, что не примитивы (числа, строки, ...) - то объекты, в том числе функции. Если б создатель языка захотел - он бы и функцию сделал примитивом. Но раз он сделал её объектом - грех не воспользоваться.

[ivandao]

longclaps, TNPTSYWWCC, 4mobile
не знаю кто из вас прав, но реализовал данным ↑ методом

Answer 2

[4mobile]

Странная задача, но можно и так ее решить

function f1 (data) {
   const a = {data: data}
   const f3 = f2.bind(a)
   f3()
}

function f2(){
  console.log(this.data)
}

f1('dddddd');

Comments

[ivandao]

this.a = data
Это аналогично var a = data ? Почему Вы используете такой способ?

Странная задача

Распространитесь, пожалуйста. Хочется понять, может мои подходы к разработке непопулярны.

[4mobile]

отредактировал свой ответ на более правильный вариант.

Распространитесь, пожалуйста. Хочется понять, может мои подходы к разработке непопулярны.

Нужно видеть задачу в общем, скорее всего есть решения по-проще.
Почему по вашим условиям нельзя передавать аргументом параметры в f2?

[ivandao]

4mobile,

const a = {data: data}
   const f3 = f2.bind(a)

Получается ссылку на константный объект привязываем к функции, спасибо, попробую.

Почему по вашим условиям нельзя передавать аргументом параметры в f2

f2 рекурсивно вызывает себя несколько раз. Можно, конечно попробовать, но мне кажется это некрасиво. и уменьшает читаемость.

Answer 3

[Andrey Suha]

Как вариант объявит ее в глобальном скопе

var a = 1
const f1 = function(data) {
   f2 (data)
}

const f2 = function(data) {
  console.log(a)
  return a
}

Comments

[ivandao]

Поправил исходную задачу

[Andrey Suha]

ivandao, собственно ниже Вам написали как

Answer 4

[Александр]

Область видимости переменной

var a;
const f1 = function(data) {
   a = 1   
   f2 ()

}

const f2 = function() {
  console.log(a)
  return a
}

Comments

[ivandao]

Александр,
я функцию f1 экспортирую в качестве модуля, поэтому заранее объявить что-то не вариант

[Александр]

может стоит тогда как-то по другому обмен данными вести.

[ivandao]

Александр, например?

[Александр]

localstorage, глобальный объект.

[ivandao]

Александр,

localstorage

У меня нода

глобальный объект

функцию f1 экспортирую в качестве модуля

Answer 5

[Sashqa]

let a;
const f1 = function (data) {
  a = 1   
  f2 ()

}

const f2 = function() {
  console.log(a)
  return a
}
f1(data);

Comments

[ivandao]

я функцию f1 экспортирую в качестве модуля