Помогите Null Pointer

Fenix78563 · 18 дек 2016

Доброго времени суток, форумчане. Ловит NullPointer, хотя объект не null

Java:

public class moveBullet extends BukkitRunnable{
  
    private Player p;
    private Firework Bullet;

    public moveBullet(Player pl, Firework bul){
        this.p=pl;
        this.Bullet=bul;
    }
    final Vector vc = p.getLocation().getDirection();
    Double y = vc.getY();
  
    @Override
    public void run() {
        p.sendMessage("as"); //специально для проверки null
        if (Bullet.isDead()){
            cancel();
        }
        cancel();
    }

}

null в строке

Java:

final Vector vc = p.getLocation().getDirection();

Когда отключаю из кода строки

Java:

final Vector vc = p.getLocation().getDirection();
    Double y = vc.getY();

то все пашет. Даже сообщение приходит. Т.е. игрок не null

В чем подвох?
Заранее спасибо

alexandrage · 18 дек 2016

Подвох в отсутствие стак трейса. Вангов нету.

Fenix78563 · 18 дек 2016

alexandrage сказал(а):
Подвох в отсутствие стак трейса. Вангов нету.

Java:

[Poligon] Task #91 for Poligon v0.1 generated an exception
java.lang.NullPointerException
    at ru.blc.poligon.utils.moveBullet.<init>(moveBullet.java:17) ~[?:?]
    at ru.blc.poligon.entity.Bullet.launchBullet(Bullet.java:28) ~[?:?]
    at ru.blc.poligon.Shooting.onShot.run(onShot.java:21) ~[?:?]
    at org.bukkit.craftbukkit.v1_10_R1.scheduler.CraftTask.run(CraftTask.java:71) ~[server.jar:git-Spigot-90f61bc-6053c5c]
    at org.bukkit.craftbukkit.v1_10_R1.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:350) [server.jar:git-Spigot-90f61bc-6053c5c]
    at net.minecraft.server.v1_10_R1.MinecraftServer.D(MinecraftServer.java:723) [server.jar:git-Spigot-90f61bc-6053c5c]
    at net.minecraft.server.v1_10_R1.DedicatedServer.D(DedicatedServer.java:400) [server.jar:git-Spigot-90f61bc-6053c5c]
    at net.minecraft.server.v1_10_R1.MinecraftServer.C(MinecraftServer.java:668) [server.jar:git-Spigot-90f61bc-6053c5c]
    at net.minecraft.server.v1_10_R1.MinecraftServer.run(MinecraftServer.java:567) [server.jar:git-Spigot-90f61bc-6053c5c]
    at java.lang.Thread.run(Unknown Source) [?:1.8.0_101]
[23:11:55] [Server thread/INFO]: Type-1[m
[23:11:55] [Server thread/WARN]: [Poligon] Task #92 for Poligon v0.1 generated an exception
java.lang.NullPointerException
    at ru.blc.poligon.utils.moveBullet.<init>(moveBullet.java:17) ~[?:?]
    at ru.blc.poligon.entity.Bullet.launchBullet(Bullet.java:28) ~[?:?]
    at ru.blc.poligon.Shooting.onShot.run(onShot.java:21) ~[?:?]
    at org.bukkit.craftbukkit.v1_10_R1.scheduler.CraftTask.run(CraftTask.java:71) ~[server.jar:git-Spigot-90f61bc-6053c5c]
    at org.bukkit.craftbukkit.v1_10_R1.scheduler.CraftScheduler.mainThreadHeartbeat(CraftScheduler.java:350) [server.jar:git-Spigot-90f61bc-6053c5c]
    at net.minecraft.server.v1_10_R1.MinecraftServer.D(MinecraftServer.java:723) [server.jar:git-Spigot-90f61bc-6053c5c]
    at net.minecraft.server.v1_10_R1.DedicatedServer.D(DedicatedServer.java:400) [server.jar:git-Spigot-90f61bc-6053c5c]
    at net.minecraft.server.v1_10_R1.MinecraftServer.C(MinecraftServer.java:668) [server.jar:git-Spigot-90f61bc-6053c5c]
    at net.minecraft.server.v1_10_R1.MinecraftServer.run(MinecraftServer.java:567) [server.jar:git-Spigot-90f61bc-6053c5c]
    at java.lang.Thread.run(Unknown Source) [?:1.8.0_101]

slavik123123123 · 18 дек 2016

Fenix78563 сказал(а):
final Vector vc = p.getLocation().getDirection();

Double y = vc.getY();

p == null?
перемести это moveBullet

alexandrage · 18 дек 2016

Пробуй так.

Java:

public class moveBullet extends BukkitRunnable{
    private Player p;
    private Firework Bullet;
    public moveBullet(Player pl, Firework bul){
        this.p=pl;
        this.Bullet=bul;
    }
    @Override
    public void run() {
        Vector vc = p.getLocation().getDirection();
        Double y = vc.getY();
        p.sendMessage("as"); //специально для проверки null
        if (Bullet.isDead()){
            cancel();
        }
        cancel();
    }
}

Deamonish · 18 дек 2016

Проверь дебагером. Как уже сказали, скорее всего p == null. Лучше сделать вот так:

Java:

public class moveBullet extends BukkitRunnable{
    private Player p;
    private Firework Bullet;
    private final  Vector vc;
    private double y;
    public moveBullet(Player pl, Firework bul){
        this.p=pl;
        this.Bullet=bul;
        if(pl =! null){
            vc = pl.getLocation().getDirection();
            y = vc.getY();
        } else {
           throw new NullPointerException("Объект player - null");
         }
    }
    @Override
    public void run() {
        p.sendMessage("as");
        if (Bullet.isDead()){
            cancel();
        }
        cancel();
    }
}

Т.е делать инициализацию в конструкторе. И проверьте, не передаете ли при создании экземпляра класса moveBullet в конструктор вместо игрока null.

Fenix78563 · 19 дек 2016

slavik123123123 сказал(а):
p == null?

Deamonish сказал(а):
Как уже сказали, скорее всего p == null.

Fenix78563 сказал(а):
Когда отключаю из кода строки
то все пашет. Даже сообщение приходит. Т.е. игрок не null

Угу, да.

Deamonish сказал(а):
Java:

private final Vector vc; //че то пропущено vc =pl.getLocation().getDirection();

Объявляете переменную как final (неизменяемая если что), а потом изменяете ее.
Я просто похлопаю

З.Ы. trow'инг игрока на null не проканал, он не null

alexandrage сказал(а):

Пробуй так. Или просто убери final и попробуй.

Java:

public class moveBullet extends BukkitRunnable{
    private Player p;
    private Firework Bullet;
    public moveBullet(Player pl, Firework bul){
        this.p=pl;
        this.Bullet=bul;
    }
    @Override
    public void run() {
        Vector vc = p.getLocation().getDirection();
        Double y = vc.getY();
        p.sendMessage("as"); //специально для проверки null
        if (Bullet.isDead()){
            cancel();
        }
        cancel();
    }
}

Спасибо, работает, но почему? В чем все таки был подвох?

slavik123123123 · 19 дек 2016

Fenix78563 сказал(а):
Спасибо, работает, но почему? В чем все таки был подвох

В том что p был null? джаве почти пофиг на порядок строк для переменных. тоесть делая
Player p;
У тебя в любой переменой p будет null.(те что находятся не в методе)
Location loc=p.getLocation();//null
Чтобы такого небыло тебе нужно сделать так.
Player p;
Location loc;
метод(Игрок, еще чтото, и еще) {
p = Игрок;
loc= p.getLocation();
}

alexandrage · 19 дек 2016

Fenix78563 сказал(а):
Угу, да.

Объявляете переменную как final (неизменяемая если что), а потом изменяете ее.
Я просто похлопаю

З.Ы. trow'инг игрока на null не проканал, он не null

Спасибо, работает, но почему? В чем все таки был подвох?

Ты работал с Player до конструктора, вот и был бублик.

Sonicxd2 · 19 дек 2016

Fenix78563 сказал(а):
Доброго времени суток, форумчане. Ловит NullPointer, хотя объект не null

Java:

public class moveBullet extends BukkitRunnable{ private Player p; private Firework Bullet; public moveBullet(Player pl, Firework bul){ this.p=pl; this.Bullet=bul; } final Vector vc = p.getLocation().getDirection(); Double y = vc.getY(); @Override public void run() { p.sendMessage("as"); //специально для проверки null if (Bullet.isDead()){ cancel(); } cancel(); } }

null в строке

Java:

final Vector vc = p.getLocation().getDirection();

Когда отключаю из кода строки

Java:

final Vector vc = p.getLocation().getDirection(); Double y = vc.getY();

то все пашет. Даже сообщение приходит. Т.е. игрок не null

В чем подвох?
Заранее спасибо

Засунь эти строки "

final Vector vc = p.getLocation().getDirection();
Double y = vc.getY();

" в "public moveBullet"

Fenix78563 · 19 дек 2016

Спасибо за ответы. В явке получается нельзя использовать методы вне методов?

Den_Abr · 19 дек 2016

Fenix78563 сказал(а):
В явке получается нельзя использовать методы вне методов?

В явке нельзя вызывать методы на нулевых объектах

Fenix78563 · 19 дек 2016

Den_Abr сказал(а):
В явке нельзя вызывать методы на нулевых объектах

Ало, он НЕ NULL

slavik123123123 · 19 дек 2016

Fenix78563 сказал(а):
Ало, он НЕ NULL

Java:

class {
Player p;
method(Player pl) {
this.p = pl; }
Location loc = p.getLocation(); //ТУТ ВСЕГДА p == null
//Хоть поставь это в конец класса, всеравно будет null; Хоть в начало. Везде null будет.

}

Fenix78563 · 19 дек 2016

slavik123123123 сказал(а):
//Хоть поставь это в конец класса, всеравно будет null; Хоть в начало. Везде null будет.

почему? мы же объявили игрока. или вы заявляете что ява нелинейный язык? (гугл заявляет что ява линейная)

Den_Abr · 19 дек 2016

Fenix78563 сказал(а):
почему? мы же объявили игрока. или вы заявляете что ДЖАВА нелинейный язык? (гугл заявляет что ДЖАВА линейная)

сначала инициализируются все поля, а только потом выполняется код конструктора

alexandrage · 20 дек 2016

Fenix78563 сказал(а):
почему? мы же объявили игрока. или вы заявляете что ява нелинейный язык? (гугл заявляет что ява линейная)

Я тебе написал уже почему. Объект Player был пустым, потому что ты использовал его ДО конструктора, где ты назначаешь значения переменным. Сперва нужно инициализировать класс MyClass myclass new MyClass(передача значений); , а потом уже дальше работать с ними myclass.getUserVector(); Или же как писали выше, получить его сразу в конструкторе, где переменные определились уже.

Fenix78563 · 20 дек 2016

Еще раз спасибо за ответы.

Помогите Null Pointer

Fenix78563

Активный участник

alexandrage

Старожил

Fenix78563

Активный участник

slavik123123123

Старожил

alexandrage

Старожил

Deamonish

Активный участник

Fenix78563

Активный участник

slavik123123123

Старожил

alexandrage

Старожил

Sonicxd2

Активный участник

Fenix78563

Активный участник

Den_Abr

Старожил

Fenix78563

Активный участник

slavik123123123

Старожил

Fenix78563

Активный участник

Den_Abr

Старожил

alexandrage

Старожил

Fenix78563

Активный участник