我编写的公司正在开发的部份代码,出于某种理由,一些操作被我处理了

feilong816 2004-07-02 09:42:50
刚入行不久,希望大家给我品一下这个代码的代码规范,谢谢.
　　首先保证的是这个代码在我公司里面算是完成的代码,可以正常使用,当然可以使用的代码未必是个好的代码,希望大家品一品
　　谢谢!

package turbine.modules.actions;

/**
* @SQL ---inlude method->(1)doCreateagent,(2)doAddcredit,(3)doAddcreditbat,
* (4)doChangepassowrd,(5)doModifystatus,(6)doSitestatus,(7)doModifymemberstatus
* (8)doCreatesite
* @Functiohn:
* create agent;create member(include in doCreateagent);
* modify password;
* agent,member status's change,
* add/withdraw credit,
* create site,
* change site status
*@param binding connection
*@author David
*Company: blueskytech
*
*/

import org.apache.velocity.context.Context;
import javax.servlet.http.*;
import org.apache.turbine.util.RunData;
import org.apache.turbine.modules.actions.VelocityAction;
import turbine.om.server.*;
import turbine.om.dataservice.*;
import turbine.modules.util.*;
import java.util.*;

import org.apache.log4j.*;



public class SQL
extends SecureAction {

public String renum = "\\d+"; //match of int character
public String restring = "\\w+"; //match of String character
public int bindingvalue = ErrorCode.ERR_UNKNOWN;



/*===========================1=Create new user=============================*/
/**
*
* add new agent,member identified by user password
* @param data RunData
* @param context Context
* @param nickname,Creates user's nickname
* @param password,Adding user's password
* @param displayname,Adding user's displayname
* @param passwordc,the up people's password,(operate password)
*
* @throws Exception
*/
public void doCreateagent(RunData data) throws Exception {

String error=null;

CleanInfoSession(data);
String nickname = data.getParameters().getString("Nick"); //adding user's nick
String displayname = data.getParameters().getString("displayname"); //adding user's diplay name
String password = data.getParameters().getString("password"); //adding user's password
String passwordoper = data.getParameters().getString("passwordc"); //operater's password
String sitenameid = data.getParameters().getString("sitenameid") ;
String language = this.getLanguage(data);

if (this.InputValidate(this.nickname, nickname, "doCreateagent", data) &&
this.InputValidate(this.displayname, displayname, "doCreateagent", data) &&
this.InputValidate(this.password, password, "doCreateagent", data) &&
this.InputValidate(this.passwordoper, passwordoper, "doCreateagent",
data)) {
try{
//code..................................
}
catch (Exception e) {
}
this.getResult(bindingvalue, "doCreateagent", data);
}
}

/*===========================2=add/reduce score in onece=======================*/
/**doAddcredit
* give score to user,identified by user password
* @param data RunData
* @param context Context
* @param agentchildid,Userid of people will add credit
* @param creditoperate,the operater of add or withdraw creidt
* @param addcreditid,credit number
* @throws Exception
*/

public void doAddcredit(RunData data) throws Exception {
CleanInfoSession(data);
String passwordoper = data.getParameters().getString("agentpassword") ;
int agentchildid = data.getParameters().getInt("agentchildid");
String addcredits = data.getParameters().getString("addcredit");
String agentnick = data.getParameters().getString("agentnick");
String creditoperate = data.getParameters().getString("creditoperate") ;

if (this.InputValidate(this.addcredits, addcredits, "doAddcredit", data) &&
this.InputValidate(this.nickname, agentnick, "doAddcredit", data)) {
try{
//code..................................
}
catch (Exception e) {
}

this.getResult(bindingvalue, "doAddcredit", data);

}

}

/*===========================4==Change Password===========================*/
/**doChangepassword
* user change password herself,identify by password(oldpassword)
* @param data RunData
* @param context Context
* @throws Exception
*/
public void doChangepassword(RunData data) throws
Exception {

CleanInfoSession(data);

String passwordoper = data.getParameters().getString("oldpassword") ;
String passwordnew = data.getParameters().getString("newpassword") ;

String nickname = agent.getNick();
if (this.InputValidate(this.passwordoper, passwordoper, "doChangepassword",
data) &&
this.InputValidate(this.password, passwordnew, "doChangepassword", data)) {



try {
//code......................

}
catch (Exception e) {}
/*----------out put the Error------------*/
this.getResult(bindingvalue, "doChangepassword", data);
}

}
...全文
870 46 打赏 收藏 转发到动态 举报
写回复
用AI写文章
46 条回复
切换为时间正序
请发表友善的回复…
发表回复
feilong816 2004-07-05
  • 打赏
  • 举报
 回复
没有,首先是那个InputValidate方法,希望有人提个好建议
另外我觉得规范方面没有做好
当然也想让人探讨一下,我从一开始就说这个代码不规范,我并没有说我的代码好.
如果说的话我当然会同意,如果说得不好,又没说出什么,我就没有必要说好
因为我也是刚毕业不久的,了解我们学校的一些情况,在学校里谁会注意这些,又多少人在呼?


I11 2004-07-05
  • 打赏
  • 举报
 回复
记得看过一个故事，说一个画家，在别人办画展的时候把自己的一副画也加在里面，希望得到别人的意见，同时附了一段话和一支笔在旁边，想让每一个看过这幅画的人标出他们认为不足的地方，结果他发现他的画上几乎全部都被标出了各种各样的问题。第二次，他把同一幅画挂出来，但是他换了一个方式，希望看过的人标出他们认为画得不错的地方，结果他发现同样的，他的画上几乎全部都被标上了赞誉。
不知道楼主打算怎么样。是希望通过这种方式得到满足感，以便在公司的大牛面前说话底气足一点还是想真正知道有没有什么不足和差距。
I11 2004-07-05
  • 打赏
  • 举报
 回复
看出来了，楼主是打算把大家说的全部否定，然后让大家一致通过这段代码精彩绝伦，那我捧场好了。
鼓掌...
feilong816 2004-07-05
  • 打赏
  • 举报
 回复
zhang21cnboy(事了抚衣去，不留身与名)
晕,apache的代码就只是这样而已???那谁还敢用tomcat更不用提turbine
feilong816 2004-07-05
  • 打赏
  • 举报
 回复
谢谢I11(雪儿) ,下回会注意

zhang21cnboy 2004-07-05
  • 打赏
  • 举报
 回复
*@param binding connection
*@author David
*Company: blueskytech


原来楼住时外国人阿。。。哥们不要把人家apache turbine的软件拿过来载着列卖弄行不行？
gernett21 2004-07-05
  • 打赏
  • 举报
 回复
注释还是少，一个好的程序注释应该至少占三分之一的我认为，还有人认为应该更多
I11 2004-07-05
  • 打赏
  • 举报
 回复
1. public class SQL
extends SecureAction {
写成
public class SQL extends SecureAction {
原因：不需要分行
参考：http://java.sun.com/docs/codeconv/html/CodeConventions.doc5.html#381

2. String displayname...
写成
String displayName...
原因：变量名采用（仅）首字母大写，其他变量名以此类推
参考：http://java.sun.com/docs/codeconv/html/CodeConventions.doc8.html#367

3. if (resultvalue == -3000) {
写成：
public static final int SUCCESS = -3000
if (resultvalue == SUCCESS) {
原因：增加代码可读性
参考：http://java.sun.com/docs/codeconv/html/CodeConventions.doc9.html#1255

其他诸如一些代码的习惯性处理方面就先不提了。既然楼主“希望大家给我品一下这个代码的代码规范”，那么有关于turbine和InputValidate就不用在这里说了，还是另开帖子吧。
再有，代码中的注释用的也不是很好，可以参考http://java.sun.com/docs/codeconv/html/CodeConventions.doc4.html#385。
Jet法拉利 2004-07-03
  • 打赏
  • 举报
 回复
还是不错的，可能是贴出来的时候，格式发生了变化，支持搂住
alienbat 2004-07-03
  • 打赏
  • 举报
 回复
对代码的评价：

非常烂。命名不规范，注释不整齐，编码无序而混乱，异常处理这块简直就是瞎搞。总之，一无是处。

这么烂的玩意也敢拿出来丢人现眼，服了你了。
duye 2004-07-03
  • 打赏
  • 举报
 回复
尽量不要在程序中特别是程序语句的后面用//来注释.看了不舒服.
知足常乐 2004-07-03
  • 打赏
  • 举报
 回复
md，这种垃圾也发！！！！！










BeiAn 2004-07-03
  • 打赏
  • 举报
 回复
很佩服楼主啊！能写出那么长的代码。我真的很怕啊！因为我以后也想做写代码方面的工作，但是现在还没有写过什么程序啊！
楼主，是不是到工作后多写就习惯了呢？你在上学时是不是也很回写了呢？
jerrykey 2004-07-02
  • 打赏
  • 举报
 回复
这个……晕了。
mbl 2004-07-02
  • 打赏
  • 举报
 回复
很好
feilong816 2004-07-02
  • 打赏
  • 举报
 回复
看代码不需要一行一行的看,只要先把握主要功能,总体思路,其他的就都一样了,需要什么再看什么了.否则的话这么长的代码发上去是浪费，不会有人看的．
feilong816 2004-07-02
  • 打赏
  • 举报
 回复
这不是主要问题,请不要发些小问题,这里估计没有一个人把代码从头看到尾,或者了解大概,或者说总体有所了解,如果有人不会看代码,那我就更不好说了,谢谢
　　这个软件是面向香港的,我英文也不好,只是JbuilderX不好调,我就不理了,全部写成英文.
　　另外这也涉及到另一方面,Turbine技术还很新,估计没有多少中文文档,如果有也是不全的,没有一定的英文,想维护也难.
wubai250 2004-07-02
  • 打赏
  • 举报
 回复

呵呵,注释最好用中文

不要以为自己英文多么好!

如果真有一个人对你说,我看不懂你的英文注释,你应该怎么对他说

难道你真的会对他说:你的英语怎么这么差啊,连这个都看不懂

也许最后这个人会向上级反映,再最后,你不得不把所有英文注释改成中文注释

所以麻烦的还是你自己!呵呵!

呵呵,写好的程序不是自己来欣赏的,是给大家用的!



eaglecoody 2004-07-02
  • 打赏
  • 举报
 回复
够规范，不过最好再加点注释...............:)
flydreams 2004-07-02
  • 打赏
  • 举报
 回复
“代码够本身够清楚的话，不用注解”…… @_@
加载更多回复（26）
【guiplan2.0教程】可视化布局一键生成web前端代码
前端可视化软件guiplan教程，无论是懂前端还是不懂前端，都可以学会基本... 支持手写代码与自动生成代码混合编写开发，比如添加过滤器filters,监听器watch等。 灵活便捷对程序员有好，是一款真正适合程序员开发的软件。
使用java关键字编写代码
使用java关键字编写代码 java的关键字 java的基本数据类型 Java是一种强类型语言，必须为每一个变量声明一种类型。Java共包含8中基本类型，其中4种整型、2种浮点型、1种用于表示Unicode编码的字符单元的字符类型char和1种用于表示真值的boolean类型。 Java包含4种整型: int:4字节 short:2字节 long:8字节 byte:1字节 长整...
软件开发标准（文档模板）
操作手册（GB8567——88）1引言1.1编写目的说明编写这份操作手册的目的，指出预期的读者。1.2前景说明：a．  这份操作手册所描述的软件系统的名称；b．  该软件项目的任务提出者、开发者、用户（或首批用户）及安装该软件的计算中心。1.3定义列出本文件中用到的专门术语的定义和外文首字母组词的原词组。1.4参考资料列出有用的参考资料，如：a． 
软件文档编写向导
综述： 在程序员的日常工作中，除了编写代码之外，还免不了需要编写各种技术文档。一个编写良好的技术文档在项目中能够很好地建立沟通与协作，起到很积极的作用。因此，编写技术文档也就成为了程序员技能提升的很重要的一面。 为此，我们特意收集了一些在项目开发过程中经常用到的文档模板，这些模板包括格式和简单的写作说明，相信能够帮助大家编写出更加高效、实用的技术文档。在收集过程中，我们十分注重其实用性
以太坊智能合约开发-《精通以太坊智能合约开发》学习总结实践
文章目录初探以太访智能合约remix小demo以太坊核心概念交易／事务（ Transaction )区块参考 初探以太访智能合约 remix小demo https://remix.ethereum.org 目前尝试 Solidity 编程的最好方式是使用 Remixo Remix 是一个基于浏览器的 Solidity IDE ， 它集成了 Solidity 编译器、运行环境 ，以及调试和发布工具。 使用浏览器打开网址 https://remix.ethereum.org 官方demo // SPDX-Lic
Web 开发

81,092

社区成员

341,716

社区内容

发帖
与我相关
我的任务
社区描述
Java Web 开发
社区管理员
  • Web 开发社区
加入社区
  • 近7日
  • 近30日
  • 至今

加载中

查看更多榜单
社区公告
暂无公告

试试用AI创作助手写篇文章吧

+ 用AI写文章