Многопоточность NullPointerException

@mrtelohranitel · Регистрация: 02.10.2012

Студворк — интернет-сервис помощи студентам

Кто-нибудь может подсказать, почему при k=2 и z=1 выбрасывается ошибка NullPointerException и как побороть данную проблему?

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
class Zain implements Runnable {
 
private int strings;
private Object[] objs; // массив объектов синхронизации
 
/**
*
* @param strings число строк вывода
* @param threads число потоков
*/
public Zain(int strings, int threads) {
this.strings = strings;
objs = new Object[threads + 1];
}
 
@Override
public void run() {
// цикл создания дочерних потоков.
// важно, что выход из цикла проверяется на длину - 1
// в последний элемент списка ляжет отдельный объект главной нити.
for (int i = 0; i < objs.length - 1; i++) {
final int index = i; // финальная переменная индекса нити для передачи в Runnable
Thread th = new Thread() {
@Override
public void run() {
 
objs[index] = new Object();
 
 
while(true) {
 
try {
synchronized (objs[index]) {
objs[index].wait();
}
} catch (InterruptedException e) {
e.printStackTrace();
}
 
System.out.print(Thread.currentThread().getName()+" ");
 
 
 
synchronized (objs[index + 1]){
 
objs[index + 1].notify();
}
 
}
}
};
 
th.start(); 
}
 
 
objs[objs.length - 1] = new Object();
// цикл вывода строк.
for (int i = 0; i < strings; i++) {
 
synchronized (objs[0]) {
    
objs[0].notify();
}
synchronized (objs[objs.length - 1]){
 
try {
objs[objs.length - 1].wait();
} catch (InterruptedException e) {
e.printStackTrace();
}
 
System.out.println();
}
}System.exit(0);
}
   }
public class Proekt4 
{
public static void main(String[] args) throws InterruptedException{
try{
    int k=(Integer.parseInt(args[0]));
    int z=(Integer.parseInt(args[1]));
    new Zain(k,z).run();
}
 
catch (NumberFormatException z)
{
System.out.println("Provertye argumenty");}
catch (ArrayIndexOutOfBoundsException z)
{System.out.println("Provertye argumenty");}
 
catch(NullPointerException z)
{System.out.println("Error with argument 2");
System.exit(1);}
}
}

@turbanoff · 10.04.2014, 20:16

mrtelohranitel, NPE вываливается так как вы пытаетесь захватить монитор на еще не созданном объекте objs[0] (строка 61).
Ваши потоки еще не успели стартануть и заполнить objs, а вы уже пытаетесь из него читать.

Обойти это можно разными способами.
Например можно создать барьер (CyclicBarrier), который будет пускать потоки в тех местах, в которых вы предполагаете.
Или просто заполнить ваш массив в главном потоке.

PS. У вас в коде еще довольно много ошибок. Например, при вызове notify вы никак не проверяете: действительно ли какой-то поток ждёт нашего notify.

@Wado-Ru · 10.04.2014, 20:17

Сообщение от mrtelohranitel

почему при k=2 и z=1 выбрасывается ошибка NullPointerException

а при при k=1 и z=2 не выбрасывается?

@mrtelohranitel · 10.04.2014, 20:29 **[ТС]**

Wado-Ru, не вылетает

Добавлено через 1 минуту
turbanoff, к сожалению по заданию можно пользоваться только wait() и notify(). Поэтому никак и не проверяется, ждет ли кто-нибудь этот notify

Добавлено через 1 минуту
turbanoff, Не подскажете, как для этого кода заполнить массив в главном потоке?

@turbanoff · 10.04.2014, 20:36

Сообщение от mrtelohranitel

Не подскажете, как для этого кода заполнить массив в главном потоке?

Сделать два цикла: один создает объекты, второй - стартует потоки.

Сообщение от mrtelohranitel

к сожалению по заданию можно пользоваться только wait() и notify()

В смысле, даже нельзя завести массив boolean переменных? Можно даже синхронизироваться на них, если использовать Boolean.

@mrtelohranitel · 10.04.2014, 20:40 **[ТС]**

turbanoff, нет нельзя, только synchronized, wait() и notify(), даже notifyAll нельзя

@turbanoff · 10.04.2014, 21:37

mrtelohranitel, ну что за глупости. Вы же используете Object? Как это так, его используете, когда можно только wait и notify?

@mrtelohranitel · 10.04.2014, 21:41 **[ТС]**

Изменил немного код, теперь просто зависает при k=2,z=1.

Java
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
45
46
47
48
49
50
51
52
53
54
55
56
57
58
59
60
61
62
63
64
65
66
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96
97
98
99
100
101
102
103
class Zain implements Runnable {
 
private int strings;
public Object[] objs; // массив объектов синхронизации
 
/**
*
* @param strings число строк вывода
* @param threads число потоков
*/
public Zain(int strings, int threads) {
this.strings = strings;
objs = new Object[threads + 1];
}
 
@Override
public void run() {
// цикл создания дочерних потоков.
// важно, что выход из цикла проверяется на длину - 1
// в последний элемент списка ляжет отдельный объект главной нити.
    for (int k=0;k<objs.length-1;k++)
    {final int indez = k;
    objs[indez] = new Object();
    }
for (int i = 0; i < objs.length - 1; i++) {
final int index = i; // финальная переменная индекса нити для передачи в Runnable
Thread th = new Thread() {
@Override
public void run() {
 
objs[index] = new Object();
 
 
while(true) {
 
try {
synchronized (objs[index]) {
objs[index].wait();
}
} catch (InterruptedException e) {
e.printStackTrace();
}
 
System.out.print(Thread.currentThread().getName()+" ");
 
 
 
synchronized (objs[index + 1]){
 
objs[index + 1].notify();
}
 
}
}
};
 
th.start(); 
}
 
 
objs[objs.length - 1] = new Object();
 
// цикл вывода строк.
for (int i = 0; i < strings; i++) {
 
synchronized (objs[0]) {
    
objs[0].notify();
 
}
synchronized (objs[objs.length - 1]){
 
try {
objs[objs.length - 1].wait();
} catch (InterruptedException e) {
e.printStackTrace();
}
 
System.out.println();
}
}System.exit(0);
}
   }
public class Proekt4 
{
public static void main(String[] args) throws InterruptedException{
try{
    int k=(Integer.parseInt(args[0]));
    int z=(Integer.parseInt(args[1]));
    new Zain(k,z).run();
}
 
catch (NumberFormatException z)
{
System.out.println("Provertye argumenty");}
catch (ArrayIndexOutOfBoundsException z)
{System.out.println("Provertye argumenty");}
 
catch(NullPointerException z)
{System.out.println("Error with argument 2");
System.exit(1);}
}
}

@turbanoff · 10.04.2014, 21:43

Сообщение от mrtelohranitel

теперь просто зависает

По той причине, которую я указал. Нельзя просто так потоку вызывать wait. Всегда необходимо какое-то условие.

@mrtelohranitel · 10.04.2014, 21:51 **[ТС]**

Object я использую, потому что не известно заранее сколько будет потоков и строк. А для синхронизации потоков можно только wait и notify. И почему, когда число потоков больше 2 то все работает нормально, а именно когда k=1 , то все зависает.

Добавлено через 1 минуту
Вы можете привести пример кода, который будет описывать ваши мысли или книжку про многопоточность, а то я пока совсем плохо ее понимаю

Новые блоги и статьи Все статьи Все блоги /
Отчёт о затраченных материалах за определенный период с макетом печатной формы Maks 21.04.2026 Отчёт из решения ниже размещён в конфигурации КА2. Задача: показать затраченные материалы за определённый период, с возможностью вывода печатной формы отчёта с шапкой и подвалом. В качестве. . .	Отчёт о спецтехнике находящейся в ремонте Maks 20.04.2026 Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .	Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)" Hrethgir 19.04.2026 Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .	Запрет удаления строк ТЧ документа при определённом условии Maks 19.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "Аккумуляторы", разработанного в конфигурации КА2. У данного документа есть ТЧ, в которой в зависимости от прав доступа. . .
Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.

@turbanoff 4094 / 3828 / 745 Регистрация: 18.05.2010 Сообщений: 9,331 Записей в блоге: 12
	10.04.2014, 20:16
	mrtelohranitel, NPE вываливается так как вы пытаетесь захватить монитор на еще не созданном объекте objs[0] (строка 61). Ваши потоки еще не успели стартануть и заполнить objs, а вы уже пытаетесь из него читать. Обойти это можно разными способами. Например можно создать барьер (CyclicBarrier), который будет пускать потоки в тех местах, в которых вы предполагаете. Или просто заполнить ваш массив в главном потоке. PS. У вас в коде еще довольно много ошибок. Например, при вызове notify вы никак не проверяете: действительно ли какой-то поток ждёт нашего notify. 0

@mrtelohranitel 11 / 7 / 11 Регистрация: 02.10.2012 Сообщений: 40
	10.04.2014, 20:29 [ТС]
	Wado-Ru, не вылетает Добавлено через 1 минуту turbanoff, к сожалению по заданию можно пользоваться только wait() и notify(). Поэтому никак и не проверяется, ждет ли кто-нибудь этот notify Добавлено через 1 минуту turbanoff, Не подскажете, как для этого кода заполнить массив в главном потоке? 0

@mrtelohranitel 11 / 7 / 11 Регистрация: 02.10.2012 Сообщений: 40
	10.04.2014, 20:40 [ТС]
	turbanoff, нет нельзя, только synchronized, wait() и notify(), даже notifyAll нельзя 0

@turbanoff 4094 / 3828 / 745 Регистрация: 18.05.2010 Сообщений: 9,331 Записей в блоге: 12
	10.04.2014, 21:37
	mrtelohranitel, ну что за глупости. Вы же используете Object? Как это так, его используете, когда можно только wait и notify? 0

@mrtelohranitel 11 / 7 / 11 Регистрация: 02.10.2012 Сообщений: 40
	10.04.2014, 21:51 [ТС]
	Object я использую, потому что не известно заранее сколько будет потоков и строк. А для синхронизации потоков можно только wait и notify. И почему, когда число потоков больше 2 то все работает нормально, а именно когда k=1 , то все зависает. Добавлено через 1 минуту Вы можете привести пример кода, который будет описывать ваши мысли или книжку про многопоточность, а то я пока совсем плохо ее понимаю 0

Опции темы