vector wykonuje się za dużo razy

Question

Witam ma taki problem z programem bo wyjścia takiego:
1 2 3 4 5
6 7 8 9 10
11 12 13 14 15
16 0 0 0 0

mam takie:
1 2 3 4 5
1 2 3 4 5
1 2 3 4 5
1 2 3 4 5
1 2 3 4 5
6 7 8 9 10
6 7 8 9 10
6 7 8 9 10
6 7 8 9 10
6 7 8 9 10
11 12 13 14 15
11 12 13 14 15
11 12 13 14 15
11 12 13 14 15
11 12 13 14 15
16 0 0 0 0

wydaje mi się ze vector idzie w mojej podwójnej pętli zamiast po i to po j. Ale nie jestem pewien bo kontenerów używam od nie dawna.

import java.util.Scanner;
import java.util.Vector;
 
public class Train {
 
    static Scanner in = new Scanner(System.in);
 
    static int answer= 0;
 
    static Vector<Compartment> v = new Vector<Compartment>();
 
    public static void show (Compartment[] compartment) {
        for (int i = 0; i < compartment.length ; i++) {
            System.out.println();
            compartment[i].showComartment();
        }
    }
 
    public static void showVector() {
        for (int i = 0; i < v.size(); i++) {
            System.out.println();
            v.elementAt(i).showComartment();
        }
    }
 
    public static void addPasarzerowie(int ilePasarzerow, int wagony) {
        int count = 1;
        for (int i = 0; i < wagony ; i++) {
            Compartment c = new Compartment();
            for (int j = 0; j < 5; j++) {
                if (ilePasarzerow%5 > 0 && count <= ilePasarzerow) {
                    c.wagon[j] = count;
                    v.addElement(c);
                }
                else if (ilePasarzerow%5 == 0 && count <= ilePasarzerow ) {
                    c.wagon[j] = count;
                    v.addElement(c);
                }
                count++;
            }
        }
    }
 
    public static void addMorePasarzerow() {
 
    }
 
    public static void main(String[] args) {
 
        int pasazerowie = 0;
        int ileWagonow = 0;
 
        do{
            System.out.println();
            System.out.println("podaj ile ma byc pasazeru");
            pasazerowie = in.nextInt();
 
            if (pasazerowie%5 > 0) {
                System.out.println("tu");
                ileWagonow = pasazerowie/5;
                ileWagonow++;
            }
            else
                ileWagonow = pasazerowie/5;
 
            addPasarzerowie(pasazerowie, ileWagonow/*, compartments*/);
            showVector();
        }while (answer == 0);
    }
}

 

Comments

Vector? Nie lepiej zapomnieć o tym i używać czegoś z kolekcji? List, setów czy map? Chociaż tutaj pewnie ArrayList byłaby lepsza.

Edit: na miłość boską. Albo używaj Angielskiego albo Polskiego. Chociaż dużo lepiej angielskiego. A Polish-english jak "addMorePasarzerow", jeszcze z błędem, sobie daruj. I nie vektor wykonuje się za dużo razy tylko jak już to metoda.

Answer 1

Linijka 33 i 37 dodajesz do wektora Compartment 5 razy zamiast tylko raz.

I takie dwie małe rady. W twoim programie czesto używasz liczby 5 jako liczba prasazerów w wagoni. Polecał bym takie zmienne wynieść gdzieś to do zewnetrznej zmiennej statycznej bo co jeśli byś chciał miec 6 pasażerów w wagonie? Musiał byś przeorać cały program zamiast zmienic w jednym miescu.

Druga. Polecam zapoznać się z pętlą for each i kolekcjami. Vectora nieużywa się od bardzo dawna. W wiekszosci przypadków lepsza jest array list.