[go: up one dir, main page]

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

建议统一引号写法 #173

Open
shirokurakana opened this issue Apr 2, 2022 · 13 comments
Open

建议统一引号写法 #173

shirokurakana opened this issue Apr 2, 2022 · 13 comments
Labels
documentation📝 Improvements or additions to documentation

Comments

@shirokurakana
Copy link

现在引号的写法比较随性,有时用单引号有时又用双引号,还有嵌套的时候,有时外面是单引号有时外面又是双引号。。
建议还是选择一个标准,然后都按照标准改,不然浏览代码的时候也太顶了(

@MingxuanGame
Copy link
Collaborator

这个确实
我早期写Java的习惯了双引号
单引号和双引号放一起真的顶
(f-string除外)

@MingxuanGame MingxuanGame added the enhancement✨ New feature or request label Apr 2, 2022
@shirokurakana
Copy link
Author

如果没啥好的想法的话,我这里提一个我常用的写法吧(仅供参考)

# 通常情况下统一使用单引号
text = 'test'

# 嵌套的情况下外面使用单引号,内部使用双引号
text = f'message: data["msg"]'

# 三引号字符串统一使用双引号(与PEP 257保持一致)
def sum(nums: List[int]) -> int:
    """calculate the sum of the given array“”“
    pass

text = f"""等级:{level}
经验值:{exp}
血量:{health}
"""

其实之前想提pr来着,但因为这个问题就一直没做(好吧其实是懒x)

如果嫌改起来麻烦,可以你们那边确定好标准以后让我来改,我只在意能有个统一的代码风格。

还有个问题是,同一个文件里塞的代码太多了,这问题不知道后面有没有什么好的方法解决。。

@lgc2333
Copy link
lgc2333 commented Apr 3, 2022

jb能做到吗
之前写js用过prettier,格式化的时候可以自动把双引号字符串换成单引号,要是prettier支持python就太方便了…

@MingxuanGame
Copy link
Collaborator

jb能做到吗
之前写js用过prettier,格式化的时候可以自动把双引号字符串换成单引号,要是prettier支持python就太方便了…

jb好像不能批量替换,但是在字符串上可以转换

@shirokurakana
Copy link
Author

之前我有做过类似的事,我当时的做法是:

  1. 先将所有双引号替换为单引号
  2. 再去报错的地方修改(应该只有嵌套的地方会报错,将内部引号改为双引号即可)
  3. 最后将三个连着的单引号替换为双引号

由于PEP没有规定引号的使用规范,所以应该是找不到这样的格式化工具的(可能有,但我没听说过)

@lgc2333
Copy link
lgc2333 commented Apr 3, 2022

好的,我去看看

@lgc2333
Copy link
lgc2333 commented Apr 3, 2022

麻了
image

@lgc2333
Copy link
lgc2333 commented Apr 3, 2022

image
……
f'(.*)\{(.*)'(.*)'(.*)\}([^\s]*)'
f'$1\{$2"$3"$4\}$5'

@lgc2333
Copy link
lgc2333 commented Apr 3, 2022

#179 #180 #181 #182

@MingxuanGame
Copy link
Collaborator
MingxuanGame commented Apr 3, 2022

感觉以后还会有这样的需求
有时间我搞个脚本替换引号

@KimigaiiWuyi KimigaiiWuyi added documentation📝 Improvements or additions to documentation and removed enhancement✨ New feature or request labels May 8, 2022
@lgc2333
Copy link
lgc2333 commented Jul 18, 2022

要不直接用black格式化得了

@KimigaiiWuyi
Copy link
Owner

要不直接用black格式化得了

使用black格式化加args--skip-string-normalization
我还是更倾向于使用单引号,包括python官方文档也更倾向于使用单引号写法
顺便快来dev分支ctrl c ctrl v

@lgc2333
Copy link
lgc2333 commented Jul 18, 2022 via email

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation📝 Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

4 participants