Ai-je le bon sens de la méthode statique

Question

J'ai classe Pays qui a des collections de villes.

Au client j'utiliser webmethod

[WebMethod] 
public void AddCity(string countryCode,string name) 
{ 
MyFacade.AddCity(countryCode,name); 
} 

à Façade Je méthode

public void AddCity(string countryCode,string name) 
{ 
Country.AddCity(countryCode,name); <-in this method is simple sql operation 
} 

et le noyau de ma question:

public class Country 
{ 
public static void AddCity(string countryCode, string cityName) 
{ 
//insert into table cities new city 
} 
} 

Il est ok? Ou je dois créer objectCountry, et il y a une méthode non statique AddCity?

Et une autre question:

Meilleure utilisation:

City[] cities= Country.GetAllCities(countryCode) 

ou

City[] cities= new Country(countryCode).GetAllCities() 

Answer 1

ACCEPTEZ à la fois countryCode et cityName comme paramètres est très bien pour la couche d'accès aux données, mais je ne voir toutes les méthodes qui devraient être statiques.

Plutôt AddCity devrait être un membre non-statique de DataConnection ou une telle, de sorte que vous pouvez facilement le railler, remplacer la base de données, etc. sans changer l'interface d'appel.

Answer 2

Voulez-vous être en mesure de tester votre code en utilisant un cadre de simulation?

Miser sur la réponse de Ben, remplacer la façade avec une interface:

[WebMethod] 
public void AddCity(string countryCode, string name) 
{ 
    ICountryDataAccess dao = GetDAOFromDI(); // basically get a DI framework to manage this object instance. 
    dao.AddCity(countryCode, name); 
} 

public interface ICountryDataAccess 
{ 
    void AddCity(string countryCode, string name); 
    ICollection<City> GetAllCities(string countryCode); 
    // OR ! 
    Country Retrieve(string countryCode); 
    // using an ORM or something Country then as a list of cities 
} 

public Country 
{ 
    public virtual string CountryCode {get;set;} 
    public virtual ICollection<City> Cities {get; protected set;} 
}