ZDoom

by **Alexagon** » Fri May 17, 2019 2:14 am

So I'm making an altfire for the revolver I have that shoots all 6 shots with one press. Everythig is working except for two things:

1. When I'm completely out of ammo in both the clip and reserve the altfire still shoots and can shoot endlessly. The odd thing is, earlier today it was working without this issue, but all of a sudden without editing the code it's no longer working as intended and this issue showed up.

2. The altfire shoots 6 shots regardless oif how much ammo is left in the clip when I use the altfire.

Here's the code I have for the revolver (it's a bit long so bear with me):

Spoiler:
Code: Select all • Expand view
ACTOR Revolver : ModWeapon replaces Pistol { Weapon.SelectionOrder 2000 Weapon.SlotNumber 2 Weapon.AmmoType "PistolMag" Weapon.AmmoUse 1 Weapon.AmmoGive 0 Weapon.AmmoType2 "clip" Weapon.AmmoGive2 6 Weapon.AmmoUse2 0 Inventory.PickupMessage "You got your Revolver back!" Inventory.PickupSound "weapons/pistol/slide" Weapon.UpSound "weapons/pistol/equip" Tag "Revolver" +WEAPON.NOALERT +WEAPON.AMMO_OPTIONAL //+WEAPON.NOAUTOFIRE States { Ready: TNT1 A 0 A_JumpIfInventory ("clip", 0, "ReadyFull") PPIS A 1 A_WeaponReady (WRF_ALLOWRELOAD) Loop Select: PPIS A 1 A_Raise TNT1 A 0 A_PlaySound("weapons/pistol/slide", CHAN_AUTO) PPIS A 1 Offset(-90,79) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-75,69) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-60,60) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-45,52) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-30,45) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-15,39) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-1,35) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) Goto Ready Deselect: TNT1 A 0 A_JumpIf(ACS_NamedExecuteWithResult("AnimToggle")==1, "DeadSelect") TNT1 A 0 A_PlaySound("weapons/deselect", CHAN_AUTO) TNT1 A 0 A_PlaySound("weapons/pistol/holster", CHAN_AUTO) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(2, 34) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(7, 39) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(10, 47) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(22, 58) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(32, 69) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(54, 81) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(67, 100) PPIS A 1 A_Lower Wait Fire: TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 1 A_FireBullets(2, 2, -1, 30, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.6)) PPIS B 1 Bright A_SetPitch (pitch - (0.06)) PPIS C 1 A_SetPitch (pitch - (0.02)) PPIS C 1 PPIS D 2 A_SetPitch (pitch + (0.02)) PPIS D 2 A_SetPitch (pitch + (0.09)) PPIS E 2 A_SetPitch (pitch + (0.15)) PPIS E 2 A_SetPitch (pitch + (0.30)) PPIS F 2 A_SetPitch (pitch + (0.09)) PPIS F 2 A_SetPitch (pitch + (0.03)) Goto Ready AltFire: TNT1 A 0 A_JumpIfNoAmmo ("dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) Goto dryfire NoReloadFire: TNT1 A 0 A_JumpIfInventory("clip", 1, 1) Goto Fire+1 TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_TakeInventory("clip", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_FireBullets(2, 2, -1, 30, "BulletPuff", 1) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS B 1 Offset(0, 33) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS B 1 Offset(0, 37) TNT1 A 0 A_SetPitch(pitch-0.4) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS C 1 Offset(0, 40) TNT1 A 0 A_SetPitch(pitch+0.2) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) TNT1 A 0 A_FireCustomMissile("GunSmokeSpawner", 0, 0, 8, 0) TNT1 A 0 A_FireCustomMissile("9mmCasingSpawner", 0, 0, 8, 0) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS C 1 Offset(0, 41) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS D 1 Offset(0, 39) TNT1 A 0 A_SetPitch(pitch+0.2) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS E 1 Offset(0, 37) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS D 1 Offset(0, 35) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS B 1 Offset(0, 34) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) TNT1 A 0 A_ReFire TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(0, 33) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) Goto Ready Dryfire: TNT1 A 0 A_PlaySound("weapons/pistol/dryf", 6) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(0, 35) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 2 Offset(0, 34) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(0, 33) Reload: TNT1 A 0 A_JumpIfInventory("PistolMag", 6, 2) TNT1 A 0 A_JumpIfInventory("clip", 1, "ReloadWork") PPIS A 1 A_WeaponReady Goto Ready ReloadWork: TNT1 A 0 A_JumpIfInventory("PistolMag", 1, "ReloadWork2") TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS GHIJKLLL 1 TNT1 A 0 A_PlaySound("weapons/pistol/cout", 6) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS MMMNNN 1 TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) Goto ChamberCheck ReloadWork2: TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS GHIJKLLL 1 TNT1 A 0 A_PlaySound("weapons/pistol/cout", 6) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS MMMNNN 1 TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) Goto ChamberCheck ChamberCheck: TNT1 A 0 A_JumpIfInventory("PistolMag", 1, "ReloadLoop2") Goto ReloadLoop ReloadLoop: TNT1 A 0 A_TakeInventory("clip", 1) TNT1 A 0 A_GiveInventory("PistolMag", 1) TNT1 A 0 A_JumpIfInventory("PistolMag", 6, "ReloadFinish") TNT1 A 0 A_JumpIfInventory("clip", 1, "ReloadLoop") Goto ReloadFinish ReloadLoop2: TNT1 A 0 A_TakeInventory("clip", 1) TNT1 A 0 A_GiveInventory("PistolMag", 1) TNT1 A 0 A_JumpIfInventory("PistolMag", 6, "ReloadFinish2") TNT1 A 0 A_JumpIfInventory("clip", 1, "ReloadLoop2") Goto ReloadFinish2 ReloadFinish: TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS OOPPPQQQRRR 1 TNT1 A 0 A_PlaySound("weapons/pistol/cin", 6) PPIS SSSTTTTUUU 1 TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) Goto Ready ReloadFinish2: TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS OOPPPQQQRRR 1 TNT1 A 0 A_PlaySound("weapons/pistol/cin", 6) PPIS SSSTTTTUUU 1 TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) Goto Ready DoPunch: TNT1 A 0 A_PlaySound("weapons/deselect", CHAN_AUTO) TNT1 A 0 A_TakeInventory("Punching",1) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-5,35) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-30,45) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-40,52) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-49,75) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-51,90) TNT1 A 1 TNT1 A 0 A_PlaySound("swing") ActualPunch: TNT1 A 0 A_JumpIfInventory("PowerStrength", 1, "BerserkPunch") PUN3 A 1 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PUN3 B 1 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) TNT1 A 0 A_SetPitch(+2 + pitch) PUN3 D 1 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) TNT1 A 0 A_FireCustomMissile("FistAttack",0,0,0,0) TNT1 A 0 A_SetPitch(+3 + pitch) PUN3 EGH 1 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PUN3 I 1 A_SetPitch(-2 + pitch) TNT1 A 0 A_SetPitch(-3 + pitch) PUN3 JKL 1 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) TNT1 A 3 Goto PunchDone BerserkPunch: TNT1 A 0 A_PlaySound("bsrkswing", CHAN_AUTO) PUN3 A 1 A_ZoomFactor(1.05) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PUN3 B 1 A_ZoomFactor(1.10) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) TNT1 A 0 A_SetPitch(+2 + pitch) PUN3 D 1 A_ZoomFactor(1.15) TNT1 A 0 A_FireCustomMissile("BerserkFistAttack",0,0,0,0) TNT1 A 0 A_SetPitch(+3 + pitch) PUN3 EGH 1 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PUN3 I 1 A_SetPitch(-2 + pitch) TNT1 A 0 A_SetPitch(-3 + pitch) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PUN3 J 1 A_ZoomFactor(1.10) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PUN3 K 1 A_ZoomFactor(1.05) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PUN3 L 1 A_ZoomFactor(1.00) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) TNT1 A 3 Goto PunchDone PunchDone: TNT1 A 1 Offset(-90,79) PPIS A 1 Offset(-90,79) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-75,69) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-60,60) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-45,52) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-30,45) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-15,39) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) PPIS A 1 Offset(-1,35) TNT1 A 0 A_WeaponReady(WRF_NoFire|WRF_NoSwitch) Goto Ready Flash: BRTF B 1 Bright A_Light2 BRTF A 1 Bright A_Light1 Goto LightDone Spawn: PIST A -1 Stop } } ACTOR PistolMag : Ammo { Inventory.PickupMessage "" Inventory.Amount 0 Inventory.MaxAmount 6 Ammo.BackpackAmount 0 Ammo.BackpackMaxAmount 6 Inventory.PickupSound "" }

Would anyone be so kind as to help me solve these issues?

by **Dan_The_Noob** » Fri May 17, 2019 4:09 am

Firstly, holy shit that's a lot of offsets O_O

may be easier to just make duplicate animation frames and adjust them that way so you can call them as a sequence.

but ALSO
your AltFire code:

Spoiler:
Code: Select all • Expand view
AltFire: TNT1 A 0 A_JumpIfNoAmmo ("dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) Goto dryfire

My change to the first few lines:

Code: Select all • Expand view: TNT1 A 0 A_JumpIfInventory("PistolMag",6,"Loaded") //checks if mag full, skips to fire sequence TNT1 A 0 A_PlaySound("dryfire") //plays click if not available goto Ready //jumps back to not firing TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) //this should be AFTER shots to make sure it takes ammo when check has been made TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1)//this is checking if the mag has at least 1 bullet so it's not needed (because you want 6 and also have taken 1 before this so it will never be 6 here) Loaded:

by **wildweasel** » Fri May 17, 2019 11:12 am

Dan_The_Noob wrote:may be easier to just make duplicate animation frames and adjust them that way so you can call them as a sequence.

Easier, but also takes up a lot more space.

(Side note: I'd be interested in finding a way to make offset-based animation easier and less messy, so one could have the best of both worlds.)

by **ramon.dexter** » Fri May 17, 2019 12:08 pm

wildweasel wrote:
(Side note: I'd be interested in finding a way to make offset-based animation easier and less messy, so one could have the best of both worlds.)

Well, I used the textures lump approach. I defined the frames as sprites in textures lump,using SLADE's wysiwyg textures editor with offsets and anything. That kept the source without hundreds of duplicate picture files, while allowing lots of frames to be defined. I think buddy Vostyok taught me this method.

sorry for the OT

by **Alexagon** » Fri May 17, 2019 9:30 pm

Dan_The_Noob wrote:Firstly, holy shit that's a lot of offsets O_O

may be easier to just make duplicate animation frames and adjust them that way so you can call them as a sequence.

but ALSO
your AltFire code:
Spoiler:
Code: Select all • Expand view
AltFire: TNT1 A 0 A_JumpIfNoAmmo ("dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) Goto dryfire

My change to the first few lines:
Code: Select all • Expand view
TNT1 A 0 A_JumpIfInventory("PistolMag",6,"Loaded") //checks if mag full, skips to fire sequence TNT1 A 0 A_PlaySound("dryfire") //plays click if not available goto Ready //jumps back to not firing TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) //this should be AFTER shots to make sure it takes ammo when check has been made TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1)//this is checking if the mag has at least 1 bullet so it's not needed (because you want 6 and also have taken 1 before this so it will never be 6 here) Loaded:

That didn't seem to change anything, and the goto ready part makes it not fire at all but still take ammo.

by **Dan_The_Noob** » Fri May 17, 2019 10:12 pm

Spoiler:
Code: Select all • Expand view
Ready: TNT1 A 0 A_JumpIfInventory ("clip", 0, "ReadyFull") PPIS A 1 A_WeaponReady (WRF_ALLOWRELOAD) Loop

I can't find "ReadyFull" but i may be blind.

also, it shouldn't take any ammo if it skips to ready... i'll look through the code it just seems really intense with all the animation frames

--EDIT--
OHHHHHH you want the revolver to fanfire when you AltFire!
i took out all the offsets n crap jus to read it and i see now ...
OK let me see.

--EDIT 2--
here's what the code looks like without all the animation offsets (just "do stuff" code left) and i removed A_AlertMonsters playing each shot and the weapon flags etc.
and honestly... I think removing +Weapon.AMMO_OPTIONAL is probably your solution. but you could cleanup things like AltFire could be a reloadcheck loop check.

or you could replace AltFire with just a "Hold:" state for Fire (since it doesn't seem too different anyway?) then use AltFire for the punch stuff.

Spoiler:
Code: Select all • Expand view
ACTOR Revolver : ModWeapon replaces Pistol { Weapon.SelectionOrder 2000 Weapon.SlotNumber 2 Weapon.AmmoType "PistolMag" Weapon.AmmoUse 1 Weapon.AmmoGive 0 Weapon.AmmoType2 "clip" Weapon.AmmoGive2 6 Weapon.AmmoUse2 0 Inventory.PickupMessage "You got your Revolver back!" Inventory.PickupSound "weapons/pistol/slide" Weapon.UpSound "weapons/pistol/equip" Tag "Revolver" States { Ready: TNT1 A 0 A_JumpIfInventory ("clip", 0, "ReadyFull") PPIS A 1 A_WeaponReady (WRF_ALLOWRELOAD) Loop Select: PPIS A 1 A_Raise TNT1 A 0 A_PlaySound("weapons/pistol/slide", CHAN_AUTO) Goto Ready Deselect: TNT1 A 0 A_JumpIf(ACS_NamedExecuteWithResult("AnimToggle")==1, "DeadSelect") TNT1 A 0 A_PlaySound("weapons/deselect", CHAN_AUTO) TNT1 A 0 A_PlaySound("weapons/pistol/holster", CHAN_AUTO) PPIS A 1 A_Lower Wait Fire: TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 1 A_FireBullets(2, 2, -1, 30, "BulletPuff", 1) Goto Ready AltFire: TNT1 A 0 A_JumpIfNoAmmo ("dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) Goto dryfire NoReloadFire: TNT1 A 0 A_JumpIfInventory("clip", 1, 1) Goto Fire+1 TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_TakeInventory("clip", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_FireBullets(2, 2, -1, 30, "BulletPuff", 1) TNT1 A 0 A_FireCustomMissile("GunSmokeSpawner", 0, 0, 8, 0) TNT1 A 0 A_FireCustomMissile("9mmCasingSpawner", 0, 0, 8, 0) Goto Ready Dryfire: TNT1 A 0 A_PlaySound("weapons/pistol/dryf", 6) Reload: TNT1 A 0 A_JumpIfInventory("PistolMag", 6, 2) TNT1 A 0 A_JumpIfInventory("clip", 1, "ReloadWork") PPIS A 1 A_WeaponReady Goto Ready ReloadWork: TNT1 A 0 A_JumpIfInventory("PistolMag", 1, "ReloadWork2") PPIS GHIJKLLL 1 TNT1 A 0 A_PlaySound("weapons/pistol/cout", 6) PPIS MMMNNN 1 Goto ChamberCheck ReloadWork2: PPIS GHIJKLLL 1 TNT1 A 0 A_PlaySound("weapons/pistol/cout", 6) PPIS MMMNNN 1 Goto ChamberCheck ChamberCheck: TNT1 A 0 A_JumpIfInventory("PistolMag", 1, "ReloadLoop2") Goto ReloadLoop ReloadLoop: TNT1 A 0 A_TakeInventory("clip", 1) TNT1 A 0 A_GiveInventory("PistolMag", 1) TNT1 A 0 A_JumpIfInventory("PistolMag", 6, "ReloadFinish") TNT1 A 0 A_JumpIfInventory("clip", 1, "ReloadLoop") Goto ReloadFinish ReloadLoop2: TNT1 A 0 A_TakeInventory("clip", 1) TNT1 A 0 A_GiveInventory("PistolMag", 1) TNT1 A 0 A_JumpIfInventory("PistolMag", 6, "ReloadFinish2") TNT1 A 0 A_JumpIfInventory("clip", 1, "ReloadLoop2") Goto ReloadFinish2 ReloadFinish: PPIS OOPPPQQQRRR 1 TNT1 A 0 A_PlaySound("weapons/pistol/cin", 6) PPIS SSSTTTTUUU 1 Goto Ready ReloadFinish2: PPIS OOPPPQQQRRR 1 TNT1 A 0 A_PlaySound("weapons/pistol/cin", 6) PPIS SSSTTTTUUU 1 Goto Ready DoPunch: TNT1 A 0 A_PlaySound("weapons/deselect", CHAN_AUTO) TNT1 A 0 A_TakeInventory("Punching",1) TNT1 A 0 A_PlaySound("swing") ActualPunch: TNT1 A 0 A_JumpIfInventory("PowerStrength", 1, "BerserkPunch") TNT1 A 0 A_FireCustomMissile("FistAttack",0,0,0,0) TNT1 A 3 Goto PunchDone BerserkPunch: TNT1 A 0 A_PlaySound("bsrkswing", CHAN_AUTO) TNT1 A 0 A_FireCustomMissile("BerserkFistAttack",0,0,0,0) TNT1 A 3 Goto PunchDone PunchDone: Goto Ready Flash: BRTF B 1 Bright A_Light2 BRTF A 1 Bright A_Light1 Goto LightDone Spawn: PIST A -1 Stop } } ACTOR PistolMag : Ammo { Inventory.PickupMessage "" Inventory.Amount 0 Inventory.MaxAmount 6 Ammo.BackpackAmount 0 Ammo.BackpackMaxAmount 6 Inventory.PickupSound "" }

by **Alexagon** » Fri May 17, 2019 10:23 pm

Dan_The_Noob wrote:
Spoiler:
Code: Select all • Expand view
Ready: TNT1 A 0 A_JumpIfInventory ("clip", 0, "ReadyFull") PPIS A 1 A_WeaponReady (WRF_ALLOWRELOAD) Loop

I can't find "ReadyFull" but i may be blind.

also, it shouldn't take any ammo if it skips to ready... i'll look through the code it just seems really intense with all the animation frames

--EDIT--

OHHHHHH you want the revolver to fanfire when you AltFire!
i took out all the offsets n crap jus to read it and i see now ...
OK let me see.

Yeah, sorry if I wasn't too clear about it. And thanks.

by **Dan_The_Noob** » Fri May 17, 2019 10:31 pm

no problem, here's what i came up with for feedback/fixes

Dan_The_Noob wrote:here's what the code looks like without all the animation offsets (just "do stuff" code left) and i removed A_AlertMonsters playing each shot and the weapon flags etc.
and honestly... I think removing +Weapon.AMMO_OPTIONAL is probably your solution. but you could cleanup things like AltFire could be a reloadcheck loop check.

or you could replace AltFire with just a "Hold:" state for Fire (since it doesn't seem too different anyway?) then use AltFire for the punch stuff.
Spoiler:
Code: Select all • Expand view
ACTOR Revolver : ModWeapon replaces Pistol { Weapon.SelectionOrder 2000 Weapon.SlotNumber 2 Weapon.AmmoType "PistolMag" Weapon.AmmoUse 1 Weapon.AmmoGive 0 Weapon.AmmoType2 "clip" Weapon.AmmoGive2 6 Weapon.AmmoUse2 0 Inventory.PickupMessage "You got your Revolver back!" Inventory.PickupSound "weapons/pistol/slide" Weapon.UpSound "weapons/pistol/equip" Tag "Revolver" States { Ready: TNT1 A 0 A_JumpIfInventory ("clip", 0, "ReadyFull") PPIS A 1 A_WeaponReady (WRF_ALLOWRELOAD) Loop Select: PPIS A 1 A_Raise TNT1 A 0 A_PlaySound("weapons/pistol/slide", CHAN_AUTO) Goto Ready Deselect: TNT1 A 0 A_JumpIf(ACS_NamedExecuteWithResult("AnimToggle")==1, "DeadSelect") TNT1 A 0 A_PlaySound("weapons/deselect", CHAN_AUTO) TNT1 A 0 A_PlaySound("weapons/pistol/holster", CHAN_AUTO) PPIS A 1 A_Lower Wait Fire: TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 1 A_FireBullets(2, 2, -1, 30, "BulletPuff", 1) Goto Ready AltFire: TNT1 A 0 A_JumpIfNoAmmo ("dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) TNT1 A 0 A_JumpIfNoAmmo ("Dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) Goto dryfire NoReloadFire: TNT1 A 0 A_JumpIfInventory("clip", 1, 1) Goto Fire+1 TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_TakeInventory("clip", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_FireBullets(2, 2, -1, 30, "BulletPuff", 1) TNT1 A 0 A_FireCustomMissile("GunSmokeSpawner", 0, 0, 8, 0) TNT1 A 0 A_FireCustomMissile("9mmCasingSpawner", 0, 0, 8, 0) Goto Ready Dryfire: TNT1 A 0 A_PlaySound("weapons/pistol/dryf", 6) Reload: TNT1 A 0 A_JumpIfInventory("PistolMag", 6, 2) TNT1 A 0 A_JumpIfInventory("clip", 1, "ReloadWork") PPIS A 1 A_WeaponReady Goto Ready ReloadWork: TNT1 A 0 A_JumpIfInventory("PistolMag", 1, "ReloadWork2") PPIS GHIJKLLL 1 TNT1 A 0 A_PlaySound("weapons/pistol/cout", 6) PPIS MMMNNN 1 Goto ChamberCheck ReloadWork2: PPIS GHIJKLLL 1 TNT1 A 0 A_PlaySound("weapons/pistol/cout", 6) PPIS MMMNNN 1 Goto ChamberCheck ChamberCheck: TNT1 A 0 A_JumpIfInventory("PistolMag", 1, "ReloadLoop2") Goto ReloadLoop ReloadLoop: TNT1 A 0 A_TakeInventory("clip", 1) TNT1 A 0 A_GiveInventory("PistolMag", 1) TNT1 A 0 A_JumpIfInventory("PistolMag", 6, "ReloadFinish") TNT1 A 0 A_JumpIfInventory("clip", 1, "ReloadLoop") Goto ReloadFinish ReloadLoop2: TNT1 A 0 A_TakeInventory("clip", 1) TNT1 A 0 A_GiveInventory("PistolMag", 1) TNT1 A 0 A_JumpIfInventory("PistolMag", 6, "ReloadFinish2") TNT1 A 0 A_JumpIfInventory("clip", 1, "ReloadLoop2") Goto ReloadFinish2 ReloadFinish: PPIS OOPPPQQQRRR 1 TNT1 A 0 A_PlaySound("weapons/pistol/cin", 6) PPIS SSSTTTTUUU 1 Goto Ready ReloadFinish2: PPIS OOPPPQQQRRR 1 TNT1 A 0 A_PlaySound("weapons/pistol/cin", 6) PPIS SSSTTTTUUU 1 Goto Ready DoPunch: TNT1 A 0 A_PlaySound("weapons/deselect", CHAN_AUTO) TNT1 A 0 A_TakeInventory("Punching",1) TNT1 A 0 A_PlaySound("swing") ActualPunch: TNT1 A 0 A_JumpIfInventory("PowerStrength", 1, "BerserkPunch") TNT1 A 0 A_FireCustomMissile("FistAttack",0,0,0,0) TNT1 A 3 Goto PunchDone BerserkPunch: TNT1 A 0 A_PlaySound("bsrkswing", CHAN_AUTO) TNT1 A 0 A_FireCustomMissile("BerserkFistAttack",0,0,0,0) TNT1 A 3 Goto PunchDone PunchDone: Goto Ready Flash: BRTF B 1 Bright A_Light2 BRTF A 1 Bright A_Light1 Goto LightDone Spawn: PIST A -1 Stop } } ACTOR PistolMag : Ammo { Inventory.PickupMessage "" Inventory.Amount 0 Inventory.MaxAmount 6 Ammo.BackpackAmount 0 Ammo.BackpackMaxAmount 6 Inventory.PickupSound "" }

by **Alexagon** » Fri May 17, 2019 11:16 pm

Removing the ammo optional part didn't change anything unfortunately. As far as the hold goes, I want to make it so that one click of the altfire makes it empty the entire magazine, to make it a commitment thing.

Would it be helpful if I upload the pk3 for you to take a look at personally?

by **Dan_The_Noob** » Sat May 18, 2019 1:16 am

I guess but you already pasted it in the post.
I think the first step is working out a loop. I would recommend a quick google for "A_CheckForReload" help

here's some code i made to give my autoshotgun a burst-fire you can adapt it if you like, i've added some comments to understand each line better.

Spoiler:
Code: Select all • Expand view
AltFire: Burst: //jump marker (has a proper name, i'm sure) ASGG A 0 A_JumpIfInventory("ASGClip",1,5) //if "has ammo" //check if mag has ammo ASGG A 0 A_ResetReloadCounter //reset "Burst counter" (start new burst) //set burst counter to 0 ASGG A 0 A_JumpIfInventory("Shell",1,"Reload") //if "has shells" //reload if have reserve ammo ASGG A 0 A_PlayWeaponSound("Weapons/asgout") //play "click" sound goto Ready ASGG A 1 //firing starts here ASGG L 0 A_GunFlash //flash ASGG A 0 A_TakeInventory("ASGClip",1) //take 1 bullet out of mag ASGG L 2 A_FireBullets(5,5,8,8,"BulletPuff",0) //fire 1 shot ASGG M 3 ASGG A 0 A_CheckForReload(3,"Burst") //counts 3-shot burst, each time it hits this mark it jumps to "Burst" and repeats until it has done it 3 times or the mag is empty (due to the earlier check right after "Burst") ASGG N 3 ASGG A 8 ASGG A 0 A_ResetReloadCounter //burst finished, set counter to 0

by **Alexagon** » Sat May 18, 2019 1:57 am

Dan_The_Noob wrote:I guess but you already pasted it in the post.
I think the first step is working out a loop. I would recommend a quick google for "A_CheckForReload" help

here's some code i made to give my autoshotgun a burst-fire you can adapt it if you like, i've added some comments to understand each line better.

Spoiler:
Code: Select all • Expand view
AltFire: Burst: //jump marker (has a proper name, i'm sure) ASGG A 0 A_JumpIfInventory("ASGClip",1,5) //if "has ammo" //check if mag has ammo ASGG A 0 A_ResetReloadCounter //reset "Burst counter" (start new burst) //set burst counter to 0 ASGG A 0 A_JumpIfInventory("Shell",1,"Reload") //if "has shells" //reload if have reserve ammo ASGG A 0 A_PlayWeaponSound("Weapons/asgout") //play "click" sound goto Ready ASGG A 1 //firing starts here ASGG L 0 A_GunFlash //flash ASGG A 0 A_TakeInventory("ASGClip",1) //take 1 bullet out of mag ASGG L 2 A_FireBullets(5,5,8,8,"BulletPuff",0) //fire 1 shot ASGG M 3 ASGG A 0 A_CheckForReload(3,"Burst") //counts 3-shot burst, each time it hits this mark it jumps to "Burst" and repeats until it has done it 3 times or the mag is empty (due to the earlier check right after "Burst") ASGG N 3 ASGG A 8 ASGG A 0 A_ResetReloadCounter //burst finished, set counter to 0

So I tried adapting this code for my thing, but it was making it wonky and not making and sounds for some reason so I undid the changes and just added the checkforreload line to the very bottom before goto dryfire and it allows it to do the burst properly which is cool since I can get rid of the copy pasted code I had, but the issues I'm having with clip amount and it shooting when I'm out of ammo are still there.

by **Dan_The_Noob** » Sat May 18, 2019 2:22 am

alright, pop up your new altfire code?

have a look, it sounds like you're missing an ammo check, or your ammo check is missing an exit.

Also, yours wouldn't make any sound because my shotgun has the sound applied as an actor property

Code: Select all • Expand view: weapon.attacksound

i think?

by **Alexagon** » Sat May 18, 2019 2:34 am

Dan_The_Noob wrote:alright, pop up your new altfire code?

have a look, it sounds like you're missing an ammo check, or your ammo check is missing an exit.

Also, yours wouldn't make any sound because my shotgun has the sound applied as an actor property
Code: Select all • Expand view
weapon.attacksound
i think?

Oh, gotcha. Oh well I suppose,

Here's what I have right now:

Spoiler:
Code: Select all • Expand view
AltFire: TNT1 A 0 A_JumpIfNoAmmo ("dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_CheckForReload(6,"Altfire") Goto dryfire

by **Dan_The_Noob** » Sat May 18, 2019 2:49 am

Alexagon wrote:Oh, gotcha. Oh well I suppose,

Here's what I have right now:
Spoiler:
Code: Select all • Expand view
AltFire: TNT1 A 0 A_JumpIfNoAmmo ("dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_CheckForReload(6,"Altfire") Goto dryfire

"TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1)" basically does nothing, it effectively says "if pistol mag has 1 bullet or more, do next step, else do next step" BUT if you change the last 1 to a 2 and add an exit "goto dryfire" it will say "if pistolmag has 1 bullet or more, jump to shooting, else goto dryfire"
also, A_TakeInventory is happening before the ammo check so it could take the last shot before it's fired. so move it further down.

TL;DR
-> move "A_TakeInventory" before "A_FireBullets"
-> make a "goto dryfire" after the "TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 2<<<changed to 2)"
so that if you have no bullets in the mag it will jump to dryfire, if you have at least 1 it will jump to the second thing after and continue to firing.

by **Alexagon** » Sat May 18, 2019 2:59 am

Dan_The_Noob wrote:
Alexagon wrote:Oh, gotcha. Oh well I suppose,

Here's what I have right now:
Spoiler:
Code: Select all • Expand view
AltFire: TNT1 A 0 A_JumpIfNoAmmo ("dryfire") TNT1 A 0 A_TakeInventory("PistolMag", 1, TIF_NOTAKEINFINITE) TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1) TNT1 A 0 A_AlertMonsters TNT1 A 0 A_GunFlash TNT1 A 0 A_PlaySound("weapons/pistol/fire") TNT1 A 0 A_FireBullets(2, 2, -1, 24, "BulletPuff", 1) PPIS B 1 Bright A_SetPitch (pitch - (0.24)) PPIS C 1 A_SetPitch (pitch - (0.12)) PPIS D 1 A_SetPitch (pitch - (0.12)) PPIS E 1 A_SetPitch (pitch - (0.40)) PPIS F 1 A_SetPitch (pitch - (0.20)) TNT1 A 0 A_CheckForReload(6,"Altfire") Goto dryfire

"TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 1)" basically does nothing, it effectively says "if pistol mag has 1 bullet or more, do next step, else do next step" BUT if you change the last 1 to a 2 and add an exit "goto dryfire" it will say "if pistolmag has 1 bullet or more, jump to shooting, else goto dryfire"
also, A_TakeInventory is happening before the ammo check so it could take the last shot before it's fired. so move it further down.

TL;DR
-> move "A_TakeInventory" before "A_FireBullets"
-> make a "goto dryfire" after the "TNT1 A 0 A_JumpIfInventory("PistolMag", 1, 2<<<changed to 2)"
so that if you have no bullets in the mag it will jump to dryfire, if you have at least 1 it will jump to the second thing after and continue to firing.

Welp, you just fixed the problems I was having with this revolver. I'm still learning this stuff so I appreciate the patience. And thanks for the help in general!

ZDoom

[SOLVED] Firing when I have no ammo to fire with?

[SOLVED] Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Re: Firing when I have no ammo to fire with?

Who is online