做了统计php代码行数的小程序，大家来批评

最新回复

shaoshuai_496 at 2008-8-04 19:55:20

shaoshuai_496 at 2008-8-04 20:01:15

怎么没人批评呀，我感觉对于没毕业的来说很有用啊，面试的时候有用，没有工作经验是很实际的，但是可以用自己写过多少代码来让对方了解你的能力呀

cers at 2008-8-04 20:05:58

嗯，比我强，问下你学的什么专业的？
那个嵌套问题你没管？

shaoshuai_496 at 2008-8-04 20:05:59

shaoshuai_496 at 2008-8-04 20:06:52

QUOTE:

原帖由 cers 于 2008-8-4 20:05 发表
嗯，比我强，问下你学的什么专业的？
那个嵌套问题你没管？

太谦虚了，我很菜的，我是学软件工程的，嵌套问题没管，是个bug

cers at 2008-8-04 20:11:58

难怪代码那么结构化
而且注释还是英文的
我一句也没看懂

cers at 2008-8-04 20:13:44

require_once('common.func.php');
这句？

shaoshuai_496 at 2008-8-04 20:18:02

刚才统计了一下我的部分代码 4000多行，
又统计了一下Discuz_6.1.0_SC_UTF8的代码49061行，哭啊，努力啊

shaoshuai_496 at 2008-8-04 20:19:04

QUOTE:

原帖由 cers 于 2008-8-4 20:13 发表
require_once('common.func.php');
这句？

哦，在我这里是4个文件，我那个获取扩展名的函数在那里

七月十五 at 2008-8-04 20:23:00

我的PHP文件全部是0行的
楼主对\n和\r欠考虑了

cers at 2008-8-04 20:24:34

UNIX文件格式
为什么php文件是0行的呀？

shaoshuai_496 at 2008-8-04 20:27:55

QUOTE:

原帖由 七月十五 于 2008-8-4 20:23 发表
我的PHP文件全部是0行的
楼主对\n和\r欠考虑了

当时考虑到了，我在windows下做得，后来没改，谢谢十五提醒

七月十五 at 2008-8-04 20:31:29

1、小功能用上了递归，楼主显然比较强，但是能不能简单问题简单解决，用普通的函数就可以了？
2、以\n或\r来判断，能不能以fgets来判断，是否效率要高一点？
3、global 和 常量DIR，一般真的不提倡在函数里用，破坏了函数的独立性、封闭性。

shaoshuai_496 at 2008-8-04 20:32:51

对了，换行符都是什么来着，windows \n  unix \r\n  mac \r    是这样的吗？

shaoshuai_496 at 2008-8-04 20:35:35

QUOTE:

原帖由 七月十五 于 2008-8-4 20:31 发表
1、小功能用上了递归，楼主显然比较强，但是能不能简单问题简单解决，用普通的函数就可以了？
2、以\n或\r来判断，能不能以fgets来判断，是否效率要高一点？
3、global 和 常量DIR，一般真的不提倡在函数里用，破坏 ...

恩，指点的很到位，十分感谢，十分感谢

七月十五 at 2008-8-04 20:37:09

[ 本帖最后由 七月十五 于 2008-8-4 20:38 编辑 ]

cers at 2008-8-04 20:41:24

额
十五统计的是一个文件的所有行数
我还是第一次看到fgets这个函数的实际应用

七月十五 at 2008-8-04 20:43:01

QUOTE:

原帖由 cers 于 2008-8-4 20:41 发表
额
十五统计的是一个文件的所有行数
我还是第一次看到fgets这个函数的实际应用

是的，没有判断PHP标志和是否为PHP文件。
所以是getFileLines不是getPhpLines
楼主不是有很好的函数么？稍改一下就可以了。

shaoshuai_496 at 2008-8-04 20:44:41

如果文件里面全都是php代码，可以用fgets，但是如果是嵌入html中的就不能这么统计了，我提出了所有<?php ?>的标签，然后再判断的换行符，这样效率低一些，但是没想出来统计html中嵌入的php更好的方法

shaoshuai_496 at 2008-8-04 20:46:51

十五的正则运用的如何，cers给我的正则有个bug，它处理不了这样嵌套的