初めまして。JavaScript初心者です。
現在、私は動画を見ながらweb上に電卓アプリを作成しているのですが、動画通りに入力しても、上手く動きませんでした。
エラーの原因はJavaScriptの
const calculator = new Calculator(previousOperandTextElement,currentOperandTextElement);
の箇所で"previousOperandTextElement"が定義されていないとエラーが出力されました。
そこで、引数を無くし
const calculator = new Calculator
と記述し、コンパイルを行いました。
そして、実際に数字をクリックしてみると、以下のようなエラーが出力されました。
script.js:29 Uncaught TypeError: Cannot set property 'innerText' of undefined
at Calculator.updateDisplay (script.js:29)
at HTMLButtonElement.<anonymous> (script.js:46)
質問内容は2つで
- インスタンス化するときの引数をどのようにすればよいのか。
- 上記のエラーを解消するにはどのようにすればよいのか。
<試したこと>
・引数に1を与えた。
⇒Uncaught TypeError: Cannot create property 'innerText' on number '1'
at Calculator.updateDisplay (script.js:29)
at HTMLButtonElement.<anonymous> (script.js:46)
・htmlでscriptの読み込みを</head>の手前に書き、deferを付け加えた。
⇒エラーの内容に変化なし
ご回答よろしくお願いします。
html
1<!DOCTYPE html> 2<html lang="ja"> 3<head> 4 <meta charset="UTF-8"> 5 <meta name="viewport" content="width=device-width, initial-scale=1.0"> 6 <title>Calculator</title> 7 <link href="style.css" rel="stylesheet"> 8</head> 9<body> 10 <div class="calculator-grid"> 11 <div class="output"> 12 <div data-previous-operand class="previous-operand"></div> 13 <div data-current-operand class="current-operand"></div> 14 </div> 15 <button data-all-clear class="span-two">AC</button> 16 <button data-delete>DEL</button> 17 <button data-operation>÷</button> 18 <button data-number>1</button> 19 <button data-number>2</button> 20 <button data-number>3</button> 21 <button data-operation>*</button> 22 <button data-number>4</button> 23 <button data-number>5</button> 24 <button data-number>6</button> 25 <button data-operation>+</button> 26 <button data-number>7</button> 27 <button data-number>8</button> 28 <button data-number>9</button> 29 <button data-operation>-</button> 30 <button data-number>.</button> 31 <button data-number>0</button> 32 <button data-equals class="span-two">=</button> 33 </div> 34 <script src="script.js"></script> 35</body> 36</html>
JavacScript
1'use strict'; 2 3class Calculator{ 4 constructor(previousOperandTextElement, currentOperandTextElement) { 5 this.previousOperandTextElement = previousOperandTextElement; 6 this.currentOperandTextElement = currentOperandTextElement; 7 this.clear(); 8 } 9 clear() { 10 this.currentOperand = ''; 11 this.previousOperand = ''; 12 this.operation = undefined; 13 } 14 15 delete() { 16 17 } 18 19 appendNumber(nubmer) { 20 this.currentOperand = this.currentOperand.toString() + numberButtons.toString(); 21 } 22 chooseOperation(operation) { 23 24 } 25 compute() { 26 27 } 28 updateDisplay() { 29 this.currentOperandTextElement.innerText = this.currentOperand; 30 } 31} 32 33const numberButtons = document.querySelectorAll('[data-number]'); 34const operationButtons = document.querySelectorAll('[data-operation]'); 35const equalsButton = document.querySelector('[data-equals]'); 36const deleteButton = document.querySelector('[data-delete]'); 37const allCrearButton = document.querySelector('[data-all-clear]'); 38const previousOperandButton = document.querySelector('[data-previous-operand]'); 39const currentOperandButton = document.querySelector('[data-current-operand]'); 40 41const calculator = new Calculator(previousOperandTextElement,currentOperandTextElement); 42 43numberButtons.forEach(button => { 44 button.addEventListener('click', () => { 45 calculator.appendNumber(button.innerText); 46 calculator.updateDisplay(); 47 }); 48});
回答1件
あなたの回答
tips
プレビュー