0

我正在尝试将以下 CoffeeScript 代码编译成 Javascript：

GetCard = ->
  do Math.floor do Math.random * 12

Results = ->
  do NewGame if prompt "You ended with a total card value of #{UserHand}. Would you like to play again?"
  else
    alert "Exiting..."

NewGame = ->
  UserHand = 0
  do UserTurn

UserTurn = ->
  while UserHand < 21
    if prompt "Would you like to draw a new card?" is "yes"
      CardDrawn = do GetCard
      UserHand += CardDrawn
      alert "You drew a #{CardDrawn} and now have a total card value of #{UserHand}."
    else
      do Results
      break

但是，如果您说是，则生成的 Javascript 会在控制台上打印以下错误：

Uncaught TypeError: number is not a function BlackJack.js:4
GetCard BlackJack.js:4
UserTurn BlackJack.js:22
NewGame BlackJack.js:14
onclick

由于某种原因，CoffeeScript 也没有将UserHand设置为0 。我对 Javascript 相当陌生，对 CoffeeScript 也很陌生。我四处搜索并阅读了 CoffeeScript 文档以及 CoffeeScript Cookbook，据我所知，CS 代码看起来是正确的，而 JS 则没有：

var GetCard, NewGame, Results, UserTurn;

GetCard = function() {
  return Math.floor(Math.random() * 12)();
};

Results = function() {
  return NewGame(prompt("You ended with a total card value of " + UserHand + ". Would you like to play again?") ? void 0 : alert("Exiting..."))();
};

NewGame = function() {
  var UserHand;
  UserHand = 0;
  return UserTurn();
};

UserTurn = function() {
  var CardDrawn, _results;
  _results = [];
  while (UserHand < 21) {
    if (prompt("Would you like to draw a new card?") === "yes") {
      CardDrawn = GetCard();
      UserHand += CardDrawn;
      _results.push(alert("You drew a " + CardDrawn + " and now have a total card value of " + UserHand + "."));
    } else {
      Results();
      break;
    }
  }
  return _results;
};

任何帮助将不胜感激。谢谢！

更新： 感谢所有答案。我只是对双括号有点困惑，错误地使用 do 关键字替换参数括号而不是函数调用括号。不过，我仍然对全球范围感到困惑。我知道您可以在纯 JS 中使用 var 关键字，但在 CoffeeScript 文档中它指定您永远不需要使用它，因为它为您管理范围？

4

2 回答 2

1

经过一点阅读，您的第一个问题似乎是由于do运营商。据我所知，它用于在循环中创建闭包以将当前值绑定到函数。不是你在这里需要的。

我不认为简单地有什么问题：

GetCard = ->
    Math.floor( Math.random() * 12 )

第二个问题是范围之一。UserHand是本地的NewGame，所以它在UserTurn（它在undefined那里）中是不可访问的。您可以将其设为全局，或将其作为参数传递，但在我看来，NewGame无论如何只需要结果：

NewGame = ->
    hand = UserTurn() // set hand to whatever the UserHand is from UserTurn

UserTurn = ->
    UserHand = 0
    while UserHand < 21
        if prompt "Would you like to draw a new card?" is "yes"
        (etc)
    UserHand // return the final UserHand value after the loop

我还建议重新考虑一些名称；有些有点令人困惑。虽然你已经在许多好的地方分解了它，但仍然有一些奇怪的选择（例如，为什么超过 21 由另一个函数处理UserTurn但停止由另一个函数处理？）

于 2013-08-25T02:19:40.793 回答
0 
GetCard = ->
  Math.floor(Math.random() * 12)

您需要那里的括号来说明您要在没有参数的情况下调用 Math.random。否则，afunctionName something表示“以某些东西作为第一个参数调用 functionName”。

一般来说，只有在函数调用非常明显时才省略括号，否则它会是一个模棱两可的事情，会妨碍可读性。

好的：

someFunc 1, 2, 3

不清楚：

someFunc someOtherFunc someArg

（在旁边...）

您应该查看do关键字。它仅对非常特定的目的有用，几乎所有的使用都是不必要的/不正确的。

于 2013-08-25T02:21:42.060 回答