Console Progress Percent in one line

Question

0.00/5 (No votes)

See more:

Hi there im trying to display the progress percentage currently it looks like this

Downloaded Percentage: 1%
Downloaded Percentage: 2%
Downloaded Percentage: 3%
Downloaded Percentage: 4%

which is wrong i want it to update with out one after another my code is below can someone please point out where im going wrong.

C#

var videoDownloader = new VideoDownloader(video,
                Path.Combine(Environment.GetFolderPath(Environment.SpecialFolder.MyVideos),
                RemoveIllegalPathCharacters(video.Title) + video.VideoExtension));

            for (int i = 0; i < videoDownloader.SavePath.Length; i++)
            {
                // Register the ProgressChanged event and print the current progress
                videoDownloader.DownloadProgressChanged += (sender, args) => Console.Write(string.Format("Downloaded Percentage: {0}", args.ProgressPercentage));
            }

Posted 13-Oct-15 1:57am

Member 11838038

Add a Solution

Comments

Tomas Takac 13-Oct-15 8:36am

Why are you registering the DownloadProgressChanged handler in a loop for each character in the SavePath? This doesn't make sense.

Member 11838038 13-Oct-15 9:03am

Sorry man still learning :)

2 solutions

Solution 2

There's a trick to this.

Console.Write("Something" + "\r")
will return the cursor back to the beginning of the current line.

So you would use
Console.Write("Downloaded Percentage: {0, -3}\r", args.ProgressPercentage);

As the line is overwritten it's important to ensure that the new text is at least as long as the old stuff. Hence the use of the 3 character left aligned field in the example.

An alternative is to just end the line with a few spaces, e.g.
Console.Write("Downloaded Percentage: {0}% \r", args.ProgressPercentage);
3 spaces will be ok as long as the new line is no more than 3 chars shorter than the original.

Alan.

Posted 13-Oct-15 2:36am

Alan N

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Richard Deeming · Accepted Answer · 2015-10-13T02:35:00

Solution 1

You need to save the cursor position before you start, and set it[^] before you write the message:

C#

int left = Console.CursorLeft;
int top = Console.CursorTop;

videoDownloader.DownloadProgressChanged += (sender, args) => 
{
    Console.SetCursorPosition(left, top);
    Console.Write("Downloaded Percentage: {0}", args.ProgressPercentage);
};

(There's no need to call string.Format when writing to the console; both Write and WriteLine have overloads which accept a composite format string and arguments.)

Posted 13-Oct-15 2:35am

Richard Deeming

Comments

Member 11838038 13-Oct-15 8:55am

Thanks so much so i didnt need the for(){} ?

Richard Deeming 13-Oct-15 8:58am

As Tomas said, the for loop didn't make much sense. You should only need to attach the event handler once.

Member 11838038 13-Oct-15 9:03am

Yes you correct thank you much for help