Проблема с "toUpperCase", которая выводит "function toUpperCase () {[собственный код]}" вместо желаемого вывода. Сделайте остановку l oop при достижении значения - PullRequest
Новая
       65

Проблема с "toUpperCase", которая выводит "function toUpperCase () {[собственный код]}" вместо желаемого вывода. Сделайте остановку l oop при достижении значения

0 голосов
/ 28 мая 2020

Итак, в основном я должен выполнить следующий фрагмент скрипта: 

for (let i = 0; i < 5; i++){
            result = window.prompt('Rock, Paper or Scissor?', 'What do you choose?') //prompt user to input choice
            let playerSelection = result.toUpperCase
            console.log(playRound(playerSelection, computerPlay())) // to see that loop for "playRound works"
        }

Он отлично работает, прежде чем я добавлю «toUpperCase» после «let playerSelection = result.toUpperCase» (что я делаю для того, чтобы пользователь мог для ввода любого варианта, например «RoCK», «ROCK», «rock» и «RocK»). После добавления «toUpperCase» я получаю неверный вывод в консоли. Вместо желаемого результата я получаю: "function toUpperCase () {[native code]}", который я хочу исправить.

Вот мой полный сценарий: 

    let computerScore = 0
let playerScore = 0


function computerPlay (){
    const choices = ["Rock", "Paper", "Scissor"];
    const randomChoice = choices[Math.floor(Math.random() * choices.length)];

    return(randomChoice);
    }

function playRound(playerSelection, computerPlay) { //This is the actual round, increments "playerScore" and "computerScore" based on who wins round
if (playerSelection == computerPlay) {
        result = `Computer also choose ${computerPlay}! It's a Draw!`;
    } else if ((playerSelection == "Paper" && computerPlay == "Rock") ||
               (playerSelection == "Rock" && computerPlay == "Scissor") ||
               (playerSelection == "Scissor" && computerPlay == "Paper")) {
                   playerScore = playerScore + 1;
                   result = `Computer choose: ${computerPlay}! ${playerSelection} beats ${computerPlay}. You get one point! Your score is: ${playerScore} Computer score is: ${computerScore}`;
               } else {
                computerScore = computerScore + 1;
                   result = `Computer choose: ${computerPlay}! ${computerPlay} beats ${playerSelection}. Computer gets one point! Your score is: ${playerScore} Computer score is: ${computerScore}`;
               } 

            return (result);
        }    

       // console.log(playRound(playerSelection, computerPlay())) // to test if function "playRound works"

        for (let i = 0; i < 5; i++){
            result = window.prompt('Rock, Paper or Scissor?', 'What do you choose?') //prompt user to input choice
            let playerSelection = result.toUpperCase
            console.log(playRound(playerSelection, computerPlay())) // to see that loop for "playRound works"
        }

        function winLose () { //to give output based on who wins or loses (or if draw)
            if (playerScore > computerScore){
                console.log(`You won over the computer with ${playerScore} points. Yay! You sure showed that computer who's boss!`) 
            } else if (computerScore > playerScore) {
                console.log(`The computer won over you with ${computerScore} points. Oh no! You are inferior to the computer!`)
            } else if (playerScore == computerScore){
                console.log(`You had ${playerScore} points and the computer had ${computerScore} points. No one wins! Boring...`)
            } else {
                console.log("You and the computer where both unlucky and had too many draws to decide who's the superior life form")
            }
        }
        winLose()

I ' Я также задаюсь вопросом, как остановить «l oop», когда «computerScore» или «playerScore» достигнет 3. Поскольку нет смысла получать подсказку о продолжении игры после того, как вы проиграли или выиграли 3 раза.

Спасибо за любые указатели и помощь!

1 Ответ

1 голос
/ 28 мая 2020

Ошибка заключается в том, что вы используете toUpperCase как свойство вместо вызова функции. Чтобы вызвать функцию в строке, вам нужно добавить круглые скобки: 

let playerSelection = result.toUpperCase()

Но, судя по вашему коду, я не думаю, что вам нужна эта функция. Что он делает, так это превращает каждый символ в его версию в верхнем регистре, поэтому бумага становится БУМАГОЙ. Я думаю, вы имели в виду что-то вроде функции капитализации, например, этот документ -> Бумага. Для того, чтобы ваш код работал, вам также нужно будет изменить функцию игры на это: 

 else if ((playerSelection == "PAPER" && computerPlay == "Rock") ||
           (playerSelection == "ROCK" && computerPlay == "Scissor") ||
           (playerSelection == "SCISSOR" && computerPlay == "Paper"))
...