Benutzereinstellungen sinnvoll speichern

sonne75 · 23. April 2014, 16:56

fichz schrieb:

Nur weil sie kompliziert aussieht, heißt es ja nicht automatisch, dass ich diese nicht verwenden will.

Dann habe ich dich falsch verstanden.

Man kann sich nicht immer (im Berufsleben) aussuchen, welche Treiber man benutzt...

fichz · 23. April 2014, 17:01

sonne75 schrieb:

Man kann sich nicht immer (im Berufsleben) aussuchen, welche Treiber man benutzt...

Wem sagst du das...

Um zu der Klasse von @ErfinderDesRades zurückzukommen:
Ich verstehe das ganze im Prinzip. Hab diese auch nun in einem Testprojekt ohne Probleme zum Laufen gebracht.
Was ich nicht verstehe, wie eine einfache IList(of String) weiß welcher Einstellung welchem Control zugeordnet ist. Ich denke wenn ich diesen Funken verstanden habe, könnt ich dies für mein Projekt verwenden.

lg

ErfinderDesRades · 23. April 2014, 18:01

hier eine bisserl verbesserte version (vbCodeProvider ist oversized, die TypeConverter rufe ich besser von TypeDescriptor ab) - mit Erklärungen: ComplexConverter auf VBP

Was du nicht verstehst ist grad der Witz an der Sache: Lese-Code und Schreib-Code ist derselbe. Wenn du also erst Button1.Text schreibst, und danach dann Checkbox2.Checked, dann wird auch in genau dieser Reihenfolge gelesen. Geht nicht anners - denn es ist derselbe Code!
Die dumme IList(Of String) weiß es tatsächlich nicht, es ist der EventhandlerCode selbst, der es weiß.

us4711 · 23. April 2014, 18:36

Guck mal, da hat Dir @ErfinderDesRades was Schönes gebastelt:

ComplexConverter: alles in einen String und zurück

sonne75 · 24. April 2014, 08:10

Ich habe jetzt das Ganze angeschaut, aber es greift doch auf My.Settings zurück und hat mit Vorgaben von @fichz (eine DB-Tabelle mit Spalten) nichts zu tun? Oder ist damit gemeint, dass dieser String einfach statt in My.Settings einfach in eine Tabellenspalte geschrieben wird?
Aber wird damit nicht geschummelt, wenn die Aufgabenstellung für @fichz eine DB-Lösung war?

ErfinderDesRades · 24. April 2014, 08:20

Wie gesagt: Der ComplexConverter persistiert in einen String. Im Tut wird dieser String in die Settings gepackt, aber wenn du ihn lieber inne DB hast

ErfinderDesRades schrieb:

Den String schreib in die DB, wenns beliebt.

sonne75 · 24. April 2014, 08:22

Wie gesagt, dafür müsste ich die Aufgabenstellung von ihm kennen, und wie genau sie eingehalten werden muss. Denn mit einer DB-Tabelle hat das Ganze ja recht wenig zu tun (außer, dass der String da reingeschrieben wird).
Muss er entscheiden, ob es als Lösung gilt.

ErfinderDesRades · 24. April 2014, 08:32

fichz schrieb:

ich habe eine eher generelle Frage zum Speichern von Benutzereinstellungen. Ich will diese in einer SQL-Server Datenbank hinterlegen.

also da nehme ich an, die DB beschäftigt sich vorwiegend mit anneren Sachen, aber es gibt auch eine Tabelle User, und wenn man den User um eine Spalte "GuiSettings" (String) erweitert, dann kriegt jeder User sein Form in genau dem Zustand hingeflackt, in dem er es verlassen hat (hab ich grad letzte Woche so verbaut).

fichz · 24. April 2014, 08:53

Wenn ich diese Klasse nun richtig verstanden habe, dann muss jedoch die Reihenfolge des Strings immer gleich bleiben oder?

Ich werde mir diese Methode anhand eines Testprojekts zu Gemüte führen, hab jedoch nun auch eine andere Idee wie es eventuell per XML gehen könnte. Werde beides mal versuchen und anschließend berichten.

lg

ErfinderDesRades · 24. April 2014, 09:03

du kannst es nicht fassen, was?

Die Reihenfolge bleibt immer gleich, das ist unausweichlich. Es ist derselbe Code, der liest und schreibt. Wie soll sich da jemals die Reihenfolge ändern können?
Die Reihenfolge kann sich nur ändern, wenn du den lese/schreib - Code zwischenzeitlich änderst, etwa statt des Treeviews nun ein Datagridview persistierst.
Also das Programm neu kompiliert hast, mit Änderungen am Persistenz-Code.
Und auch das hat keine annere Auswirkung, als dass dann - dank des Rollbacks - wieder bei den Default-Werten angefangen wird.

fichz · 24. April 2014, 09:09

Die Reihenfolge bleibt dann gleich wenn ich sie gleiche belasse ja

Ich denke hier eher an "Die eine Checkbox wird nicht mehr gebraucht -> weg damit" Im "Gesamtstring" ist der "True" Wert jedoch an dritter Stelle gewesen.
Somit könnte ich den gespeicherten String nicht wieder herstellen. Wenn ich, wie gesagt, das korrekt verstanden habe.

Ok jetzt weiß ich was du meinst. Da das Backup (was vorher passiert) erstellt wird und im Fehlerfall dann eingetragen wird ist es egal wenn sich sowas in der Art ändert.
Beim nächsten speichern wird der String wieder korrekt geschrieben und kann beim nächsten Start dann wieder korrekt restauriert werden.

lg

ErfinderDesRades · 24. April 2014, 09:12

so isses

fichz · 24. April 2014, 12:05

So ich mach es nun doch nicht mit dem ComplexConverter sondern habe mir nun was ähnliches gebastelt.
Das ganze läuft nun über XML und Serialisierung (Hinweis war von @ThuCommix -> ComplexConverter: alles in einen String und zurück).

Ich erstelle mir nun für jede Form eine eigene Settingsklasse welche serialisierbar ist. Dieser geb ich beim Schließen der Form die Einstellungen welche halt für diese Form so passen.
Die Klasse wird dann mit XML serialisiert und der ganze XML-String was dabei zustande kommt wird dann in die Datenbank gespeichert (ID, BedienerID, Formname, Daten).

Beim erneuten Start wird versucht diese Klasse wiederherzustellen und die Settings dann wieder zu setzen.

Warum mach ich das nun so?

Das hat eigentlich mehrere Gründe:

Irgendwie stoßt mir das ungut auf, dass es alles Strings sind. Hab ich das in einer Klasse hab ich gezielt die Datentypen die ich benötige
Ich hab teilweise kompliziert generierte Settings zu speichern. Als Beipsiel ein Datengrid welches Expander beinhaltet wo der "Expanded-Status" mitgesichert wird (Dieses Grid ist jedoch an keine DataSource gebunden)
Ich hab einige Tests gemacht wenn ich die Klasse verändere. Darunter Eigenschaften entfernen, Eigenschaftsnamen ändern, Datentypen ändern. Das Ergebnis: Es wird keine Exception geworfen sondern einfach die Sachen die gar nicht mehr passen ignoriert.

lg

ErfinderDesRades · 24. April 2014, 12:19

wie du wolle

nur hast du jetzt ebenfalls strings, und ein vielfaches an Code, weil du für jedes Form eine Extra-Klasse basteln musst, und recht komplizierten Code, um die Extra-Klasse zu befüllen und auszulesen und anzuwenden.
Und zur Komplexität: thuCommix tönt ja, Serialisierung sei iwie besser, oder zumindest gleichwertig (ohne es zu belegen). Aber versuch mal wirklich die im Tut gegebene Persistenz mit deinem Ansatz umzusetzen - bin ich gespannt, ob du das schaffst, und wie das dann aussieht.
Also wenn du von "kompliziert" reden willst.

fichz · 24. April 2014, 12:24

Das stimmt. Ich bin aber generell lieber der Typ der etwas mehr Code schreibt, um das ganze dann halt besser lesbar/verständlicher zu machen und man auch problemlos erweitern kann. Von dem her ist mir diese Möglichkeit lieber.
Und zum Speichern + Auslesen sind leider die Settings so verschieden, dass ich das fast nicht verallgemeinern kann. Deswegen leg ich lieber in jeder Form selbst Hand an.

Trotzdem danke für die Tipps! Nun hab ich (zumindest für mich) ein System welches sehr flexibel Benutzer-Daten speichern kann

lg

ErfinderDesRades · 24. April 2014, 12:33

fichz schrieb:

Und zum Speichern + Auslesen sind leider die Settings so verschieden, dass ich das fast nicht verallgemeinern kann.

Was ist verschieden? Bei mir ists ein String, bei sinds 4 Strings (ID, BedienerID, Formname, Daten).
Da mein Sample nur einen Bediener und nur 1 Form hat, und die ID Datenbank-Sache ist, entfallen ID, BedienerID, Formname.
Also soo verschieden finde ich das nicht.

fichz · 24. April 2014, 12:36

Nein nein. Es nicht verschieden. Sogar sehr ähnlich. Nur verwende ich halt ein XML und du einen mit Seperator getrennten String.
Gefühlsmäßig gefällt mir die XML Variante halt mehr. Wobei ich nicht sage, dass deine Methode schlecht ist

Ist rein persönlicher Natur.

lg

simpelSoft · 24. April 2014, 14:51

Hier auch noch mal zur Vollständigkeit ein XML-Ansatz zum serialisieren von Controls:

Die Klasse "ControlSerializer":

Spoiler anzeigen

C#-Quellcode

using System;
using System.Collections.Generic;
using System.Drawing;
using System.IO;
using System.Windows.Forms;
using System.Xml.Serialization;
public class ControlSerializer
{
private Control _control;
private List<MyControlProp> _props = new List<MyControlProp>();
private List<MyControlProp> Props
{
get { return _props; }
set { _props = value; }
}
public Control Control
{
get { return _control; }
set { _control = value; }
}
public ControlSerializer(Control ctrl)
{
this.Control = ctrl;
}
public void Serialize(string filePath)
{
Serialize(this.Control, filePath);
}
public void Serialize(Control ctrl, string filePath)
{
if (ctrl != null)
{
this.Props.Clear();
ReadPropes(ctrl);
XmlSerializer serializer = new XmlSerializer(typeof(List<MyControlProp>));
using (FileStream stream = new FileStream(filePath, FileMode.Create))
serializer.Serialize(stream, this.Props);
}
else
{
throw new Exception();
}
}
public void Deserialize(string filePath)
{
Deserialize(this.Control, filePath);
}
public void Deserialize(Control ctrl, string filePath)
{
if (ctrl != null)
{
XmlSerializer serializer = new XmlSerializer(typeof(List<MyControlProp>));
using (FileStream stream = new FileStream(filePath, FileMode.Open))
{
object obj = serializer.Deserialize(stream);
this.Props = (List<MyControlProp>)obj;
WritePropes(ctrl);
}
}
else
{
throw new Exception();
}
}
private void WritePropes(Control ctrl)
{
foreach (MyControlProp item in this.Props)
{
Control[] chCtrllist = ctrl.Controls.Find(item.Name, true);
if (chCtrllist.Length > 0)
item.SetControlValues(chCtrllist[0]);
}
}
private void ReadPropes(Control ctrl)
{
MyControlProp prop = new MyControlProp(ctrl);
this.Props.Add(prop);
if (ctrl.HasChildren)
{
foreach (Control item in ctrl.Controls)
ReadPropes(item);
}
}
[Serializable]
public class MyControlProp
{
private string _name;
private string _text;
private int _selectedIndex;
private bool _disbaled;
private bool _visible;
private Color _foreColor;
private Color _backColor;
private bool _checked;
private int _checkState;
public int CheckState
{
get { return _checkState; }
set { _checkState = value; }
}
public bool Checked
{
get { return _checked; }
set { _checked = value; }
}
public string Name
{
get { return _name; }
set { _name = value; }
}
public string Text
{
get { return _text; }
set { _text = value; }
}
public int SelectedIndex
{
get { return _selectedIndex; }
set { _selectedIndex = value; }
}
public bool Disbaled
{
get { return _disbaled; }
set { _disbaled = value; }
}
public bool Visible
{
get { return _visible; }
set { _visible = value; }
}
public Color ForeColor
{
get { return _foreColor; }
set { _foreColor = value; }
}
public Color BackColor
{
get { return _backColor; }
set { _backColor = value; }
}
/// <summary>
/// Parameterless ctr is required to make
/// class serializable
/// </summary>
public MyControlProp()
{
}
/// <summary>
///
/// </summary>
/// <param name="ctrl"></param>
public MyControlProp(Control ctrl)
{
GetControlValues(ctrl);
}
/// <summary>
/// Method to read control values and assign it to object
/// </summary>
/// <param name="ctrl"></param>
public void GetControlValues(Control ctrl)
{
if (ctrl == null)
throw new Exception("UnhandledException");
this.Name = ctrl.Name;
this.Text = ctrl.Text;
this.Disbaled = ctrl.Enabled;
this.Visible = ctrl.Visible;
this.ForeColor = ctrl.ForeColor;
this.BackColor = ctrl.BackColor;
if (ctrl.GetType() == typeof(ComboBox) || ctrl.GetType() == typeof(ListBox))
{
ListControl lst = (ListControl)ctrl;
this.SelectedIndex = lst.SelectedIndex;
}
else if (ctrl.GetType() == typeof(CheckBox))
{
CheckBox ch = (CheckBox)ctrl;
this.Checked = ch.Checked;
this.CheckState = (int)ch.CheckState;
}
else if (ctrl.GetType() == typeof(RadioButton))
{
RadioButton btn = (RadioButton)ctrl;
this.Checked = btn.Checked;
}
}
/// <summary>
/// Method to set values to controls
/// </summary>
/// <param name="ctrl"></param>
public void SetControlValues(Control ctrl)
{
if (ctrl == null)
throw new Exception("UnhandledException");
ctrl.Text = this.Text;
ctrl.Enabled = this.Disbaled;
ctrl.Visible = this.Visible;
ctrl.ForeColor = this.ForeColor;
ctrl.BackColor = this.BackColor;
if (ctrl.GetType() == typeof(ComboBox) || ctrl.GetType() == typeof(ListBox))
{
ListControl lst = (ListControl)ctrl;
lst.SelectedIndex = this.SelectedIndex;
}
else if (ctrl.GetType() == typeof(CheckBox))
{
CheckBox ch = (CheckBox)ctrl;
ch.Checked = this.Checked;
ch.CheckState = (CheckState)this.CheckState;
}
else if (ctrl.GetType() == typeof(RadioButton))
{
RadioButton btn = (RadioButton)ctrl;
btn.Checked = this.Checked;
}
}
}
}

Aufruf zum Speichern einer ganzen Form mit allen Controls:

C#-Quellcode

ControlSerializer serializer = new ControlSerializer(this);
serializer.Serialize(@"D:\test.xml");

Und wieder zurück:

C#-Quellcode

if (File.Exists(@"D:\test.xml"))
{
ControlSerializer serializer = new ControlSerializer(this);
serializer.Deserialize(@"D:\test.xml");
}

ErfinderDesRades · 24. April 2014, 15:57

Das sind jetzt über 250 Zeilen, aber etwa SplitterDistance eines SplitContainers, mit der der User sich grad die linke oder rechte Seite für ihn richtig eingestellt hat, kannst du damit nicht persistieren.

auch bezweifel ich, dass das die Write-Methode auch Controls findet und restauriert, die auf einem Panel liegen, oder gar in einem eingebunden komplexen UserControl - hast du das getestet?

Benutzereinstellungen sinnvoll speichern

fichz schrieb:

sonne75 schrieb:

ErfinderDesRades schrieb:

fichz schrieb:

fichz schrieb:

C#-Quellcode

C#-Quellcode

C#-Quellcode

Tags

Ähnliche Themen

5 Benutzer haben hier geschrieben