Python 正規表現

Question

前提

3桁ごとにカンマで区切った数字にのみマッチするプログラムで思うようにいかなかったので質問させていただきました。下記の数字を参考にマッチする値だけ表示する正規表現を考えたのですがうまくいきませんでした。

マッチする
42
1,234
6,368,745
マッチしない
12,34,567
1234

該当のソースコード

import re
import pyperclip


number_regex = re.compile(r'((\d,)?(\d,|\d{3},)?\d{1,3})')

#クリップボードのテキスト検索する
text = str(pyperclip.paste())
matches = []
for groups in number_regex.findall(text):
    matches.append(groups[0])

#検索結果をクリップボードに貼り付ける
if len(matches) > 0:
    pyperclip.copy('\n'.join(matches))
    print('クリップボードにコピーしました')
    print('\n'.join(matches))
else:
    print('マッチする数字がありませんでした')

ターミナル上の表示

クリップボードにコピーしました
42
1,234
6,368,745
12
34
567
123
4

###実現したいこと

12
34
567
123
4

どのようにすれば上の数字を表示できなくすることができるでしょうか。ご教授頂けると助かります。

Answer 1

これでどうでしょう。

number_regex = re.compile(r'^\d{1,3}(?:,\d{3})*$', flags=re.MULTILINE)

---

#####_eito_さんの書いた正規表現

この正規表現だと、1,2,3とかもマッチしますね。

#####私の書いた正規表現

MULTILINEモードを活用して、行内で一回だけ判定を行うようにしています。

---

図はDebuggex: Online visual regex tester. JavaScript, Python, and PCRE.で作成

Comments

[_eito_]

すみません、上記のコードを試してみたのですが、マッチする数字がありませんでした...

[LouiS0616]

text = str(pyperclip.paste())
matches = number_regex.findall(text) だとどうなりますか。

[_eito_]

同じくマッチする数字がありませんでした。

[LouiS0616]

text = str(pyperclip.paste())
print(text) の実行結果を教えて下さい。

[LouiS0616]

一応手元では想定どおり動いているのです。
https://wandbox.org/permlink/NFWxtjP9P55qrl7v

[_eito_]

42
1,234
6,368,745

12,34,567
1234
1,2,3
という結果が出ました

[_eito_]

一応
text = str(pyperclip.paste())
matches = [number_regex.findall(text)]
print(matches)
でも実行してみたのですがその時の結果は
[[]]
でした

[LouiS0616]

re.MULTILINEは忘れず指定していますか？

[_eito_]

指定しています

[LouiS0616]

再現しました。pypertextが改行文字を\r\nにしているのが原因のようです。
どのように解決するのがスマートかちょっと検討しますので、少々お待ちください。

[LouiS0616]

ちょっとダサいですけど、text = str(pyperclip.paste()).replace('\r\n', '\n') がどうやら正解です。

[_eito_]

なぜかマッチする数字の先頭の部分しか取れませんでした。下記がターミナル上の表示です。
クリップボードにコピーしました
4
1
6

[LouiS0616]

for groups in number_regex.findall(text): matches.append(groups[0]) を
for groups in number_regex.findall(text): matches.append(groups) に修正して下さい。

[LouiS0616]

(?:...)を使うことで不必要なキャプチャを取らないよう工夫しているので、re.findallは単にマッチした文字列だけを返してくれます。

[_eito_]

1つ1つの質問に丁寧に解説してくださり、本当にありがとうございました!おかげで解決することができました。