Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

修复: 判断语言是否为中文 -> 判断语言是否为中文(中国大陆) #5293

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

PlayMcBKuwu
Copy link

@PlayMcBKuwu PlayMcBKuwu commented Dec 15, 2024

Fixes #5292
PCL2/Plain Craft Launcher 2/Modules/Base/ModBase.vb 文件中的语言判断进行了修改,使得其不再只是检查系统语系是否为中文而是细分到 中文(中华人民共和国)(即 中文(中国大陆)zh-CN)。
用于解决「仅中国大陆可以免费游玩 Minecraft」的问题,以防止「所有中文地区都可免费游玩 Minecraft」的歧义。

@3gf8jv4dv
Copy link
Collaborator

原始 Issue: #5292fixes #5292

@3gf8jv4dv 3gf8jv4dv added · Bug 等待确认 已经过社区确认,等待开发者确认 labels Dec 15, 2024
@PlayMcBKuwu PlayMcBKuwu changed the title 修复: 判断语言是否为中文 -> 判断语言是否为中文(中国) 修复: 判断语言是否为中文 -> 判断语言是否为中文(中国大陆) Dec 15, 2024
Copy link
Collaborator

@3gf8jv4dv 3gf8jv4dv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM. 看龙猫怎么说。

@Deep-Dark-Forest
Copy link
Collaborator

Deep-Dark-Forest commented Dec 15, 2024

下次要是这样的话,建议直接发 PR 就行,没必要再发一个多余的 Issue……

@PlayMcBKuwu
Copy link
Author

下次要是这样的话,建议直接发 PR 就行,没必要发一个 Issue……

找代码找了半天没找到判断函数所以才发了个 Issue -w-

shimoranla

This comment was marked as resolved.

@@ -1975,12 +1975,13 @@ RetryDir:
End Function

''' <summary>
''' 判断当前系统语言是否为中文。
''' 判断当前系统语言是否为中文(中国大陆)
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

不是很认同。新加坡和马来西亚一样使用中文,且都是简体字。
除此之外只是为了解决「仅中国大陆可以免费游玩 Minecraft」的问题。

@3gf8jv4dv
Copy link
Collaborator

应该是中文(简体) 而非中文(中国大陆) ,应该没有谁这么写

  1. 区分地区很有必要,而且与 IETF 语言标签中的 zh-CN 相对应。
  2. 来看下 Microsoft 的新做法:Windows 11 24H2 简体中文语言下,在切换输入法的地方写的是「简体中文(中国大陆)」,英语中为「Chinese (Simplified, Mainland China)」——并不是没人写。

@LTCatt LTCatt added 处理中 开发者正在调查或处理该项 🟨 中 优先度:中 and removed 等待确认 已经过社区确认,等待开发者确认 labels Dec 15, 2024
@shimoranla
Copy link

来看下 Microsoft 的新做法:Windows 11 24H2 简体中文语言下,在切换输入法的地方写的是「简体中文(中国大陆)」,英语中为「Chinese (Simplified, Mainland China)」——并不是没人写。

嘶...
刚刚看了预览版本的设置,好像确实是这样
但是总感觉 中文(中国大陆) 说起来很别扭(

@PlayMcBKuwu
Copy link
Author

PlayMcBKuwu commented Dec 16, 2024

来看下 Microsoft 的新做法:Windows 11 24H2 简体中文语言下,在切换输入法的地方写的是「简体中文(中国大陆)」,英语中为「Chinese (Simplified, Mainland China)」——并不是没人写。

嘶... 刚刚看了预览版本的设置,好像确实是这样 但是总感觉 中文(中国大陆) 说起来很别扭(

Microsoft 在 Windows 10 的决定是这样的: 中文(简体, 中华人民共和国)
事实上我第一个 Commit 也是打算这么写的,不过从某些角度听起来很「敏感」就是了(
不过我最后提交 Commit 的时候还是用的 Google 的做法,也就是 中文(中国)

@Chiloven945
Copy link
Contributor

Chiloven945 commented Dec 16, 2024

我的建议是最好不要包含国,尽量使用中国大陆
使用中国可能会出现一些对比情况下会产生的事情,嗯……
应该是简体中文(中国大陆)

@PlayMcBKuwu
Copy link
Author

我的建议是最好不要包含国,尽量使用中国大陆 使用中国可能会出现一些对比情况下会产生的事情,嗯…… 应该是简体中文(中国大陆)

没必要特地强调「简体」,中国大陆规范汉字就是简体字……

@Chiloven945
Copy link
Contributor

没必要特地强调「简体」,中国大陆规范汉字就是简体字……

……这个当然知道
但是在这里真正涉及到名字的就只有代码的注释部分,如果可以甚至简化为检查语言检查是否为zh-cn都是可行的,真正重要的是检测是否准确。
除此之外,在此之外的很多地方均会使用简体中文(中国大陆),比如 Minecraft 内就是如此。如果仅有简体与繁体之分,那么此处的简体中文基本上指大陆,而繁体指台湾。但就像上述所说,这并不是一个必须纠结到底的问题。

@PlayMcBKuwu
Copy link
Author

没必要特地强调「简体」,中国大陆规范汉字就是简体字……

……这个当然知道 但是在这里真正涉及到名字的就只有代码的注释部分,如果可以甚至简化为检查语言检查是否为zh-cn都是可行的,真正重要的是检测是否准确。 除此之外,在此之外的很多地方均会使用简体中文(中国大陆),比如 Minecraft 内就是如此。如果仅有简体与繁体之分,那么此处的简体中文基本上指大陆,而繁体指台湾。但就像上述所说,这并不是一个必须纠结到底的问题。

总会有人看不懂的.png

@Silverteal
Copy link
Contributor

Silverteal commented Dec 25, 2024

这涉及到遵循哪种类型的语言标签规范的问题。根据 IETF BCP 47 (RFC 4647 和 RFC 5646)的话,应该识别 *-CN,即任意语言(任意变体,中华人民共和国)(万一有人在中国用英语呢,或者少数民族语言甚至文言)。但是实际上 BCP 47 完全没有人用,你还是得老老实实全字匹配(zh-CN,zh-Hans),此即名实的分离。

这只是个注释,别惦记它了。直接把操作去掉改成目的都行,我看判断是否处于离线功能可用区域也挺好。

@Hex-Dragon Hex-Dragon locked as too heated and limited conversation to collaborators Dec 25, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
· Bug 处理中 开发者正在调查或处理该项 🟨 中 优先度:中
Projects
None yet
Development

Successfully merging this pull request may close these issues.

PCL II 错误地认为非中国大陆的中文地区可以免费游玩 Minecraft
7 participants