Lee Mac Posted April 29, 2010 Posted April 29, 2010 Nice one Buzzard, Bear in mind though that as it stands, the program will still try to execute, even if the load_dialog fails, due to how you have structured the last Cond statement. Also, the COND and IF together are redundant when testing the same variable in a few of your functions, as both are conditionals, I would either use one or the other Again, just suggestions for improvement. Lee Quote
The Buzzard Posted April 29, 2010 Author Posted April 29, 2010 I guess I did not fix that correctly? Quote
Lee Mac Posted April 29, 2010 Posted April 29, 2010 The Buzzard said: I guess I did not fix that correctly? Not quite. Again, let me re-iterate though, there wasn't too much wrong with your original code, as always, there are many ways to code the same thing, and I wouldn't like you to think that I am nagging you into changing parts of your program, I'm just offering ideas as to how I would approach the coding, I will not be offended if you decide not to incorporate my suggestions Quote
The Buzzard Posted April 29, 2010 Author Posted April 29, 2010 I think you are referring to where the conditional has ended? I am not quite sure where it is suppose to end. At the end of the whole Main Function I think? Quote
Lee Mac Posted April 29, 2010 Posted April 29, 2010 Ok, the Cond is correct now - but do you understand why we are using it in that way? I wouldn't want you to use code just because I recommend it. Quote
The Buzzard Posted April 29, 2010 Author Posted April 29, 2010 Now I see what you are saying about the cond & if being redundant. I did not know I could combine them, But it makes good sense. I have now fixed that. Quote
The Buzzard Posted April 29, 2010 Author Posted April 29, 2010 Lee Mac said: Ok, the Cond is correct now - but do you understand why we are using it in that way? I wouldn't want you to use code just because I recommend it. The light bulb lit up the last minute. I understand it now as I had to figure it out for myself. They share the same variable, so they should be under the same conditional. Thanks Quote
The Buzzard Posted April 30, 2010 Author Posted April 30, 2010 Attention All! Version V0.2 has been uploaded to the first post. Added Non-Coded Flanged Only Head. Enjoy the code. The Buzzard Quote
ReMark Posted April 30, 2010 Posted April 30, 2010 No fair! You're changing it faster than I can use it! LOL Thank you Buzzard. Already shared it with one fellow CAD user and he went crazy with glee! Quote
The Buzzard Posted April 30, 2010 Author Posted April 30, 2010 ReMark said: No fair! You're changing it faster than I can use it! LOL Thank you Buzzard. Already shared it with one fellow CAD user and he went crazy with glee! Sorry about that, I will put it to rest for while. I have other codes to start. Quote
Tankman Posted April 30, 2010 Posted April 30, 2010 Nice job Buzzard. Saving for future use. VA Hospital, Walter Reed, my father passed away there after open heart surgery. It was 1956, pneumonia, after spending many hours packed in ice. Guess that was the old way to slow the blood flow. Quote
The Buzzard Posted May 1, 2010 Author Posted May 1, 2010 Tankman said: VA Hospital, Walter Reed, my father passed away there after open heart surgery.It was 1956, pneumonia, after spending many hours packed in ice. Guess that was the old way to slow the blood flow. Very sorry to hear about that. Tankman said: Nice job Buzzard. Saving for future use. Thanks, But save for future use?... With a name like Tankman, I would think you would get a lot of use out of this program soon. Quote
Tankman Posted May 1, 2010 Posted May 1, 2010 Future use? Looks great! I used appload to load the DH lisp file, loaded with no problem. I then tried to bring up the menu, using "DH" on the command line, nada, zip, nothin'. At this point, I really feel "stupid." What or how do I get the dialog box to open? Help! I need this! Quote
The Buzzard Posted May 1, 2010 Author Posted May 1, 2010 Tankman said: Future use? Looks great! I used appload to load the DH lisp file, loaded with no problem. I then tried to bring up the menu, using "DH" on the command line, nada, zip, nothin'. At this point, I really feel "stupid." What or how do I get the dialog box to open? Help! I need this! Hey Tankman, Just keep both files together and make sure that the directory they are in is in the ACAD Support Search Path. You can do this under Options file tab (See Image Below) or you could just put the files in any of ACADs subdirectories, But I would not recommend that. This link also has information on loading lisp files with dcl: Refer to Lee Macs section in post 2 on additional loading instructions. http://www.cadtutor.net/forum/showthread.php?t=1390 Quote
ReMark Posted May 1, 2010 Posted May 1, 2010 I put both files in C:\Program Files\AutoDesk 2010\Support. Quote
The Buzzard Posted May 1, 2010 Author Posted May 1, 2010 ReMark said: I put both files in C:\Program Files\AutoDesk 2010\Support. That will work, But as I said I do not recommend doing it that way. If you ever upgrade your CAD version, You could run the risk of losing the programs. Quote
ReMark Posted May 1, 2010 Posted May 1, 2010 Only if you do not keep a copy of the original zip file and the two files it contains in the c:\unzipped\DH_0V folder. Gotcha. Quote
The Buzzard Posted May 1, 2010 Author Posted May 1, 2010 ReMark said: Only if you do not keep a copy of the original zip file and the two files it contains in the c:\unzipped\DH_0V folder. Gotcha. I guess you did. Just hope you have it in a place that is safe and one you will remember. Just to mention, The file could also be compiled in to a VLX or FAS and then you do not have to worry about directories at all. Quote
ReMark Posted May 1, 2010 Posted May 1, 2010 I also backup to my flash drive and have a copy on my home computer. The state of Connecticut would have to disappear from the face of the earth for me to lose it. LOL I would like to hear more about compiling in to a VLX or FAS if you would care to elaborate. Thanks. Got to go. Promised the wife I'd take her to lunch. But...I'll be back. Quote
Recommended Posts
Join the conversation
You can post now and register later. If you have an account, sign in now to post with your account.
Note: Your post will require moderator approval before it will be visible.