Java-Gaming.org
Play Revenge of the Titans! The situation is critical. We need fancy commanders to defend Earth, the moon, Mars!
Featured games (78)
games approved by the League of Dukes
Games in Showcase (408)
games submitted by our members
Games in WIP (293)
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  
  Not Greater Than Problem  (Read 454 times)
0 Members and 1 Guest are viewing this topic.
Offline tyeeeee1

Senior Member


Medals: 3
Projects: 1



« Posted 2013-02-03 15:11:59 »

I'm making a few random classes that do various things at the moment but I'm slightly stuck on this method. Every time I run the below program it works perfectly except it throws an array index out of bounds exception before it finishes; I know the cause of the problem but the only way I can think of fixing it would be to find some way to have an 'is not greater than' operator to make sure that z never goes past the end of the char array.

If anyone can figure out a fix to this it would help. Maybe I'll get lucky and figure it out myself soon. =)

1  
2  
3  
4  
5  
6  
7  
8  
9  
10  
11  
12  
13  
14  
15  
16  
17  
18  
19  
20  
21  
22  
23  
class MessageDisplay
{
   public MessageDisplay()
   {
   }
   
   public static void typeWriterDisplay(String x)
   {
      char[] array = x.toCharArray();
      int i = 0;
      int z = 0;
      while( i <= x.length() )
      {
         System.out.println( array[z] );
         z++;
      }
   }
   
   public static void main(String args[])
   {
      typeWriterDisplay("Lololol");
   }
}
Offline matheus23

JGO Wizard


Medals: 72
Projects: 3


You think about my Avatar right now!


« Reply #1 - Posted 2013-02-03 15:27:48 »

instead of
i <= x.length()
use
i < x.length();
...

But this is strange code anyways ^^

Better use this code:

public static void typeWriterDisplay(String str) {
    for (int i = 0; i < str.length(), i++) {
        System.out.println(str.charAt(i));
    }
}

See my:
    My development Blog:     | Or look at my RPG | Or simply my coding
http://matheusdev.tumblr.comRuins of Revenge  |      On Github
Offline tyeeeee1

Senior Member


Medals: 3
Projects: 1



« Reply #2 - Posted 2013-02-03 15:34:04 »

instead of
i <= x.length()
use
i < x.length();
...

But this is strange code anyways ^^

Better use this code:

public static void typeWriterDisplay(String str) {
    for (int i = 0; i < str.length(), i++) {
        System.out.println(str.charAt(i));
    }
}


Ah, I figured I could use either a for or while loop but I guess the for loop is better. It's all working now, thanks!

Now to add a thread to finish the effect!
Games published by our own members! Check 'em out!
Play the free demo of Revenge of the Titans!
Offline ClickerMonkey

Senior Member


Medals: 11


Game Engineer


« Reply #3 - Posted 2013-02-03 15:59:01 »

1  
2  
3  
4  
5  
6  
7  
8  
9  
10  
      char[] array = x.toCharArray();
      int i = 0;
      int z = 0;
      while( i <= x.length() )
      {
         System.out.println( array[z] );
         z++;
      }

}


You're not incrementing i, you're incrementing z. You also need to exlude x.length();

1  
2  
3  
4  
5  
      while( z < x.length() )
      {
         System.out.println( array[z] );
         z++;
      }

Offline Cero
« Reply #4 - Posted 2013-02-03 18:20:50 »

Rule of thumb: almost always use for-loops

every loop can do everything, use "for" most of times

Offline Jimmt
« Reply #5 - Posted 2013-02-03 18:35:30 »

Can't remember the last time I used while...oh yeah, Java4K.
Offline tyeeeee1

Senior Member


Medals: 3
Projects: 1



« Reply #6 - Posted 2013-02-03 20:23:44 »

Can't remember the last time I used while...oh yeah, Java4K.

Yeah, it's not the most used loop ever... I usually just use it for 'while(variable= true)'.
Offline sproingie
« Reply #7 - Posted 2013-02-03 21:27:55 »

I usually just use it for 'while(variable= true)'.

I see what you did there.

Offline tyeeeee1

Senior Member


Medals: 3
Projects: 1



« Reply #8 - Posted 2013-02-03 21:42:21 »


O.o So did I thanks to you, lol...
Pages: [1]
  ignore  |  Print  
 
 

Play Revenge of the Titans! The situation is critical. We need fancy commanders to defend Earth, the moon, Mars!
 
Browse for soundtracks for your game!

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

The first screenshot will be displayed as a thumbnail.

The invasion has landed! On Mars! And you're there to beat 'em!
cubemaster21 (126 views)
2013-05-17 21:29:12

alaslipknot (134 views)
2013-05-16 21:24:48

gouessej (164 views)
2013-05-16 00:53:38

gouessej (157 views)
2013-05-16 00:17:58

theagentd (169 views)
2013-05-15 15:01:13

theagentd (154 views)
2013-05-15 15:00:54

StreetDoggy (198 views)
2013-05-14 15:56:26

kutucuk (222 views)
2013-05-12 17:10:36

kutucuk (221 views)
2013-05-12 15:36:09

UnluckyDevil (225 views)
2013-05-12 05:09:57
Complex number cookbook
by Roquen
2013-04-24 12:47:31

2D Dynamic Lighting
by Oskuro
2013-04-17 16:46:12

2D Dynamic Lighting
by Oskuro
2013-04-17 16:45:57

2D Dynamic Lighting
by Oskuro
2013-04-17 16:23:20

Noise (bandpassed white)
by Roquen
2013-04-05 17:36:01

Noise (bandpassed white)
by Roquen
2013-04-03 16:17:38

Java Data structures
by Roquen
2013-03-29 13:21:12

Topic Request
by kutucuk
2013-03-22 21:42:01
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!
Page created in 0.078 seconds with 21 queries.