0

1.20Ghzプロセッサと1GBのRAMを搭載したネットブックを持っています。

その上でC#WinFormsアプリを実行しています。このアプリは、5分間隔で、テキストファイルのすべての行を読み取り、その行の内容に応じて、スキップするか、xmlファイルに書き込みます。場合によっては、約2000行を処理している可能性があります。

このタスクを開始すると、プロセッサは最大になり、100%使用されます。ただし、2.40Ghzプロセッサと3GB RAMを搭載したデスクトップでは、(明らかな理由で)手つかずのままです...このプロセッサの問題を実際に劇的に減らす方法はありますか?コードは複雑ではなく、コーディングも苦手ではなく、ファイルを開いたり、読み書きしたりすることもありません...すべてが一挙に行われます。

助けていただければ幸いです!?

サンプルコード

***タイマー.....

#region Timers Setup

            aTimer.Tick += new EventHandler(OnTimedEvent);
            aTimer.Interval = 60000;
            aTimer.Enabled = true;
            aTimer.Start();
            radioButton60Mins.Checked = true;

            #endregion Timers Setup


private void OnTimedEvent(object source, EventArgs e)
        {
            string msgLoggerMessage = "Checking For New Messages " + DateTime.Now;
            listBoxActivityLog.Items.Add(msgLoggerMessage);
            MessageLogger messageLogger = new MessageLogger();
            messageLogger.LogMessage(msgLoggerMessage);

            if (radioButton1Min.Checked)
            {
                aTimer.Interval = 60000;
            }
            if (radioButton60Mins.Checked)
            {
                aTimer.Interval = 3600000;
            }
            if (radioButton5Mins.Checked)
            {
                aTimer.Interval = 300000;
            }

            // split the file into a list of sms messages
            List<SmsMessage> messages = smsPar.ParseFile(smsPar.CopyFile());

            // sanitize the list to get rid of stuff we don't want
            smsPar.SanitizeSmsMessageList(messages);

            ApplyAppropriateColoursToRecSMSListinDGV();
        }

public List<SmsMessage> ParseFile(string filePath)
        {
            List<SmsMessage> list = new List<SmsMessage>();
            using (StreamReader file = new StreamReader(filePath))
            {
                string line;
                while ((line = file.ReadLine()) != null)
                {
                    var sms = ParseLine(line);
                    list.Add(sms);
                }
            }
            return list;
        }

        public SmsMessage ParseLine(string line)
        {
            string[] words = line.Split(',');

            for (int i = 0; i < words.Length; i++)
            {

                words[i] = words[i].Trim('"');
            }
            SmsMessage msg = new SmsMessage();
            msg.Number = int.Parse(words[0]);
            msg.MobNumber = words[1];
            msg.Message = words[4];
            msg.FollowedUp = "Unassigned";
            msg.Outcome = string.Empty;

            try
            {

                //DateTime Conversion!!!
                string[] splitWords = words[2].Split('/');
                string year = splitWords[0].Replace("09", "20" + splitWords[0]);
                string dateString = splitWords[2] + "/" + splitWords[1] + "/" + year;
                string timeString = words[3];
                string wholeDT = dateString + " " + timeString;
                DateTime dateTime = DateTime.Parse(wholeDT);
                msg.Date = dateTime;

            }
            catch (Exception e)
            {
                MessageBox.Show(e.ToString());
                Application.Exit();
            }
            return msg;
        }

        public void SanitizeSmsMessageList(List<SmsMessage> list)
        {
            // strip out unwanted messages
            // list.Remove(some_message); etc...
            List<SmsMessage> remove = new List<SmsMessage>();
            foreach (SmsMessage message in list)
            {
                if (message.Number > 1)
                {
                    remove.Add(message);
                }
            }
            foreach (SmsMessage msg in remove)
            {
                list.Remove(msg);
            }
            //Fire Received messages to xml doc
            ParseSmsToXMLDB(list);
}

public void ParseSmsToXMLDB(List<SmsMessage> list)
        {
            try
            {
                if (File.Exists(WriteDirectory + SaveName))
                {
                    xmlE.AddXMLElement(list, WriteDirectory + SaveName);
                }
                else
                {
                    xmlE.CreateNewXML(WriteDirectory + SaveName);
                    xmlE.AddXMLElement(list, WriteDirectory + SaveName);
                }
            }
            catch (Exception e)
            {
                MessageBox.Show(e.ToString());
                Application.Exit();
            }
        }

public void CreateNewXML(string writeDir)
        {
            try
            {
                XElement Database = new XElement("Database");
                Database.Save(writeDir);
            }
            catch (Exception e)
            {
                MessageBox.Show(e.ToString());
            }
        }

        public void AddXMLElement(List<SmsMessage> messages, string writeDir)
        {
            try
            {
                XElement Database = XElement.Load(writeDir);
                foreach (SmsMessage msg in messages)
                {
                    if (!DoesExist(msg.MobNumber, writeDir))
                    {
                        Database.Add(new XElement("SMS",
                              new XElement("Number", msg.MobNumber),
                              new XElement("DateTime", msg.Date),
                              new XElement("Message", msg.Message),
                              new XElement("FollowedUpBy", msg.FollowedUp),
                              new XElement("Outcome", msg.Outcome),
                        new XElement("Quantity", msg.Quantity),
                        new XElement("Points", msg.Points)));

                        EventNotify.SendNotification("A New Message Has Arrived!", msg.MobNumber);

                    }
                }
                Database.Save(writeDir);
                EventNotify.UpdateDataGridView();
                EventNotify.UpdateStatisticsDB();
            }
            catch (Exception e)
            {
                MessageBox.Show(e.ToString());
            }
        }

public bool DoesExist(string number, string writeDir)
        {
            XElement main = XElement.Load(writeDir);
            return main.Descendants("Number")
                      .Any(element => element.Value == number);
        }
4

5 に答える 5

4

プロファイラーおよび/またはパフォーマンスモニターおよび/\\live.sysinternals.com\tools\procmon.exeまたはリソースモニターを使用して、何が起こっているかを判断します

于 2009-08-07T11:19:17.763 に答える
1

5分間のプロセスがバックグラウンドタスクである場合は、スレッド優先度を利用できます。

ここでMSDN 。

別のスレッドで処理を行い、タイマーをSystem.Threading.Timerに変更し、コールバックイベントを使用する場合、そのスレッドに他のアプリケーションよりも低い優先度を設定できるはずです。

于 2009-08-07T11:52:38.027 に答える
0

ParseFileループ内で、Thread.SleepApplication.DoEvents()呼び出しを追加して、それが役立つかどうかを確認できます。解析でこれを行う方が良いのは別のスレッドですが、少なくともこの簡単なテストを試して、それが役立つかどうかを確認することができます。

于 2009-08-07T11:19:58.323 に答える
0

キャッチ内のMessageBoxがクロススレッドの問題に遭遇している可能性があります。トレース出力に書き込むためにそれらを交換してみてください。

いずれにせよ、あなたは(小さな)プログラム全体を投稿しましたが、それはあなたが特定のアドバイスを得るのを助けません。メソッド本体を(一度に1つずつ、科学的に)削除してみて、問題が発生する/発生を停止するようにしてください。これは、問題を特定し、質問の無関係な部分(自分自身とSOの両方)を排除するのに役立ちます。

于 2009-08-07T11:22:24.590 に答える
0

現在の処理モデルはバッチベースです。解析を実行してから、メッセージを処理します。

Linqスタイルの「プル」アプローチに切り替えると、メモリのオーバーヘッドが削減される可能性があります。

たとえば、次のようにParseFile()メソッドを変換できます。

public IEnmerable<SmsMessage> ParseFile(string filePath)
{
    using (StreamReader file = new StreamReader(filePath))
    {
        string line;
        while ((line = file.ReadLine()) != null)
        {
            var sms = ParseLine(line);
            yield return sms;
        }
    }
}

利点は、すべてのメッセージを一度に解析してからすべてを処理するのではなく、各SmsMessageを生成時に処理できることです。

これにより、メモリオーバーヘッドが低下します。これは、ネットブックとデスクトップのパフォーマンスの違いの原因の1つです。

于 2009-08-08T07:25:08.117 に答える