質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

89.21%

if文で新しい文字と色の変更を同時にreturnしたいです。

解決済

回答 3

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 281

katopa165

score 18

現在、ボタンをクリックする度に大吉や中吉などを返してくれるプログラムを書いています。
その中でクリックする度にランダムに値を返すことは出来るのですが、同時に値のfontColorを変えられずに困っています。
最初if文内にreturnを二つ書いてみたのですが、実行されませんでした。
次に色だけ変えてみようと思い下記のコードだけreturnしてみたら値に赤文字のredが返ってくるようになってしまいました。
document.getElementById('id').sytle.color = "red";

どなたか解決方法を教えて頂けないでしょうか?
よろしくお願いいたします。

<div id="id"></div>
<button id="btn"></button>
document.getElementById('btn').onclick = function(){
function func(){
  var ran = Math.random() * 10;
  var sum = Math.floor(ran) % 3;
  if(sum == 0){

      return document.getElementById("id").style.color="red";
  }else if(sum == 1){
      return "中吉";
  }else{
      return "吉";
  }
  }
    document.getElementById("id").innerHTML = func();
}
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 過去に投稿した質問と同じ内容の質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • m.ts10806

    2019/08/06 17:14 編集

    ファイルが別なのでしたらコードブロック分けてもらえたらと
    ※原則「コピペで再現できること」です。

    キャンセル

  • katopa165

    2019/08/06 17:16

    アドバイス頂きありがとうございます。
    変更いたします。

    キャンセル

  • m.ts10806

    2019/08/06 17:23

    sum==0のときは何をしたいのでしょうか

    キャンセル

  • katopa165

    2019/08/06 17:49

    sum==0の時は赤文字で大吉と返したいです。
    ご回答いただきありがとうございます。

    キャンセル

回答 3

+4

出目の出現確率の調整とかあるのでしょうか?
普通にやるなら文字と背景色の組み合わせで配列を設定しておけばよいでしょう

<script>
window.addEventListener('DOMContentLoaded', function(e){
  document.querySelector('#btn').addEventListener('click',function(){
    var ran = parseInt(Math.random()*3);
    var list=[
      {"text":"大吉","bgColor":"red"},
      {"text":"中吉","bgColor":"yellow"},
      {"text":"吉","bgColor":"lime"},
      ];
    document.querySelector("#id").textContent=list[ran]["text"];
    document.querySelector("#id").style.backgroundColor=list[ran]["bgColor"];
  });
});
</script>

<div id="id"></div>
<button id="btn">button</button>

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/06 17:58

    回答ありがとうございます。
    自分には知らないメソッドが多いので一つずつ調べて使えるようにしたいと思います。
    またよろしくお願いいたします。

    キャンセル

+3

ちょっと意味がわからないので的外れだったらすみません。
まず、乱数発生部分ですが特別な意図がなければvar sum = Math.floor( Math.random() * 10) ;で良いのでは?
で本題でreturn "<span style=color:red;>大吉</span>";という風に指定すれば良いのでは?

<div id="id"></div>
<button id="btn"></button>
document.getElementById('btn').onclick = function(){
  function func(){
    var sum = Math.floor( Math.random() * 10) ;
    if(sum == 0){
      retuen "<span style=color:red;>[大吉とか?]</span>";
    }else if(sum == 1){
      return "中吉";
    }else{
      return "吉";
    }
  }
  document.getElementById("id").innerHTML = func();
}


Codepen - ソース
Codepen - フルビュー

投稿

編集

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2019/08/06 17:57

    回答頂きありがとうございます。
    var sum = Math.floor( Math.random() * 10) ;
    上記の書き方を知らなかったので今後使っていこうと思います。

    キャンセル

checkベストアンサー

+2

returnしているからですね。
私も初めて知ったのですが、document.getElementById("id").style.color="red"で返される結果って式のtrue/falseじゃなくおさめた文字そのものだったんですね。

console.log(document.getElementById("id").style.color="red");

本題:

return document.getElementById("id").style.color="red";


これだと#idstyle=color:redを付与したうえでredという文字を返していることになります。

何をしたいのかによりますが、document.getElementById("id").style.color="red"だけで1つの指示が成り立っているので、returnする必要ありません。

document.getElementById("id").style.color="red";
return "表示したい文字列";

あとは、出目に対して何の色にしたいかで決めてください。
※ただし、念のためfunc()冒頭で色の解除(初期化)はしておいたほうが良いです。

ただ、冗長なコードになるので、出目と対応する色を配列に持っておいて取り出すのが良いと思います。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 89.21%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる