mirror of
https://gitee.com/xmz7891/ykxmz.git
synced 2026-10-18 16:00:43 +00:00
用户这一轮报了两个问题,都修了。
== 一、Win7 上语音播报不能用 ==
用户原话:"我 win7 上的语音播报好像不能用,你再试一下别的版本能不能用"
**原来只有一条路**:
powershell -NoProfile -WindowStyle Hidden -Command
"Add-Type -AssemblyName System.Speech; $s = New-Object ...; $s.Speak('...')"
这在 Win7 上有三个坑,任何一个都能让它彻底不出声:
1. **`powershell` 只按名字找**
Win7 的 powershell.exe 在 `%SystemRoot%\System32\WindowsPowerShell\v1.0\`。
精简版系统、或从服务/计划任务拉起时 PATH 里可能没有 ——
CreateProcess 按名字找不到时是**静默失败**的。
2. **Win7 默认 PowerShell 2.0,它按 ANSI 代码页解析 `-Command`**
PS 3.0 起才按 UTF-16 处理。中文经 `-Command` 传进去容易乱码 ——
表现是念出来是乱码,或者干脆不念。
3. **`System.Speech` 需要 .NET Framework 3.0+**
而系统自带的 **SAPI5** 从 Windows XP 就有,**根本不需要 .NET**。
**改成多方案降级**,按"依赖越少越优先"排:
1. cscript + VBScript + SAPI5 <- Win98 起就有,不用装任何东西
2. wscript + VBScript + SAPI5 <- 上一条被安全软件拦时的备选
3. powershell + SAPI5 (COM) <- 绕开 .NET
4. powershell + System.Speech <- 原来的做法,改用绝对路径
5. powershell(PATH) + System.Speech <- 最原始的,兜底
一条通了**记住它**,以后直接用(不用每次从头试)。
**顺带修掉一个"骗人"的行为**:原来 `say()` 用 `Popen` 不等进程结束,
失败了也返回"已播报" —— 明明没出声却报成功。
现在等退出码,**全失败会明确报错并列出试过哪几条**。
代价是同步等待(念多久等多久),但"报成功其实没出声"比"慢一点"糟糕得多。
还加了 `_build/diag_tts.py`:在一台机器上把 5 条方案全试一遍并报告
哪条能用 —— 下次再有"某台机器上不行",跑一下就知道。
VBScript 的细节:**必须用 ANSI(mbcs) 写**,写成 UTF-8 的话
cscript 读出来中文是乱码(VBScript 的老规矩,很隐蔽)。
长文本还要分句 —— VBScript 单行有长度上限。
== 二、快捷栏按钮太小 + 4 格都要能换 ==
用户原话:"屏幕最上面这个按钮有些按不到,按钮太小了,做大一点,
然后呢,这个每每个四格算换一下吧,第一个第二个第三个
第四个都能换东西"
* **按钮做大**:30dp -> 46dp(接近 Android 建议的最小可点区域 48),
字号 11 -> 14。30dp 在表盘上手指按上去一半在栏外。
**栏高做成常量 QUICK_BAR_DP,栏高 / 面板 topMargin / addView
三处用同一个值** —— 分三处写的话改一个忘一个,
表现是"内容被盖住"或"上面空一条"。
* **4 格全部可自定义**。用户最早说"其中有一个按钮不能改,
功能就是一键返回桌面",第一版就按"第 4 格写死 desktop"做的。
现在他改主意了 —— 所以 desktop 变成**普通可选动作**,
默认仍放第 4 格(习惯不用改),但想换也能换。
可选动作从 7 个变成 8 个。
**教训(写进测试注释了)**:别把"用户当时的一句话"当成永久约束
焊进结构里;真是约束的话,注释里要写清是什么时候的要求,
下次他改主意时才知道该动哪儿。
== 三、修一个"测试自己没擦干净"造成的假 bug ==
全量回归报:
[FAIL] 配对码正确写入 实际 '(旧的残留值)'
而 `(旧的残留值)` 是**上一次测试遗留**的配对码 —— 数据目录
`C:\ProgramData\ykxmz\data` 在卸载时**故意不删**(那里存着用户的
中转地址、配对码等,卸载不该删),所以机器上会留着上一轮的
settings.json。测试装的时候虽然传了 `/CODE=517284`,但环境没擦干净时
就可能读到旧值 —— 表现成"安装包有 bug"。
**修法**:`test_installer_cycle.py` 加一步"装前先清残留"
(正规卸载 -> 杀进程 -> 删安装目录 -> **删数据目录**)。
数据目录那一步是关键 —— 卸载器不删它。
== 四、其他 ==
* `docs/项目规矩.md` 第一节改成**按预案执行**:
预案一 = 做完测试没问题就上传;预案二 = 做完测试但不上传,直接给用户。
用户没说预案几时默认按预案一,并在交付时说明。
* 顺手把那份文档里的老域名 `relay.xxmxxs.top` 改成现在的
`yk.xxmxxs.top`(老服务器早就退役了)。
* 修掉自己造的重复定义:往 actions.py 插降级链时,
`list_voices` / `say` 各插了两份(pyflakes 报 redefinition)。
* 一次危险的误操作:`Path.write_text(..., newline='\n')` 会**先截断文件
再校验参数**,参数非法时文件已经空了 —— 我用它把
`_tests/test_quickbar.py` 清空了。已重写。
**以后不给 write_text 传 newline。**
== 验证 ==
全量回归(21 关)全部 PASS
新增关卡:3a13 手表快捷栏 / 3a14 语音播报降级
安装包周期 23 项全过(装 -> 验 -> 卸 -> 再验)
Java 编译通过(android-34, --release 8)
本机实测语音播报:cscript 那条通了(2.4 秒)
全部产物重建,Win7 兼容校验通过
225 lines
9.5 KiB
Python
225 lines
9.5 KiB
Python
# -*- coding: utf-8 -*-
|
||
r'''验「手表快捷栏」和「允许新设备连接」。
|
||
|
||
== 验什么 ==
|
||
|
||
一、快捷栏的形状
|
||
* 一共 4 格,**每格都能自定义**
|
||
* 「显示桌面」是普通可选动作(默认放第 4 格,但能换掉)
|
||
* 8 个动作都在
|
||
* 挂进了 onCreate、加进了 showOnly 的数组
|
||
* 设置页里有入口
|
||
* 鼠标触摸板在
|
||
|
||
二、按钮大小
|
||
用户反馈原话:"屏幕最上面这个按钮有些按不到,按钮太小了,做大一点"
|
||
|
||
* 栏高做成常量(>= 44dp),而且**栏高 / 面板 margin / addView
|
||
三处用同一个常量** —— 分三处写的话改一个忘一个,
|
||
表现是"内容被盖住"或"上面空一条"
|
||
|
||
三、电脑端 allow_new_pairing 这个 feature
|
||
|
||
== 这条断言改过一次,值得记下来 ==
|
||
|
||
用户最早说:"最多可以加 4 个,其中有一个按钮不能改,
|
||
功能就是一键返回桌面"。
|
||
第一版就按"第 4 格写死 desktop"实现,也按那个断言测。
|
||
|
||
后来用户看到实物改了主意:"第一个第二个第三个第四个都能换东西"。
|
||
|
||
所以现在验的是**相反**的事。教训:
|
||
**别把"用户当时的一句话"当成永久约束焊进结构里**;
|
||
真是约束的话,注释里要写清是什么时候的要求,
|
||
下次他改主意时才知道该动哪儿。
|
||
'''
|
||
from __future__ import annotations
|
||
|
||
import re
|
||
import sys
|
||
import tempfile
|
||
from pathlib import Path
|
||
|
||
ROOT = Path(__file__).resolve().parent.parent
|
||
sys.path.insert(0, str(ROOT / "windows"))
|
||
|
||
FAILS: list = []
|
||
|
||
|
||
def check(name: str, ok: bool, why: str = "") -> None:
|
||
print(" [%s] %s%s" % ("PASS" if ok else "FAIL", name,
|
||
(" —— " + why) if why else ""))
|
||
if not ok:
|
||
FAILS.append(name)
|
||
|
||
|
||
def main() -> int:
|
||
print("=" * 58)
|
||
print(" 手表快捷栏 + 允许新设备连接")
|
||
print("=" * 58)
|
||
print()
|
||
|
||
main_java = (ROOT / "android" / "src" / "com" / "lanremote" / "watch"
|
||
/ "MainActivity.java").read_text(encoding="utf-8")
|
||
|
||
# ---------------- 一、快捷栏 ----------------
|
||
print("一、快捷栏(手表端)")
|
||
for token, label in [
|
||
("private void buildQuickBar()", "有 buildQuickBar"),
|
||
("private void refreshQuickBar()", "有 refreshQuickBar"),
|
||
("private void onQuickClick(int slot)", "有点击分发 onQuickClick"),
|
||
("private void runQuickAction(String id)", "有动作执行 runQuickAction"),
|
||
("private void loadQuickSlots()", "有读取设置"),
|
||
("private void saveQuickSlots()", "有保存设置"),
|
||
]:
|
||
check(label, token in main_java)
|
||
|
||
# 4 格,全部可自定义。这条断言改过一次,见文件头。
|
||
check("★ 一共 4 格", "QUICK_SLOTS = 4" in main_java, "用户要求 4 格")
|
||
check("★ 旧的 3+1 分法已经不在了",
|
||
"QUICK_CUSTOM_SLOTS" not in main_java, "还有残留说明没改干净")
|
||
check("★ 没有把 desktop 写死在某格",
|
||
': "desktop";' not in main_java,
|
||
"还有地方把 desktop 写死成某一格")
|
||
check("★ desktop 在可选动作里(能换到任意一格)",
|
||
bool(re.search(r'\{"desktop",\s*"显示桌面"\}', main_java)),
|
||
"不在 QUICK_ACTIONS 里的话用户没法把它放到别的格")
|
||
check("四格平等:都从 quickSlots 取",
|
||
"String id = quickSlots[i];" in main_java
|
||
and "String id = quickSlots[slot];" in main_java,
|
||
"还有特殊分支")
|
||
check("默认值就是那 4 个",
|
||
'new String[]{"file", "mirror", "sfx", "desktop"}' in main_java)
|
||
check("desktop 走的是 show_desktop 命令",
|
||
'"cmd", "show_desktop"' in main_java)
|
||
|
||
print()
|
||
print(" 8 个可选动作:")
|
||
for aid, label in [("file", "传文件"), ("mirror", "屏幕镜像"),
|
||
("mouse", "鼠标控制"), ("sfx", "音效"),
|
||
("power", "关机"), ("tts", "语音播报"),
|
||
("image", "投图到电脑"), ("desktop", "显示桌面")]:
|
||
check(" %s(%s)" % (aid, label),
|
||
('{"%s", "%s"}' % (aid, label)) in main_java)
|
||
|
||
# ---------------- 二、按钮大小 ----------------
|
||
print()
|
||
print("二、按钮大小(用户反馈「太小了、按不到」)")
|
||
m = re.search(r"QUICK_BAR_DP = (\d+)", main_java)
|
||
check("★ 栏高做成常量 QUICK_BAR_DP", m is not None,
|
||
"没有常量的话栏高和面板 margin 会各写各的")
|
||
h = int(m.group(1)) if m else 0
|
||
check("★ 栏高 >= 44dp(接近 Android 建议的可点区域 48)", h >= 44,
|
||
"现在是 %d" % h)
|
||
check("★ 面板 topMargin 用同一个常量",
|
||
"flp.topMargin = dp(QUICK_BAR_DP);" in main_java,
|
||
"分两处写的话改一个忘一个")
|
||
check("★ addView 也用同一个常量",
|
||
"MATCH_PARENT, dp(QUICK_BAR_DP)));" in main_java)
|
||
check("★ 按钮字号 >= 13(原来是 11)",
|
||
"b.setTextSize(14f);" in main_java)
|
||
# ⚠ 只查**代码**,不查注释。
|
||
# 第一版这里是 `"dp(30)" not in main_java`,结果被
|
||
# applyPanelTopMargin 的 javadoc 里那句旧说明误报了 ——
|
||
# 注释里提一下旧值是很正常的事("以前是 30dp"),
|
||
# 不该因为它变红。
|
||
code_lines = [ln for ln in main_java.splitlines()
|
||
if not ln.strip().startswith(("*", "//", "/*"))]
|
||
hard = [ln.strip() for ln in code_lines if "dp(30)" in ln]
|
||
check("没有残留的 dp(30)(旧栏高)", not hard,
|
||
"还有代码用旧的 30dp:%s" % hard)
|
||
|
||
# ---------------- 三、挂接 ----------------
|
||
print()
|
||
print("三、挂接(漏了就白写)")
|
||
i0 = main_java.find("protected void onCreate")
|
||
oncreate = main_java[i0:i0 + 7000]
|
||
check("★ onCreate 里调了 buildQuickBar()", "buildQuickBar();" in oncreate,
|
||
"没挂上去栏根本不显示")
|
||
check("★ onCreate 里调了 loadQuickSlots()", "loadQuickSlots();" in oncreate)
|
||
check("★ 快捷栏最后 addView(保证在最上层)",
|
||
oncreate.find("root.addView(quickBar")
|
||
> oncreate.find("root.addView(mousePanel"),
|
||
"加在面板之前会被面板整个盖住")
|
||
check("★ showOnly 的数组里有 mousePanel",
|
||
"settingsPanel, mousePanel};" in main_java,
|
||
"漏了的话鼠标面板永远不隐藏")
|
||
check("有 applyPanelTopMargin(面板给栏让位)",
|
||
"private void applyPanelTopMargin" in main_java)
|
||
|
||
# ---------------- 四、设置页 ----------------
|
||
print()
|
||
print("四、设置页入口(没入口用户改不了)")
|
||
check("有 fillSettingsExtra",
|
||
"private void fillSettingsExtra()" in main_java)
|
||
check("有 pickQuickSlot(选动作)",
|
||
"private void pickQuickSlot(final int slot)" in main_java)
|
||
check("设置页里加了「快捷栏」小节",
|
||
'sectionTitle("快捷栏")' in main_java)
|
||
check("设置页里加了「允许新设备连接」小节",
|
||
'sectionTitle("允许新设备连接")' in main_java)
|
||
check("界面上说明了四格都能换", "每一个都能换" in main_java,
|
||
"不说明的话用户不知道第四格也能改")
|
||
|
||
# ---------------- 五、鼠标触摸板 ----------------
|
||
print()
|
||
print("五、鼠标触摸板")
|
||
check("有 buildMousePanel", "private void buildMousePanel()" in main_java)
|
||
check("用 GestureDetector(单击/双击/拖动)",
|
||
"GestureDetector" in main_java
|
||
and "onSingleTapConfirmed" in main_java
|
||
and "onDoubleTap" in main_java
|
||
and "onScroll" in main_java)
|
||
check("单击 -> 左键", 'sendMouse("click", "left", 0)' in main_java)
|
||
check("双击 -> 右键", 'sendMouse("click", "right", 0)' in main_java)
|
||
check("滚轮上/下", 'sendMouse("scroll", "", 3)' in main_java
|
||
and 'sendMouse("scroll", "", -3)' in main_java)
|
||
check("拖动 -> moveby", '"moveby"' in main_java)
|
||
check("isConnected() 存在(第一版漏了,编译才发现)",
|
||
"private boolean isConnected()" in main_java)
|
||
|
||
# ---------------- 六、电脑端 allow_new_pairing ----------------
|
||
print()
|
||
print("六、电脑端 allow_new_pairing feature")
|
||
from lanremote.config import Store
|
||
from lanremote.features import Features
|
||
|
||
class FakeActions:
|
||
"""Features 要的是 actions 对象,不是 store(第一版就是这里传错了)。"""
|
||
|
||
def __init__(self, store):
|
||
self.store = store
|
||
self.ui = None
|
||
|
||
st = Store(Path(tempfile.mkdtemp()))
|
||
feats = Features(FakeActions(st))
|
||
|
||
check("能查到当前状态", "允许" in feats.run("allow_new_pairing", {}))
|
||
out = feats.run("allow_new_pairing", {"allow": False})
|
||
check("能关掉", "禁止" in out, "拿到 %r" % out)
|
||
check(" ★ 存储里真的变了", st.get("allow_new_pairing") is False,
|
||
"拿到 %r" % st.get("allow_new_pairing"))
|
||
check(" 关掉之后再查也是关",
|
||
"不允许" in feats.run("allow_new_pairing", {}))
|
||
out = feats.run("allow_new_pairing", {"allow": True})
|
||
check("能打开", "已允许" in out, "拿到 %r" % out)
|
||
check(" 存储里也变了", st.get("allow_new_pairing") is True)
|
||
|
||
try:
|
||
feats.run("allow_new_pairing_typo", {})
|
||
check("不认识的名字会报错", False, "它竟然没报错")
|
||
except ValueError:
|
||
check("不认识的名字会报错", True)
|
||
|
||
print()
|
||
print("=" * 58)
|
||
if FAILS:
|
||
print(" 失败 %d 项:%s" % (len(FAILS), ", ".join(FAILS)))
|
||
return 1
|
||
print(" 全部通过")
|
||
return 0
|
||
|
||
|
||
if __name__ == "__main__":
|
||
sys.exit(main())
|