Re: Creating & writing to a text file from two different forms - C# Discussion Boards

best mothod for save file database C#

4-Nov-18 6:47

As you can see from the responses here, there are many ways you could implement this; without understanding your UI goals in more detail, I think we're limited in terms of how helpful we can be.

Issues to consider in using multiple Forms:

1. shown modal ? allow user to cancel while incomplete ?
2. data in each field cleared with use, or persistent ?
3. validation ? allow empty fields ? required fields ?

My preference is to offload all serialization, or printing. functions, to a single static Class which might look something like this:

using System;
using System.IO;
using System.Text;

namespace YourNameSpace
{
    public static class WriteManager
    {
        public static string FileName = "Test";

        public static string FilePath = Environment.GetFolderPath(Environment.SpecialFolder.Desktop);

        public static string ExtensionType = ".txt";

        private static string CurrentPath;

        static WriteManager()
        {
            SetWritePath(FileName, FilePath, ExtensionType);
        }

        public static void SetWritePath(string name, string path, string extension)
        {
            FileName = name;
            FilePath = path;
            ExtensionType = extension;
            CurrentPath = Path.Combine(path, name + extension);
        }

        static StringBuilder stringBldr = new StringBuilder();

        public static void WriteFile(StringBuilder sb1, StringBuilder sb2)
        {
            stringBldr.Clear();

            stringBldr.Append(sb1.Append(sb2));

            File.WriteAllText(CurrentPath, stringBldr.ToString());
        }

        public static void ReadFile()
        {
            // you write this
        }
    }
}

In this model, the 'WriteFile method requires two 'StringBuilder instances as inputs. I'd implement a 'GetData method in each of the Forms that has the responsibility to read the fields and turn them into a ready to save 'StringBuilder. The call in the main Form that triggers saving might look like:

// in the Main Form
StringBuilder sb1 = new StringBuilder();
public StringBuilder GetData()
{
    sb1.Clear();

    sb1.AppendLine(textBox1.Text);
    sb1.AppendLine(textBox2.Text);
    sb1.AppendLine(textBox3.Text);
    sb1.AppendLine(textBox4.Text);

    return sb1;
}

private void SaveFileButton_Click(object sender, EventArgs e)
{
    WriteManager.WriteFile(this.GetData(), form2.GetData());
}

Keep in mind this is a sketch, but, the architecture is one that has proved useful foe me.

«Where is the Life we have lost in living? Where is the wisdom we have lost in knowledge? Where is the knowledge we have lost in information?» T. S. Elliot

arman02-Nov-18 4:40

arman0

2-Nov-18 4:40

hi
What is the best method to store files in the database?
Storing large files in the database takes a few seconds
now for this witch method is best to control on insert method
When the size of a file is greater, it should monitor the storage process
Which one is best for you?
backgroundworker
Async
Threading
or ?

string filePath = "";
        string connectionString = "";
        FileStream stream = new FileStream(filePath, FileMode.Open, FileAccess.Read);
        BinaryReader reader = new BinaryReader(stream);
        byte[] file = reader.ReadBytes((int)stream.Length);
        reader.Close();
        stream.Close();

        SqlCommand command;
        SqlConnection connection = new SqlConnection(connectionString);
        command = new SqlCommand("INSERT INTO FileTable (File) Values(@File)", connection);
        command.Parameters.Add("@File", SqlDbType.Binary, file.Length).Value = file;
        connection.Open();
        command.ExecuteNonQuery();

Re: best mothod for save file database C#

OriginalGriff2-Nov-18 5:16

OriginalGriff

2-Nov-18 5:16

Large files probably shouldn't be stored in a DB at all: instead, store them on an accessible file server, and store the path to the file in the DB.

The way I do it is to use a GUID file name for the actual file, store that path to that in eth DB, and store the "real" file name with that. That way, multiple users can have ABC.DOCX without them conflicting.

Sent from my Amstrad PC 1640
Never throw anything away, Griff
Bad command or file name. Bad, bad command! Sit! Stay! Staaaay...
AntiTwitter: @DalekDave is now a follower!

Re: best mothod for save file database C#

Gerry Schmitz2-Nov-18 6:06

Gerry Schmitz

2-Nov-18 6:06

Sometimes I just store the "name and location" in the database; the files get added to a separate folder on the server (e.g. receipt images); available at any time regardless of database status.

Meaningful file names are important (e.g. time stamped; invoice #; etc.).

"(I) am amazed to see myself here rather than there ... now rather than then".
― Blaise Pascal

Re: best mothod for save file database C#

Eddy Vluggen3-Nov-18 2:07

Eddy Vluggen

3-Nov-18 2:07

For Sql Server (which I assume you're using since the code references it) look in the docs;

UPDATE (Transact-SQL) | Microsoft Docs[^], under the heading "Updating Large Value Data Types".

--edit
An example of the usage can be found here[^]. Combine those and you can show a progressbar while reading/writing.

Bastard Programmer from Hell Suspicious | :suss:

If you can't read my code, try converting it here[^]
"If you just follow the bacon Eddy, wherever it leads you, then you won't have to think about politics." -- Some Bell.

modified 3-Nov-18 9:24am.

List of surrounding points

Bernhard Hiller2-Nov-18 3:16

2-Nov-18 3:16

I have a list of System.Drawing.Point, e.g. selected pixels in a bitmap. Now I'd like to get a list of points which are nearby, say less than 10 pixels (simply the sum of distances in x and y direction) away from the former points.
When the number of points is small, that's easy. But when more than 1000 pixels are selected, the performance becomes important. Can you suggest some algorithms for this task?

Currently, I use a simple method:

public static IReadOnlyList<Point> GetSurroundingCloud(this IEnumerable<Point> _points, int _maximumDistance)
{
    ISet<Point> result = new HashSet<Point>();
    foreach (Point p in _points)
    {
        for (int x = 0; x <= _maximumDistance; x++)
        {
            for (int y = 0; y + x <= _maximumDistance; y++)
            {
                result.Add(new Point(p.X - x, p.Y - y));
                result.Add(new Point(p.X - x, p.Y + y));
                result.Add(new Point(p.X + x, p.Y - y));
                result.Add(new Point(p.X + x, p.Y + y));
            }
        }
    }
    return result.ToList();
}

As you can see, there are some features which won't scale well:
- the iteration over the input list (though O(n) could be acceptable)
- the loops with maximumDistance parameter - O(n^2) with that parameter
- HashSet.Add - the larger the list gets, the more points it has to look up for preventing duplicates, so something like O(n^2) or worse.

Oh sanctissimi Wilhelmus, Theodorus, et Fredericus!

modified 5-Nov-18 3:50am.

Dar Brett2-Nov-18 5:15

Dar Brett

2-Nov-18 5:15

Trigonometry to the rescue! You can use Pythagorean theorem to calculate the hypotenuse which is basically the length from one point to another.
Hypotenuse - Wikipedia[^]

The trick to make it efficient is to avoid using the Square Root function.

Basically...

10 > sqrt(x^2 + y^2)

is a lot slower to calculate than

10^2 > x^2 + y^2

As some bonus trivia I found the Fast inverse square root - Wikipedia[^] really fascinating:

Bernhard Hiller4-Nov-18 21:44

4-Nov-18 21:44

Thanks, Dar, but that's not the thing I am looking for. For the purpose of getting that list of points surrounding a given list of points, the sum of x and y will just do for the distance.

It is the generation of the list which I think of.

Oh sanctissimi Wilhelmus, Theodorus, et Fredericus!

Gerry Schmitz2-Nov-18 5:50

Gerry Schmitz

2-Nov-18 5:50

C# System.Drawing.Point - Coding Horrors - GameDev.net

Use your own structs or lists of "primitives"; e.g. ints or longs that translate to coordinates.

"(I) am amazed to see myself here rather than there ... now rather than then".
― Blaise Pascal

BillWoodruff9-Nov-18 13:28

9-Nov-18 13:28

The problem with using 'structs in this case would be their mutability will require you to update your list with the new copy of the struct created by modifying it.

«Where is the Life we have lost in living? Where is the wisdom we have lost in knowledge? Where is the knowledge we have lost in information?» T. S. Elliot

Daniel Pfeffer4-Nov-18 23:53

Daniel Pfeffer

4-Nov-18 23:53

Analytical Geometry to the rescue!

Given point (X₀, Y₀), return all points (X, Y) such that |X - X₀| + |Y - Y₀| <= R.

For the row Y == Y₀, this is trivially all the points (X, Y₀), X₀ - R <= X <= X₀ + R.
For the rows Y == Y₀+/-1, this is trivially all the points (X, Y₀+/-1), X₀ - R + 1 <= X <= X₀ + R - 1.
For the rows Y == Y₀+/-n, this is trivially all the points (X, Y₀+/-n), X₀ - R + n <= X <= X₀ + R - n.
And for Y == Y₀+/-R, this is trivially the points (X₀, Y₀+/-R).

No coordinates are explicitly calculated, and no point is visited more than once.

Note that this approach would work for the more conventional distance, as well. Use Bresenham's circle-drawing algorithm to calculate the end-points of each row.

Freedom is the freedom to say that two plus two make four. If that is granted, all else follows.
-- 6079 Smith W.

BillWoodruff9-Nov-18 13:26

9-Nov-18 13:26

@bhiller Without some 'unsafe voodoo, I don't see any way but "brute force;" in this example, I chose to generate a set of rectangles:

public static IEnumerable<Rectangle> GetNearbyPoints(int maxDist, List<Point> points, int nRows, int nColumns)
{
    int x, y, w, h;
    int half = maxDist / 2;

    List<Rectangle> rects = new List<Rectangle>();
        
    for (int i = 0; i < points.Count(); i++)
    {
        Point pt = points[i];
        x = pt.X;
        y = pt.Y;

        if (x < 0) x = 0;
        w = maxDist;
        if (x + half > nColumns)
        {
            w  = nColumns - x;
        }

        if (y < 0) y = 0;

        h = maxDist;
        if (y + half > nRows)
        {
            h = nRows - y;
        }

        yield return (new Rectangle(x,y,w,h));
    }
}

«Where is the Life we have lost in living? Where is the wisdom we have lost in knowledge? Where is the knowledge we have lost in information?» T. S. Elliot

Bernhard Hiller11-Nov-18 21:33

11-Nov-18 21:33

Thanks, Bill. That could pave the way to a solution.
I really need some kind of "List" of those points - IEnumerable is necessary, Count would be nice, mutability is not required at all. That "list" will be used as a parameter in further calculations.
So, a solution could be:
- create a rectangle enclosing all original points (that's O(n) for number of points)
- create a 2-dimensional array of bool, and map those points (i.e. x and y of the point become indices of the array, and there the value is set to true)
- create a 2-dimensional array of bool for the result. It's bigger, and the size can easily be determined.
- either
- - iterate over all result array and find out if a point of the original array can be reached
- - iterate over the original array and mark all points which can be reached in the result array. In contrast to the HashSet implementation, there is no need to check for duplicates: it does not matter if an item already true is set to true again.
- re-map the array to a List of Point
That should be O(n) for the number of original points (which is my main concern), but stil O(n^2) for the distance (which is a configurable value, and should not vary to much).

Oh sanctissimi Wilhelmus, Theodorus, et Fredericus!

BillWoodruff11-Nov-18 22:26

11-Nov-18 22:26

off the top of my head: from the IEnumerable<Rectangle>

public static IEnumerable<Point> RectanglesToPoints(IEnumerable<Rectangle> rects)
{
    return rects.Select(r => RectToPoints(r)).SelectMany(pt => pt);
}

public static IEnumerable<Point> RectToPoints(Rectangle rect)
{
    for (int i = rect.X; i < rect.Width + rect.X; i++)
    {
        for (int j = rect.Y; j < rect.Height + rect.Y; j++)
        {
            yield return new Point(i,j);
        }
    }
}

I suspect there's a better way; is there something about when you need to go from IEnumerable to List, or other, that is critical here ?

«Where is the Life we have lost in living? Where is the wisdom we have lost in knowledge? Where is the knowledge we have lost in information?» T. S. Elliot

Bernhard Hiller13-Nov-18 21:57

13-Nov-18 21:57

Thanks for your input. I think I could have enough information now to improve the original solution (don't need to do it now, but I am sure I'll have to do it some when later).

Regarding your latest code, there a two points to consider: the resulting list will be used in a few places. So I'd have to do a ToList or ToArray call to prevent the Linq statement from being executed multiple times. That's an easy fix.
Also, when I need the amount of points in the resulting list, it should be free of duplicates. A call to Distinct in RectanglesToPoints should do the trick easily again.

Oh sanctissimi Wilhelmus, Theodorus, et Fredericus!

BillWoodruff13-Nov-18 22:05

13-Nov-18 22:05

Hi Bernhard, It was fun to think about this.

I was puzzling over the issue of duplication, since two rectangles could overlap, without being identical. My only thought was to generate the full list of points, and call 'Distinct on that: I tried to ignore the thought of graphic paths that could represent rectangles with chunks taken out of them Smile | :)

cheers, Bill

«Where is the Life we have lost in living? Where is the wisdom we have lost in knowledge? Where is the knowledge we have lost in information?» T. S. Elliot

Bernhard Hiller13-Nov-18 22:22