如何用 map 和 re 来优化下面的 if..else..

[ladypxy]

代码如下

long_string ="blablabl"

my_dict = {"a": "string1", "b": "string2" , "c": "string3", "d":"string4"}

if "Andy" in long_string:
    return mydict["a"]
elif "Jack" in long_string:
    return mydict["b"]
elif "Jim" in long_string and "Mike" not in long_string:
    return mydict["c"]
else:
    return mydict["d"]

感觉这样效率有点低，感觉用 map 处理比较好。但是条件 3 实际上是 2 个条件，这样用 map 反而不好写了。

请问大家有啥好的建议？

谢谢

[BeautifulSoap]

写个循环遍历 long_string ，一次循环就能判定了

[ladypxy]

@BeautifulSoap 一次？求范例，谢谢

[westoy]

就就几个条件， 这样写挺好的

[ccde8259]

为什么是 map 和 re 而不是 DFA 啊……

[luozic]

原始需求是啥？ 你这算字符串转义？

[ladypxy]

@luozic 需求就是查询字符串 long_string 里有没有指定的字符串，然后返回字典的数值。
要求用 map 和 re 解决

[LoNeFong]

long_string = "blablabl"

my_dict = {"a": "string1", "b": "string2", "c": "string3", "d": "string4"}

result = my_dict["a"] if "Andy" in long_string else my_dict["b"] if "Jack" in long_string else my_dict["c"] if "Jim" in long_string and "Mike" not in long_string else my_dict["d"]

result

[tairan2006]

让 gpt 优化就行 专业对口

[BeautifulSoap]

@luozic 看了 lz 的回复有一点不太明白了，你想改善的到底是那一坨 if 的判断逻辑，还是想改善那一坨 if 里的多个 in 查找？

[JasonLaw]

@LoNeFong #7 这代码没有一点可读性🤕

[ladypxy]

@BeautifulSoap 就是不要用 if 而是用 map 来实现

[blankmiss]

gpt 给的答案 def get_value(long_string):
string_map = {
"Andy": "string1",
"Jack": "string2",
"Jim": "string3",
"Mike": "not_found",
}

return next(map(lambda key: string_map[key], filter(lambda key: key in long_string and (key != "Jim" or "Mike" not in long_string), string_map)), "string4")

可是这样真的好吗

[Jakarta]

@blankmiss 优雅，俺们 v2 就是追求这样的代码

[lhbc]

先遍历找出所有的关键词是否存在，存到一个字典里，然后判断返回

但没必要这样优化
你的逻辑是满足其中一个条件就 return 了，其他都没必要找
先找出所有的反而效率低

除非你的字符串极长

[ladypxy]

@tairan2006 试了下，需要多次才能给出差不多的答案，但是个人感觉这种可读性还不如 if else

import re

long_string = "blablabl"

my_dict = {"a": "string1", "b": "string2", "c": "string3", "d": "string4"}

keyword_mapping = {
"Andy": "a",
"Jack": "b",
"Jim": "c",
"Mike": "d"
}

matched_keywords = set(filter(lambda x: re.search(x, long_string) is not None, keyword_mapping.keys()))
matched_values = [my_dict[keyword_mapping[keyword]] for keyword in matched_keywords]

if "Jim" in matched_keywords and "Mike" not in matched_keywords:
result = my_dict[keyword_mapping["Jim"]]
else:
result = matched_values[0] if matched_values else my_dict["d"]

print(result)

[Pastsong]

不可读吗？比正则可读吧

[liantian]

看了楼上各种解决方案…

还是原始方案最可读啊。

[qwq11]

https://paste.ubuntu.com/p/pFwm9WPymf/
https://paste.ubuntu.com/p/zWr2WHhsN2/

感觉本帖像《回字的四种写法》

[luozic]

@BeautifulSoap 看这 lz 还是有点故意把能优化成 DFA 的，用 re+map 处理？ 关键这种字典映射，只取一个就返回？

[luozic]

https://paste.ubuntu.com/p/pFwm9WPymf/
@qwq11 拼写单词错误，语法错误