Loop ArrayList<String> in Order

Question

I have an a ArrayList which I would like to display in order and loop continuously, but I'm not sure how to do so. I seem to only be able to use the specific type of String used in the current code, but if you know of a workaround so I can iterate it, that would be great.

The current code for the ArrayList and string is:

  public void showBarChanging(final Player p)
  {
    getServer().getScheduler().scheduleSyncRepeatingTask(this, new Runnable()
    {
      public void run()
      {
        ArrayList<String> list = new ArrayList<String>();
        list.add(DCBar.this.getConfig().getString("1").replaceAll("&", "§"));
        list.add(DCBar.this.getConfig().getString("2").replaceAll("&", "§"));
        list.add(DCBar.this.getConfig().getString("3").replaceAll("&", "§"));
        list.add(DCBar.this.getConfig().getString("4").replaceAll("&", "§"));
        list.add(DCBar.this.getConfig().getString("5").replaceAll("&", "§"));
        list.add(DCBar.this.getConfig().getString("6").replaceAll("&", "§"));
        list.add(DCBar.this.getConfig().getString("7").replaceAll("&", "§"));

        String message = (String)list.get(list.size());

        BarAPI.setMessage(p, message);
      }
    }, 0L, 100L);
  }

Config file (where integers come from) looks like this:

1: '&bTest 1'
2: '&bTest 2'
3: '&bTest 3'
4: '&bTest 4'
5: '&bTest 5'
6: '&bTest 6'
7: '&bTest 7'

It currently infinitely displays the seventh integer's message.

EDIT:

Jason suggested the following code, which works, but it does not cycle through the integers. The "5" in BarAPI.setMessage(p, message, 5); is equal to five seconds.

    ArrayList<String> list = new ArrayList<String>();
    for (int i=1;i<=7;i++){
        list.add(DCBar.this.getConfig().getString(Integer.toString(i)).replaceAll("&", "§"));
    }

    String message = (String)list.get(list.size()-1);

    BarAPI.setMessage(p, message, 5);

list.get(list.size()) should always trigger an exception, since list.size() is one more than the maximum legal index. What are you trying to accomplish? What's the problem with looping through an array list either by index or with an iterator (or with an enhanced for loop, which uses an iterator)? — Ted Hopp
– Ted Hopp, Commented Feb 11, 2014 at 2:41
What is the return type of getConfig() and what methods does that type have, other than getString()? — Dawood ibn Kareem
– Dawood ibn Kareem, Commented Feb 11, 2014 at 2:45

Jason · Accepted Answer · 2014-02-11 03:34:56Z

1

Is this what you wanted?

  public void showBarChanging(final Player p)
  {
    getServer().getScheduler().scheduleSyncRepeatingTask(this, new Runnable()
    {

      private static int count=1;
      public void run()
      {
           if (count>7) count=1;

           String string = DCBar.this.getConfig().getString(Integer.toString(count++));
           string = string.replaceAll("&", "§");

           BarAPI.setMessage(p, string, 5);   

      }
    }, 0L, 100L);
  }

edited Feb 11, 2014 at 3:34

answered Feb 11, 2014 at 2:42

Jason

14k15 gold badges80 silver badges131 bronze badges

Sign up to request clarification or add additional context in comments.

7 Comments

Ted Hopp Over a year ago

The loop index needs to be for (int i = 1; i <= 7; i++) to replicate the behavior of OP's code.

Jason Over a year ago

Good catch. Is this what he wanted? I can't tell.

Ted Hopp Over a year ago

I don't know what OP is asking. But you still need to start the loop at 1, not at 0.

swiftsly Over a year ago

@Jason Closer, but it only displays the first one, and I should have been more clear about this, but it needs to display one at a time, and change every five seconds (which is built into the "BarAPI"), which it is not doing. It only seems to display number 7 from the ArrayList.

Jason Over a year ago

Updated the code. Think I understand what you want now.

|

ChriskOlson · Accepted Answer · 2014-02-11 03:05:31Z

0

Can't comment yet, but I know that using the .sort() method would put your arrayList in order. And then you'd loop

answered Feb 11, 2014 at 3:05

ChriskOlson

5131 gold badge3 silver badges12 bronze badges

Collectives™ on Stack Overflow

Loop ArrayList<String> in Order

2 Answers 2

7 Comments

Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

7 Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Related