Diskuze: Přidání řádku do tabulky
V předchozím kvízu, Test znalostí C# .NET online, jsme si ověřili nabyté zkušenosti z kurzu.
Tvůrce
Zobrazeno 8 zpráv z 8.
V předchozím kvízu, Test znalostí C# .NET online, jsme si ověřili nabyté zkušenosti z kurzu.
Nechápu ty this ve tvém kódu. DataSet si přidáš k projektu jako nový item, potom na něj přetaháš z Server Exploreru tabulky. Ale opět ti radím, používej raději LINQ.
Ale já chci lokální databázi... To vše jsem samozřejmě udělal (přecijen s tím dělám již dělší dobu).
Pokud jsi dataset takto založil, tak to musí fungovat.
using System;
using System.IO;
using System.Windows.Forms;
using Necseat.DataSetForCDSTableAdapters;
namespace Necseat
{
class ClipboardParser
{
private DataSetForCDS dataSetForCDS1 = new DataSetForCDS();
private SectionFilesTableAdapter sectionFilesTableAdapter = new SectionFilesTableAdapter();
private SectionTextsTableAdapter sectionTextsTableAdapter = new SectionTextsTableAdapter();
private SectionImagesTableAdapter sectionImagesTableAdapter = new SectionImagesTableAdapter();
private string AppData = Environment.GetFolderPath(Environment.SpecialFolder.ApplicationData) + @"\ClipboardDataStorage";
public ClipboardParser()
{
var di = new DirectoryInfo(AppData);
if (!di.Exists)
di.Create();
this.dataSetForCDS1.DataSetName = "dataSet1";
this.dataSetForCDS1.SchemaSerializationMode = System.Data.SchemaSerializationMode.IncludeSchema;
this.sectionFilesTableAdapter.ClearBeforeFill = true;
this.sectionFilesTableAdapter.Fill(dataSetForCDS1.SectionFiles);
this.sectionTextsTableAdapter.ClearBeforeFill = true;
this.sectionTextsTableAdapter.Fill(dataSetForCDS1.SectionTexts);
this.sectionImagesTableAdapter.ClearBeforeFill = true;
this.sectionImagesTableAdapter.Fill(dataSetForCDS1.SectionImages);
}
public void Actualize(string toDo)
{
MessageBox.Show(AppData);
switch (Clipboard.ContainsData(DataFormats.FileDrop) ? 0 : Clipboard.ContainsText() ? 1 : Clipboard.ContainsImage() ? 2 : 3)
{
case 0:
foreach (string file in (string[])Clipboard.GetData(DataFormats.FileDrop))
{
var destination = AppData + @"\" + Path.GetFileName(file);
if (toDo == "cut")
File.Move(file, destination);
else
File.Copy(file, destination);
sectionFilesTableAdapter.Insert(destination, file, DateTime.Now);
}
this.sectionFilesTableAdapter.Fill(this.dataSetForCDS1.SectionFiles);
break;
case 1:
sectionTextsTableAdapter.Insert(Clipboard.GetText(), DateTime.Now);
sectionTextsTableAdapter.Fill(dataSetForCDS1.SectionTexts);
break;
case 2:
var ms = new MemoryStream();
var im = Clipboard.GetImage();
im.Save(ms, im.RawFormat);
sectionImagesTableAdapter.Insert(ms.ToArray(), DateTime.Now);
sectionImagesTableAdapter.Fill(dataSetForCDS1.SectionImages);
break;
case 3:
break;
}
}
}
}
Můžeš nám vysvětlit, proč dáváš proměnným taková hnusná, nicneříkající a dlouhá jména odvozená od názvu třídy? Kdo se v tom má vyznat? Vždyť přece každá proměnná má nějaký význam.
Mohlo by mi to být jedno, ale tvůj postup je proti logickému myšlení při programování. Pak se v tom zbytečně ztrácíš. Je to stejné, jako kdyby se tvůj pes jmenoval Pes.
Já se v tom ale neztrácím. A pojmenovávám co nejpřehledněji. Jak bys chtěl pojmenovat DataSet, tak by to mělo smysl? Potřebuji přece vědět, že je to DataSet, takže funkci specifikuji až dále pomocí ForCDS1. Co je na tom nelogického? Napsal jsem vlastně, že je to DataSet pro soubor ClipboardDataSet1... Dále tabulky jsem rozdělil do sekcí - tedy SectionFiles, SectionImages... No a pro adaptéry jsem za to akorát dopsal "adaptér"... A kdybych měl dopsat, že se jedná o instance, tak by to bylo ještě delší... Místo toho jsem to rozdělil zmenšením počátečního písmena - jelikož takto se proměnné pojmenovávají. A tím se v tom vyznám...
DataSet bych pojmenoval podle toho, co se v něm nachází. Není to "množina dat", to je příliš obecný pojem, ale něco jiného. Například "seznam skladeb na CD" a z toho ten název odvodit a zkrátit. Názvy privátních objektů se dělají krátké, aby se odlišily od veřejných.
Dále jsem si všiml, že tam máš velmi divně udělaný switch. Možná
jsi to převzal z jiného vlákna dnešní diskuze, ale tohle je spíš na
klasiku if ... elseif ... else. Bude to určitě
přehlednější.
Zobrazeno 8 zpráv z 8.