Re: Perhaps a threading problem? - C# Discussion Boards

Re: Service does not start

Jordanwb15-Dec-09 2:18

Jordanwb

15-Dec-09 2:18

In my opening post I stated I was running Ubuntu 9.04

Re: Service does not start

Jimmanuel15-Dec-09 3:26

Jimmanuel

15-Dec-09 3:26

Yes, but a service is a service. I would expect mono to mimic the behavior of services on their native platform, which is what running without the --debug flag seems to do.

Natively they have no console or UI so if you want output, the easiest way is to write it to a file. Like I said, though, your code works on my 9.10 installation with the --debug flag.

Badger | [badger,badger,badger,badger...]

Badger | [badger,badger,badger,badger...]

Re: Service does not start [modified]

Jordanwb15-Dec-09 5:31

Jordanwb

15-Dec-09 5:31

I'll try writing to a file.

*Edit*

On Ubuntu 9.10 the output of Console.WriteLine, Console.Out.WriteLine and Console.Error.WriteLine goes to the user log. Why do I have to killall mono in order to stop the service? Why can't I just call mono-service2 ServiceTest.exe to stop it?

modified on Tuesday, December 15, 2009 12:02 PM

Re: Service does not start

Jimmanuel15-Dec-09 6:33

Jimmanuel

15-Dec-09 6:33

From what it looked like when I tried it killing the mono process didn't give it a chance to cleanup the lock file that it created for the service that it was running. Any further attempts to run the service would fail because it would see the dead lock file and think it was still running.

The docs for mono-service[^] tell you how to start a service where you know where the lock file is and then use that to stop the service.

Badger | [badger,badger,badger,badger...]

Re: Service does not start

Jordanwb15-Dec-09 7:17

Jordanwb

15-Dec-09 7:17

Okay Thanks.

Perhaps a threading problem?

George Quarton14-Dec-09 10:12

George Quarton

14-Dec-09 10:12

I have two classes, my main class (in which a form resides) and a file copying class. (I have other classes they just aren't important).

I have a proc event that determines when a USB drive is inserted and when caught it will fire a WorkerThread which will run the file copying method so that while the file operations are running the Form is still able to be updated.

protected override void WndProc(ref Message m)
        {
            if ((m.Msg == WM_DEVICECHANGE) && (m.WParam == DBT_DEVICEARRIVAL))
            {
                int deviceType = Marshal.ReadInt32(m.LParam, sizeof(int));


                if (deviceType == LOGICAL_VOLUME)
                {
                    uint unitMask = (uint)Marshal.ReadInt32(m.LParam, sizeof(int) * 3);


                    if (unitMask != 0)
                    {
                        char driveLetter = DriveLetterFromUnitMask(unitMask);
                        // some more code here but irrelevant.
                        backgroundWorker1.RunWorkerAsync(copy);
                        // the copy is a new object class containing data, however it is irrelevant
                    }
                }
             }
         }

The thread is not hard coded but simply added using the form controls (BackgroundWorker control).
The thread methods are so (added from double clicking the controls in the designer view)
After adding code I had this;

public void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
        {
            CopyObject a = e.Argument as CopyObject;
            RecursiveCopy.FullCopy(a.sConfigLocation, a.sourceFolder, a.destFolder, a.sDriveName, a.iniFile);
            e.Result = "1";
        }
     public void backgroundWorker1_ProgressChanged(object sender, ProgressChangedEventArgs e)
        {  
            string message = e.UserState as string;
            fileCopyingLbl.Text = message; // Update the label control.
        }

So the BackgroundWorker will call FullCopy as shown here;

public class RecursiveCopy
    {
        public static void FullCopy(string sConfigLocation, string sourceFolder, string destFolder, string sDriveName, object iniFile)
        {
            if (!Directory.Exists(destFolder))
                Directory.CreateDirectory(destFolder);
  
            int countFile = 0;
            string[] files = Directory.GetFiles(sourceFolder);
            foreach (string file in files)
            {
                string name = Path.GetFileName(file);
                string dest = Path.Combine(destFolder, name);
                DateTime WriteTime = File.GetLastWriteTime(dest);
                IniFunctions.WriteToIni(sConfigLocation, sDriveName, file, WriteTime.ToString(), iniFile);
                object fileCopying1 = file;
                PenDriveBackup myClass = new PenDriveBackup();
                myClass.backgroundWorker1.ReportProgress(0, file); // This will call backgroundWorker1_ProgressChanged method in the main class.
                                                                   //  If I were to put a MessageBox.Show(message); into the ProgressChanged method
                                                                   //  then I can see the correct files changing however the label does not get updated.
                                                                   //  I'm not sure what the problem is but I'm guessing it's something to do with the
                                                                   //  PenDriveBackup myClass = new PenDriveBackup(); not taking control of the form?
                File.Copy(file, dest, true);
            }
            string[] folders = Directory.GetDirectories(sourceFolder);
            foreach (string folder in folders)
            {
                string name = Path.GetFileName(folder);
                string dest = Path.Combine(destFolder, name);
                Console.WriteLine("Copying " + folder);
                FullCopy(sConfigLocation, folder, dest, sDriveName, iniFile);
            }
        }
     }

It's been annoying me a lot, trying a lot of different methods and to no avail none of them work.
I don't really understand threading all that much so I'm shooting in the dark here a little bit.

Thanks in advance George.

Re: Perhaps a threading problem?

Luc Pattyn14-Dec-09 11:56

Luc Pattyn

14-Dec-09 11:56

Hi George,

while you clearly attempted to explain things well, you did manage to confuse me: what is the name of the class holding most of the code shown? and what is inside PenDriveBackup?

are you sure you want a new PenDriveBackup() instance created in your FullCopy() method, which is after all a recursive method?

you might answer these by providing a simple class schematic, or by editing your initial post.

Smile | :)

Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles]

I only read code that is properly indented, and rendered in a non-proportional font; hint: use PRE tags in forum messages

Re: Perhaps a threading problem?

George Quarton14-Dec-09 12:17

George Quarton

14-Dec-09 12:17

Okay sorry for not explaining this clearly. My goal is simple really. What I want to achieve is;

1. Detect a USB Insertion (this works fine)
2. Upon detection begin copying the contents from the USB drive onto the hard drive. (this works fine)
3. Update a label control on my form

Now if we call the FileCopying method in the same thread the form can not be updated (the form just freezes as copying is taking place)

To count-act this I went into the Designer View and added a 'BackgroundWorker' control which allowed me to run the FileCopying method in that thread so
that the form is able to be updated. (This works, the copying takes place and the form isn't frozen)

Now as point 3. on my list is to update the control this is what I need to do.
Hopefully this can be explained better in code than words so;

public void backgroundWorker1_DoWork(object sender, DoWorkEventArgs e)
        {
            CopyObject a = e.Argument as CopyObject;
            RecursiveCopy.FullCopy(a.sConfigLocation, a.sourceFolder, a.destFolder, a.sDriveName, a.iniFile);
            e.Result = "1";
        }

This is the 'BackgroundWorker' control as added before and the RecursiveCopy.FullCopy is called in this function (which is called in another thread).
This works fine, the file copy operation is called succesfully as shown here;

public class RecursiveCopy
    {
        public static void FullCopy(string sConfigLocation, string sourceFolder, string destFolder, string sDriveName, object iniFile)
        {
            if (!Directory.Exists(destFolder))
                Directory.CreateDirectory(destFolder);
            //System.Windows.Forms.ToolTip ToolTip1 = new System.Windows.Forms.ToolTip();
            int countFile = 0;
            string[] files = Directory.GetFiles(sourceFolder);
            foreach (string file in files)
            {
                string name = Path.GetFileName(file);
                string dest = Path.Combine(destFolder, name);
                // below here I am trying to update the label control on my form
                // Perhaps this is the wrong way to go about it?
                PenDriveBackup myClass = new PenDriveBackup(); // the PenDriveBackup class is where the form resides.
                myClass.backgroundWorker1.ReportProgress(0, file); // the backgroundWorker1 function sits in the PenDriveBackup class.
                // The ReportProgress() method will pass the file name into the ProgressChanged method which I will document below.
                File.Copy(file, dest, true);
            }
            string[] folders = Directory.GetDirectories(sourceFolder);
            foreach (string folder in folders)
            {
                string name = Path.GetFileName(folder);
                string dest = Path.Combine(destFolder, name);
                Console.WriteLine("Copying " + folder);
                FullCopy(sConfigLocation, folder, dest, sDriveName, iniFile);
            }
        }
    }

public void backgroundWorker1_ProgressChanged(object sender, ProgressChangedEventArgs e)
        {
            string message = e.UserState as string;
            fileCopyingLbl.Text = message; // Update the label control.
        }

So from what I know the ProgressChanged method is there so you can update your Form. So from what I believed should work from calling this method is that
the label will be updated as shown in the code.

If I were to put a MessageBox in the method then I can see that the file names are passed into the method properly.
For example;

public void backgroundWorker1_ProgressChanged(object sender, ProgressChangedEventArgs e)
        {
            string message = e.UserState as string;
            MessageBox.Show(message); // Here
            fileCopyingLbl.Text = message; // Update the label control.
        }

Everytime a file gets copied in the RecursiveCopy.FullBackup method it will send the data into that function and I can see the MessageBoxes for each file.
So I know the data is getting sent between the methods correctly just that the label control is not getting updated.

Perhaps I am tackling the problem the wrong way? Maybe with the way I'm calling it?

PenDriveBackup myClass = new PenDriveBackup();
     myClass.backgroundWorker1.ReportProgress(0, file);

I hope the information this time is enough to hel you get a grasp of the project I am under-going. Thanks for replying, George.

Re: Perhaps a threading problem?

Luc Pattyn14-Dec-09 12:41

Luc Pattyn

14-Dec-09 12:41

Hi George,

you did not really answer the three questions I asked you, however you added a comment which probably points to the main problem:

George Quarton wrote:
PenDriveBackup myClass = new PenDriveBackup(); // the PenDriveBackup class is where the form resides.

so you create a NEW form; this new form is not the one you already had and probably are looking at, and it is never shown; it has its own instance of everything, including a backgroundworker and a label; that label now gets updated, which will not do anything on the original form, the only one that is visible.

I was somehow expecting this. As I already said before, creating new things in a recursive method often is a mistake.

BTW: You should be more careful when choosing names; I'd suggest PenDriveBackupForm (normally class names are nouns and most often their name suggests what they really are or represent, here a form; methods on the other hand are best named using verbs as they are doing something).

Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles]

I only read code that is properly indented, and rendered in a non-proportional font; hint: use PRE tags in forum messages

Re: Perhaps a threading problem?

George Quarton14-Dec-09 12:45

George Quarton

14-Dec-09 12:45

Ah! I see. Silly mistake. How would I go about updating the form directly then?

I can not call the backgroundWorker1_ProgressChanged method as its apart of another class and is not able to use the static identifier. (I dont have much experience in c# coming from a scripting language things are a little different)

Thanks again for your help and advice.

Re: Perhaps a threading problem? [modified]

Luc Pattyn14-Dec-09 13:00

Luc Pattyn

14-Dec-09 13:00

I would organize things a bit differently:

1) simple scheme

1A. have your PenDriveBackupForm as it is, however without the BackgroundWorker, as it does not belong to the form.

1B. have your RecursiveCopy class, with a constructor that creates a single (and private) BackgroundWorker (with the "new" keyword), to be used inside that class only; and give it a Label parameter, which you store in a private class member, so the BGW inside that class can access it.

2) better scheme (more object-oriented; keywords are: delegate, event)

2A. same as 1A.

2B. have your RecursiveCopy class, with a constructor that creates a single (and private) BackgroundWorker, to be used inside that class only; don't give it a Label parameter; give the class a public event of type Action<string> which almost means "function pointer to a function that takes a string and returns nothing". Now let your BGW fire that event.

2C. Add a "SetProgressLabel()" method to PenDriveBackupForm to accept a string and set the Label; then also add a delegate for that method to the public event you provided in RecursiveCopy.

The net result is: RecursiveCopy will execute a method of which it does not know much, and SetProgressLabel() will be called when necessary; the coupling between both classes is minimal, you can change the Label to something else without telling RecursiveCopy at all (which can't be done in the simple scheme).

Smile | :)

Luc Pattyn [Forum Guidelines] [Why QA sucks] [My Articles]

I only read code that is properly indented, and rendered in a non-proportional font; hint: use PRE tags in forum messages

modified on Monday, December 14, 2009 7:35 PM

vtchris-peterson14-Dec-09 8:38

vtchris-peterson

14-Dec-09 8:38

Using SMO with SQL Express 2005, I'm experiencing some frustrating behavior.

I'm using SMO Server and Database classes to script my database using Database.ExecuteNonQuery. The database objects are getting scripted correctly, but I'm having problems re-connecting to the database (using SqlConnection.Open). I get a SqlException ("System.Data.SqlClient.SqlException: Cannot open database \"MyDatabase\" requested by the login. The login failed.\r\nLogin failed for user 'MyDomain\\MyUser'.\r\n at System.Data.ProviderBase.DbConnectionPool.GetConnection(DbConnection owningObject)\r\n at System.Data.ProviderBase.DbConnectionFactory.GetConnection(DbConnection owningConnection)\r\n at System.Data.ProviderBase.DbConnectionClosed.OpenConnection(DbConnection outerConnection, DbConnectionFactory connectionFactory)\r\n at System.Data.SqlClient.SqlConnection.Open()\r\n at MyCodePath\\DataAccessFactory.cs:line 57")

The problem "goes away" if I just wait before trying to create the second connection e.g. if I use a breakpoint and step through the code it works fine.

I am closing my previous connection (the one where I scripted the database) and everything is sequential on a single (the UI) thread.

I'd hate to have to code some delay or retry logic for something that seems pretty standard.

Migounette14-Dec-09 20:44

Migounette

14-Dec-09 20:44

It's normal, you may use a connection pool in order to avoid problems of connection.

myConnection.ConnectionString = "Persist Security Info=False;Integrated Security=SSPI;database=MyDatabase;server=MyServer;Connect Timeout=30";

The value is expressed in seconds:

Connect Timeout=30

vtchris-peterson15-Dec-09 3:56

vtchris-peterson

15-Dec-09 3:56

That's kind of annoying. I've noticed these types of subtleties even when using standard MS tools like SSMS. My first reaction when something fails is to change some variable of the experiment before trying again, but with SQL Server, a lot of times, you just need to act like a crazy person and try the EXACT same thing expecting different results.

Thanks for your tip, I had managed to workaround the issue, but I might implement this suggestion as well.

C# RichTextBox strange behaviour

ikurtz14-Dec-09 8:13

ikurtz

14-Dec-09 8:13

greetings,

for the first time ever im investigating RichTextBox control in C# windows forms. i know i need this control in my app as textBox is to simple for my needs.

i have the followig code:

<br />
using System;<br />
using System.Collections.Generic;<br />
using System.ComponentModel;<br />
using System.Data;<br />
using System.Drawing;<br />
using System.Linq;<br />
using System.Text;<br />
using System.Windows.Forms;<br />
<br />
namespace _19_richtextbox<br />
{<br />
    public partial class Form1 : Form<br />
    {<br />
        public Form1()<br />
        {<br />
            InitializeComponent();<br />
        }<br />
<br />
        private void richTextBoxHome_KeyDown(object sender, KeyEventArgs e)<br />
        {<br />
            if (e.KeyValue == (char)(Keys.Return))<br />
            {<br />
                richTextBoxChat.AppendText("Home:" + richTextBoxHome.Text + "\n");<br />
                richTextBoxHome.Clear();<br />
            }<br />
        }<br />
<br />
    }<br />
}<br />

for the moment i just want whatever is typed in one richtextbox to be displayed on the other richtextbox on hitting return.

the issue is everytime i hit return the data is being transfered to the other control but the first control is left with a carriage return before the cursor. this happens everytime i hit return.

how do i make it stop doing this?

both the controls accept multiline input.

please advise if i need to provide more info.

thank you for your time.

Re: C# RichTextBox strange behaviour

OriginalGriff14-Dec-09 8:42

OriginalGriff

14-Dec-09 8:42

If you handle the return key and don't tell the OS you have done it, it passes it through to the default handler afterwards. Add "e.Handled = true;" to your code, all will be fine.

[edit]I can't spell "the" correctly - fixed.[/edit]

All those who believe in psycho kinesis, raise my hand.

Re: C# RichTextBox strange behaviour

ikurtz14-Dec-09 8:50

ikurtz

14-Dec-09 8:50

thank you for your response. it has solved my problem.

may i introduce you to my new situation?

i would like a way to make the "Home:" part in bold.

i have found very little info on this on my searches. the following are the only actual code that i could understand.

rtb1.Rtf = @"{\rtf1\ansi {\b hello} {\i World}}" ;

richTextBox1.Rtf = @"{\rtf1\ansi This is in \b bold\b0.}";

-----------

im not sure how to proceed with this info. i just want the richtextbox to display "Home:" and "Away:" in bold and be able to handle URLs in text.

please advise what should i specify when searching this potic on google or any reference your could think of would be of great help.

Re: C# RichTextBox strange behaviour

OriginalGriff14-Dec-09 9:09

OriginalGriff

14-Dec-09 9:09

You seem to have most of what you need there already, but have a look at this[^] - it's in VB, but it's pretty clear.

Just think of rtf as an unreadable HTML file and you are just about there! Laugh | :laugh:

All those who believe in psycho kinesis, raise my hand.

Re: C# RichTextBox strange behaviour

ikurtz14-Dec-09 9:14

ikurtz

14-Dec-09 9:14

thank you!

WONDER

farokhian14-Dec-09 7:58

farokhian

14-Dec-09 7:58

Hi Am i ask a hard question about 4hours ago.

Re: WONDER

loyal ginger14-Dec-09 8:08

loyal ginger

14-Dec-09 8:08

Your question is indeed tough. You have to wait patiently for an answer by somebody who knows the answer.

GOLDEN WONDER

Keith Barrow14-Dec-09 8:09

Keith Barrow

14-Dec-09 8:09

The reason you did not get an answer is because

a) It was a repost.

b) Your original was replied to, and you reposted last time
http://www.codeproject.com/Messages/3302834/Re-backup-database.aspx[^]

c) There are many examples on CP about to achieve what you want to achieve.

CCC solved so far: 2 (including a Hard One!)
37!?!! - Randall, Clerks

Re: GOLDEN WONDER

farokhian14-Dec-09 8:16

farokhian

14-Dec-09 8:16

My question was changed.
I asked that how can i backup from database.
I got my answer and check it when my database attach to microsoft sql.
but it is necessary for me to use SQLEXPRESS and that way doesn't work when i add my database to SQLEXPRESS and detach from microsoft sql.
I explain more about 4 hours ago.
If somebody know the answer please help me.

Re: WONDER

OriginalGriff14-Dec-09 8:39

OriginalGriff

14-Dec-09 8:39

Does shouting and hassling people work in your company? Because in all the ones I've worked in, it just gets peoples backs up, and (certainly with me) reduces the amount of help given to the minimum they can get away with. Just thought I'd mention it...

All those who believe in psycho kinesis, raise my hand.

Re: WONDER

farokhian14-Dec-09 8:59

farokhian

14-Dec-09 8:59

please !
I don't think that i'm shouting or... .
I thought that i used wrong sentence or wrong structure because my first language is not english.
but if i told in bad way , i'm sorry.

Use Ctrl+Left/Right to switch messages, Ctrl+Up/Down to switch threads, Ctrl+Shift+Left/Right to switch pages.