回答編集履歴
2
説明を修整
answer
CHANGED
|
@@ -17,6 +17,8 @@
|
|
|
17
17
|
|
|
18
18
|
に書き換えて試してみてください。
|
|
19
19
|
**追記 変更するif分の条件に ! を付けました**
|
|
20
|
+
条件４つとも整数値であった場合にtrueを返すので整数値でエラーメッセージが出てしまうので
|
|
21
|
+
条件４つに!をつけて 整数値でなかったらエラーメッセージが出るように修正しました
|
|
20
22
|
|
|
21
23
|
ただ、
|
|
22
24
|
```JavaScript
|
1
コードの修正
answer
CHANGED
|
@@ -1,4 +1,4 @@
|
|
|
1
|
-
**HTML全体が提示されてないので動作保障は出来かねますが**
|
|
1
|
+
~~**HTML全体が提示されてないので動作保障は出来かねますが**~~
|
|
2
2
|
|
|
3
3
|
```JavaScript
|
|
4
4
|
// 0以上の整数のみ
|
|
@@ -12,7 +12,24 @@
|
|
|
12
12
|
```JavaScript
|
|
13
13
|
if(Number.isInteger(FizzNum)||Number.isInteger(BuzzNum)){
|
|
14
14
|
↓
|
|
15
|
-
if(isNumber(FizzNum) || isNumber(BuzzNum) || Number.isInteger(FizzNum) || Number.isInteger(BuzzNum)){
|
|
15
|
+
if(!isNumber(FizzNum) || !isNumber(BuzzNum) || !Number.isInteger(FizzNum) || !Number.isInteger(BuzzNum)){
|
|
16
16
|
```
|
|
17
17
|
|
|
18
|
-
に書き換えて試してみてください。
|
|
18
|
+
に書き換えて試してみてください。
|
|
19
|
+
**追記 変更するif分の条件に ! を付けました**
|
|
20
|
+
|
|
21
|
+
ただ、
|
|
22
|
+
```JavaScript
|
|
23
|
+
if(index % FizzNum === 0 && index % BuzzNum === 0) {
|
|
24
|
+
n.textContent=FizzBuzz ${index};
|
|
25
|
+
}else if(index % FizzNum === 0) {
|
|
26
|
+
n.textContent=Fizz ${index};
|
|
27
|
+
}else if(index % BuzzNum === 0) {
|
|
28
|
+
n.textContent=Buzz ${index};
|
|
29
|
+
```
|
|
30
|
+
の部分、FizzBuzz、Fizz 、Buzz って定義されて無いように見受けられますが・・・
|
|
31
|
+
あと 後ろの```${index}```って何でしょう？ jQueryにしても意味がわからないしHTMLでも
|
|
32
|
+
jQueryを読み込んでないようですが・・・
|
|
33
|
+
一応jQueryを読み込ませてみましたがやはり```${index}```の部分でスクリプトエラーになります
|
|
34
|
+
提示いただいたJavaScriptコードは実際に使ってるコードですか？
|
|
35
|
+
なにかプラグインなど使ってませんか？ とにかく上のコードの部分でスクリプトエラーで動きません
|