Zenerid hat geschrieben:Deswegen hätte ich mal ein paar kleine Fragen.
Ich bin weder C# Junkie (obwohl offiziell C#-Entwickler ^^) und dass ich das letzte Mal C# mit Datenbanken nutzte ist ziemlich genau 10 Jahre her. ^^
Darum war ich gestern auch etwas zurückhaltender mit der Antwort, falls jemand kompetenteres sich äußern mag.
Zenerid hat geschrieben:1. Ich habe eine einzige Klasse, die die Datenbankoperationen darstellt, also das Hinzufügen von Kunden, Löschen, Bearbeiten usw., vermutlich könnte man das ganze noch besser aufteilen aber wie? Derzeit ist halt wie gesagt alles in dieser einen Klasse.
Ich würde eine Klasse nehmen, die die Datenbank-Transaktionen kapselt und deren Instanz Du entweder Deine Datenobjekte übergibst oder die Du an Deine Datenobjekte übergibst, wohin sie sich schreiben oder initialisieren können.
z.B.
Im umgekehrten Fall kannst Du eventuell mit Reflection arbeiten und Funktionalität abstrahieren, aber wäre dann eher was für fortgeschrittene Entwickler.
Zenerid hat geschrieben:2. Ich habe in dieser einen Klasse, die übrigens statisch ist eine Verbindung, die überall genutzt wird und zudem ein paar andere. Ist es empfehlenswert eine Verbindung in der Klasse zu haben, die überall genutzt wird? Die meisten Methoden in der Klasse sind ebenfalls static.
Das sieht so aus:
Code: Alles auswählen
private static MySqlConnection sql_connection = new MySqlConnection();
private static MySqlCommand sql_command = new MySqlCommand();
private static MySqlDataReader sql_reader;
static Database()
{
sql_connection.ConnectionString = Login();
sql_command.Connection = sql_connection;
}
Warum machst Du die Sachen statisch?
Du nimmst Dir damit doch die Möglichkeit flexibler mit mehreren Datenbanken zu arbeiten!?
Zenerid hat geschrieben:Jetzt ist muss man natürlich auch immer sql_connection.Open() machen. Könnte man die Offene Verbindung auch irgendwie übergeben oder anderweitig irgendwie das ganze kürzer fassen?
So sieht dann z.B eine der Methoden aus:
Code: Alles auswählen
public static bool DeleteAllData()
{
sql_command.CommandText = "Delete from Kunde;";
try
{
using (sql_connection)
{
sql_connection.Open();
sql_command.ExecuteNonQuery();
}
}
catch (MySqlException mysqlex)
{
// Behandlung
return false;
}
return true;
}
Wenn ich mich recht entsinne muss man die Verbindung nur einmal öffnen. Aber das programmiere ich auch nicht täglich auf's Neue, da müsste ich nachsehen.
Zenerid hat geschrieben:3. Zudem habe ich auch keine interne Repräsentation eines Kunden in meinem Projekt, sondern nur in der DB eine Tabelle für den Kunden. Brauche ich eine eigene Klasse dafür in meinem Projekt, wenn ja wie könnte diese aussehen? Wäre das einfach eine eigene Klasse?
Das sollte eine eigene Klasse sein, die Du zum Beispiel von einer Klasse 'PrimaryKey' ableitest, so dass jeder Kunde weiß, wo er in der Datenbank steht oder dass er noch eingefügt werden muss (bei einem ungültigen PrimaryKey)
Zenerid hat geschrieben:4. Ich habe auch eine eigene Klasse geschrieben, die es ermöglicht die Daten aus der Datenbank in ein XML-File zu exportieren und zu importieren und zudem befindet sich dort auch noch eine Methode, die die Fehler protokolliert (Exceptions) und diese eine Datei schreibt. Derzeit handelt es sich dabei um ein txt.-File. Ist das so üblich?
Ja, das ist jedenfalls nicht unüblich, Fehler in ein .txt-File zu schreiben oder eben an std::cerr zu melden, wenn es sich um ein kleiens Tool handelt. Dann kann der Nutzer flexibler mit den Ausgaben umgehen.
Zenerid hat geschrieben:Ich würde mir auch wahnsinning gerne mal irgendwo fertige Projekte angucken, allerdings sind viele dieser Open-Source Projekte so gigantisch und auch so komplex, das ich fast nichts verstehe.
Hat hier evtl. jemand eine Empfehlung für ein kleineres Projekt, wo man sich auch mal einen guten Stil angucken kann oder eine Seite, wo es solche Projekte gibt? Github und Codeplex etc. kenne ich schon aber wie kann man da auch mal kleienr Projekte finden, die aber trotzdem gut umgesetzt sind?
Diese müssen auch nicht unbedingt eine Datenbankanwendung sein.
![Wink ;)](./images/smilies/icon_e_wink.gif)
Nur ein guter Stil ist mir wichtig, wobei sowas ja auch teilweise Ansichstsache ist aber es gibt ja auch sehr schlecht gibt und zumindest sowas wie "Akzeptabel".
Heutzutage ist alles, was eine erwähnenswerte Funktionalität besitzt bereits 'groß'.
Weiterhin ändert sich die Bedeutung von Codequalität, wenn Projekte groß werden - bei kleinen Projekten spielt sie in der Regel keine Rolle.
Statt Projekte empfehle ich Dir Styleguides zu suchen und Dir anzusehen. Du wirst darin viele gute Tipps finden, die meist auf den Erfahrungen professioneller Entwickler beruhen. Und Du wirst feststellen, dass in vielen widersprüchliches steht und gerade am Anfang wirst Du feststellen, dass manche Styleguides viel zu viel regeln. Das sind die Styleguides aus denen Du Dich bedienen solltest.
Auch wenn Dir die Frage ob es
oder
heißt für Dich erstmal unwichtig ist, da sind bereits drei Fragen drin, über die sich erfahrene Entwickler lange und ausführlich streiten können... und auch schon seit vielen Dekaden tun, ohne zu einem Ergebnis zu kommen.
Da es den einen guten Stil nicht gibt, musst Du Dir selbst einen Kopf machen, was für Dich gut lesbar ist und funktioniert.
Merke: Wer Ordnung hellt ist nicht zwangsläufig eine Leuchte.
Ich beantworte keine generellen Programmierfragen per PN oder Mail. Dafür ist das Forum da.