Text Parser Snippet [v.1.1]

[Moderator]

12th Dec 2015
IcemanBo: Too long time as NeedsFix. Rejected.

11:19, 9th Feb 2011
Bribe: See the post I wrote for you and Nestharus' descriptions

 

[ShadowFlare]

Subscribed

 

[Adiktuz]

I think this fits more on the jass sections... ^_^

 

[Cihparg]

Can I ask you one question... Why is this vJass and 3 functions as it could be done with Jass in one function?

 

[ShadowFlare]

Can I ask you one question... Why is this vJass and 3 functions as it could be done with Jass in one function?

Privitization

 

[Cihparg]

Shortened the 2 unnecessary functions:

function GetMaxLines takes nothing returns integer
    
    return MaxLines
    
endfunction

function GetLine takes integer line returns string
    
    return Lines[line]
    
endfunction

I'm writing the new functions *Cough*

 

[ShadowFlare]

updated

 

[Cihparg]

Can you write this in Jass or is Jass too difficult for you? Because there's no need for vJass in this function.

 

[Nestharus]

This could easily just return one string by inputting \n throughout it.

I don't see a need to be able to access specific lines. If you need to access specific lines, could still be done in one function by returning a stack.

Furthermore, this seems to do word wrapping. You should rename it for more clarity. String parsing could be anything to do with parsing, so this is pretty vague and ambiguous.

edit
Wrote this out

library WrapString
    function WrapString takes string stringToWrap, integer maxLineLength returns string
        local integer i = 1
        local integer length = StringLength(stringToWrap)
        local integer space = 0
        local string new = ""
        local integer last = 0
        local integer cur = 0
        loop
            set cur = cur + 1
            if ((SubString(stringToWrap, i-1, i)) == " ") then
                if (i-1 == last) then
                    set cur = cur - 1
                else
                    set space = i
                endif
            endif
            if (i == length) then
                if (new != "") then
                    set new = new + "\n"
                endif
                return new + SubString(stringToWrap, last, i)
            elseif (cur >= maxLineLength and space > last) then
                if (new != "") then
                    set new = new + "\n"
                endif
                set new = new + SubString(stringToWrap, last, space)
                set last = space
                set cur = 0
            endif
            set i = i + 1
        endloop
        
        return new
    endfunction
endlibrary

and demo

private struct test extends array
    private static method tests takes nothing returns boolean
        call DisplayTimedTextToPlayer(GetLocalPlayer(), 0, 0, 100000, WrapString(GetEventPlayerChatString(), 9))
        return false
    endmethod
    
    private static method onInit takes nothing returns nothing
        local trigger t = CreateTrigger()
        call TriggerAddCondition(t, Condition(function thistype.tests))
        call TriggerRegisterPlayerChatEvent(t, Player(0), "", false)
    endmethod
endstruct

The above essentially has the same output as yours without the nested loops, extra globals, 2 extra functions, and array locals : P

I say essentially because this one cuts off the first space of a new line if that line begins with a space. In your screenshot, the last line starts with a space. With this code, the last line wouldn't start with a space ; ).

Feel free to use the above code in your lib = P.

 

[Scythy Dervish]

Can you write this in Jass or is Jass too difficult for you? Because there's no need for vJass in this function.

I don't know why NOT to do this in vJass

In vJass you can make most things shorter and more efficient than in Jass or GUI.

How many mappers (which are making maps and not learning how to map) do you know which do not have the JNGP?

 

[Adiktuz]

@OffGraphic - if you want the systems in GUI or JASS why not do them yourself rather than going through the resources and blabbing about, this is doable in GUI, write this in JASS, etc...

+I like vJASS systems because you can just copy them into a blank trigger rather than copying a JASS system into the map header (which could lead to a harder readability when you import a lot of systems)...

anyway, 3/5

 

[Bribe]

Consider the recommendations from Nestharus, if anyone knows how to do string parsing, he does. I have to agree the name itself is just not descriptive for what this does. Nestharus wrote StringParser, take a look at his library (in the JASS Forum) and you will see THAT is dynamic string parsing, worthy of its name, and this is definitely a snippet that provides only a single type of parsing.