Revision: 1058
Author: ffaber
Date: Fri Aug  7 10:19:47 2009
Log: responding to helpful comments pointing out typos.
http://code.google.com/p/google-guice/source/detail?r=1058

Modified:
  /wiki/KeepConstructorsHidden.wiki

=======================================
--- /wiki/KeepConstructorsHidden.wiki   Thu Aug  6 10:20:35 2009
+++ /wiki/KeepConstructorsHidden.wiki   Fri Aug  7 10:19:47 2009
@@ -11,7 +11,7 @@

  It's a common reflex to implement this interface with a public class:
  {{{
-public class DatabaseDataReader {
+public class DatabaseDataReader implements DataReader {

     private final ConnectionManager connectionManager;

@@ -40,7 +40,7 @@

  Perhaps worst of all, any direct use of a constructor circumvents Guice's  
object instantiation.

-As a correction, simple limit the visibility of both your implementation  
classes, and their constructors.  Typically package private is preferred  
for both, as this facilitates:
+As a correction, simply limit the visibility of both your implementation  
classes, and their constructors.  Typically package private is preferred  
for both, as this facilitates:

     * binding the class within a `Module` in the same package
     * unit testing the class through means of direct instantiation

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"google-guice-dev" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/google-guice-dev?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to