Re: Creating a new user control using decorator pattern - C# Discussion Boards

Re: Creating a new user control using decorator pattern

18-Jun-10 7:36

Hello all,

To demonstrate the decorator pattern, many articles gives as an example the ability to extend user controls (e.g. a window) with another ability (e.g. scrollable bar).

I'm trying to use it in winforms and extend the ProgressBar.

I'm building a utility function that copy some files from one network drive to another network drive, and I would like to supply also a progress bar that will show (as accurate as possible) the progress of the copy.

In order to that I want to extend the progress bar with a full amount property, and a sniffing method (supplied by the user, e.g. querying the network adapter), and thus the progress bar would display the size so far divided by the known full amount.

Decorator pattern seems the perfect choice but i can't seems to implement in winforms.

Thanks,
Berlus.

Ennis Ray Lynch, Jr.18-Jun-10 7:49

Ennis Ray Lynch, Jr.

18-Jun-10 7:49

While you would normally use the File.Copy method to copy files in Windows using the File.OpenRead method will allow you to manually copy using streams and have the file size and stream location immediately available and thus negate the need for a custom progress bar that uses a sniffer. Just remember, never load the full stream in memory.

Need custom software developed? I do custom programming based primarily on MS tools with an emphasis on C# development and consulting.

A man said to the universe:
"Sir I exist!"
"However," replied the universe,
"The fact has not created in me
A sense of obligation."
--Stephen Crane

Re: Creating a new user control using decorator pattern

Berlus18-Jun-10 8:25

Re: Creating a new user control using decorator pattern

18-Jun-10 8:25

Thanks for your answer.

The problem is that the file copying is in 3rd party Dll, and I'm aware only to the side effects of the copying process.

And, I would like to use this opportunity to exercise the decorator pattern.

thanks anyway,
Berlus

Ennis Ray Lynch, Jr.18-Jun-10 9:09

Ennis Ray Lynch, Jr.

18-Jun-10 9:09

Patterns are suggestions when problems arise that they are suited for. Using a pattern for a given problem because you want to use the pattern is not a recipe for success. Sometimes the problem is not with the answer but with the desired approach.

Since a progress bar is inheritable this is the simplest method. Note, BeginCopyNotifyChange will call your third party tool and poll. However, keep in mind, that you cannot technically know when the transfer is finished unless you poll the locks placed on the file for copying nor will you know the size, assuming the file copy tool is a third party tool.

public class FileCopyProgressBar : System.Windows.Forms.ProgressBar {
    public void BeginCopyNotifyChange(string source, string destination);
}

Re: Creating a new user control using decorator pattern

T M Gray18-Jun-10 8:33

T M Gray

18-Jun-10 8:33

Where are you getting your information about patterns? Decorator involves changing the behaviour of a class on the fly at runtime. The stuff you want to do can and probably should be done at compile time. AJAX extenders are good examples of decorator. They add functionality. But what you are wanting to do is more of a change in how the progress bar works than an addition of a feature. You want to change how it calculates progress. That sounds more like a subclass to me.

Re: Creating a new user control using decorator pattern

Berlus18-Jun-10 11:07

Re: Creating a new user control using decorator pattern [modified]

18-Jun-10 11:07

Thanks for your answer, it is execatly what I was aiming for, beacause I need some help understanding the pattern.

I've looked up decorator pattern in wikipedia, and i quote:

http://en.wikipedia.org/wiki/Decorator_pattern

"In object-oriented programming, the decorator pattern is a design pattern that allows new/additional behaviour to be added to an existing object dynamically."

dynamically, as you said.

but in the motivation part:

"As an example, consider a window in a windowing system. To allow scrolling of the window's contents, we may wish to add horizontal or vertical scrollbars to it, as appropriate. Assume windows are represented by instances of the Window class, and assume this class has no functionality for adding scrollbars. We could create a subclass ScrollingWindow that provides them, or we could create a ScrollingWindowDecorator that adds this functionality to existing Window objects. At this point, either solution would be fine."

which seemed like what i'm trying to do.

Thanks, Berlus

Berlus18-Jun-10 11:09

Re: Creating a new user control using decorator pattern

18-Jun-10 11:09

After rereading the article i'm attaching a sample class:

public abstract class ProgressBarDecorator : ProgressBar
    {
        protected ProgressBar m_decoratedProgressBar; // the Window being decorated

        public ProgressBarDecorator()
        {
            
        }

        public ProgressBarDecorator(ProgressBar p_decoratedProgressBar)
        {
            this.m_decoratedProgressBar = p_decoratedProgressBar;
    }

    }

    // the first concrete decorator which adds functionality
    public class NotifiableProgressBarDecorator : ProgressBarDecorator
    {

        public NotifiableProgressBarDecorator()
        {
            
        }
        public NotifiableProgressBarDecorator(ProgressBar p_decoratedProgressBar)
            : base(p_decoratedProgressBar)
        {
        }

        protected override void OnCreateControl()
        {
            base.OnCreateControl();
            this.BackColor = Color.Green;
            this.Value = 50;
        }
    }

}

It is the base design fot my implementation, if it will be any good, i might consider posting as an article.

Your thoughts ans suggestions are most welcome.

Thanks,
Belrus

modified on Friday, June 18, 2010 5:17 PM

DaveyM6918-Jun-10 11:14

DaveyM69

18-Jun-10 11:14

First thought is to use the 'code block' widget above the edit box when posting so your code is surrounded by <pre></pre> tags so the code is legible.

Dave

If this helped, please vote & accept answer!

Binging is like googling, it just feels dirtier. (Pete O'Hanlon)

BTW, in software, hope and pray is not a viable strategy. (Luc Pattyn)

Re: Creating a new user control using decorator pattern

Berlus18-Jun-10 11:18

Re: Creating a new user control using decorator pattern

18-Jun-10 11:18

Thanks

Luc Pattyn18-Jun-10 11:58

Luc Pattyn

18-Jun-10 11:58

Right.

Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles] Nil Volentibus Arduum

Please use < PRE > tags for code snippets, it preserves indentation, and improves readability.

Re: Creating a new user control using decorator pattern

T M Gray18-Jun-10 11:41

T M Gray

18-Jun-10 11:41

That isn't what you are trying to do. Adding a scrollbar to a window doesn't change anything about the underlying functionality of the window. How the window is closed, moved, z-ordered etc. all remain intact. Changing the methodology a progress bar uses to determine the amount of progress is a fundamental change to the existing underlying functionality, not an addition. Adding a user preference for the color of the progress bar would be more of a decorator.

I see from one of your other responses that it is a 3rd party control you are dealing with. Decorator pattern seems to be what you want only because you can't do it with subclassing because you don't have the source to do it. You are shoehorning your design requirement into a pattern based on other external factors.

Why are you using a control that doesn't do what you want it to?

Re: Creating a new user control using decorator pattern

Berlus18-Jun-10 12:17

always visible splash screen

18-Jun-10 12:17

T M Gray wrote:
Why are you using a control that doesn't do what you want it to?

Form where do i get such control ?

anyway, here is the first version of the class, I hope i'm not abusing this forum ...

using System;
using System.Drawing;
using System.Net.NetworkInformation;
using System.Windows.Forms;

namespace TestProgressBar
{
    public abstract class ProgressBarDecorator : ProgressBar
    {
        protected ProgressBar m_decoratedProgressBar; // the Window being decorated

        public ProgressBarDecorator()
        {

        }

        public ProgressBarDecorator(ProgressBar p_decoratedProgressBar)
        {
            this.m_decoratedProgressBar = p_decoratedProgressBar;
        }

    }

    // the first concrete decorator which adds vertical scrollbar functionality
    public class NotifiableProgressBarDecorator : ProgressBarDecorator
    {
        public long FullAmount;

        public void InformOnNewData(long p_newDelta)
        {
            float normalizedSize = Maximum - Minimum;

            if (FullAmount != 0)
            {
                int newValue = Value + (int) (p_newDelta/(float) FullAmount*normalizedSize);

                if ((Value + newValue) <= Maximum)
                {
                    Value += newValue;
                }
                else
                {
                    Value = Maximum;
                }
            }
        }

        public void Clear()
        {
            FullAmount = 0;
        }

        public NotifiableProgressBarDecorator()
        {
        }

        public NotifiableProgressBarDecorator(ProgressBar p_decoratedProgressBar)
            : base(p_decoratedProgressBar)
        {
        }
    }

    public class NetworkNotifiableProgressBarDecorator : NotifiableProgressBarDecorator
    {
        private NetworkInterface adapter;
        private Timer m_timer = new Timer();
        private long m_lastSentBytes;

        public NetworkNotifiableProgressBarDecorator()
        {
            adapter = NetworkInterface.GetAllNetworkInterfaces()[1];

            m_timer.Interval = 500;
            m_timer.Tick += onTimerTick;
        }

        public void Init(int p_totalNofBytes)
        {
            FullAmount = p_totalNofBytes;
            m_lastSentBytes = adapter.GetIPv4Statistics().BytesSent;
            m_timer.Start();
        }

        private void onTimerTick(object p_sender, EventArgs p_args)
        {
            long sentBytes = adapter.GetIPv4Statistics().BytesSent;
            if ( sentBytes - m_lastSentBytes > 0)
            {
                InformOnNewData(sentBytes - m_lastSentBytes);
            }
            m_lastSentBytes = sentBytes;
        }

        public void Stop()
        {
            m_timer.Stop();
        }
    }

}

Jassim Rahma18-Jun-10 6:54

Jassim Rahma

18-Jun-10 6:54

how can i show a splash screen which should be visible until the main form is fully loaded and shown?

Henry Minute18-Jun-10 7:11

18-Jun-10 7:11

There are many, many articles here on Code Project that show how to do what you want. Use the Search Box on the home page. You'll find loads of examples.

Henry Minute

Do not read medical books! You could die of a misprint. - Mark Twain
Girl: (staring) "Why do you need an icy cucumber?"
“I want to report a fraud. The government is lying to us all.”
Why do programmers often confuse Halloween and Christmas? - Because 31 Oct = 25 Dec.
Business Myths of the Geek #4 'What you think matters.'

Luc Pattyn18-Jun-10 8:18

Luc Pattyn

18-Jun-10 8:18

And yet I haven't seen a splash screen implementation I really like; I do it differently and I'm afraid I really should write yet another article on the subject.

Smile | :)

Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles] Nil Volentibus Arduum

Please use < PRE > tags for code snippets, it preserves indentation, and improves readability.

modified on Friday, June 18, 2010 3:00 PM

Henry Minute18-Jun-10 8:20

18-Jun-10 8:20

You continental types always were too picky! Big Grin | :-D

On the rare occasions that I use a splash screen, I too do it differently.

determine sql server 2008 express size

Muammar©

18-Jun-10 8:06

Here:

public partial class Splash : Form
    {

        private void Splash_Load(object sender, EventArgs e)
        {
            tmrWasteTime.Start();
        }
        
        private void tmrFader_Tick(object sender, EventArgs e)
        {
            if(this.Opacity >0.01)
                this.Opacity = this.Opacity - 0.03;
            else
                kill_tmrFader();
        }

        Form1 form1;
        private void kill_tmrFader()
        {
            tmrFader.Stop();
            form1 = new Form1();
            this.Hide();
        }

        int waste;
        private void tmrWasteTime_Tick(object sender, EventArgs e)
        {
            if (waste < 2)
                waste++;
            else
                kill_tmrWasteTime();

        }

        private void kill_tmrWasteTime()
        {
            tmrWasteTime.Stop();
            tmrFader.Start();
        }

    }

In this, example, you realize I'm using some components like timers, you can modify it to fit your needs, however, make sure you call the Splash in this example from your Program.cs and modify form1 here to your called form.

Cheers.
Muammar.

modified on Friday, June 18, 2010 2:15 PM

Jassim Rahma18-Jun-10 6:12

Jassim Rahma

18-Jun-10 6:12

as sql server express 2008 is limited, how can I determine the current size of the database using c#?

dan!sh 18-Jun-10 6:24

dan!sh

18-Jun-10 6:24

You can use the system procedure sp_helpdb for this.

Henry Minute18-Jun-10 7:13

18-Jun-10 7:13

If you get the latest download, they have increased the size from 4 to 10GB.

Jassim Rahma19-Jun-10 5:20

Jassim Rahma

19-Jun-10 5:20

wow!! really?
where is this article? and where is the URL for the download? I have SQL Server 2008 Express.

Henry Minute19-Jun-10 5:45