Java-Gaming.org Hi !
Featured games (83)
games approved by the League of Dukes
Games in Showcase (542)
Games in Android Showcase (133)
games submitted by our members
Games in WIP (604)
games currently in development
News: Read the Java Gaming Resources, or peek at the official Java tutorials
 
    Home     Help   Search   Login   Register   
Pages: [1]
  ignore  |  Print  
  Efficient parsing?  (Read 468 times)
0 Members and 1 Guest are viewing this topic.
Offline Tyecon

Senior Newbie





« Posted 2013-04-02 17:28:40 »

http://pastebin.java-gaming.org/224c73c2b59
This is my Compatiblity.shader file, and I'm combining the .vert and .frag shaders into one file. My current code for parsing is:
1  
2  
3  
4  
5  
6  
7  
8  
9  
10  
String shader=AssetHandler.readFile(path+".shader");
            String version=shader.substring(
                    (shader.indexOf("<VERSION>")==-1) ? 0 : shader.indexOf("<VERSION>")+9,
                    (shader.indexOf("</VERSION>")==-1) ? 0 : shader.indexOf("</VERSION>")).trim();
            vertShader=createShader(
                    ((version.isEmpty()) ? "" : ("#version "+version))
                    + shader.substring(shader.indexOf("<VERT>")+6, shader.indexOf("</VERT>")), GL_VERTEX_SHADER);
            fragShader=createShader(
                    ((version.isEmpty()) ? "" : ("#version "+version))
                    + shader.substring(shader.indexOf("<FRAG>")+6, shader.indexOf("</FRAG>")), GL_FRAGMENT_SHADER);


Is this ok? I feel like it could be a lot cleaner and nicer. I'm going to add a <varying> tag next.

Please have a baggy ready before looking at my code.
Offline quew8

JGO Coder


Medals: 33



« Reply #1 - Posted 2013-04-03 15:12:53 »

If your going to be doing some serious parsing, might I suggest using JavaCC. I find it to be a wonderful tool that is as simple as it needs to be. If your unaware of it, you write a syntax for it and it will generate the source code for a java parser that will parse your input and perform any actions you add in.
Offline StrideColossus
« Reply #2 - Posted 2013-04-03 15:32:41 »

Is this ok? I feel like it could be a lot cleaner and nicer. I'm going to add a <varying> tag next.

Couple of suggestions you may want to consider:

1. Encapsulate the code that extracts the text from between the XML tags into a helper method:
- you can then write a unit-test to make sure that chunk works
- the code that's doing the actual parsing will then be a lot cleaner (and therefore less buggy / easier to maintain)
- and you won't be cut-and-pasting and hard-coding the end index when you add further functionality, further reducing chances of cock-ups Smiley

2. Avoid duplicating the indexOf() call - OK the performance of doing it twice is probably hardly game-breaking but it's good practice.

3. Is there a reason you're not just throwing exceptions for the cases where the file is not valid (no <version> tag)?  You'll have to check elsewhere that the vertex/fragment shader is potentially empty which (presumably) is a pretty fatal scenario, so you might as well handle that explicitly with an exception.

-stride
Pages: [1]
  ignore  |  Print  
 
 
You cannot reply to this message, because it is very, very old.

 

Add your game by posting it in the WIP section,
or publish it in Showcase.

The first screenshot will be displayed as a thumbnail.

CopyableCougar4 (16 views)
2014-12-28 02:10:29

BurntPizza (17 views)
2014-12-27 22:38:51

Mr.CodeIt (13 views)
2014-12-27 04:03:04

TheDudeFromCI (17 views)
2014-12-27 02:14:49

Mr.CodeIt (25 views)
2014-12-23 03:34:11

rwatson462 (56 views)
2014-12-15 09:26:44

Mr.CodeIt (46 views)
2014-12-14 19:50:38

BurntPizza (92 views)
2014-12-09 22:41:13

BurntPizza (113 views)
2014-12-08 04:46:31

JscottyBieshaar (86 views)
2014-12-05 12:39:02
How do I start Java Game Development?
by gouessej
2014-12-27 19:41:21

Resources for WIP games
by kpars
2014-12-18 10:26:14

Understanding relations between setOrigin, setScale and setPosition in libGdx
by mbabuskov
2014-10-09 22:35:00

Definite guide to supporting multiple device resolutions on Android (2014)
by mbabuskov
2014-10-02 22:36:02

List of Learning Resources
by Longor1996
2014-08-16 10:40:00

List of Learning Resources
by SilverTiger
2014-08-05 19:33:27

Resources for WIP games
by CogWheelz
2014-08-01 16:20:17

Resources for WIP games
by CogWheelz
2014-08-01 16:19:50
java-gaming.org is not responsible for the content posted by its members, including references to external websites, and other references that may or may not have a relation with our primarily gaming and game production oriented community. inquiries and complaints can be sent via email to the info‑account of the company managing the website of java‑gaming.org
Powered by MySQL Powered by PHP Powered by SMF 1.1.18 | SMF © 2013, Simple Machines | Managed by Enhanced Four Valid XHTML 1.0! Valid CSS!