C# WinApp BackgroundWorker ถ้าอยากให้แสดงผลอย่างอื่นนอกจาก ProgressPercentage ด้วยครับ
ลองแบบนี้ได้ไหมครับ
ประกาศตัวแปรมาสักตัว
แล้วเราก็ update ตัวแปรนั้น
แล้วใช้ Timer คอยเอาค่านั้นส่งไปที่ form/label/textbox อีกที
Date :
2017-04-26 10:52:50
By :
fonfire
ตอบความคิดเห็นที่ : 1 เขียนโดย : fonfire เมื่อวันที่ 2017-04-26 10:52:50
รายละเอียดของการตอบ ::
ถ้าแบบนั้นผมทำอยู่ครับ
แต่ไม่ใช้ Timer เพราะ Thread ควบคุมได้ง่ายกว่า เยอะกว่า แถม โค้ดก็สั้นกว่า ด้วยครับ
อย่างถ้าเรามี 26 Thread เราต้องเขียนโค้ดซ้ำๆกัน 26 Timer
แต่ถ้าใช้ Thread เราเขียน 1 method แล้วเรียกใช้ผ่าน Thread ได้เลยครับ
สักครู่ได้ทดลองทำเกี่วกับ % ปรากฎว่า ProgressPercentage ไม่สามารถทำให้เป็น Double ได้อีก กรรมแท้ๆ
ตัวอย่างโค้ดเก่าก็ประมาณนี้ครับ ใช้ 26 Thread
เพื่อที่จะจัดการ
ตามความคิดผมคือ ถ้าเราจะเปลี่ยนแปลงอะไรก็อยากให้มันดีขึ้นครับ
อย่าง โค้ดลดลง เขียนง่ายขึ้น ประมาณนี้ครับ
Code (C#)
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
namespace CheckFileNCN
{
public partial class frmCheckFileNCN : Form
{
public frmCheckFileNCN()
{
InitializeComponent();
}
int sngPer =0, sngRun = 0;
double startCounter;
List<string> list;
string PathKARA;
int maxThread = 26;
int ChkThread;
void GetNCN()
{
list = new List<string>();
System.Collections.Generic.Stack<string> stack = new System.Collections.Generic.Stack<string>();
stack.Push(txtSouce.Text);
while ((stack.Count > 0))
{
string dir = stack.Pop();
list.AddRange(System.IO.Directory.GetFiles(dir, "*.lyr"));
this.Invoke(new Action(() =>{this.Text = "Check Folder " + dir;})) ;
lblEMKDetail.Invoke(new Action(() => {lblEMKDetail.Text= "NCN File; " + (list.Count - 1); })) ;
string directoryName = null;
foreach (string directoryName_loopVariable in System.IO.Directory.GetDirectories(dir))
{
directoryName = directoryName_loopVariable;
stack.Push(directoryName);
}
}
if (list.Count - 1 < 1) { Text = "Complete ..."; return; }
sngPer = (list.Count - 1) / 26;
progressBar2.Invoke(new Action(() => { progressBar2.Maximum = list.Count - 1; }));
PathKARA = txtTarget.Text;
startCounter = DateTime.Now.TimeOfDay.TotalSeconds;
ChkThread = 0;
for (int i = 0; i < maxThread; i++)
{
new System.Threading.Thread(CheckFile).Start(new object[] { Convert.ToChar(65 + i).ToString(), i * sngPer, (i + 1) * sngPer - 1 });
}
}
#region _Select path
private void button4_Click(object sender, EventArgs e)
{
System.Threading.Thread Thd = new System.Threading.Thread(new System.Threading.ThreadStart(GetNCN)) { IsBackground = true };
Thd.Start();
}
private void button1_Click(object sender, EventArgs e)
{
FolderBrowserDialog fd = new FolderBrowserDialog();
fd.ShowDialog();
txtSouce.Text = fd.SelectedPath;
}
private void button2_Click(object sender, EventArgs e)
{
FolderBrowserDialog fd = new FolderBrowserDialog();
fd.ShowDialog();
txtTarget.Text = fd.SelectedPath;
}
#endregion
private void frmCheckFileNCN_Load(object sender, EventArgs e)
{
}
private void CheckFile(object obj)
{
string Fol_Lyrics, Fol_Song, Fol_Cursor;
string s_S = ""; string t_S = "";
string s_L = ""; string t_L = "";
string s_C = ""; string t_C = "";
int _sngNCN = 0;
string FolderCode = (string)((object[])obj)[0];
int min = (int)((object[])obj)[1];
int max = (int)((object[])obj)[2];
int i = min;
// set path ต่างๆ
Fol_Lyrics = PathKARA + "\\Songs\\MIDI\\NCN\\Lyrics\\" + FolderCode;
Fol_Song = PathKARA + "\\Songs\\MIDI\\NCN\\Song\\" + FolderCode;
Fol_Cursor = PathKARA + "\\Songs\\MIDI\\NCN\\Cursor\\" + FolderCode;
MIDI_Dll.MIDI_Dll.CreatePath(Fol_Song);
MIDI_Dll.MIDI_Dll.CreatePath(Fol_Lyrics);
MIDI_Dll.MIDI_Dll.CreatePath(Fol_Cursor);
while (true)
{
s_L = list[i].ToLower();
s_S = s_L.Replace("\\lyrics\\", "\\song\\").Replace(".lyr", ".mid");
s_C = s_L.Replace("\\lyrics\\", "\\cursor\\").Replace(".lyr", ".cur");
if (System.IO.File.Exists(s_C) && System.IO.File.Exists(s_S))
{
do
{
t_S = Fol_Song + "\\" + FolderCode + string.Format("{0:00000}", _sngNCN) + ".mid";
t_C = Fol_Cursor + "\\" + FolderCode + string.Format("{0:00000}", _sngNCN) + ".cur";
t_L = Fol_Lyrics + "\\" + FolderCode + string.Format("{0:00000}", _sngNCN) + ".lyr";
_sngNCN++;
} while (System.IO.File.Exists(t_L) || System.IO.File.Exists(t_C) || System.IO.File.Exists(t_S));
System.IO.File.Move(s_S, t_S);
System.IO.File.Move(s_C, t_C);
System.IO.File.Move(s_L, t_L);
}
try
{
if (sngRun > 0)
{
double t = DateTime.Now.TimeOfDay.TotalSeconds - startCounter;
double strt = (double)((list.Count - sngRun) * t / sngRun);
this.lblEMKDetail.Invoke(new Action(() =>
{
this.lblEMKDetail.Text = "Check File " + s_S + Environment.NewLine + t_S + Environment.NewLine +
"File: " + sngRun + "/" + (list.Count - 1) + Environment.NewLine +
"Completion Ratio:" + string.Format("{0:0.0000}", Convert.ToDouble(100 * (double)sngRun / (double)(list.Count - 1))) + " %" + Environment.NewLine +
"Speed:" + string.Format("{0:0.00}", ((double)sngRun / t)) + "file/s" + Environment.NewLine +
"Elapsed time:" + MIDI_Dll.MIDI_Dll.Time(t) + Environment.NewLine +
"Remaining time:" + MIDI_Dll.MIDI_Dll.Time(strt);
}));
}
this.progressBar2.Invoke(new Action(() => { this.progressBar2.Value = sngRun; }));
}
catch { }
sngRun++;
System.Threading.Thread.Sleep(5);
i++;
if (i >= max)
{
ChkThread++;
if (ChkThread > maxThread && checkBox1.Checked == true)
{
System.Diagnostics.Process.Start(new System.Diagnostics.ProcessStartInfo("shutdown.exe", "-s"));
}
else
{ return; }
}
}
}
}
}
Date :
2017-04-26 11:15:42
By :
lamaka.tor
ที่ให้ลองใช้ Timer จะเป็นแค่ในส่วนของการ update gui ครับ
ไม่ได้เกี่ยวกับการทำงานแต่อย่างใด
ผมเข้าใจว่าตัว thread ไม่สามารถเข้าถึง GUI ที่อยู่บน main thread ได้อยู่แล้ว
แต่ถ้าเรา update แค่ตัวแปร แล้วเอา timer ถึงทำงานบน main thread มา update gui ก็น่าจะทำได้น่ะครับ
เห็นปกติเขาจะใช้ตัว delegate เอา
แต่ผมก็ไม่เคยทำเลยครับ T___T
หากผิดไปต้องขออภัยด้วยครับ
Date :
2017-04-26 11:25:25
By :
fonfire
ตอบความคิดเห็นที่ : 3 เขียนโดย : fonfire เมื่อวันที่ 2017-04-26 11:25:25
รายละเอียดของการตอบ ::
ตามความเข้าใจแบบ้านๆของผมเลยนะครับ
ถ้าเรามีงานอยู่ 5 งานที่อยากทำพร้อมกัน
ถ้าTimer จะเขียนแบบนี้
Code (C#)
public partial class Form1 : Form
{
public Form1()
{
InitializeComponent();
}
Timer timer1 = new Timer();
Timer timer2 = new Timer();
Timer timer3 = new Timer();
Timer timer4 = new Timer();
Timer timer5 = new Timer();
private void timer1_Tick(object sender, EventArgs e)
{
for (int i = 1; i <= 1000; i++)
Console.WriteLine("timer1_" + i);
}
private void timer2_Tick(object sender, EventArgs e)
{
for (int i = 1001; i <= 2000; i++)
Console.WriteLine("timer2_" + i);
}
private void timer3_Tick(object sender, EventArgs e)
{
for (int i = 2001; i <= 3000; i++)
Console.WriteLine("timer3_" + i);
}
private void timer4_Tick(object sender, EventArgs e)
{
for (int i = 3001; i <= 4000; i++)
Console.WriteLine("timer4_" + i);
}
private void timer5_Tick(object sender, EventArgs e)
{
for (int i = 4001; i <= 5000; i++)
Console.WriteLine("timer5_" + i);
}
private void Form1_Load(object sender, EventArgs e)
{
timer1.Tick+= new EventHandler(this.timer1_Tick);
timer2.Tick += new EventHandler(this.timer2_Tick);
timer3.Tick += new EventHandler(this.timer3_Tick);
timer4.Tick += new EventHandler(this.timer4_Tick);
timer5.Tick += new EventHandler(this.timer5_Tick);
}
}
ส่วนถ้าใช้ thread จะเขียนแบบนี้
Code (C#)
public partial class Form2 : Form
{
public Form2()
{
InitializeComponent();
}
private void Form2_Load(object sender, EventArgs e)
{
new System.Threading.Thread(testThread).Start(new object[] { 1, "Threading_1" });
new System.Threading.Thread(testThread).Start(new object[] { 1001, "Threading_2" });
new System.Threading.Thread(testThread).Start(new object[] { 2001, "Threading_3" });
new System.Threading.Thread(testThread).Start(new object[] { 3001, "Threading_4" });
new System.Threading.Thread(testThread).Start(new object[] { 4001, "Threading_5" });
}
void testThread(object obj)
{
int _min = (int)((object[])obj)[0];
string str = (string)((object[])obj)[1];
for (int i = _min; i <= _min+1000; i++)
Console.WriteLine(str + i);
}
}
ความเห็นส่วนตัวผมนะครับ
Thread ยืดหยุ่นกว่า timer ระดับหนึ่ง
โดยจะเห็นได้ว่าถ้าผมอยากจะเพิ่ม งาน เป็น 100 งาน ก็แค่เรียกใช้ๆไปเรื่อยๆ
ในขณะที่ timer เราต้องประกาศตัวแปรแล้วยังเรียกใช้ event Tick ด้วยแถมยังต้องเขียนโค้ดเพิ่มจนกว่าจะครบ 100
ส่วนการเข้าถึง display ของ GUI ทั้ง timer และ thread ยังคงต้องใช้การ invoke เข้าไปอยู่
Code (C#)
private void timer1_Tick(object sender, EventArgs e)
{
for (int i = 1; i <= 1000; i++)
this.Invoke(new Action(()=>{this.Text="timer1_" + i;}));
}
private void timer2_Tick(object sender, EventArgs e)
{
for (int i = 1001; i <= 2000; i++)
this.Invoke(new Action(()=>{this.Text="timer2_" + i;}));
}
private void timer3_Tick(object sender, EventArgs e)
{
for (int i = 2001; i <= 3000; i++)
this.Invoke(new Action(()=>{this.Text="timer3_" + i;}));
}
private void timer4_Tick(object sender, EventArgs e)
{
for (int i = 3001; i <= 4000; i++)
this.Invoke(new Action(()=>{this.Text="timer4_" + i;}));
}
private void timer5_Tick(object sender, EventArgs e)
{
for (int i = 4001; i <= 5000; i++)
this.Invoke(new Action(() => { this.Text = "timer5_" + i; }));
}
Code (C#)
void testThread(object obj)
{
int _min = (int)((object[])obj)[0];
string str = (string)((object[])obj)[1];
for (int i = _min; i <= _min+1000; i++)
this.Invoke(new Action(() => { this.Text = str + i; }));
}
ในส่วนของ BackgroundWorker จะเป็น Worker ที่สามารถเข้าถึง display ของ GUI ได้อยู่แล้วครับ
แต่มันติดปัญหาว่า มันดันมีแค่ ProgressPercentage แถมยังเป็น int อีกด้วย (กรรมแต้ๆน่)
เวลาผมทำงานเป็นล้านๆ มันอ่านไปแค่พันเดียวก็ไม่ขึ้น % แล้วละครับ
ผมเอาเท่าที่ความรู้ผมจะแกะมาได้นะครับ
แต่บางท่านอาจจะมีวิธีที่ดีกว่าอยู่แล้วในการจัดการ display ของ GUI BackgroundWorker
Date :
2017-04-26 12:29:08
By :
lamaka.tor
Load balance : Server 00