how to improve cannons so that they display the correct result ?

import java.util.LinkedList;

import static java.lang.System.*;

class Cannon {
    String name;
    public boolean loaded;
    public boolean unloaded;


    public Cannon(String name, boolean loaded) {
        this.name = name;
        this.loaded = loaded;
    }
    public String getName() {
        return name;
}
    public boolean fire() {
        if (loaded = true) {
            System.out.println("Fire!");
        }
        return loaded = false;
    }

    public boolean load() {
        if (loaded = false) {
            System.out.println("Load cannon");
        }
        return loaded = true;
    }
    public boolean isLoaded() {
       return loaded;
    }


    }


public class Program {
    public static void main(String[] args) {
        LinkedList<Cannon> cannons = new LinkedList<>();
        cannons.add(new Cannon("cannon1", true));
        cannons.add(new Cannon("cannon2", false));
        cannons.add(new Cannon("cannon3", false));
        cannons.add(new Cannon("cannon4", true));
        cannons.add(new Cannon("cannon5", false));
        cannons.add(new Cannon("cannon6", true));

                out.println("Unloaded cannons: ");
        for (Cannon cannon : cannons) {
            if (cannon.isLoaded()) {
                out.println(cannon.getName());
            }
        }

    }
}

out.println("Unloaded cannons: ");
        for (Cannon cannon : cannons) {
            if (cannon.isLoaded() == false) {
                out.println(cannon.getName());

public boolean fire() {
    // you do not really need the check here as you already check that in main, it's the functionality of the isLoaded
    // method... similar to the hasNext() method of an iterator
    // hence you could make the methods void... as the loaded state is returned by isLoaded()
    if (loaded == true) { // PROBLEM, you assigned a value and did not do a comparison = is not ==, or just write if (loaded) {
        System.out.println("Fire!");
    }
    return loaded = false;
}

public void load() {
    // so this would do (depending on your needs -  if you need the additional check your way is just perfect)
    // made it void here as well
    System.out.println("Load cannon " + name);
    loaded = true;
}

for (Cannon cannon : cannons) {
    if (cannon.isLoaded()) {
        out.println(cannon.getName());
        // yep, the cannon is loaded... fire
        // cannon.fire();
    } else {
        // here the cannon is not loaded
        // cannon.load();
    }
}