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

名前付きキャプチャ部分の冗長な正規表現 #943

Open
xtetsuji opened this issue Dec 2, 2024 · 2 comments · May be fixed by #960
Open

名前付きキャプチャ部分の冗長な正規表現 #943

xtetsuji opened this issue Dec 2, 2024 · 2 comments · May be fixed by #960
Labels
stage2:着手可 着手できるフェーズのissueです。プルリクエストを作成できます。 type:誤植 日本語やサンプルコードなどの記載ミス volume:S 数文字〜1文程度の小さめの執筆量。

Comments

@xtetsuji
Copy link

xtetsuji commented Dec 2, 2024

サバイバルTypeScript、TypeScript の学習で一度通読した後、わからないたびに何度も参照して助けていただいています 🙇

何をどうしたいか?

RegExp > Named capturing groupsについての中にある正規表現に、軽微なミスがあったので修正したいです。

const regex = /(?<pref>.+[都|道|府|県])(?<ward>.+[市|区|町|村])/gu;

ここの部分、文字クラス [〜〜〜] に関しては文字を並べるだけで「その中の任意の一文字」という OR 条件になるため、縦棒は不要だと思いました(これがグループ化の場合は (?:都|道|府|県) が正)。

そうしたい理由は何か?(誤植の修正など理由が自明な場合は不要)

着手可能になったら自分でプルリクエストを作成したいか?

自分で作成したい

※ 着手可になってから、1ヶ月以上経過している場合は他の人が着手する可能性があります。

@suin suin added type:誤植 日本語やサンプルコードなどの記載ミス stage2:着手可 着手できるフェーズのissueです。プルリクエストを作成できます。 volume:S 数文字〜1文程度の小さめの執筆量。 labels Dec 3, 2024
@suin
Copy link
Contributor

suin commented Dec 3, 2024

@xtetsuji ご指摘のとおりですね。ご報告ありがとうございます。PRもいただけたら嬉しいです!

@xtetsuji xtetsuji linked a pull request Jan 6, 2025 that will close this issue
@xtetsuji
Copy link
Author

xtetsuji commented Jan 6, 2025

@suin 遅くなってしまいましたが、PR #960 作成しました 🙇

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
stage2:着手可 着手できるフェーズのissueです。プルリクエストを作成できます。 type:誤植 日本語やサンプルコードなどの記載ミス volume:S 数文字〜1文程度の小さめの執筆量。
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants