Skip to content

Download File Progress#5

Open
Berkel256 wants to merge 1 commit intoSanftMonster:masterfrom
Berkel256:patch-1
Open

Download File Progress#5
Berkel256 wants to merge 1 commit intoSanftMonster:masterfrom
Berkel256:patch-1

Conversation

@Berkel256
Copy link

The progress is reporting the number of downloaded pieces, which can be larger than 100. To download the progress in percentage (ranging from 0 to 100) devide size downloaded size by total file size and multiply by 100 (for percentage). Use Math.Floor to not report a progress of 100 when a byte is still missing.

The progress is reporting the number of downloaded pieces, which can be larger than 100.
To download the progress in percentage (ranging from 0 to 100) devide size downloaded size by total file size and multiply by 100 (for percentage). Use Math.Floor to not report a progress of 100 when a byte is still missing.
await fs.WriteAsync(buffer, 0, bytesRead);
newResumeSize += bytesRead;
progress?.Report((int)(newResumeSize * pieces / totalFileLengths));
progress?.Report((int)Math.Floor(newResumeSize / totalFileLengths * 100));
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

make sure that this is actually compiling because of type conversions

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant