不要在程序中用System.out.println()

在log4j大行其道的今天，居然还有很多程序员在程序中到处写System.out.println()。这样做带来的影响是极其恶劣的。

1.最要命的就是影响程序的性能

2.浪费磁盘空间，如果你用的是tomcat，catalina.out文件用不了多久就会上G

3.影响程序的调试开发，一个用户请求发过去，控制台哗啦啦像下雨一样的打印日志，根本不晓得自己要看的输出语句在何方

4.就这么多了。。。

 

如果你不喜欢用log4j这样的日志类库，那么最次也要写个Debug类封装一下System.out.println()，然后加个开关控制日志是否输出。

评论

12 楼 fangwei 2009-08-21  

night_stalker 写道

这样？ 好像有点麻烦 ...

PrintStream oldOut = System.out;
PrintStream newOut = new PrintStream(new File("e:/log"));
System.setOut(newOut);

System.out.println("hello");

System.setOut(oldOut);
newOut.close();

另外直接打印屏幕的扩展性很好的，linux 下面很多命令的源代码都是直接 printf，调用的时候用 > 重定向就可以了。

ps -e > processes.log

windows 也支持重定向和管道

dir | find "java" > javaSrcList.txt

多谢分享

11 楼 fangwei 2009-08-21  

treblesoftware 写道

在DUBUG的时候System.out.println()何罪之有？请指教。

这样会有一个风险，你一不小心就会忘记删除或注掉类似的调试打印语句，然后就会在运行时不停的输出到控制台。
另外，即使你很小心翼翼的每次调试完毕后都删除或注掉了类似的调试打印语句，但是下次你再次需要调试时，你又得重新写一次或重新打开注释。其实也不麻烦，但是使用开关是更好的方式。举一个现实中的例子，你回到家打开灯，睡觉前关灯，但是你不会回到家把灯泡装上，睡觉前把灯泡取下来，System.out.println()就是这个灯泡。

10 楼 night_stalker 2009-08-21  

这样？ 好像有点麻烦 ...

PrintStream oldOut = System.out;
PrintStream newOut = new PrintStream(new File("e:/log"));
System.setOut(newOut);

System.out.println("hello");

System.setOut(oldOut);
newOut.close();

另外直接打印屏幕的扩展性很好的，linux 下面很多命令的源代码都是直接 printf，调用的时候用 > 重定向就可以了。

ps -e > processes.log

windows 也支持重定向和管道

dir | find "java" > javaSrcList.txt

9 楼 treblesoftware 2009-08-21  

在DUBUG的时候System.out.println()何罪之有？请指教。

8 楼 fangwei 2009-08-21  

night_stalker 写道

正确的方法是用 setOut 重定向，没必要封装。

多谢指教

System.setOut(new PrintStream(new File("e:/log")));
System.out.println("hello");

这样确实可以做到重定向输出到文件，请问如何做到关闭输出呢？我查了一下api未果，下面这样直接空指针异常

System.setOut(null);

7 楼 fangwei 2009-08-21  

1986jade 写道

受教

只是一些小感悟，愿与大家分享

6 楼 fangwei 2009-08-21  

feigofigo3 写道

LZ，简直在乱说，应该是新人吧，新人是可以原谅的

我请求你不要原谅我

5 楼 fangwei 2009-08-21  

conservatism 写道

4 楼 night_stalker 2009-08-20  

正确的方法是用 setOut 重定向，没必要封装。

3 楼 1986jade 2009-08-20  

受教

2 楼 feigofigo3 2009-08-20  

LZ，简直在乱说，应该是新人吧，新人是可以原谅的

1 楼 conservatism 2009-08-20