Spec checking?

[Stree]

As much as i like you, Stree. It wouldnt hurt to ask before publishing any shape or form of my work in any forum section.

Sorry, I checked if you were online before posting and assumed it was alright being that anyone can see the code if they open the file with notepad.

 

[Stree]

I don't know if (unit.IsTargetingMyPartyMember || unit.IsTargetingMyRaidMember) will return true if its targeting you. I could be wrong. But if im right and you are tanking, then it could return no adds if you have full aggro.

hmm. Well, unit.IsTargetingMe don't work. Would unit.IsHostile or something like that?

 

[CodenameG]

hmm. Well, unit.IsTargetingMe don't work. Would unit.IsHostile or something like that?

is hostile isnt the same thing as, in combat with you and pissed.

 

[Stree]

Gotcha.

Think I might have it with unit.IsTargetingMeOrPet as that is not coming back with any errors on HB loadup.

I did start using this

if (detectAdds().Count >= 3)
						    if	(CastSpell("Shockwave") == true && Me.CurrentTarget.Distance <= 10)
							    Logging.Write(Color.Green, ">> Shockwave <<");

                            else if (detectAdds().Count >= 3 && Me.CurrentTarget("Thunder Clap") == true)
						    if	(CastSpell("Cleave") == true)
							    Logging.Write(Color.Green, ">> Cleave <<");

and am getting an error on the "else if" line.

Line: 172 Error: Non-invocable member 'Styx.WoWInternals.WoWObjects.WoWUnit.CurrentTarget' cannot be used like a method.

Not really sure what it means.

 

[Shaddar]

Gotcha.

Think I might have it with unit.IsTargetingMeOrPet as that is not coming back with any errors on HB loadup.

I did start using this

if (detectAdds().Count >= 3)
						    if	(CastSpell("Shockwave") == true && Me.CurrentTarget.Distance <= 10)
							    Logging.Write(Color.Green, ">> Shockwave <<");

                            else if (detectAdds().Count >= 3 && Me.CurrentTarget("Thunder Clap") == true)
						    if	(CastSpell("Cleave") == true)
							    Logging.Write(Color.Green, ">> Cleave <<");

and am getting an error on the "else if" line.


Not really sure what it means.

Perhaps remove the space between Targe and T? That might get rid of the error

 

[Stree]

Perhaps remove the space between Targe and T? That might get rid of the error

That's the problem. In the code there is no space. It only shows that way in the error.

EDIT: Never mind. I cant believe i didn't notice sooner.

Me.CurrentTarget to Me.CurrentTarget.HasAura

 

[Ama]

I think unit.CurrentTarget.IsMe would be better since you don't have a pet.

Also, make sure you add the references for Honorbuddy.EXE, and Tripper.Tools.dll to you project. This way you can see not only compiling errors you are going to have, but you can also see properties. Idk if you were the same person I talked to a few days ago about Visual Studio express. I used to love Notepad2, but Visual Studio express has really sped up my learning and saved me tons of time.

 

[Stree]

I think unit.CurrentTarget.IsMe would be better since you don't have a pet.

Also, make sure you add the references for Honorbuddy.EXE, and Tripper.Tools.dll to you project. This way you can see not only compiling errors you are going to have, but you can also see properties. Idk if you were the same person I talked to a few days ago about Visual Studio express. I used to love Notepad2, but Visual Studio express has really sped up my learning and saved me tons of time.

Yeah I was just opening the original .cs file of Shaddar's CC with it which didn't allow me to add references to it that I could find. I have since then setup a project for it so that I can.
The biggest thing I don't like about Visual Studio is there are no line numbers, so finding what line errors are generating from is a pain. So I just copy and paste all the code into Notepad++ to find lines then make the changes in Visual Studio.


Thanks for that snippet of code by the way, I have added that to the detection code in addition to the other way I found to get it to work via some of CodeNameG's great work!