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

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

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

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

React.js

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

Q&A

解決済

1回答

901閲覧

最初から表示されているボタンをイベントが発生してから表示されるようにしたい

yuiei

総合スコア2

JavaScript

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

React.js

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

0グッド

0クリップ

投稿2021/05/05 02:24

編集2021/05/05 10:55

前提・実現したいこと

エラーが出ているわけではないのですが、Buttonコンポーネントにあるボタンを押すとtask,dead, timeと削除ボタン、詳細ボタンが表示されるTodoアプリを作っています。task,dead,timeについては初期値を''にすることで一番最初は表示されないようになっているのですが、削除ボタンと詳細ボタンについては最初から表示されてしまいます。ボタン(Buttonコンポーネント)を押したら新たに作成されるようにしたいです。何もしていない状態から(Buttonをクリックする前から)削除ボタンと詳細ボタンが出てきてしまっているので、これをButtonをクリックしてから出てくるようにしようと思うのですが、調べても解決法が見つかりませんでした。何か良い方法はありますでしょうか。

発生している問題

エラーは特に出ているわけではないのですが、Buttonを押す前に詳細ボタンと削除ボタンが表示されてしまっているのを消したいです。消したいのは最初だけであとから追加する分はtask,dead,timeと一緒に追加していきたいです。

該当のソースコード

Javascript

1 2 3 4 5 6import classes from './taskinput.module.css' 7import React, { useState } from 'react'; 8 9import Button from './button' 10import styled from 'styled-components'; 11import { withRouter } from 'react-router-dom' 12 13const getKey = () => Math.random().toString(32).substring(2); 14 15function Taskinput({ history }) { 16 17 const moveToTaskDetail = (e) => { 18 history.push("/taskDetail"); 19 } 20 21 22 //タスクの追加 23 const initialState = [ 24 { 25 task: '', 26 time: '', 27 dead: '', 28 message: '', 29 arr: '', 30 isCompleted: false 31 }, 32 33 ] 34 35 const [todos, setTodo] = useState(initialState); 36 37 38 const [task, setTask] = useState('') 39 40 const handleNewTask = (event) => { 41 setTask(event.target.value) 42 } 43 44 //タスクにかかる時間について 45 const [time, setTime] = useState('1h') 46 47 /* const options = [ 48 49 ] */ 50 51 52 53 54 55 console.log(time) 56 //taskにかかる時間のselect 57 const dead_Time = [ 58 { value: "1", label: "1h", id: "1", }, 59 { value: "2", label: "2h", id: "2", }, 60 { value: "3", label: "3h", id: "3", }, 61 { value: "4", label: "4h", id: "4", }, 62 { value: "5", label: "5h", id: "5", }, 63 { value: "6", label: "6h", id: "6", } 64 ] 65 66 67 68 const Change = e => { 69 setTime(e.target.value) 70 } 71 72 73 let slicetime = time.slice(0, 1) 74 let getSlicetime = slicetime * 3600000 75 //taskの締め切りについて 76 const deadTask = dead_Time.map(deadtask => { 77 return ( 78 <option key={deadtask.value}>{deadtask.id}h</option> 79 ) 80 }) 81 //締め切りの日にち 82 const [dead, setDead] = useState('') 83 let today = new Date() 84 let nowdate = today.getTime() 85 let deadTime = Date.parse(new Date(dead)) 86 87 let differdate = deadTime - nowdate - getSlicetime 88 89 /* console.log(today) */ 90 let classAdd = false 91 92 93 //differdateを並べておく配列 94 const [arr, setArr] = useState('') 95 96 /* const [message, setMessage] = useState(initialState) */ 97 98 //plusbuttonクリック時のイベント 99 const handleSubmit = (event) => { 100 event.preventDefault() 101 classAdd = true 102 if (task === '') return 103 const tmpTodo = [...todos, { task, time, dead, arr: differdate,/* message: `${year}`"年", */ isCompleted: false }] 104 105 setTask('') 106 setArr(tmpTodo) 107 tmpTodo.sort(function (a, b) { 108 console.log(a.arr) 109 if (a.arr < b.arr) { 110 return -1; 111 } 112 if (a.arr > b.arr) { 113 return 1 114 } 115 return 0; 116 }) 117 setTodo(tmpTodo) 118 119 console.log(classAdd) 120 } 121 122 123 124 /* console.log(message) */ 125 //task削除 126 const handleRemoveTask = index => { 127 const newTodos = [...todos] 128 newTodos.splice(index, 1) 129 setTodo(newTodos) 130 131 } 132 133 134 135 136 console.log(Date.parse(new Date(dead))) 137 138 return ( 139 <> 140 141 <div className={classes.todolist}> 142 143 <div> 144 <form > 145 Add Task : <input value={task} placeholder="Add New Task" onChange={handleNewTask} /> 146 147 </form> 148 </div> 149 150 <div> 151 <p>想定される時間</p> 152 <select 153 onChange={Change} > 154 {deadTask} 155 </select> 156 157 158 </div> 159 <div> 160 161 <input type="date" 162 onChange={e => setDead(e.target.value)} /> 163 <div>{dead}</div> 164 </div> 165 <div><Button clicked={handleSubmit} /></div> 166 </div> 167 <ul> 168 <TaskWrap> 169 {todos.map((todo, index) => ( 170 <div className={classes.tasklist}> 171 <Item key={getKey()}>{todo.task} </Item> 172 <Item key={getKey()}>{todo.time}</Item> 173 <Item key={getKey()}>{todo.dead}</Item> 174 {/* <li key={getKey()}>締め切りまで{todo.message}</li> */} 175 <Buttontask onClick={moveToTaskDetail}>詳細</Buttontask> 176 <input 177 className={classAdd ? classes.play : classes.none} 178 type="button" 179 onClick={() => handleRemoveTask(index)} 180 value="削除" /> 181 </div> 182 ))} 183 </TaskWrap> 184 185 186 </ul> 187 188 </> 189 ); 190} 191 192 193 194 195 196const TaskWrap = styled.div` 197 198width: 100%; 199background-color: #C0C0C0; 200` 201const Item = styled.div` 202margin-left: 20px; 203margin-top: 30px; 204` 205const Buttontask = styled.button` 206 207margin-top: 30px; 208margin-left: 20px; 209` 210 211export default withRouter(Taskinput)

Javascript

1import React from 'react' 2import classes from './button.module.css' 3 4const Button = (props) => { 5 return ( 6 <div 7 className={`${classes.icon} ${classes.iconplus}`} 8 onClick={props.clicked} > 9 <span className={classes.icon__mark}></span> 10 </div> 11 ) 12} 13 14export default Button;

CSS

1.todolist { 2 display: flex; 3 justify-content: space-between; 4} 5 6.tasklist{ 7 display: flex; 8 justify-content: space-between; 9 list-style: none; 10} 11 12.none{ 13 display: none; 14} 15 16.play{ 17 background-color: aquamarine; 18}

試したこと

三項演算子でclassNameの付け替えをすればいいのではと思いましたが、クラスをつけた途端、結局出てきてしまうのでできないと思いました。しかし、そもそもこのコードではclassAddは変化していてもCSSが反映されませんでした。
cleateElementを試そうかと思ったのですが、できませんでした。

補足情報(FW/ツールのバージョンなど)

windows10

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

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

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

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

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

coolwind0202

2021/05/05 10:41

classAddを使ってclassNameを付け替えようとしたときのソースを見たいです 三項演算子を使うのはいい判断だと思いますが、CSSが反映されないというのが気になります
yuiei

2021/05/05 10:57

coolwindさん!ありがとうございます! 記事を更新しました。 className={classAdd ? classes.play : classes.none}が該当箇所です。cssはmodule.cssをimportしました。しかし、console.log(classAdd)で確認したところ、trueが入っているのですが、実際の画面上にはボタンが表示されませんでした。cssが反映されないというのはこのことについてです。
guest

回答1

0

ベストアンサー

classAdd が生の変数になっているのが原因だと思います。
質問された部分を実際に実行して確認しましたが、 useState() で包んだことで 削除 ボタンが正常に表示されるようになりました。

js

1let classAdd = false;

js

1const [classAdd, setClassAdd] = useState(false);

js

1classAdd = true;

js

1setClassAdd(true);

なお、ご存知かもしれませんが参考までに、 className に三項演算子を適用するのではなく、React要素そのものを三項演算子で「条件付きレンダー」する方法もあります。

条件付きレンダー - React

js

1<div className={classes.tasklist}> 2 <Item key={getKey()}>{todo.task} </Item> 3 <Item key={getKey()}>{todo.time}</Item> 4 <Item key={getKey()}>{todo.dead}</Item> 5 { classAdd && 6 ( 7 <> 8 <Buttontask onClick={moveToTaskDetail}>詳細</Buttontask> 9 <input className={classes.play} 10 type="button" 11 onClick={() => handleRemoveTask(index)} 12 value="削除" /> 13 </> 14 ) 15 } 16</div>

投稿2021/05/05 14:39

coolwind0202

総合スコア708

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

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

yuiei

2021/05/06 01:49

本当にありがとうございます!助かります。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.35%

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

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

質問する

関連した質問