동적으로 정의되지 않은 전역 변수에 액세스하는 경우 (D3의 예) DRY

Question

this question에서이어서 영국 신문의 일별 정서 분석을 나타내는 D3 그래프의 Js 변수를 만드는 DRY 방법을 만들고 싶습니다.

var allObjects = { guardian : {line : gLine,chart : gChart}, 
independent : {line : iLine,chart : iChart}}// and so on for each newspaper     

다음과 같이 내가 함수를 사용하여 변수를 할당 :

var guardian,independent; // many more here 
var gLine,gChart; // many more here 
var iLine,iChart; // many more here 

내가 객체에 신문 특정 변수를 저장하고 다음은

내 스크립트의 몇 가지 예제 코드입니다

function makeLine(name){return d3.svg.line().y(function(d) { return y(d[name]); }); } 
// and so on for each newspaper attribute in AllObjects 

항상 내 자신을 반복하는 대신 각 개체를 ind ividually : 잘 작동

makeLine('guardian'); makeLine('independent'); // etc 

가 ..., 내가 좋아하는 것

은 온통 신문을 반복하고, 모든 신문 하나의 기능으로 개체를 할당 할 수 있으려면, 뭔가 같은 :

var allFunctions = {line: makeLine(),chart: makeChart()}; 

    function make(type){ 
    var myFunc = allFunctions.type; 
    for(var prop in allObjects){prop.type = myFunc(type);} 
} 

make(line)이 gLine, iLine 할당 것이다 그래야 등

문제는 allObjects.guardian에서 변수가 정의되지 않은 한,이 방법이 작동하지 않는 것입니다.

이러한 방식으로 리팩토링하는 방법에 대한 제안 사항은 무엇입니까?

Answer 1

오히려 자신에게 모든 시간을 반복하기보다는 각 개체를 개별적으로 :

이

makeLine('guardian'); makeLine('independent'); // etc 

... 잘 작동하는, 내가 이상 모든 신문을 반복 할 수 및 개체를 지정하려는 모든 신문

내가 그 권리를 읽고 있어요 경우에 대해 하나의 기능으로, 당신의 "같은 것을"정말 가까운, 의견을 참조하십시오

var allFunctions = {line: makeLine, chart: makeChart}; 
// Note no() here ----------------^ or here --------^ 
// We want the reference to the function, we don't want to call it (yet) 

// Assuming `type` is "line", "chart", etc. 
function make(type){ 
    // Note brackets: We want the property whose name is in the type 
    // variable, not a property actually called "type" 
    var myFunc = allFunctions[type]; 
    //      ^----^------ We want the property whose name is in 
    //         the `type` variable, not a property 
    //         actually *called* "type" 
    for (var prop in allObjects) { 
     allObjects[prop][type] = myFunc(prop); 
     //  ^----^-----^----------- Brackets again as above 
    } 
}