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

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

新規登録して質問してみよう
ただいま回答率
85.50%
JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

React.js

Reactは、アプリケーションのインターフェースを構築するためのオープンソースJavaScriptライブラリです。

Q&A

解決済

1回答

2028閲覧

react windowクリックでstateを変更したい

cheche0830

総合スコア187

JavaScript

JavaScriptは、プログラミング言語のひとつです。ネットスケープコミュニケーションズで開発されました。 開発当初はLiveScriptと呼ばれていましたが、業務提携していたサン・マイクロシステムズが開発したJavaが脚光を浴びていたことから、JavaScriptと改名されました。 動きのあるWebページを作ることを目的に開発されたもので、主要なWebブラウザのほとんどに搭載されています。

React.js

Reactは、アプリケーションのインターフェースを構築するためのオープンソースJavaScriptライブラリです。

0グッド

0クリップ

投稿2018/08/08 02:20

下記のようにbuttonをクリックしたら文字が出るようにして、
画面のどこでもいいから再度クリックすると
文字が消えるようにしたいのですが(stateを変更する)、
うまくいきません。
どうしたらよいでしょうか?

class Application extends React.Component { constructor(props) { super(props); this.state = { messageDisplay: 'none', }; } click = () => { this.setState({ messageDisplay:"block" }); } componentDidMount = () => { window.addEventListener("click",function() { this.setState({ messageDisplay:"none" }.bind(this)); }); } render() { return <div> <h1 style={ { display: this.state.messageDisplay } }>Hello</h1> <button onClick={this.click.bind(this)}>btn</button> </div>; } } /* * Render the above component into the div#app */ React.render(<Application />, document.getElementById('app'));

気になる質問をクリップする

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

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

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

まず、.bindsetStateの引数に対して呼ばれているので、正しくbindされていません。function(){...}.bindとするか、自動でthisが固定されるArrow Functionを使いましょう。

それが解決しても、clickイベントはボタンを押した際にも発生するので、これを弾いておく必要があります。

javascript

1 componentDidMount = () => { 2 window.addEventListener("click",function(e) { 3 if(e.target.tagName=== 'BUTTON') return; 4 this.setState({ 5 messageDisplay:"none" 6 }); 7 }.bind(this)); 8 }

2点を直して動作させるようにしたCodepen

投稿2018/08/08 02:34

maisumakun

総合スコア145121

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

cheche0830

2018/08/08 02:37

ああ、bindの位置が違いましたね・・・ しかもボタンの除外までしていただいてありがとうございました・・・!
maisumakun

2018/08/08 02:47 編集

「バブリングさせたイベントにもうっかり反応してしまう」ミスは、けっこうやらかしがちな気もします。 あと、通常は問題ないかもしれませんが、componentWillUnmountでセットしたイベントの後始末もしておいたほうがいいと思います。
cheche0830

2018/08/08 02:54

あ、そうですね。。ありがとうございました!!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問