Classe Design Question

Question

Ceci est une question super débutant, j'ai été la programmation depuis un certain temps, mais je viens d'apprendre OOP. J'ai une classe qui fonctionne avec l'entrée de l'utilisateur via la console C#.
Il existe différentes méthodes dans cette classe pour rassembler différents ensembles d'entrées.

J'ai une autre classe qui prend ces ensembles d'entrée et les met dans une base de données.
Quelle est la meilleure façon de transmettre l'entrée de ma classe d'entrée à ma classe d'insertion de base de données?

Je dirais:

Y at-il quelque chose de mal avec cela ou est-il une meilleure façon de le faire? Devrais-je même avoir deux classes (La classe de base de données pourrait également prendre l'entrée de l'utilisateur)?

Answer 1

Vous devriez avoir une classe "data", qui représente tous vos paramètres.

Votre classe GetParameters doit créer une instance de cette classe.

Votre classe InsertDatabase doit accepter une instance de cette classe.

public class Data 
{ 
    public string value1 {get;set;} 
    // add more properties here 
} 

public class GetInputParameters 
{ 
    public Data GetParameters() 
    { 
     var d = new Data(); 
     d.value1 = Console.ReadLine(); 
     return d; 
    } 
} 


public class InsertToDatabase 
{ 
    public void InsertRecord(Data value) 
    { 
     // database persistance code 
    } 
} 

De plus, vous pouvez utiliser une liste générique pour passer plus d'une fois instance de la classe de données, vous pouvez utiliser un tableau, mais une liste générique est beaucoup plus facile de travailler avec.

Answer 2

Le type de sécurité est le premier problème que je vois avec ceci. Une meilleure approche consisterait à envelopper votre base de données à l'aide de LINQ to SQL, puis à simplement passer l'objet métier dans une méthode Save() et Delete(). De cette façon, la mise en œuvre réelle de la base de données pourrait théoriquement être remplacée, mais vos objets de gestion seraient certainement utiles à long terme.

Answer 3

Au moins deux classes est certainement une bonne idée. Vous voulez essayer d'encapsuler des fonctionnalités dans une classe. Dans une application de console standard, je suggère de créer une classe pour les E/S de console, une classe pour l'accès à la base de données et une classe qui leur permettra de se parler et éventuellement de manipuler les données (c'est-à-dire une classe de service). Ainsi, votre classe d'E/S de console peut attendre des données, puis appeler votre classe de service pour enregistrer les données, et votre service appelle alors votre base de données pour enregistrer les données.

Answer 4

En général, je pense que la séparation de votre code en différentes couches est une bonne idée. À l'heure actuelle, vous avez votre couche d'interface utilisateur (celle qui fonctionne avec l'entrée de la console) et votre couche de données (celle qui insère des données). C'est un bon début.

Quel type de données collectez-vous et insérez-vous? Ce pourrait être un bon candidat pour une autre classe. Disons que c'est l'information de l'utilisateur, et un utilisateur entre son nom, son âge, son sexe, etc. Tout ce que vous collectionnez peut probablement être emballé dans un objet. Vous pouvez ensuite simplement passer cet objet à votre classe Data. La classe de données peut ensuite digérer cette information comme elle le doit.

Dans votre classe d'entrée:

User user = new User(); 
//get all user info from console, assigning it to your user object 
DatabaseInsertClass.InsertIntoDatabase1(user);