I have a question - C# Discussion Boards

14-Sep-06 5:59

I'm trying to set a windows timer event to fire, and when it does it will close the current drawing (Autocad) and open a new one. I can get the event to fire, but for some reason it doesn't seem to want to close the drawing. It knows the stuff for Autocad, as I put in messages boxes to make sure it does, it just won't do what I want it to do. Any body tried this before? Knows (has ideas) as to how I can do this?

The program is supposed to open a set of drawings, one at a time, for a specified time period.

Any/all help is appreciated. I'm pretty new to C#, just so you know.

If you want to see the code, I can post it.

Thanks.

Tim

Nader Elshehabi14-Sep-06 9:52

14-Sep-06 9:52

Hello

T.Willey wrote:
If you want to see the code, I can post it.

Sure!! Posting the code that closes/opens the drawings would do a great help.

Regards

T.Willey14-Sep-06 11:19

14-Sep-06 11:19

Thanks for the response. Here is the code. It will open the first drawing, issue the '3dorbit' command, fire the timer event, but then it stops. It will not close the current drawing, nor open the next one.

using System;
using System.Diagnostics;
using System.Timers;
using System.Windows.Forms;
using System.Runtime.InteropServices;

using Autodesk.AutoCAD.ApplicationServices;
using Autodesk.AutoCAD.Runtime;

using AcadApp = Autodesk.AutoCAD.ApplicationServices.Application;

[assembly: CommandClass(typeof(Test.TimeLimitOpenv02))]

namespace Test
{
/// <summary>
/// Description of TimeLimitOpen.
/// </summary>
public class TimeLimitOpenv02
{

[DllImport("acad.exe", CallingConvention = CallingConvention.Cdecl,
EntryPoint = "?acedPostCommand@@YAHPBD@Z")]
extern static public int acedPostCommand(string cmd);
public string[] DwgList;
public Document NewDoc;
DocumentCollection DocMan = AcadApp.DocumentManager;
System.Timers.Timer CountDownTimer;
int cnt = 1;
//- Constructor: sets up var's etc for this class before use.
public TimeLimitOpenv02()
{
CountDownTimer = new System.Timers.Timer();
CountDownTimer.Elapsed += new ElapsedEventHandler(CancelCommand);
CountDownTimer.Interval = 10000;
}

~TimeLimitOpenv02() // <-- Destructor, do clean up here.
{
CountDownTimer.Elapsed -= new ElapsedEventHandler(CancelCommand);
}
public void CancelCommand(object sender, ElapsedEventArgs e)
{
acedPostCommand ("CancelCmd");
//MessageBox.Show (DocMan.ToString());
NewDoc = DocMan.MdiActiveDocument;
//MessageBox.Show ("Testing");
//MessageBox.Show (cnt.ToString());
NewDoc.CloseAndDiscard();
NewDoc = DocMan.Open (DwgList[cnt], true);
if (NewDoc != DocMan.MdiActiveDocument) {
DocMan.MdiActiveDocument = NewDoc;
}
++cnt;
if (cnt > DwgList.Length) {
CountDownTimer.Enabled = false;
CountDownTimer.Dispose();
}
NewDoc.SendStringToExecute ("_.3dorbit", false, false, true);
}

[CommandMethod("TestTimer", CommandFlags.Session)]
public void Main()
{
Autodesk.AutoCAD.Windows.OpenFileDialog Dia =
new Autodesk.AutoCAD.Windows.OpenFileDialog(
"Select drawings to update Cloud layer", "", "dwg", "",
Autodesk.AutoCAD.Windows.OpenFileDialog.OpenFileDialogFlags.AllowMultiple);
Dia.ShowDialog();
string[] DwgList = Dia.GetFilenames();
if (DwgList.Length > 1)
{
CountDownTimer.Enabled = true;
Document NewDoc = DocMan.Open (DwgList[0], true);
if (NewDoc != DocMan.MdiActiveDocument) {
DocMan.MdiActiveDocument = NewDoc;
}
NewDoc.SendStringToExecute ("_.3dorbit\n", false, false, true);
}
}
}
}

I can't seem to keep the formatting. Nevermind, I think I did it right.

Tim

Nader Elshehabi14-Sep-06 11:37

14-Sep-06 11:37

Hello

Well, I wonder a bit about how your code did compile at all!!

your main method is public while it should have been static private. This -I believe- should make a compile error!!

Anyway, I don't understand much of Autocad's interop method calls. I hope you are sure they should do the work. You said that the elapsed event is fired?? Does that mean that your CancelCommand is called and exctuted properly?

BTW:
You've declared DwgList twice. The one in main method is not the one you are accessing in the CancelCommand method. Try to debug through your code and see its value in the first time the event is fired.

Regards

T.Willey14-Sep-06 12:06

14-Sep-06 12:06

I don't know why/how it compiles either, but it does. =D

The CancelCommand does fire, but you are correct in that my DwgList isn't anything in the CancelCommand part. I thought by adding it to the main part of the class, that it could be used inside the code, and shared with other parts of code as long as it was within the same class. Guess I was wrong, back to reading some more.

I guess I need to figure out how to have two global variable. One for DwgList and one for cnt. Or figure out a way to pass them between the calls to CommandCancel.

Thanks for your help, I don't think I would have ever got this far without it.

Tim

Nader Elshehabi14-Sep-06 12:26

14-Sep-06 12:26

Hello

Well, forgive me if I say that your code sin't Object oriented. It looks like you have good experience in languages before C++. Your style is very procedural and it sin't that bad. It's just that your applications doesn't even have an interface?!! All you do is that you show a dialog, ake an input then do some interop. Well, either you make a full functional windows application or wrap your code in a dll or something. I didn't mean to ofend you - sincerely-, I just wanted to be as honest as I can with you.

Anyway, I wouldn't like to disrupt your code structure, so, the closest solution to get your code to work -though it would be better to remodel your code later to be more OO-, is simply to declare your variables as static thus you'd be able to call them by the class.
Here is your code, modified:

namespace Test
{
    /// <summary>
    /// Description of TimeLimitOpen.
    /// </summary>
    public class TimeLimitOpenv02
    {
        
        [DllImport("acad.exe", CallingConvention = CallingConvention.Cdecl,
            EntryPoint = "?acedPostCommand@@YAHPBD@Z")]
        extern static public int acedPostCommand(string cmd);
        private static string[] DwgList;//<---Modification
        public Document NewDoc;
        DocumentCollection DocMan = AcadApp.DocumentManager;
        System.Timers.Timer CountDownTimer;
        private static int cnt = 1; //<---Modification
        //- Constructor: sets up var's etc for this class before use.
        public TimeLimitOpenv02()
        {
            CountDownTimer = new System.Timers.Timer();
            CountDownTimer.Elapsed += new ElapsedEventHandler(CancelCommand);
            CountDownTimer.Interval = 10000;
        }

        ~TimeLimitOpenv02() // <-- Destructor, do clean up here.
        {
            CountDownTimer.Elapsed -= new ElapsedEventHandler(CancelCommand);
        }
        public void CancelCommand(object sender, ElapsedEventArgs e)
        {
            acedPostCommand ("CancelCmd");
            //MessageBox.Show (DocMan.ToString());
            NewDoc = DocMan.MdiActiveDocument;
            //MessageBox.Show ("Testing");
            //MessageBox.Show (cnt.ToString());
            NewDoc.CloseAndDiscard();
            NewDoc = DocMan.Open (DwgList[cnt], true);
            if (NewDoc != DocMan.MdiActiveDocument) {
                DocMan.MdiActiveDocument = NewDoc;
            }
            ++TimeLimitOpenv02.cnt;//You access it by the class name
            if (TimeLimitOpenv02.cnt > TimeLimitOpenv02.DwgList.Length) {
                CountDownTimer.Enabled = false;
                CountDownTimer.Dispose();
            }
            NewDoc.SendStringToExecute ("_.3dorbit", false, false, true);
        }

        [CommandMethod("TestTimer", CommandFlags.Session)]
        public void Main()
        {
            Autodesk.AutoCAD.Windows.OpenFileDialog Dia =
                new Autodesk.AutoCAD.Windows.OpenFileDialog(
                "Select drawings to update Cloud layer", "", "dwg", "",
                Autodesk.AutoCAD.Windows.OpenFileDialog.OpenFileDialogFlags.AllowMultiple);
            Dia.ShowDialog();
            TimeLimitOpenv02.DwgList = Dia.GetFilenames();
            if (DwgList.Length > 1)
            {
                CountDownTimer.Enabled = true;
                Document NewDoc = DocMan.Open (DwgList[0], true);
                if (NewDoc != DocMan.MdiActiveDocument) {
                    DocMan.MdiActiveDocument = NewDoc;
                }
                NewDoc.SendStringToExecute ("_.3dorbit\n", false, false, true);
            }
        }
    }
}

BTW:
You still can access DwgList from your main, and your CancelCommand if you didn't redeclare it. Your redeclaration made a second local variable with the same name that masked the original one -I hope I'm making sense!;P-. Anyway, even though I totally don't like this static workaround, but it would relieve the ambiguity, and get your code to work. But if you want to make a windows application, at least it should have a form, shouldn't it? Well, Maybe you are making a command line tool -never liked them though-. Good luck! I hope it works this time.

Regards

T.Willey14-Sep-06 12:55

14-Sep-06 12:55

Thanks for you help, and you can be as blunt as you want to be. I know I'm learing as I go, and my only background with programming is within the AutoCAD enviroment, and that was with Lisp.

It is going to be run for a presentation on 3d drawings. It will only allow the people to use the 3dorbit command from within Acad for a limited amout of time per drawing. It will be ran from the command line, and loaded in as a dll file.

Thanks again for all the help. I will let you know when I get it working.

Tim

T.Willey14-Sep-06 13:10

14-Sep-06 13:10

It looks like the problem now is that it doesn't know how to get the current drawing NewDoc = DocMan.MdiActiveDocument;. Even if I put it in the long way NewDoc = Autodesk.AutoCAD.ApplicationServices.Application.DocumentManager.MdiActiveDocument;. It still won't be able to find it, and in turn execute the commands on it.

Is the timer taking control away from Acad? Is that why it can't find it?

If you have any ideas, I'm will to try them, as I'm totally lost now. Thanks.

Tim

Nader Elshehabi14-Sep-06 13:41

14-Sep-06 13:41

Ooops!!:-> I missed that
Docman and NewDoc too should be declared as static if you want to share them between main and CancelCommand. Don't forget to call them by the class name.

Regards

T.Willey14-Sep-06 14:02