New erevator#109
Conversation
| <View style={styles.container} > | ||
| <Image | ||
| source={require('../../../assets/images/elevator_setting_big.png')} | ||
| style={styles.leftContainer} |
There was a problem hiding this comment.
containerって名前なのに中に何持ってないのかw
There was a problem hiding this comment.
containerというクラス名だとその中に
なにかしらの要素を含んでいるような気がした。
という意味ですね
There was a problem hiding this comment.
あれ要素とは？
今入っているとかとは別のものですか？
There was a problem hiding this comment.
leftContainer
の中って何もなさそうって話ですね！
| alignItems: 'center', | ||
| marginTop: 20, | ||
| marginTop: height * 0.01, | ||
| justifyContent: 'center', |
There was a problem hiding this comment.
これ、8＋で最適化してるけどいっちーの端末とかでも確認できてる？？
There was a problem hiding this comment.
壱の端末ではガッツリ死んでいます
There was a problem hiding this comment.
iphoneXの対応はしてないので動作保証外ですねw
| elevatorObjectPoints: ObjectPoint[]; | ||
| startGate: Gate; | ||
| endGate:Gate; | ||
| elevatorListChangeFlg: () => void; |
There was a problem hiding this comment.
Flgって名前だとどちらかというとboolを想定しそうですねぇ…
| elevator: undefined, | ||
| }; | ||
|
|
||
| async componentDidMount () { |
|
|
||
| private indoorChanges = (items: any) => { | ||
| if (items == undefined) return; | ||
| if (!Array.isArray(items)) { |
There was a problem hiding this comment.
もともとは配列できたものの - を B に変換していたのですが、配列以外のものが使えなかったので、配列じゃないときは違う処理にするための判定です
There was a problem hiding this comment.
配列じゃないものがこの関数を呼ぶのってそもそも実装的になんか違うんじゃないかなぁと感じていて、
配列じゃないやつがこれを呼ぶパターンっていつでしょう…
There was a problem hiding this comment.
startGateとendGateです
| ); | ||
| } | ||
|
|
||
| private elevatorObjectPointPickOut = () => { |
There was a problem hiding this comment.
なんか関数名が一周回ってわかりにくくなってる気がします？
There was a problem hiding this comment.
getElevatorObjectPointはどうです？
|
全体的に不要になったコメントは削除してください… |
2 participants
css汚い気がしますが見た目はだいたいあっています
やったこと