Javascript.RU

Создать новую тему Ответ
 
Опции темы Искать в теме
  #1 (permalink)  
Старый 11.01.2012, 16:11
sinistral
Посмотреть профиль Найти все сообщения от melky
 
Регистрация: 28.03.2011
Сообщений: 5,418

как лучше всего организовать код?
такс. попробую обьяснить проблему на как можно более абстрактном примере.

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

Эти две функции разрастаются ещё на две (раньше на три, пока я не смекнул, как можно совместить две из них в одну, это неважно).

В итоге получается аж 4 функции!

это введение. теперь, собственно, перейду к абстрактному коду на примере одной из двух функций :

итак, есть две функции (т.к. корневая разрастается) : A и B.
function A(){};
function B(){};

функция B оперирует с переданным ей аргументом с переданными её настройками. Так сказать, "опциональное декорирование".
function B(obj, opt1, opt2){
    /*её тело совсем неважно, но пусть будет так :*/
    obj[opt1] = opt2;
};

Функция A, в свою очередь, принимает аргументы для настройки, генерирует аргументы для B и вскармливает всё это функции B :
function A(opt, another_opt){
    var z = "";
    z = "my string is : "+opt+" and it contains "+another_opt;
    var d = {};
    B(d, "options", z);
    /*.....*/
};

Обе функции являются процедурными, т.е. не возвращают результата.
как видите, первая функция является "надстройкой" над второй. Проблема заключается в том, что необходимо сохранить последовательность действий A и B.

Вопрос заключается в том, как можно их совместить в одну. Нет, не тупо сделать вторую свойством первой, а именно совместить.

В голове крутится аргумент-переключатель Boolean-типа, который будет находиться в начале функции и выполнять определённые действия, но в этом случае функция превращается в рекурсивную. На примере кода это будет :
function A(switcher,args/*1,2,3*/){
    if(switcher === true){
        /*привычные для A действия*/
        var d = {};
        A(true, d, arg2, arg3);
        // дальшейшие операции с d.
    } else {
        /*тут код функции B*/
        arg1[arg2] = arg[3];
    }
};


Вопрос к вам, о форумчане : как бы это сделали вы и как это логичнее сделать?

Последний раз редактировалось melky, 11.01.2012 в 16:22.
Ответить с цитированием
  #2 (permalink)  
Старый 11.01.2012, 16:17
Аватар для Nekromancer
Профессор
Отправить личное сообщение для Nekromancer Посмотреть профиль Найти все сообщения от Nekromancer
 
Регистрация: 06.05.2009
Сообщений: 1,163

melky,
а чего, по порядку выполнять нельзя?

function(args...){
//выполняется первая часть, код функции A, сохраняет локальные переменные
//далее сразу идёт код функции B, который пользуется этими переменными
}


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

Последний раз редактировалось Nekromancer, 11.01.2012 в 16:19.
Ответить с цитированием
  #3 (permalink)  
Старый 11.01.2012, 16:17
Аватар для trikadin
Модератор
Отправить личное сообщение для trikadin Посмотреть профиль Найти все сообщения от trikadin
 
Регистрация: 27.04.2010
Сообщений: 3,417

Если ты не вызываешь напрямую B - сделай её локальной для A...
__________________
Читайте:
Ты любопытный) Всё-таки, ничему в этом мире не помешает хорошая доля юмора)
Как спросить, чтобы вам ответили
Часто Задаваемые Вопросы (FAQ)
Ответить с цитированием
  #4 (permalink)  
Старый 11.01.2012, 16:24
sinistral
Посмотреть профиль Найти все сообщения от melky
 
Регистрация: 28.03.2011
Сообщений: 5,418

Сообщение от Nekromancer Посмотреть сообщение
melky,
а чего, по порядку выполнять нельзя?

function(args...){
//выполняется первая часть, код функции A, сохраняет локальные переменные
//далее сразу идёт код функции B, который пользуется этими переменными
}


Если я правильно понял, что какая то основная функция будет много раз вызывать эти две, то так вполне нормально.
то, что ты описал, называется каррингом. тут иное, т.к. A передает в B непостоянные аргументы, которые нельзя сохранять в замыкании.

Сообщение от trikadin Посмотреть сообщение
Если ты не вызываешь напрямую B - сделай её локальной для A...
угумс. по сути пока что так и делается. но хочется уменьшить обьём кода без потери понятливости. да и использвуется B только в двух случаях, так что её не особо желательно сохранять.
Ответить с цитированием
  #5 (permalink)  
Старый 11.01.2012, 16:44
Аватар для Nekromancer
Профессор
Отправить личное сообщение для Nekromancer Посмотреть профиль Найти все сообщения от Nekromancer
 
Регистрация: 06.05.2009
Сообщений: 1,163

melky,
Да не, причём тут замыкания. Ведь функцией B будет пользоваться только A, правильно? Ну тогда так:

function(a1, a2){
var b1, b2, b3;

//код A, он меняет переменные b1, b2, b3. Использует a1, a2

//код B, он использует переменные b1, b2, b3
}

Ну а если не только функция A будет пользоваться B, то так:
function(a1, a2, flag){
var b1, b2, b3;

if(!flag) { // false или опущен флаг, по умолчанию используем функцию A
//код A, он меняет переменные b1, b2, b3. Использует a1, a2
}
//код B, он использует переменные b1, b2, b3
}


Если против подхода с !flag, как вы когда то писали про isNaN, то можно так:
function(a1, a2, flag){
var b1, b2, b3;
A:{
if(flag) break A;
//код A, он меняет переменные b1, b2, b3. Использует a1, a2
}
//B:{ условно блок B
//код B, он использует переменные b1, b2, b3
//{
}


UPD:
Да, чтот поспешил с 2мя последними вариантами, но в этом потоке, что бы можно было передавать аргументы B. Сейчас исправлюсь.
UPD2:
function(flag){
var b1, b2, b3, a1, a2, i = 0, args, flag;
typeof arguments[0] == 'boolean'  && ++i && (flag = arguments[0]);
args = Array.prototype.slice.call(arguments, i);
if(flag) {
//выставляем значения для функции B
b1 = args[0], b1 = args[1], b2 = args[2];
}else{
a1 = args[0], a2 = args[1];
//код A, он меняет переменные b1, b2, b3. Использует a1, a2
}
//код B, он использует переменные b1, b2, b3
}


Ну это замудренно
__________________
Нужно равняться на лучших, а не оправдываться за счёт худших.

Последний раз редактировалось Nekromancer, 11.01.2012 в 16:58.
Ответить с цитированием
  #6 (permalink)  
Старый 11.01.2012, 16:56
sinistral
Посмотреть профиль Найти все сообщения от melky
 
Регистрация: 28.03.2011
Сообщений: 5,418

Сообщение от Nekromancer Посмотреть сообщение
melky,
Да не, причём тут замыкания. Ведь функцией B будет пользоваться только A, правильно? Ну тогда так:
и ещё одной внутренней функцией.

Сообщение от Nekromancer Посмотреть сообщение
Ну а если не только функция A будет пользоваться B, то так:
function(a1, a2, flag){
var b1, b2, b3;

if(!flag) { // false или опущен флаг, по умолчанию используем функцию A
//код A, он меняет переменные b1, b2, b3. Использует a1, a2
}
//код B, он использует переменные b1, b2, b3
}
похоже на переключатель из первого поста наверное, так и буду делать. точнее, так и буду делать, если не найду более понятного варианта.
Ответить с цитированием
  #7 (permalink)  
Старый 11.01.2012, 16:59
Аватар для Nekromancer
Профессор
Отправить личное сообщение для Nekromancer Посмотреть профиль Найти все сообщения от Nekromancer
 
Регистрация: 06.05.2009
Сообщений: 1,163

melky,
Я там выше исправился, но уже слишком, хотя я бы сделал так
__________________
Нужно равняться на лучших, а не оправдываться за счёт худших.
Ответить с цитированием
  #8 (permalink)  
Старый 15.01.2012, 09:14
Аватар для x-yuri
Отправить личное сообщение для x-yuri Посмотреть профиль Найти все сообщения от x-yuri
 
Регистрация: 27.12.2008
Сообщений: 4,201

Сообщение от melky
теперь, собственно, перейду к абстрактному коду на примере одной из двух функций
на том уровне абстракции, на котором ты объяснил проблему, ничего сделать нельзя. А зачем эта надстройка вообще нужна? Почему сразу не передавать в B нормальные опции?
Ответить с цитированием
  #9 (permalink)  
Старый 15.01.2012, 10:03
Аватар для Aetae
Тлен
Отправить личное сообщение для Aetae Посмотреть профиль Найти все сообщения от Aetae
 
Регистрация: 02.01.2010
Сообщений: 6,480

function A(obj, opt1, opt2){
  switch (!opt2) {
     case true:
        opt2="my string is : "+obj+" and it contains "+opt1;
        obj={};
        opt1="options";
     case false:
     alert(obj.a)
        obj[opt1] = opt2;
     break;
  }
}

Так чтоли?
__________________
29375, 35
Ответить с цитированием
  #10 (permalink)  
Старый 15.01.2012, 16:50
sinistral
Посмотреть профиль Найти все сообщения от melky
 
Регистрация: 28.03.2011
Сообщений: 5,418

Сообщение от Aetae Посмотреть сообщение
Так чтоли?
то же. с флагом мне легче читать.
всем спасибо )
Ответить с цитированием
Ответ



Опции темы Искать в теме
Искать в теме:

Расширенный поиск


Похожие темы
Тема Автор Раздел Ответов Последнее сообщение
Можно ли как то вставить произвольный код в texarea foreach jQuery 3 30.08.2011 00:59
Как лучше сделать глобальный массив skyfish AJAX и COMET 4 17.02.2009 18:05
как узнать html код документа, подгруженного в iframe alexKniaz Events/DOM/Window 7 03.12.2008 12:37
как разодрать код scuter Общие вопросы Javascript 1 25.04.2008 21:39
Стоит ли пользоваться библиотеками или лучше писать свой код? lancer Общие вопросы Javascript 5 24.03.2008 19:31