Zwei -Punkte -Crossover in Java

Question

Ich habe wie folgt einen One -Point -Crossover implementiert.

public void onePointCrossover(Individual indi) {
    if (SGA.rand.nextDouble() < pc) {

        int xoverpoint = SGA.rand.nextInt(length);


        int tmp;
        for (int i=xoverpoint; i<length; i++){
            tmp = chromosome[i];
            chromosome[i] = indi.chromosome[i];
            indi.chromosome[i] = tmp;
        }   
    }   
}

Ein Punkt Crossover - Crossover -Punkt wird ausgewählt, eine binäre Schnur von Anfang des Chromosoms bis zum Übergangspunkt wird von einem Elternteil kopiert, der Rest wird vom zweiten Elternteil kopiert.

Eltern 1 = Chromosom und Eltern 2 = Indi.

Ich verwandle die Eltern in Kinder an Ort und Stelle.

Ich muss jetzt auch einen Two -Point -Crossover machen, aber ich habe einige Probleme. Das habe ich bisher, aber ich glaube, die untere Hälfte des Codes tut das Gleiche wie ein One -Point -Crossover, anstatt die mittleren Abschnitte auszutauschen.

       public void twoPointCrossover(Individual indi) {
        if (SGA.rand.nextDouble() < pc) {

            int xoverpoint = SGA.rand.nextInt(length);
            int xoverpoint2 = SGA.rand.nextInt(length);



            int tmp;

            if (xoverpoint > xoverpoint2){
                tmp = xoverpoint;
                xoverpoint = xoverpoint2;
                xoverpoint2 = tmp;
            }

            for (int i=xoverpoint; i<xoverpoint2; i++){
                tmp = chromosome[i];
                chromosome[i] = indi.chromosome[i];
                indi.chromosome[i] = tmp;
            }   
        }   
    }
}

Dies scheint nicht richtig zu sein und jede Hilfe wird so sehr geschätzt! Vielen Dank!

Answer 1

Sie sollten nachsehen i < (or <=) xoverpoint2 statt i<length in der Schleife.

Answer 2

Ich arbeite jetzt an dem gleichen Problem. Hier ist meine Lösung:

// Two-Point Crossover function
public Genome twoPtCrossover(Genome partner) {
    Genome child = new Genome(partner.genome.length);

    int crosspoint1 = xd.nextInt(genome.length);
    int crosspoint2 = xd.nextInt(genome.length);

    // Ensure crosspoints are different...
    if (crosspoint1 == crosspoint2){
        if(crosspoint1 == 0){
            crosspoint2++;
        } else {
            crosspoint1--;
        }
    }
    // .. and crosspoint1 is lower than crosspoint2
    if (crosspoint2 < crosspoint1) {
        int temp = crosspoint1;
        crosspoint1 = crosspoint2;
        crosspoint2 = temp;
    }

    for (int i = 0; i < genome.length; i++) {
        if (i < crosspoint1 || i > crosspoint2)
            child.genome[i] = genome[i];
        else
            child.genome[i] = partner.genome[i];
    }
    return child;
}