Vydělávej až 160.000 Kč měsíčně! Akreditované rekvalifikační kurzy s garancí práce od 0 Kč. Více informací.
Hledáme nové posily do ITnetwork týmu. Podívej se na volné pozice a přidej se do nejagilnější firmy na trhu - Více informací.
Avatar
matesax
Tvůrce
Avatar
matesax:16.9.2012 10:46

Dobrý den,
našel jsem si funkční způsob jak na to. Tedy:

tableScoreTableAdapter.Insert(textBox1.Text, int.Parse(textBox2.Text), DateTime.Now);
this.tableScoreTableAdapter.Fill(this.dataSet1.TableScore);

Toto bez problému přidá řádek do tabulky. Ovšem když jsem tento postup zopakoval v jiném programu, nefunguje.

sectionTextsTableAdapter.Insert(Clipboard.GetText(), DateTime.Now);
               this.sectionTextsTableAdapter.Fill(this.dataSetForCDS1.SectionTexts);

Jediné co mne napadá, z čeho má být odvozen dataSet? Myslel jsem, že ze souboru, ale tak to není. Mám tedy soubor dataSet, jak mám vytvořit proměnnou, kterou jeje budu ovládat? Děkuji.

Editováno 16.9.2012 10:48
 
Odpovědět
16.9.2012 10:46
Avatar
David Hartinger
Vlastník
Avatar
Odpovídá na matesax
David Hartinger:16.9.2012 15:04

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.

Editováno 16.9.2012 15:05
Nahoru Odpovědět
16.9.2012 15:04
New kid back on the block with a R.I.P
Avatar
matesax
Tvůrce
Avatar
Odpovídá na David Hartinger
matesax:16.9.2012 15:09

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).

 
Nahoru Odpovědět
16.9.2012 15:09
Avatar
David Hartinger
Vlastník
Avatar
Odpovídá na matesax
David Hartinger:16.9.2012 15:11

Pokud jsi dataset takto založil, tak to musí fungovat.

Nahoru Odpovědět
16.9.2012 15:11
New kid back on the block with a R.I.P
Avatar
matesax
Tvůrce
Avatar
Odpovídá na David Hartinger
matesax:16.9.2012 15:15
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;
            }
        }
    }
}
 
Nahoru Odpovědět
16.9.2012 15:15
Avatar
Kit
Tvůrce
Avatar
Odpovídá na matesax
Kit:16.9.2012 15:54

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.

Nahoru Odpovědět
16.9.2012 15:54
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Avatar
matesax
Tvůrce
Avatar
Odpovídá na Kit
matesax:16.9.2012 16:11

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 ClipboardData­Set1... 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...

 
Nahoru Odpovědět
16.9.2012 16:11
Avatar
Kit
Tvůrce
Avatar
Odpovídá na matesax
Kit:16.9.2012 16:43

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ší.

Nahoru Odpovědět
16.9.2012 16:43
Vlastnosti objektů by neměly být veřejné. A to ani prostřednictvím getterů/setterů.
Děláme co je v našich silách, aby byly zdejší diskuze co nejkvalitnější. Proto do nich také mohou přispívat pouze registrovaní členové. Pro zapojení do diskuze se přihlas. Pokud ještě nemáš účet, zaregistruj se, je to zdarma.

Zobrazeno 8 zpráv z 8.