i feel im almost done fixing

[monkeymaster69]

if there is someone much more exispirianced than me ik i ask help already but i still don't know whats this error means i fixed the rest but idk whats this is?

[smeghammer]

Check the code syntax.

The error indicates where it is breaking. It looks like a mismatch with some braces in the DECORATE code. i.e. check that all your opening braces have matching closing ones.

 

 

[monkeymaster69]

13 minutes ago, smeghammer said:

Check the code syntax.

The error indicates where it is breaking. It looks like a mismatch with some braces in the DECORATE code. i.e. check that all your opening braces have matching closing ones.

 

 

uhm idk how to do it its my first time tryin im trying to put everything in order still doesn't work :,[

Edited February 4 by monkeymaster69

[smeghammer]

Ah - I think I see:

 

You have a space in the name "BJ Knife"

 

Try changing it to "BJ_Knife".

[monkeymaster69]

i think i fix that but now i have this problem tho

[smeghammer]

This:

 

https://zdoom.org/wiki/New_User's_Guide_to_editing_with_DECORATE

 

gives a good intro to DECORATE. 

 

I suspect again you have a syntax error in the bit of code hidden by the GZ console.

[monkeymaster69]

21 minutes ago, smeghammer said:

This:

 

https://zdoom.org/wiki/New_User's_Guide_to_editing_with_DECORATE

 

gives a good intro to DECORATE. 

 

I suspect again you have a syntax error in the bit of code hidden by the GZ console.

i tried follow it.. doesn't work :,[

 

[smeghammer]

You have the actor class line on ln 14 inside the curly brace. Swap lines 13 and 14.

 

 

 

 

[monkeymaster69]

4 minutes ago, smeghammer said:

You have the actor class line on ln 14 inside the curly brace. Swap lines 13 and 14.

 

 

 

 

i did and still show error masseges

[smeghammer]

and what is the error logged now? The key is to use the error output to guide your debugging. It's important to share that with us so we can help too.

[monkeymaster69]

okay sure sorry i didn't show im coding this damm knife from yesterday :,( soo im bit tired but here the error

 

here the other error i think i fixx

Edited February 4 by monkeymaster69

[smeghammer]

https://forum.zdoom.org/viewtopic.php?t=21446

 

Just past the fold.

 

Try taking out the commas after the slot num.

[monkeymaster69]

12 minutes ago, smeghammer said:

https://forum.zdoom.org/viewtopic.php?t=21446

 

Just past the fold.

 

Try taking out the commas after the slot num.

now it says keyfog is wrong

Edited February 4 by monkeymaster69

[ChippiHeppu]

There's quite a lot wrong with your DECORATE code. You should carefully read the ZDoom Wiki page that smeghammer posted earlier. I'd also suggest opening up other mods in SLADE and seeing how they format their DECORATE lumps.

[smeghammer]

https://zdoom.org/wiki/Classes:DoomPlayer

 

is better example. Uses the same syntax as yours (with the comma...). However, this is using the weapon names quoted. Try that.

[monkeymaster69]

1 minute ago, ChippiHeppu said:

There's quite a lot wrong with your DECORATE code. You should carefully read the ZDoom Wiki page that smeghammer posted earlier. I'd also suggest opening up other mods in SLADE and seeing how they format their DECORATE lumps.

ill try thx everyone to have the patience :,)

[monkeymaster69]

guys is it okay i post the code and you see whats wrong with it? ive been coding this stuff since yesterday and can't take it

 

decorate here

 

Actor Knifer : DoomPlayer
{
Player.WeaponSlot 1, Knife, Chainsaw
Player.WeaponSlot 2, Pistol
Player.WeaponSlot 3, Shotgun, SuperShotgun
Player.WeaponSlot 4, Chaingun
Player.WeaponSlot 5, RocketLuncher
Player.WeaponSlot 6, PlasmaRifle
Player.WeaponSlot 7, BFG9000
}

 ACTOR knife : Knife Weapon 1 Replace Fists 3700 {
 
      Weapon.SelectionOrder 50
      Weapon.AmmoUse N/A
      Weapon.AmmoGive N/A
      Weapon.AmmoType N/A
      +WEAPON.MELEEWEAPON "Knife"
}
      ACTOR KNIFE: REPLACE Weapon : fists 3700
{
      States 
      Select:
      KNIF A 1 A_Raise
      Loop
      
      Deselect:
      KNIF A 1 A_Lower
      Loop
      
      Ready:
      KNIF A A_WeaponReady
      Loop
      
      Fire:
      KNIF BCD 4
      
      KNIF B 4
      KNIF C 4
      KNIF D 4
      Goto KnifeFire
}

 

map info here:

 

GameInfo {
    Playerclasses = "Knifer"

 

idk how to fix this i feel tired and stressed just for replacing the fists

[realjohnmadden]

Here's an attempt at fixing the knife weapon code:

ACTOR Knife : Weapon 3700
{ 
      Weapon.SelectionOrder 50
      +WEAPON.MELEEWEAPON
      States
	  {
          Select:
          KNIF A 1 A_Raise
          Loop

          Deselect:
          KNIF A 1 A_Lower
          Loop

          Ready:
          KNIF A A_WeaponReady
          Loop

          Fire:
		  KNIF A 0
          KNIF BC 4
		  KNIF D 4 A_Punch
          KNIF CB 4
          Goto Ready
	  }
} 

You defined the knife twice and messed up multiple weapon properties.

The GAMEINFO definition should also have an } at the end, like this:

 

GameInfo

{
    Playerclasses = "Knifer"

}

[ChippiHeppu]

This should work. Replace all of your DECORATE with the code below. KEYCONF and MAPINFO are not needed.

 

Quote

ACTOR Knife : Fist Replaces Fist {
    Weapon.SlotNumber 1
    Obituary "%o got stabbed by %k."
    Tag "Knife"
    
    States {
        Select:
            KNIF A 1 A_Raise
            Loop
        Deselect:
            KNIF A 1 A_Lower
            Loop
        Ready:
            KNIF A 1 A_WeaponReady
            Loop
        Fire:
            KNIF B 4
            KNIF C 4 A_Punch
            KNIF D 4
            Goto Ready
    }
}

 

I removed all the unnecessary lines and replaced the broken ones. I know all this modding stuff can be difficult for a beginner, we've all been there at one point. Don't give up though, keep learning and you'll get it eventually! The ZDoom wiki is an excellent learning resource.

Let us know if you have any more questions.

[monkeymaster69]

2 minutes ago, realjohnmadden said:

Here's an attempt at fixing the knife weapon code:

ACTOR Knife : Weapon 3700
{ 
      Weapon.SelectionOrder 50
      +WEAPON.MELEEWEAPON
      States
	  {
          Select:
          KNIF A 1 A_Raise
          Loop

          Deselect:
          KNIF A 1 A_Lower
          Loop

          Ready:
          KNIF A A_WeaponReady
          Loop

          Fire:
		  KNIF A 0
          KNIF BC 4
		  KNIF D 4 A_Punch
          KNIF CB 4
          Goto Ready
	  }
} 

You defined the knife twice and messed up multiple weapon properties.

The GAMEINFO definition should also have an } at the end, like this:

 

GameInfo

{
    Playerclasses = "Knifer"

}

thx ill try it now sorry i made you fix it but i letteraly can't do this ive been trying this since 2:00 since now :,) thx you very much

[monkeymaster69]

good news the game activates and loads properly bad new when i try to switch to the knife it doesnt appear like i press one doomguy refueses to switch to it

[ChippiHeppu]

Assuming you used realjohnmadden's solution, try adding (without quotes) "Weapon.SlotNumber 1" on a new line after the first curly bracket.

[monkeymaster69]

16 minutes ago, ChippiHeppu said:

Assuming you used realjohnmadden's solution, try adding (without quotes) "Weapon.SlotNumber 1" on a new line after the first curly bracket.

ok kinda like this

Edited February 4 by monkeymaster69

[ChippiHeppu]

Create a new line between lines 13 and 14, then move it there. Remove the quote characters and the comma.