음 저도 잘하는 편은 아니라서 이렇다! 라고 말할수는 없지만

1번 같은경우 every 1 seconds 보다 while을 쓰는게 좋은걸로 알고 있습니다

구문 반복같은 경우 같은 변수면 배열변수 사용으로 줄일수 있지 않을까 생각을 해봅니다!

• command /목축.소환1:command /목축.소환2: 반복의 경우 중첩 배열변수({목축.쿨타임::%player%::1}) 의 사용으로 개선할 수 있습니다. GUI 역시 이 방법으로 1개 버튼으로 만들 수도 있을껍니다.
• on inventory click의 경우 구현이 잘못 되었습니다. 일반 인벤토리 에서도 GUI가 작동 할 껍니다.
• /기본목축 명령어의 경우 무한 실행이 가능해 악용의 여지가 큽니다.
• /목축.초기화 명령어는 2,3번째 쿨타임과 닭 허기를 초기화하지 않습니다.
• 목축 소환 명령어는 변수 이름은 닭이라고 되어 있지만 실제로는 소를 소환합니다. 실제와 맞지 않는 변수 명은 변경을 고려하세요.
• 쿨타임의 경우 every second를 쓰는 게 아니라 시작 지점을 기록해두고 명령 실행 시 시작 지점에서 얼마나 지났는지를 판단해서 구현합니다. 이렇게 하면 1초마다 불필요하게 변수를 업데이트할 필요가 없습니다.
• 아이템이나 엔티티의 이름은 모루 또는 이름표로 변경 가능하므로 안전하지 않습니다. name of tool contains "&b간단한 장갑": 같은 구현은 nbt 등으로 대체될 수 있습니다.
• 쿨타임 길이(360), 허기 값(2000)의 경우 option으로 스크립트 상단에 빼는 것이 좋습니다. 이런 경우를 하드코딩 이라고 합니다. 일반적으로 변경이 필요할 시 일일히 다 변경해야 해서 좋지 못합니다.
• set name of last spawned entity 의 경우 처음부터 이름 값과 함께 소환하는 것이 더 안전합니다. SkBee 등을 참고하세요.
• 전반적으로 every 1 second를 많이 쓰신다면 그건 성급한 판단일 수는 있지만 대체적으로 실력 부족의 결과입니다. 일반적으로 every 1 second는 필요가 없습니다.
• while도 every second를 대체하는 용도로는 일반적으로 필요가 없습니다. 그냥 그런 반복 자체가 불필요 합니다.