Archive / jolly / jr-web
PR

【8/30】デッキ更新マージ #3622

wada opened this at 2024-07-26T15:39:48; closed

Description

opened by wada at 2024-07-26T15:39:48; updated 2024-10-03T13:35:53

概要

デッキ周りのPR

マージPR

  • 7202【本日中】配列 Warning修正とグラデーション背景色の修正
  • 7811【3/31】デッキ取得周りをがんばって削減
  • 7873【即】配列を初期化

Comment

#35820 by wada at 2024-07-26T15:39:58; updated 2024-07-26T15:39:58

@kouho
レビューお願いします。

wada change_title at 2024-08-05T11:51:25: 【8/5】デッキ更新マージ デッキ更新マージ
wada change_title at 2024-08-23T16:33:46: デッキ更新マージ 【8/30】デッキ更新マージ
kouho add_label at 2024-09-26T16:01:30: 皇甫 LGTM

Comment

#36644 by kouho at 2024-09-26T16:07:11; updated 2024-09-26T16:07:11

@wada
確認OKです

本番のユーザーでmax_holderが最高のユーザーのuser_card_holderをコピーしてデッキ編集ページを開き
S_GetSubDeck関数のメモリ使用量と実行時間を計測したところ、以下の結果となっており大幅な改善が行われておりました。

修正前
メモリ使用量:3836336Byte
実行時間:0.340161562秒

修正後
メモリ使用量:3455840Byte
実行時間:0.02156243324秒

wada merge at 2024-10-03T13:32:52: 【8/30】デッキ更新マージ
wada close at 2024-10-03T13:32:52: Close
wada delete_branch at 2024-10-03T13:35:53: merge_deck_edit

Commit range

base7423843923b21cee4900be62dc6168586ef8bc6e
head9408075f4338138cdf1770e7458cf802f1a83c6b
base branchjolly/jr-web:master
head branchjolly/jr-web:merge_deck_edit

View commits

Commit details are loaded from Git when a dynamic archive server is running, or can be embedded with --include-commits.
Files changed6 filesView patch
statusfile
modifiedlib/SubFunc.inc
modifiedlib/class/controller/JrControllerDeck.class.php
modifiedpublic_html/template/Deck-CardDetail.tpl
modifiedpublic_html/template/Deck-Home.tpl
modifiedpublic_html/template/Deck-SelectChangeCard.tpl
modifiedpublic_html/template/parts_deck_replace_modal.tpl
Patch body is stored as a separate file to keep this page lightweight.