Golang regexp MatchString()方法不具备幂等性问题排查

问题原因 #

Go语言中遍历map的顺序是随机的，这是语言规范明确规定的行为。你的输入卡号5103901404433835同时符合Mastercard和VisaMaster的正则规则，而resolvePaymentNetwork函数里每次遍历PAYMENT_NETWORKS时，只要匹配就会覆盖payNet的值。由于遍历顺序不固定，最后一次匹配到的支付网络可能是两者中的任意一个，导致返回结果不稳定。

而Node.js的现代引擎（比如V8）遍历对象时会遵循属性插入顺序，所以每次都是先检查Mastercard再检查VisaMaster，最后VisaMaster会覆盖之前的结果，因此输出始终一致。

解决方案 #

有两种常用的修正方式，根据业务需求选择：

1. 使用有序结构替代map，固定遍历顺序

如果希望保持当前“遍历所有规则，最后匹配的结果生效”的逻辑，把PAYMENT_NETWORKS改成切片（slice），确保每次遍历顺序一致：

package main

import (
    "fmt"
    "regexp"
)

type PaymentNetworkData struct {
    Regex string
    Name  string
}

// 用切片替代map，固定遍历顺序
var PAYMENT_NETWORKS = []PaymentNetworkData{
    {
        Regex: "^5[1-5][0-9]{14}|^(222[1-9]|22[3-9]\\d|2[3-6]\\d{2}|27[0-1]\\d|2720)[0-9]{12}$",
        Name:  "Mastercard",
    },
    {
        Regex: "^(?:4[0-9]{12}(?:[0-9]{3})?|5[1-5][0-9]{14})$",
        Name:  "VisaMaster",
    },
}

func resolvePaymentNetwork(cardIn string) string {
    payNet := "Unknown"
    for _, v := range PAYMENT_NETWORKS {
        regex := regexp.MustCompile(v.Regex)
        if regex.MatchString(cardIn) {
            payNet = v.Name
        }
    }
    return payNet
}

func main() {
    in := "5103901404433835"
    for i := 1; i < 100; i++ {
        payNet := resolvePaymentNetwork(in)
        fmt.Println("Payment Network is: ", payNet)
    }
}

2. 匹配到第一个符合规则的就返回（推荐，更高效）

如果业务上支付网络有优先级（比如Mastercard优先于VisaMaster，或者反之），可以找到第一个匹配的结果就直接返回，同时预编译正则表达式提升性能：

package main

import (
    "fmt"
    "regexp"
)

type PaymentNetworkData struct {
    Regex *regexp.Regexp
    Name  string
}

// 预编译正则，避免每次调用重复编译
var PAYMENT_NETWORKS = []PaymentNetworkData{
    {
        Regex: regexp.MustCompile("^5[1-5][0-9]{14}|^(222[1-9]|22[3-9]\\d|2[3-6]\\d{2}|27[0-1]\\d|2720)[0-9]{12}$"),
        Name:  "Mastercard",
    },
    {
        Regex: regexp.MustCompile("^(?:4[0-9]{12}(?:[0-9]{3})?|5[1-5][0-9]{14})$"),
        Name:  "VisaMaster",
    },
}

func resolvePaymentNetwork(cardIn string) string {
    for _, v := range PAYMENT_NETWORKS {
        if v.Regex.MatchString(cardIn) {
            return v.Name // 找到第一个匹配的直接返回
        }
    }
    return "Unknown"
}

func main() {
    in := "5103901404433835"
    for i := 1; i < 100; i++ {
        payNet := resolvePaymentNetwork(in)
        fmt.Println("Payment Network is: ", payNet)
    }
}

额外优化说明 #

原代码中每次调用resolvePaymentNetwork都会重新编译正则表达式，这会带来不必要的性能开销。上面的第二种方案预编译了正则，在程序启动时就完成编译，后续调用直接使用，效率更高。

内容的提问来源于stack exchange，提问作者rogrp6