C#: How should I save my data?

Question

I have two structs like so:

public struct KeyLog
{
    Keys key;
    DateTime time;
}

public struct MouseLog
{
    MouseEvents mouse;
    Point coordinates;
    DateTime time;
}

For every keyboard key press and mousebutton click I wish to save this data but I do not know which way would be the most efficient to store/handle it in? Are there better ways to handle the data - I'm not sure if using two structs is better than merging both into one?

Edit: I'm doing a keyboard & mouse statistic application that'll store the amount of key press & mouse clicks as well as what button was pressed, where and when for my computer and I would want to save this data every time a button is pressed. Not necessarily write to disk every time but at least store it in memory till I want to save it to disk.

Edit: I thought that if I keep the two structs separate I won't create too much dead data when I store them, and then I can easily search/sort if I keep them separate. Thoughts?

Answer 1

Unless there is a good reason to keep them separate you should merge them. KISS applies here I believe.

Answer 2

er.... where are you planning to store it? If you're just wanting an in-memory log that you can refer to, a List<X> should be fine.

If you want to stream it to disk, you'll want to serialize it. Binary serialization is the most efficient in terms of total disk space, but XML serialization is "human readable".

Then there's database storage...

Answer 3

A BinaryFormatter would give you the smallest compression. Also if you change them to classes, you could have a base class with a DateTime time field.

Answer 4

I would store it in one struct, beacuse it will most likely save you time as a programmer. It will be simpler to deal with, especially when saving/loading the file later on. There is little to be gained from using structs which you have described.

And if that should become a performance problem, profile the code see where any bottlenecks are and then fix them.

Answer 5

Do you have to worry about Unicode or can this be ASCII? I would say XML is your friend. Do not merge the two, but make both classes, which are both Children of an abstract EventLog. EventLog class must force a toString() method on you, maybe a few others. Then have a list of EventLogs, where each EventLog is a KeyLog or a MouseLog. I wonder if you are reinventing the wheel ...

Answer 6

Merging them will make them easy to program with..

BUt as you are more conscious about saving disk space

You could use a generic collection of MouseLog and Keylog

eg List List for in memory representation

AS for writing it to a disk you could put them in a class and save them as one Big object

like

  [serializable]
    class BigObject 
    {

    List<Mouselog> MouseLogLst;


    List<KeyLog>  KeyLogLst ;

    }

and use Binary Formmater to store them