 |
|
 |
I've had this issue when I used the wrong Timer before. Never have figured this out, but it seems to me if you are using a System.Windows.Forms.Timer vs. a System.Timers.Timer you get different functionality, especially in a service.
Try out the opposite timer (probably you'll want to stick with System.Timers.Timer).
|
|
|
|
 |
|
 |
Thankx a lot Lee.
You saved me hell lots of time.
Simple but simply great !
jimmy
Jimmy S
|
|
|
|
 |
|
 |
Often the simplest things are the most amazing...I'm obviously easily impressed.
Suggestions offered by jriesen below were also very useful.
Thanks very much
|
|
|
|
 |
|
 |
Nice job!
Thanks
fsdafsda
|
|
|
|
 |
|
|
 |
|
 |
Your solution looks like you actually thought about it.
I originally also borrowed the first idea of using a #DEBUG directive for debugging a service. Unfortunately the rest of that code was a bit too much 'crude' rather than 'effective'. But it has been interesting to watch the pieces come together on this issue. Actually that's kind of worrying considering how long ago I first wrote this article. Maybe Macroslop will *borrow* your idea as part of the default template for windows service projects.
|
|
|
|
 |
|
 |
Hah! That'd be something.
The solution I came up with has a flaw though: it only tests the methods you implement on the service, not its ability to live in the Windows Services environment. At least accurately, anyway.
For example, if you (for some reason) didn't know you couldn't put UI code in a service without first checking "Allow Service to Interact with the Desktop" (a terrible practice anyway), then you wouldn't find this bug until you installed the service with installutil.exe and ran it from there. Similarly, you are going to have problems with calls to this.EventLog.Write* because the service won't be installed and therefore the event log won't have information for your service registered with it yet.
These kinds of issues come up no matter what solution you try. It's sad that they can't have special support in VS for debugging these things. Maybe in Orcas?
|
|
|
|
 |
|
 |
I like to use System.Environment.MachineName to base my debugging on. if I'm in development environment then I do one thing else I use production resources.
|
|
|
|
 |
|
 |
In more recent versions of this project I've gone and done the same.
|
|
|
|
 |
|
 |
myFeedback = "Thank you, thank you, thank you ...";
|
|
|
|
 |
|
 |
Thanks, Lee - I love elegant and simple solutions. I am going to incorporate this into a template so the code is automatically inserted when I spin up a new service project!
-Dave Ferreira
It IS what you say, not just how you say it!
|
|
|
|
 |
|
 |
I don't know about elegant - but simple, yep I'd agree with that.
I'd also suggest reading some of the more recent comments below. Some others have made some great contributions that add the missing elegance to my code above.
Next, a XAML management console where if you highlight the rotating spotlight beam while licking the USB port on your DNS server (may require virtual tongue - that's another project) you get a mocked up terminal services window showing the state of your service in glorious surround sound.
|
|
|
|
 |
|
 |
That would only rate a 3. You should use a Firewire 800 virtual tongue instead of a USB, and while surround sound is nice in a late 80's kind of way, you should try creating a VR world using Aero. Of course, that's just my opinion - I could be wrong.
Everyone's a critic.....
-Dave Ferreira
"When everyone is out to get you, paranoia is just good thinking...."
|
|
|
|
 |
|
 |
Quite true about the surround sound vs. Aero. Although I think the firewire is overkill, after all the tongue (virtual or real) won't be doing anything as intensive as saliva streaming ( oow yuck - very bad mental imagery - anyway what's typed is typed ).
|
|
|
|
 |
|
 |
I just sat down to eat when I read that. Score one for the diet.
-Dave Ferreira
"Time flies, but never on Delta."
|
|
|
|
 |
|
 |
Perhaps I shouldn't have started that thread - in the end it was like trying to put out a flamewar by hoicking up virtual phlegm.
|
|
|
|
 |
|
 |
Until now, I've always opted for the "rip your code logic into an assembly and have both your windows service call it for deployment & a command line app call it for debugging" approach. This is much nicer!
|
|
|
|
 |
|
 |
Thanks,
The only real downside I've ever found with this approach is if you forget to do an actual release build before putting your service into production.
- But that soon shows up as something wrong...;)
|
|
|
|
 |
|
 |
You can access the command-line arguments during the service startup (just change the "static void Main()" into "static void Main(string[] args)").
My service will run as a normal command-line app, similar to what yours does in a Debug build, if you give it the "/DEBUG" argument on the command-line. This lets me run it manually inside and outside of the debugger, whether I've done a release or a debug build, and not have to worry about shipping the wrong version either.
The only additional change is to edit the project settings to add "/DEBUG" to the command-line arguments for debugging.
|
|
|
|
 |
|
 |
That was an elegant solution to the problem, even if it seems a little quick-and-dirty. What's funny is that I'm actually using this approach (with modifications) in a real-life production application at the moment. It got a little more complicated because I ended up wanting the ability to launch the service from other assemblies (e.g. from unit tests), but it works great for me... so I figured I'd share. Not like you necessarily want to use this code directly, but it shows how to mash the OnStart method, rather than calling a public method in the service. (I use it in this case because my service is so utterly short I'd rather not even bother with private methods.) using System.Reflection; using System.ServiceProcess; public static class Program { // Note that the real service has been renamed to MyService to protect the innocent. private static MyService service = new MyService(); internal static void Main() { #if (!DEBUG) ServiceBase.Run( new ServiceBase[] { service } ); #else Start(); System.Threading.Thread.Sleep(System.Threading.Timeout.Infinite); #endif } public static void Start() { InvokeServiceMethod("OnStart"); } public static void Stop() { InvokeServiceMethod("OnStop"); } private static void InvokeServiceMethod(string methodName, params object[] parameters) { if (parameters == null || parameters.Length < 1) { parameters = new object[] { null }; } service.GetType().InvokeMember(methodName, BindingFlags.Instance | BindingFlags.Public | BindingFlags.NonPublic | BindingFlags.InvokeMethod, null, service, parameters); } }
Note the use of reflection to smash the OnStart method (or, really, any instance method you would like). Might be overkill, but I like reflection. Also, you can probably scrap the compile-time test and replace it with one at runtime. I'm thinking about switching my code to do this (but be warned, it's not fully tested yet): if (System.Diagnostics.Debugger.IsAttached) { Start(); System.Threading.Thread.Sleep(System.Threading.Timeout.Infinite); } else { ServiceBase.Run(new ServiceBase[] { service }); } Naturally, this assumes that you only want it to start this way while running in the Visual Studio (or other unmanaged) debugger. If you need to launch the service from the command line in the debugging mode, then it won't work. There -may- be some way to detect whether the app is being launched as a service (perhaps, say, through execution credentials - if you're running as LOCALSYSTEM it would be a good indicator), but I don't really need that functionality so I won't be pursuing it here.
Anyway, thanks for your help! Thought I'd try and give a little back too.
|
|
|
|
 |
|
 |
Thanks for the reply, and the suggestions, I'll give them a go myself - Yes, I use my code in production systems myself.
I've been seriously thinking about rewriting this to do things *better* - i.e. less 'crude' but still 'effective'.
PS. For those who don't know CBE stands for 'Commander of the Order of the British Empire', which ranks just below a knighthood.
|
|
|
|
 |
|
|
 |
|
 |
It works great. I did have to add a method service.OnDebugStart() that takes the place of the overridden OnStart method but with the same functionality... for debug only purposes. It makes your code: service.<Your Service's Primary Method Here>(); obsolete. the line: System.Threading.Thread.Sleep(System.Threading.Timeout.Infinite); is still necessary I also got it to work with SQL Server... no problem -- modified at 16:37 Tuesday 25th April, 2006
|
|
|
|
 |
|
 |
This is a very quick way to debug classes which are wrapped by a windows service.
|
|
|
|
 |
|
 |
System.Diagnostics.Debugger.Break();
Is there anything wrong with this option?
|
|
|
|
 |