Enemy AI | it works but? - Godot Community Forums

Enemy AI | it works but?

newMorningKingdoM

Hi there people, i made a crude ai for my enemies. its work but im sure it can be better. and im new at codding and game dev. , so am not sure what is a good ai and what is not. so i need suggestions.

this is my ai code: note : i did use 3 diffrent timer for 3 diffrent state.


var rotate_y_speed = 0.5
var rotate_y_degree
var enemy_speed = -0.1


enum {
	forward,
	rotate,
	stop
}

var state = forward

var forward_timer_random = RandomNumberGenerator.new()
var forward_timer_range

var stop_timer_random = RandomNumberGenerator.new()
var stop_timer_range

var rotate_timer_random = RandomNumberGenerator.new()
var rotate_timer_range


func _on_enemy1_forward_timeout():
	state = forward
func _on_enemy1_stop_timeout():
	state = stop
func _on_enemy1_rotate_timeout():
	state = rotate

func _process(delta):
	randomize()
	
	forward_timer_range = forward_timer_random.randf_range(0,1)
	stop_timer_range = stop_timer_random.randf_range(0,1.5)
	rotate_timer_range = stop_timer_random.randf_range(0,3)
	
	$enemy1_forward.start(forward_timer_range)
	$enemy1_stop.start(stop_timer_range)
	$enemy1_rotate.start(rotate_timer_range)
	    	
	match state:
		forward:
			translate(Vector3(0,0,enemy_speed))
		stop:
			translate(Vector3(0,0,0))
		rotate:
			rotate_y_degree = rotate_y_speed * delta
			rotate_y(rotate_y_degree)

and this is the output:

https://streamable.com/beo01i

sent44

I am also not good game AI programmer but here (also with non AI): 1. Enum, you can name it

enum abc{
	a,
	b = 2,
	c
}

But if you still want to use that way, use it like constant variable, I mean name it with UPPER_CASE.

enum { A, B, C }

2. Use type hints, make it more easy to read and auto complete

var some_node: Node2D
var rng: RandomNumberGenerator = RandomNumberGenerator.new()

3. Private variable/function name with _ at start

var _health: int = 25
func _is_dead() -> bool:
	return _health < 0

4. RandomNumberGenerator, I don't know why you store it in 3 difference variables. Just 1 should be enough in this case. Or even just don't need it, just use global scope rand_range(). Like what you already did, randomize(). 5. Field of View, make the AI not somewhat blind(where they can some what only see what only straight in front of them). For starter, just use a lot of ray cast is already ok. 6. Stage Machine. Yes, you already use it with forward, rotate, stop things, but I mean such idle, wander, chase, meelee_attack, range_attack, flee, search and such with proper state transition (some state can't directly move to another state without move to other state first).