Problem With HUD: Cannot read null.screen in Developer Help

ID:139561
 
Nov 22 2010, 7:05 pm
I've had a problem with an HUD before that had worked up until now for some reason:
obj/Wright
    icon = 'Phoenix Wright Table Slam.dmi'
    icon_state = ""
    layer = 999
    New(client/c)
        screen_loc="10,5"
        c.screen+=src
    Del(client/c)
        c.screen-=src
mob/proc
    Wright()
        src << sound(null)
        src << sound('Investigation ~ Cornered 2004 Remix.mid')
        sleep(10)
        var/obj/Wright/W = new/obj/Wright
        W.icon_state = "1"


runtime error: Cannot read null.screen
proc name: New (/obj/Wright/New)

I have also tried:
obj/Wright
    icon = 'Phoenix Wright Table Slam.dmi'
    icon_state = ""
    layer = 999
    New(client/c)
        screen_loc="10,5"
        c.screen+=src
    Del(client/c)
        c.screen-=src
obj/var
    screen
mob/proc
    Wright()
        src << sound(null)
        src << sound('Investigation ~ Cornered 2004 Remix.mid')
        sleep(10)
        var/obj/Wright/W = new/obj/Wright
        W.icon_state = "1"

And got a different runtime error, but no errors on the compiler.

What I need to know is what is wrong with it. I do not need any "spoon feeding". I recall on another post I wished for an explanation, not a string of code and got trolled by almost everyone saying I want to be spoon fed.
#1 Nov 22 2010, 7:20 pm
Replace <small>var/obj/Wright/W = new/obj/Wright</small> with <small>src.client.screen += new/obj/Wright</small> and set its icon state somewhere else like the <small>New()</small> proc.
#2 Nov 22 2010, 7:23 pm
In response to DarkeWarrior (#1)
DarkeWarrior wrote:
Replace <small>var/obj/Wright/W = new/obj/Wright</small> with <small>src.client.screen += new/obj/Wright</small> and set its icon state somewhere else like the <small>New()</small> proc.

Sorry, but the only issue with that is, I want it's icon state to constantly change. I also want it set to a variable, which requires the var/obj/Wright/W in it, doesn't it?
#3 Nov 22 2010, 7:43 pm
In response to Narutorox123456 (#2)
Not necessarily. If you ever wanted to change it's icon state at any given time you could simply use <small>for(var/obj/Wright/w in client.screen)</small>
#4 Nov 22 2010, 7:44 pm
In response to DarkeWarrior (#3)
DarkeWarrior wrote:
Not necessarily. If you ever wanted to change it's icon state at any given time you could simply use <small>for(var/obj/Wright/w in client.screen)</small>

Oh, Ok. I didn't realize that! I'm going to re-do the code to fit your description the best I can and tell you if I get any errors or runtime errors!
#5 Nov 22 2010, 7:47 pm
In response to DarkeWarrior (#3)
DarkeWarrior wrote:
Not necessarily. If you ever wanted to change it's icon state at any given time you could simply use <small>for(var/obj/Wright/w in client.screen)</small>

Ok, so far it is doing good and is showing up. The only problem: It gives a runtime error:

runtime error: Cannot read null.screen
proc name: New (/obj/Wright/New)
#6 Nov 22 2010, 7:54 pm
In response to Narutorox123456 (#5)
Instead of <small>client.screen += new/obj/Wright</small>, try <small>var/obj/Wright/W = new(src.client)</small>. Condolences to Calus CoRPS for pointing that out.
#7 Nov 22 2010, 8:00 pm
In response to DarkeWarrior (#6)
DarkeWarrior wrote:
Instead of <small>client.screen += new/obj/Wright</small>, try <small>var/obj/Wright/W = new(src.client)</small>. Condolences to Calus CoRPS for pointing that out.

Thanks! It works fine now!