Return-Path: Delivered-To: apmail-jakarta-commons-dev-archive@www.apache.org Received: (qmail 65965 invoked from network); 9 Oct 2006 23:24:05 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (209.237.227.199) by minotaur.apache.org with SMTP; 9 Oct 2006 23:24:05 -0000 Received: (qmail 90710 invoked by uid 500); 9 Oct 2006 23:23:58 -0000 Delivered-To: apmail-jakarta-commons-dev-archive@jakarta.apache.org Received: (qmail 90628 invoked by uid 500); 9 Oct 2006 23:23:58 -0000 Mailing-List: contact commons-dev-help@jakarta.apache.org; run by ezmlm Precedence: bulk List-Unsubscribe: List-Help: List-Post: List-Id: "Jakarta Commons Developers List" Reply-To: "Jakarta Commons Developers List" Delivered-To: mailing list commons-dev@jakarta.apache.org Received: (qmail 90615 invoked by uid 99); 9 Oct 2006 23:23:58 -0000 Received: from asf.osuosl.org (HELO asf.osuosl.org) (140.211.166.49) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 09 Oct 2006 16:23:58 -0700 X-ASF-Spam-Status: No, hits=0.0 required=10.0 tests= X-Spam-Check-By: apache.org Received: from [209.237.227.198] (HELO brutus.apache.org) (209.237.227.198) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 09 Oct 2006 16:23:57 -0700 Received: from brutus (localhost [127.0.0.1]) by brutus.apache.org (Postfix) with ESMTP id C2AF67142D6 for ; Mon, 9 Oct 2006 16:23:19 -0700 (PDT) Message-ID: <33126080.1160436199794.JavaMail.root@brutus> Date: Mon, 9 Oct 2006 16:23:19 -0700 (PDT) From: "Aldrin Leal (JIRA)" To: commons-dev@jakarta.apache.org Subject: [jira] Created: (LANG-283) Visitors for Builders MIME-Version: 1.0 Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-Virus-Checked: Checked by ClamAV on apache.org X-Spam-Rating: minotaur.apache.org 1.6.2 0/1000/N Visitors for Builders --------------------- Key: LANG-283 URL: http://issues.apache.org/jira/browse/LANG-283 Project: Commons Lang Issue Type: New Feature Affects Versions: 2.2 Environment: Any Reporter: Aldrin Leal Priority: Trivial While doing commons-lang stuff, I found the repetitive nature of Builder stuff. Therefore, i decided to: a) Create a SingleBuilder (for ToString / HashCode) and a DualBuilder (CompareTo / Equals) interface, and make their classes implementing; b) Create a SingleBuilderVisitor and DualBuilderVisitor; c) Do the proper wiring, while creating new constructors using the visitors; d) Create new tests and ensuring javadoc plays nice along; There's a patch attached in this e-mail, done against the latest svn STABLE url for 2.2 (http://svn.apache.org/repos/asf/jakarta/commons/proper/lang/branches/LANG_2_2_X/) There might be errors, but methinks it's a great idea to declare a constant like this in your source code. private static final EQUALS_VISITOR = new DualBuilderVisitor() { public void visit(DualBuilder builder, Object lhs, Object rhs) { Type lhsObj = (Type) lhs; Type rhsObj = (Type) rhs; builder.append(lhsObj.getId(), rhsObj.getId()).append(lhsObj.getName(), rhsObj.getName()); } }; And then, declare and instantiate builders like as, say: public boolean equals(Object anotherObj) { return new EqualsVisitor(EQUALS_VISITOR, this, anotherObj); } Please note this implementation, overrall, also lets you easily use a single visitor for either equals and compareto, thus creating a single point of management for all your field-maintenance-in-value-objects-needs. -- This message is automatically generated by JIRA. - If you think it was sent incorrectly contact one of the administrators: http://issues.apache.org/jira/secure/Administrators.jspa - For more information on JIRA, see: http://www.atlassian.com/software/jira --------------------------------------------------------------------- To unsubscribe, e-mail: commons-dev-unsubscribe@jakarta.apache.org For additional commands, e-mail: commons-dev-help@jakarta.apache.org