Please help (urgent)

[nty777]

I am confused in general by this and would really appreciate some help.

Create a struct called enemy which will contain information such as enemy health and damage, etc. (The user must be able to choose how many enemy's he wants)

This is what I had in mind

 struct enemy {
int health; 
int damage; 
};
struct enemy enemy1;

Create a struct that will hold two lists.
1. a list representing the enemy's killed.
2. a list representing the enemy's that are still alive.

Assuming enemy1 is alive how do I add it to the still alive list??

[JordanWatson]

Maybe you could have a dead list instead so all enemies are always in use until in that list. So every time the user attacks an enemy it goes through something like:

enemy1.health -= 'usersattack'
if(enemy1.health <= 0)
'add enemy1 to dead list'

[Nominal Animal]

The description uses list, not array, so I would use

struct enemy {
    struct enemy *next;
    int health; 
    int damage; 
};

struct enemy *alive = NULL; /* No alive enemies yet */
struct enemy *dead = NULL; /* No dead enemies yet */

Because the enemies are in a singly-linked list, when damaging an enemy kills it, we also need a pointer to the list pointer to be able to kill the enemy. For example, if we have:

void kill_enemy(struct enemy **alive_list, struct enemy **dead_list, struct enemy *target)
{
    /* If no target, do nothing. */
    if (target == NULL)
        return;

    if (alive_list != NULL) {
        /* Find target in alive list, and detach it. */

        if (*alive_list == target) {
            *alive_list = target->next;
            target->next = NULL;

        } else {
            struct target *prev = *alive_list;

            while (prev->next != target && prev->next != NULL)
                prev = prev->next;

            if (prev_next) {
                prev->next = target->next;
                target->next = NULL;
            }
        }
    }

    if (dead_list) {
        /* Prepend target to dead list. Most recently deceased first. */
        target->next = *dead_list;
        *dead_list = target;
    }
}

then we could kill a known target enemy foo using

    kill_enemy(&alive, &dead, foo);

assuming the global lists are used in the first snippet.

The reason I didn't use the global lists directly in kill_enemy() is that this way you can have multiple alive and dead lists in a single program. It's not like providing a pointer to the list pointer is that annoying.

[laserlight]

The description uses list, not array

Good point, though I would recommend clarification of the requirements rather than immediately going for a linked list. A list is not necessarily a linked list, e.g., an array can also be used to implement a list.

[Nominal Animal]

Good point, though I would recommend clarification of the requirements rather than immediately going for a linked list.

You're right.

I guess I got excited thinking how this would be a pretty good way to exercise pointers and linked lists, especially when you are going to move enemies from one list to another. Maybe adding additional lists like bystander or turned_to_friendly further on