[JAVA] Problem z generatorem r.nextInt(int n);

Question

Witam, dziś napotkałem dosć dziwny błąd... Mianowicie generator pseudolosowy losuje mi lekko poza przedziałem. Przykładowo mam tablice o długości 11, i chcę by losowało do niej numerek od 1 do 11. Stwarzam obiekt klasy random i metodą nextInt losuję. r.nextInt(długość-1)+1;
Tu zamieszczę kod, wyszczególnię linijki, w których moim zdaniem coś nie gra. 

public void okretyCPU(String[][] poleCPU, int[] iloscStatkow) {
		Random r = new Random();

		do {
			for (int n = 1; n < iloscStatkow.length; n++) { // różne statki
				for (int m = 1; m <= iloscStatkow[n]; m++) { // te same statki
					for (int k = 1; k <= n; k++) { // po ilości masztów

						do {
							sprawdz = false;
							warunek = true;
							
							a = r.nextInt(poleCPU.length-1)+1; //<----------------------------
							b = r.nextInt(poleCPU[0].length-1)+1; //<-----------------------------------
							
							if (k != 1) {
								konwersja(propozycje2.get(r.nextInt(propozycje2.size())));
							}

							if ((!uzyte2.contains(konwersja2(a, b)) && propozycje2.contains(konwersja2(a, b)))
									|| (k == 1 && !(uzyte2.contains(konwersja2(a, b))))) {
								outerLoop: for (int i = a - 1; i <= a + 1; i++) {
									for (int j = b - 1; j <= b + 1; j++) {

										if ((i >= 1 && i <= (poleCPU.length - 1)) && j >= 1
												&& j <= (poleCPU[0].length - 1)) {
											if (poleCPU[i][j].equals(" ")) {
												sprawdz = true;
											} else if (poleCPU[i][j].equals("X")
													&& !(uzyte2.contains(konwersja2(i, j)))) {
												sprawdz = false;
												warunek = false;
												break outerLoop;
											}
										}
									}
								}
							} else if (uzyte2.contains(konwersja2(a, b))) {
								//System.out.println("Na tej pozycji znajduje sie już statek!");
								warunek = false;
							} else if (!uzyte2.contains(konwersja2(a, b)) && !(propozycje2.contains(konwersja2(a, b)))
									&& k != 1) {
								//System.out.println("Maszty statku muszą znajdować sie koło siebie!");
								warunek = false;
							}
							
							
						} while (sprawdz == false);
						System.out.println(a);
						System.out.println(b);
						poleCPU[a][b] = postawMaszt();//<-----------indeks za duży 
						uzyte2.add(konwersja2(a, b));
						propozycje2.remove(tekst);
						for (int l = -1; l <= 1; l += 2) {
							if (!(propozycje2.contains(konwersja2(a + l, b)))
									&& !(uzyte2.contains(konwersja2(a + l, b))))
								propozycje2.add(konwersja2(a + l, b));
							if (!(propozycje2.contains(konwersja2(a, b + l)))
									&& !(uzyte2.contains(konwersja2(a, b + l))))
								propozycje2.add(konwersja2(a, b + l));
						}

						if (propozycje2.isEmpty() && k != iloscMasztow) {
							for (int i = (3 - 2); i < poleCPU.length; i++) {
								for (int j = 1; j < poleCPU.length; j++) {
									poleCPU[i][j] = " ";
									sprawdz3 = false;
								}
							}
						}

					}
					propozycje2.clear();
				}
			}
			
			if (sprawdz == false){
				for(int i=1; i<poleCPU.length; i++){
					for(int j=0; j<poleCPU[i].length; j++){
						poleCPU[i][j] = " ";
					}
				}
				uzyte.clear();
			}
				
			
		} while (sprawdz3 == false);
		
	}

Obiekt klasy Random losuje mi o 1 za dużo i próbuje wcisnąć indeks 11 do tablicy i wszystko się sypie. 

Answer 1

Jeżeli masz tablicę o długości 11 tzn, że posiada ona indeksy od 0 do 10.

Używając funkcji nextInt(dlugosc-1)+1; - podajesz właściwie przedział od 0 do 9 i do wyniku dodajesz 1, a więc od 1 do 10.

Jeżeli chcesz, aby losowało w przedziale tablicy to : nextInt(dlugoscTablicy);

Natomiast jeżeli chcesz, aby z jakichś względów losowało przedział 1 do 11 przy 11 elementowej tablicy to : nextInt(długoscTablicy)+1;

Comments

Po prostu chcę, by losowało od 1 do 10. czyli robię r.nextInt(dlugosc-1)+1; A co mi konsola wypluje jak tak zrobie:

10
1
10
1
9
2
6
1
6
2
5
4
6
3
6
3
5
3
4
5
10
5
11
Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 11
    at RozmieszczenieStatkow.okretyCPU(RozmieszczenieStatkow.java:167)
    at graWStatki.main(graWStatki.java:40)

(te randomowe cyferki to właśnie wrzutka na konsole wartości  a i b)

Answer 2

Tak jak Ci adas94 napisał. Indeksowanie tablic zaczyna się od 0.

Jeżeli będziesz chciał wylosowac liczbę, to losujesz ja z zakresu [0,dlugoscTablicy). Jeżeli chcesz pokazać, ew. zrobić żeby użytkownik wprowadzał 1, a program by Ci sprawdzał dla 0 czy jest tam statek czy cokolwiek to odejmij 1 od tej jedynki.

Btw w code istnieje coś takiego jak Java. W dodatku (przynajmniej dla mnie) wrzucasz w cholerę niepotrzebnego kodu, a nie wrzucasz kodu z  funkcji. Co robi konwersja2()? Itd

Comments

1. Pole do gry w statki zorganizowałem tak, by w 1 rzędzie były wypisane 1, 2, 3, 4, 5, ... a w 1 kolumnie a b c d e ...    Chcąc zrobić pole do gry 10x10 używam tablicy 11x11 by nagłówki mieć. No i wiem, ze indeksowane od 0 są. Powiem Ci że to moje rozwiązanie nawet wygodne jest. Indeksy z zerami są nagłówkami i ich nie ruszam. A chcąc uderzyć w pole A1, odwołuje się do komórki 1,1

2. Umknęło mi wrzucenie tej javy, poprawione.

3. konwersja2(a, b) zmienia mi wartości a, b (czyli powiedzmy 1, 1) ciąg znaków String (na A1).
postawMaszt() stawia mi na macierzy X w miejscu o kordach a, b, czyli robię tab[a][b] = postawMaszt();

private String konwersja2(int a, int b) {
		return tekst2 = Character.toString((char)(a+64)) + Integer.toString(b);
	}

	private String postawMaszt() {
		return "X";
	}

4. Co do ilości kodu... według mnie również za dużo, jednak pisząc niejedno pytanie, proszono mnie o załączanie całości kodu a nie urywków.

---

Sam generator powinien działać ok. Możesz jeszcze dac żeby pokazywał a i b, n m k  po wylosowaniu b. Wtedy będziemy wiedzieć czy ten błąd występuje zawsze w tym samym miejscu, czy generator dobrze losuje i czy nigdzie nie jest modyfikowana wartość a albo b. Bo jeszcze zakładam możliwość że a lub b są gdzieś modyfikowane przez przypadek.

---

to zaraz wrzuce na wklejke zawartość konsoli

 

@Edit: zły trop. Wystarczyło że ledwo 2x puściłem i widac różnicę:

http://www.wklejto.pl/286249

http://www.wklejto.pl/286252

 

Może jakiś składnik mojego kodu coś miesza z a  i b nim dochodzi do wstawienia X?

 

Mam tu głównego podejrzanego:

 outerLoop: for (int i = a - 1; i <= a + 1; i++) {

czy to nie zmienia wartości zarówno i jak i a?

http://www.wklejto.pl/286278 coś mi jednak podmienia, no ale dziwne że aż o 6 zmienia.

Answer 3

Tam ma być numerek od 1 do 10 - chodziło o przedział [1,11). Co do metod typu konwersja2(), postawMaszt() to działają na 100% prawidłowo- mam napisaną bliźniaczą metodę dla gracza i działa jak należy.