在 Jquery 中单击带有推文按钮的随机报价

Question

我正在做一个“随机引用机器”，它从存储数组中获取随机引用，并让用户发推文。我创建了随机报价生成器，但无法发布相同的推文。需要一些关于推文按钮的帮助。

我的 Jquery

var quote = new Array()
quote[0] = 'this is first quote'
quote[1] = 'this is second quote'

function tweet() {
window.open('https://twitter.com/intent/tweet?hashtags= freecodecamp&text='   + encodeURIComponent($(".message").html(quote[randomquote]));
}


$(document).ready(function() {
$("#getMessage").on("click", function(){
  var randomquote = Math.floor(Math.random()*(quote.length-1));      
  $(".message").html(quote[randomquote]);      
});

$('#twitter-share-button').on('click', tweet)

})

一点html

<div class = "col-xs-12 well message">
  <h3>Click to get Random Quotes</h3>
</div>
<button id = "tweetMessage" class = "btn btn-default btn-lg" target="_blank"><i class="fa fa-twitter fa-fw"></i> Tweet it</button>
<button id = "getMessage" class = "btn btn-default btn-lg"><i class="fa fa-fire fa-fw"></i> Get Quote</button>

Answer 1

  $('#twitter-share-button').on('click', tweet)

你没有id="twitter-share-button"的元素

随机时也不要减 1。由于您正在执行Math.floor 并且由于Math.random 永远不会完全返回1，因此它永远不会达到上限。

您的randomquote 变量也用于tweet 函数内部，但仅存在于您的document.ready 块内部。你需要想出一种方法来传递它。更有意义的是传递消息本身而不是索引。

对于文本，您应该使用.text(...) 而不是.html(...)，因为这将有助于提高安全性（如果您有兴趣，请查找跨站点脚本）。

我看到你在做 freecodecamp。冷静下来，继续努力！

Answer 2

终于明白了。 var 报价 = 新数组（） 报价[0] = '

“哲学和保险杠贴纸是有区别的。”
-Charles M. Schulz

'

function tweet(message) {
window.open('https://twitter.com/intent/tweet?hashtags= freecodecamp&text='   + encodeURIComponent(message));
}

var msg;
function tweetHandler() {
tweet($(msg).text()); 

}
$(document).ready(function() {
$("#getMessage").on("click", function(){
var randomquote = Math.floor(Math.random()*(quote.length-1));
msg = quote[randomquote];
$(".message").html(msg);
});

$('#tweetMessage').on('click', tweetHandler);

});

我的整个代码在

http://codepen.io/nabendu82/full/GZRaQw/