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

计科:week1 代码风格检查 #2

Open
wujr5 opened this issue Sep 24, 2015 · 12 comments
Open

计科:week1 代码风格检查 #2

wujr5 opened this issue Sep 24, 2015 · 12 comments

Comments

@wujr5
Copy link
Owner

wujr5 commented Sep 24, 2015

week1 代码风格检查

一、用cpplint.py进行代码风格检查

Sicily系统没有在编程风格上作要求,只要你的代码能运行,输入输出没问题,就可能可以通过。但是这样会写出可能可读性很差的代码。因此我们TA强烈要求同学们一定要注意自己的代码编写的风格。

下面介绍一个工具的使用来进行代码风格检查。

步骤1:安装python

教程:安装python

要注意的是,教程中说的系统变量就是python的安装路径。

步骤2:检查python是否安装成功

  1. 打开命令行窗口(win键+R;输入cmd;回车)如下:
  2. 如果安装成功,在命令行窗口输入python将会出现以下的输出:

步骤3:下载cpplint.py

点击下载cpplint.py

步骤4:代码风格检查

  1. cpplint.py文件复制到你的程序代码所在的目录。比如:

  2. 打开命令行窗口,方法就是步骤2的第一点。导航到代码所在目录。

    cd命令的含义是打开目录,dir命令的含义是查看该目录下地文件和文件夹

  3. 运行命令

    python cpplint.py --filter=-whitespace/tab filename.cpp
    

    注意finename.cpp是你的源代码的文件名

    然后就会有如下的输出:

    注意错误,如test.c:7: Line ends in whitespace. Consider deleting these extra spaces. [whitespace/end_of_line] [4]test.c指的是文件名,7指的是行号,后面的 Line ends in whitespace. Consider deleting these extra spaces. [whitespace/ end_of_line] [4]指的是代码中出现的错误。

步骤5:根据输出的改善代码

根据输出的错误,逐行改进代码。直到错误数目为0。如Total errors found: 0

步骤6:提交到Sicily

把通过google style的cpplint.py测试的代码,也就是Total errors found: 0,再把代码提交到Sicily。

总结

进行google style风格检测的目的是让同学们写出可读性良好的代码。按照上面的流程编写代码,能帮助你养成良好的编程习惯。希望同学们重视。

有任何问题欢迎评论。我们会持续改进。

参考

  1. cmd命令行大全
  2. google style guide
  3. cpplint
@wujr5 wujr5 changed the title 计科:week1 代码风格检查 数计:week1 代码风格检查 Sep 25, 2015
@wujr5 wujr5 changed the title 数计:week1 代码风格检查 计科:week1 代码风格检查 Nov 1, 2015
@wujr5
Copy link
Owner Author

wujr5 commented Nov 1, 2015

第一次课课件:点击下载

@SchroDeCat
Copy link

请问do not use namespace using-directives 是什么意思?

@wujr5
Copy link
Owner Author

wujr5 commented Nov 9, 2015

@SchroDeCat 这个错误可以不用管

@SchroDeCat
Copy link

还有win10系统改了环境变量也还是显示python 不是内部或外部命令,只能每次set PATH=%PATH%;C:\Python34有什么解决方案吗?

@wujr5
Copy link
Owner Author

wujr5 commented Nov 9, 2015

@SchroDeCat 是不是使用了中文的分号啊

@wujr5
Copy link
Owner Author

wujr5 commented Nov 9, 2015

@SchroDeCat 将系统变量放上来看看

@SchroDeCat
Copy link

%SystemRoot%\system32;%SystemRoot%;%SystemRoot%\System32\Wbem;%SYSTEMROOT%\System32\WindowsPowerShell\v1.0\;C:\Python34;

@SchroDeCat
Copy link

ok好了

@wujr5
Copy link
Owner Author

wujr5 commented Nov 9, 2015

明显使用了中文分号啊!改成这样:

%SystemRoot%\system32;%SystemRoot%;%SystemRoot%\System32\Wbem;%SYSTEMROOT%\System32\WindowsPowerShell\v1.0\;C:\Python34;

@SchroDeCat
Copy link

改好啦(win10自带输入法切换还不太习惯……

@HELLKING66666
Copy link

qq 20151112142104
qq 20151112142059

有时候为了美化,可以用空白行吗?如上图,谷歌不允许也。

@SgLy
Copy link

SgLy commented Nov 12, 2015

@i771645310 这个空白行太多了吧……一行应该可以?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants