0

このコードは機能しますが、変数リストが大きくなるとすぐに非効率になります（すでにそうですか？）。

新しいAPIを使用してTwitterストリームをページに追加しています。つまり、手動で情報を提供する必要があり、ID文字列が返されます。それはOKです。回避策はありません。

この例では、3つのストリームに必要な情報を使用していますが、リストはすぐに数十以上に増えます。

function isValidTweets(ts_titletweets) {
    var widgetid = $('#TweetStuff').attr('data-widget-id');

    var jackID = '243441836335697920'; //IDs associated with streams
    var jillID = '243398621595312128';
    var harryID = '243398621595312130';


    var ts_titletweets = title.toLowerCase(); //title is parsed from URL elsewhere
    validtweets = "jack|jill|harry"; // if the title doesn't match then no streams should be used
    if (validtweets.indexOf(ts_titletweets.toLowerCase() + "|") > -1) {

        console.log('TweetsEnabled');

        if (ts_titletweets == "jack")
        {widgetid = jackID;
        console.log(widgetid + title)}
        else if (ts_titletweets == "jill")
        {widgetid = jillID;
        console.log(widgetid + title)}
        else if (ts_titletweets == "harry")
        {widgetid = harryID;
        console.log(widgetid + title)};


        $('#TweetStuff').html("<a class=\"twitter-timeline\" width=\"500\" height=\"600\" data-widget-id=" + widgetid + "></a>");

        $("#newTweetsButton").delay(3000).show(0);
        $("#filters").css({'margin' : '-30px 0px 0px 0px'});
        return true;

    } console.log('no Tweets'); 
    return false;

}

私は手動で再入力していると確信しており、リストが大きくなるにつれてそれを避けたいと思っています。

ありとあらゆる提案は、ありがたい初心者に大いに感謝されます。

ありがとう！

更新されたタイプミス

4

4 に答える 4

1

コードで何が起こっているのかよくわかりませんが、使用できるのはルックアップ テーブルです。

    var ids = {
        'jack': '243441836335697920',
        'jill': '243398621595312128',
        'harry': '243398621595312130'
    };


    if (/*whatever*/) {

        widgetid = ids[ts_titletweets];
        console.log(widgetid + title);
    }
于 2012-09-28T01:23:29.757 に答える
1

あなたが何を求めているのかよくわかりませんが、ここに役立つかもしれないいくつかのことがあります：

// define an object with name: id pairs
var twitterIds = {
  'jack': '243441836335697920',
  'jill': '243398621595312128',
  'harry': '243398621595312130'
};

foreach(var name in twitterIds) {
  // name = jack
  // twitterIds[name] = '243441836335697920'
}

function loadTweets ( name ) {
  if( name && twitterIds[name] ) {
    // load stream here.  Use twitterIds[name] to get the twitter id
  }
}
于 2012-09-28T01:24:09.650 に答える
1

個人の名前に対応するプロパティを持つオブジェクトを作成します。次に、それに対してIDを保存します。次に、それが存在するかどうかを確認し、コードを実行できます。

http://jsfiddle.net/7PSNt/1/

var title = 'jill'; //hard coded for demo purposes
var myObject = {
    'jack': 1,
    'jill': 2,
    'harry': 3
};

//I assume title is just one of the keys above (jack, jill, harry)                 
var ts_titletweets = title.toLowerCase();

//if the object contains a property for the title then we can proceed
var tweetID = myObject[ts_titletweets];
if (tweetID != undefined) {
    widgetid = tweetID;
    console.log(widgetid + ':' + title)
}
​
于 2012-09-28T01:25:06.640 に答える
1

効率のために、これを試してください：

//cache of static jQuery objects
var $$ = { 
    'TweetStuff': $('#TweetStuff'),
    'newTweetsButton': $("#newTweetsButton"),
    'filters': $("#filters")
};

//define the ids in an outer scope - this gives isValidTweets less work to do each time it is called
var ids = {
    jack: '243441836335697920',
    jill: '243398621595312128',
    harry: '243398621595312130'
};

function isValidTweets(title) {
    var t = title.toLowerCase();
    var widgetid = ids[t] || $$.TweetStuff.attr('data-widget-id');
    $$.TweetStuff.html("<a class=\"twitter-timeline\" width=\"500\" height=\"600\" data-widget-id=" + widgetid + "></a>");
    $$.newTweetsButton.delay(3000).show(0);
    $$.filters.css({'margin' : '-30px 0px 0px 0px'});
    console.log('Tweets: ' + (ids[t] ? (widgetid + ' - ' + title) : 'none')); 
    return !!ids[t];//true or false
}
于 2012-09-28T01:52:59.693 に答える