W języku Java, czy mogę skonsolidować dwie podobne funkcje, w których używa JspWriter i innych PrintWriter?

Question

Mam następujące klasy, która jak widać ma raczej dość nadmiarową metodę formatNameAndAddress:

package hu.flux.helper; 
import java.io.PrintWriter; 

import javax.servlet.jsp.JspWriter; 

// A holder for formatting data 
public class NameAndAddress 
{ 
public String firstName; 
public String middleName; 
public String lastName; 
public String address1; 
public String address2; 
public String city; 
public String state; 
public String zip; 

// Print out the name and address. 
public void formatNameAndAddress(JspWriter out) 
    throws java.io.IOException 
    { 
    out.println("<PRE>"); 
    out.print(firstName); 

    // Print the middle name only if it contains data. 
    if ((middleName != null) && (middleName.length() > 0)) 
    {out.print(" " + middleName);} 

    out.println(" " + lastName); 

    out.println(" " + address1); 

    if ((address2 != null) && (address2.length() > 0)) 
    out.println(" " + address2); 

    out.println(city + ", " + state + " " + zip); 
    out.println("</PRE>"); 
    } 

public void formatName(PrintWriter out) 
{ 
    out.println("<PRE>"); 
    out.print(firstName); 

    // Print the middle name only if it contains data. 
    if ((middleName != null) && (middleName.length() > 0)) 
    {out.print(" " + middleName);} 

    out.println(" " + lastName); 

    out.println(" " + address1); 

    if ((address2 != null) && (address2.length() > 0)) 
    out.println(" " + address2); 

    out.println(city + ", " + state + " " + zip); 
    out.println("</PRE>"); 
} 
} 

Chciałbym przepisać klasę użyć metody rodzajowe jak:

 // Print out the name and address. 
private void genericFormatNameAndAddress(Object out) 
{ 
    out.println("<PRE>"); 
    out.print(firstName); 

    // Print the middle name only if it contains data. 
    if ((middleName != null) && (middleName.length() > 0)) 
    {out.print(" " + middleName);} 

    out.println(" " + lastName); 

    out.println(" " + address1); 

    if ((address2 != null) && (address2.length() > 0)) 
    out.println(" " + address2); 

    out.println(city + ", " + state + " " + zip); 
    out.println("</PRE>"); 
} 

Ale nie mogę tego zrobić dokładnie tak, ponieważ Object nie ma metod print() i println(). Jeśli wyrzucę dane wyjściowe do JspWriter lub PrintWriter, czasami będę je rzucał w niewłaściwy sposób.

Wyobrażam sobie, co muszę zrobić, to przekazać typ obiektu jako zmienną, a następnie użyć zmiennej do określenia sposobu rzutowania. czy to możliwe? Jeśli tak to jak? Jeśli nie, jakie byłoby dobre rozwiązanie?

Answer 1

to prawdopodobnie będzie działać:

public void formatNameAndAddress(JspWriter out) throws java.io.IOException { 
    formatNameAndAddress(new PrintWriter(out)); 
} 

Answer 2

Ty niby muddling dwa różne zadania z tych metod, a łamiąc zasadę OO specjalizacji. Oznacza to, że masz metody, które są odpowiedzialne za formatowanie jednego z dwóch typów łańcuchów ... ORAZ odpowiedzialne za przesłanie ich do jednego z dwóch typów docelowych wyników.

Lepszym podejściem może być uelastycznienie metod. Oznacza to, że TYLKO będą odpowiedzialni za budowanie łańcucha "Nazwa" lub "Nazwa i Adres" i zwracanie String jako typu zwrotu metod.

W punkcie kodu, w którym wywołuje się te metody, oczywiście masz już obiekt JspWriter lub PrintWriter ... ponieważ teraz przekazujesz go jako argument metody. Tak więc byłoby czysto po prostu zostawić ten obiekt w kodzie i wydrukować go, co jest zwracane przez wyspecjalizowaną metodę agnostyczną.

Answer 3

Jeśli chcesz oddać obiekt do prawidłowego Writer można spróbować coś takiego:

private void genericFormatNameAndAddress(Object out){ 
    if (obj instanceof Printwriter){ 
     //cast to printwriter 
    } else { 
     //cast to JspWriter 
    } 

Barkers rozwiązanie wydaje się być lepiej przystosowane ponieważ PrintWriter może być zbudowana z pomocą JspWriter.

Answer 4

Zarówno JspWriter, jak i PrintWriter są podklasami java.io.Writer. Ponieważ nie używać żadnych funkcji, które są specyficzne dla żadnej z tych dwóch, można zadeklarować jako metodę robienia java.io.Writer:

public void formatNameAndAddress(Writer out) throws java.io.IOException 
{ 
    [...] 

Następnie przekazać JspWriter lub PrintWriter w miarę potrzeb.

Edit: To nie będzie działać właściwie bez modyfikowania kodu, ponieważ, jak podkreślił innymi Writer nie posiada print i println metod, natomiast JspWriter i PrintWriter zarówno zapewnić im.