用户 2026-10-13 七条。拍板:影院只记地址、不存经纬度(后续用高德反查);
日程状态只删界面与展示、数据库列保留(「后续会有一个整体清理的工作」)。
①🔴 读书页慢 —— 真根因是**会话锁**,不是封面(我先判断错了方向)
用户原话:「读书现在还是非常慢,就是进入读书这个页面,刚开始还是非常的慢。」
线上实测(真公网、同一会话连打 3 次):
第 1 次 TTFB 0.08s ← 触发微信读书同步
第 2 次 TTFB **8.73s** ← 用户说的"刚开始非常慢"
第 3 次 TTFB 0.07s
插桩拿到的服务端时间线把因果钉死:
21:18:05.483 QUEUED ← 第 1 个请求排队
21:18:05.501 FINISH 2ms ← 响应确实提前发出去了(这步是好的)
21:18:14.674 SYNC 9.2s ← 但它仍占着这个人的会话文件锁跑完 9.2 秒
21:18:14.682 QUEUED ← 第 2 个请求此刻才真正开始执行
⇒ **`fastcgi_finish_request()` 只断开客户端连接、不释放 PHP 文件会话锁。**
后台同步要 9.2 秒(真打微信读书接口),同一个人的下一个请求一路等到它跑完。
修法(缺一不可):
· `WeRead::queueBackgroundSync()` 回调里 **先 `session_write_close()`、
再 `fastcgi_finish_request()`**(顺序不能反);
· `index.php` 两处"渲染前排同步"的入口排完**立刻放锁**
(shutdown 回调不一定执行:客户端中断/服务器超时才不放过);
· 顺带修 **`Holiday::queueBackgroundFetch()` 的同型问题**(联网 12 秒)。
为什么以前没发现:v29 引入"后台化"时只验证了"响应是不是提前发出去",
**单次请求测量永远看不到这个 bug** —— 必须同一会话连打两次才现形。
线上复验:修复后连打 4 次 = 0.15 / 0.15 / 0.09 / 0.15 秒(修复前第 2 次 8.73s);
后台同步仍正常(`weread_sync_at` 已更新)。
② 封面:顺手也修了(但这不是用户说的那个问题)
· `media.php` 读 OSS 前先放会话锁(原来从不放,35 张封面串行);
· 新增 `Storage::cacheLocally()` 磁盘二级缓存(原来 `read()` 只读不写回);
· 让 304 不再先拉整张图(原来先取回 body、再判 `If-None-Match`)。
线上实测:冷取 149ms(走 OSS)→ 热取 35ms(本地缓存、字节一致)
→ 带 `If-None-Match` 返回 **304 / 19.8ms / 空体**;35 张封面 103ms → 20ms(5 倍)。
③ 地址联想:抽成全站共用一份(`assets/js/addr_suggest.js`)+ 接入三处
· 影院管理页、观影新增页的「新建影院」弹框、日程地点、行程出发/到达;
· `layout.php` 按需加载(`[data-addr]` 出现才拉 SDK + 我们的脚本),
并暴露 `fbLoadAddr(scope)` 给 AJAX 片段(v42 踩过"内联 script 不执行");
· 安全密钥在 SDK 之前设置;回填支持 `data-addr-fill` / `data-addr-for`、
**只补空白字段**(不覆盖用户手写)、直辖市 `province==city` 去重;
· 字段名有两套(影院 `address`/`place`,行程 `from_addr`/`to_addr` 缩写 `_addr`),
`fieldFor()` 带同义词表 + 后缀匹配;
· 影院**只存地址、不存经纬度**(按用户要求)。
④ 笔记两条
· 「点保存没反应」真因:附件区是个 `<form>` **套在保存表单里** ——
HTML 不允许 form 套 form,浏览器丢掉内层标签、保留它的 `</form>`,
于是**保存按钮成了孤儿**(`button.form === null`),点了不提交也不报错。
真浏览器实测:修前 `btnInsideForm:false` / 点后 POST=0;修后 true / POST=1。
改法:附件区改 `<div id="noteAttForm">`,上传按钮用 `formaction`。
· 详情页宽度:`.note-read { max-width: 820px }` → `none`
(内容卡 820px vs 下方共享卡 1032px;修后两者都是 1032px)。
⑤ 日程状态下线
· 删表单里的「状态」下拉、`Calendar::statusWord()`、浮层 JSON 的 `status_word`;
· **保留** `events.status` 列与 `Calendar::statusOf()`(契约不动,等整体清理)。
验证(新增 5 个套件):
· `_fb_v47_addr.py`(接线 24 条)/ `_fb_v47_addr_sim.py`(真浏览器 + 假高德,16 条)
· `_fb_v47_readperf.py`(缓存与会话锁,23 条)
· `_fb_v47_notes.py`(真浏览器,9 条)
· `_b47_readperf_falsify.py`(反向验证 9 个注入)—— **9/9 全抓住**
· 全量 e2e **1510/0**;vCard 规格+往返 40/0;TB 探针 44/0;
v44 反向验证 17/0;全流水线 **PUSH_RC=0**(含"第47批形态"静态断言)
· **线上终验**:读书页整页 81ms、封面 103→20ms、media 冷 149→热 35→304 19.8ms、
修复前第 2 次请求 8.73s → 修复后 0.15s、地址联想 meta[fb-addr] 带真高德 key
顺带修掉三处**探针自身**的假红/假绿(详见 pitfalls.md):
· `_b44_falsify.py` 的"整小时"注入在**整点**与正确实现等价 ⇒ 换成恒错值;
· `_fb_b44_ui.py` 3b 的并发计时断言在单进程 `php -S` 下必然假红 ⇒ 改断言确定事实;
· 多处探针的"清理删除"撞沙箱删除门禁 ⇒ 探针**中途死掉**看起来像产品坏了
⇒ 统一改成 `_safe_rm()` / 不删。
164 lines
7.8 KiB
PHP
164 lines
7.8 KiB
PHP
<?php
|
||
/**
|
||
* 影院管理(v9,MovieLog cinemas.php 同款):上面加/编表单,下面列表。
|
||
*
|
||
* 颜色用「取色器 + 50 色预设板」两路输入(内联小段 JS 只是取色联动,
|
||
* 不依赖 JS 也能手填 hex 或直接用取色器)。删除是 POST 小表单 + confirm;
|
||
* 删掉的影院只把关联观影记录的 cinema_id 置空,记录本身不动。
|
||
*
|
||
* ⚠️ $editing 是路由传进来的「被编辑影院的记录数组」(或 null)。
|
||
* 曾经这里写成 `$editing = !empty($editing);` —— 数组被原地变成布尔值,
|
||
* 表面看标题还会显示「编辑影院」,但 hidden id 变 0、四个字段全部回填为空,
|
||
* 点「编辑」就像没带入一样(v9 线上实测)。布尔化结果必须存进**另一个**变量。
|
||
*/
|
||
$isEditing = !empty($editing);
|
||
$canWrite = \App\Auth::canWrite();
|
||
?>
|
||
<div class="page-head">
|
||
<h1>影院管理</h1>
|
||
<span class="sub">共 <?= count($cinemas) ?> 家影院</span>
|
||
<div class="actions">
|
||
<a class="btn btn-ghost" href="<?= e(url('movies')) ?>">返回观影</a>
|
||
<?php $setCurrent = 'cinemas'; $setCanWrite = $canWrite; require __DIR__ . '/_settings_menu.php'; ?>
|
||
</div>
|
||
</div>
|
||
|
||
<?php if (!$canWrite): ?>
|
||
<div class="card card-pad empty"><b>只读账号</b>访客只能查看影院列表。</div>
|
||
<?php else: ?>
|
||
<div class="card card-pad cinema-form-card">
|
||
<h2 class="cinema-h2"><?= $isEditing ? '编辑影院' : '添加影院' ?></h2>
|
||
<form method="post" action="<?= e(url('movies/cinemas_save')) ?>" id="cinemaForm">
|
||
<?= csrf_field() ?>
|
||
<?php if ($isEditing): ?>
|
||
<input type="hidden" name="id" value="<?= (int)$editing['id'] ?>">
|
||
<?php endif; ?>
|
||
<div class="form-grid">
|
||
<div class="field">
|
||
<label for="cname">影院名称 *</label>
|
||
<input class="input" id="cname" name="name" required
|
||
value="<?= e(old('name', $editing['name'] ?? '')) ?>" placeholder="如:万达影城(IMAX 店)"
|
||
<?php /* 🔴 v47(用户 2026-10-13):「输入影院的名称的时候,应该自动的帮我补齐
|
||
所在的地方和详细地址。」⇒ 名称框挂高德联想,选中后自动填
|
||
`place`(所在地方)与 `address`(详细地址)。
|
||
⚠️ 只填**空白**的字段(见 addr_suggest.js 的注释)——
|
||
用户已经手写过的地址不该被覆盖。 */ ?>
|
||
data-addr data-addr-fill="place,address" data-addr-for="#cinemaForm">
|
||
</div>
|
||
<div class="field">
|
||
<label for="cplace">所在地方</label>
|
||
<input class="input" id="cplace" name="place"
|
||
value="<?= e(old('place', $editing['place'] ?? '')) ?>" placeholder="如:青岛 · 崂山区">
|
||
</div>
|
||
<div class="field full">
|
||
<label for="caddress">详细地址</label>
|
||
<input class="input" id="caddress" name="address"
|
||
value="<?= e(old('address', $editing['address'] ?? '')) ?>">
|
||
</div>
|
||
<div class="field full">
|
||
<label for="ccolor">主题颜色</label>
|
||
<div class="color-row">
|
||
<input type="color" id="ccolor" name="color" class="color-picker"
|
||
value="<?= e(old('color', $editing['color'] ?? '#888888')) ?>">
|
||
<span class="color-hex" id="colorHex"><?= e(old('color', $editing['color'] ?? '#888888')) ?></span>
|
||
<span class="preset-palette">
|
||
<?php foreach (\App\Cinema::PRESET_COLORS as $pc): ?>
|
||
<span class="preset-color" data-color="<?= $pc ?>" style="background: <?= $pc ?>;" title="<?= $pc ?>"></span>
|
||
<?php endforeach; ?>
|
||
</span>
|
||
</div>
|
||
</div>
|
||
</div>
|
||
<div class="btn-row">
|
||
<button type="submit" class="btn"><?= $isEditing ? '保存' : '添加影院' ?></button>
|
||
<?php if ($isEditing): ?>
|
||
<a class="btn btn-ghost" href="<?= e(url('movies/cinemas')) ?>">取消编辑</a>
|
||
<?php endif; ?>
|
||
</div>
|
||
</form>
|
||
</div>
|
||
<?php endif; ?>
|
||
|
||
<div class="card card-pad">
|
||
<h2 class="cinema-h2">影院列表</h2>
|
||
<?php if (!$cinemas): ?>
|
||
<p class="empty-hint">还没有影院,先在上面添加一家。</p>
|
||
<?php else: ?>
|
||
<div class="table-wrap">
|
||
<?php /* 🔴 v40(用户 2026-10-10):「影院管理部分,注意控制列的宽度,因为最右侧的
|
||
操作、编辑和删除两个按钮,已经被挤成两行了,尝试让编辑和删除排在一行,
|
||
宽度给它留够。编辑使用蓝色的背景,删除使用红色的背景。」
|
||
⇒ ① 表格加 `.cn-table`:给「操作」列一个**最小宽度 + 不换行**
|
||
(`.cn-ops` 上 `flex-wrap: nowrap` + `white-space: nowrap`)——
|
||
两个按钮被挤成两行正是因为该列没有下限、被长地址挤压;
|
||
② 编辑 → `.btn-edit`(实心蓝)、删除 → `.btn-del`(实心红)。
|
||
⚠️ 用 `.btn-edit` / `.btn-del` 而不是复用 `.btn-danger`:后者是**淡红描边**的
|
||
"危险动作"语义,全站十来处删除都在用;用户这次明确要的是**实心色块**。 */ ?>
|
||
<table class="table table-fixed cn-table">
|
||
<thead>
|
||
<tr>
|
||
<th class="cn-col-name">名称</th>
|
||
<th class="cn-col-place">所在地方</th>
|
||
<th class="cn-col-addr">详细地址</th>
|
||
<th class="cn-col-count">观影</th>
|
||
<?php if ($canWrite): ?><th class="cn-col-ops">操作</th><?php endif; ?>
|
||
</tr>
|
||
</thead>
|
||
<tbody>
|
||
<?php foreach ($cinemas as $c): ?>
|
||
<tr>
|
||
<td>
|
||
<span class="color-preview" style="background: <?= e((string)($c['color'] ?: '#888888')) ?>;"></span>
|
||
<b><?= e($c['name']) ?></b>
|
||
</td>
|
||
<td><?= e((string)($c['place'] ?? '')) !== '' ? e($c['place']) : '—' ?></td>
|
||
<td class="cn-addr"><?= !empty($c['address']) ? e($c['address']) : '—' ?></td>
|
||
<td><?= (int)$c['movie_count'] ?> 场</td>
|
||
<?php if ($canWrite): ?>
|
||
<td class="cn-col-ops">
|
||
<div class="actions cn-ops">
|
||
<a class="btn btn-edit btn-sm" href="<?= e(url('movies/cinemas', ['edit' => (int)$c['id']])) ?>">编辑</a>
|
||
<form method="post" action="<?= e(url('movies/cinemas_del')) ?>"
|
||
onsubmit="return confirm('确定删除「<?= e($c['name']) ?>」吗?关联观影记录的影院信息将置空。')">
|
||
<?= csrf_field() ?>
|
||
<input type="hidden" name="id" value="<?= (int)$c['id'] ?>">
|
||
<button type="submit" class="btn btn-del btn-sm">删除</button>
|
||
</form>
|
||
</div>
|
||
</td>
|
||
<?php endif; ?>
|
||
</tr>
|
||
<?php endforeach; ?>
|
||
</tbody>
|
||
</table>
|
||
</div>
|
||
<?php endif; ?>
|
||
</div>
|
||
|
||
<?php if ($canWrite): ?>
|
||
<script>
|
||
(function () {
|
||
var picker = document.getElementById('ccolor');
|
||
var hex = document.getElementById('colorHex');
|
||
if (!picker || !hex) return;
|
||
function setActive(color) {
|
||
Array.prototype.forEach.call(document.querySelectorAll('.preset-color'), function (el) {
|
||
el.classList.toggle('active', el.getAttribute('data-color') === color);
|
||
});
|
||
}
|
||
picker.addEventListener('input', function () {
|
||
hex.textContent = picker.value;
|
||
setActive(picker.value);
|
||
});
|
||
Array.prototype.forEach.call(document.querySelectorAll('.preset-color'), function (el) {
|
||
el.addEventListener('click', function () {
|
||
picker.value = el.getAttribute('data-color');
|
||
hex.textContent = picker.value;
|
||
setActive(picker.value);
|
||
});
|
||
});
|
||
setActive(picker.value);
|
||
})();
|
||
</script>
|
||
<?php endif; ?>
|