Java'da döngü için son yineleme

Döngünün son kez yinelenip yinelenmediğini belirlemenin bir yolu var mı? Kodum şuna benzer:

int[] array = {1, 2, 3...};
StringBuilder builder = new StringBuilder();

for(int i : array)
{
    builder.append("" + i);
    if(!lastiteration)
        builder.append(",");
}

Şimdi, son yinelemede virgül eklemek istemiyorum. Şimdi son yineleme olup olmadığını belirlemek için bir yolu var mı yoksa for döngüsü veya takip etmek için harici bir sayaç kullanarak sıkışıp kaldım.

Başka bir alternatif, ilk yinelemede değil, i eklemeden önce virgül eklemektir. (Bu arada lütfen kullanmayın "" + i- burada gerçekten birleştirme istemezsiniz ve StringBuilder mükemmel bir ek (int) aşırı yüklemeye sahiptir.)

int[] array = {1, 2, 3...};
StringBuilder builder = new StringBuilder();

for (int i : array) {
    if (builder.length() != 0) {
        builder.append(",");
    }
    builder.append(i);
}

Bununla ilgili güzel olan şey, herhangi biriyle çalışacağıdır Iterable- her zaman bir şeyler dizine ekleyemezsiniz. ("Virgül ekleyin ve sonra sonunda kaldırın" gerçekten StringBuilder kullandığınızda hoş bir öneridir - ancak akışlara yazma gibi şeyler için işe yaramaz. )

Bunu yapmanın başka bir yolu:

String delim = "";
for (int i : ints) {
    sb.append(delim).append(i);
    delim = ",";
}

Güncelleme: Java 8 için artık Koleksiyonerleriniz var

Her zaman eklemek daha kolay olabilir. Ve sonra, döngü ile işiniz bittiğinde, son karakteri kaldırın. Bu şekilde ton daha az koşullu.

Sen kullanabilirsiniz StringBuilders' deleteCharAt(int index)endeks varlık ilelength() - 1

Belki de İş için yanlış aracı kullanıyorsunuzdur.

Bu, yaptığınızdan daha el kitabı ama biraz "eski okul" olmasa bile bir şekilde daha zarif

 StringBuffer buffer = new StringBuffer();
 Iterator iter = s.iterator();
 while (iter.hasNext()) {
      buffer.append(iter.next());
      if (iter.hasNext()) {
            buffer.append(delimiter);
      }
 }

Bu neredeyse bu StackOverflow sorununun tekrarıdır . İstediğiniz StringUtils ve join yöntemini çağırmaktır .

StringUtils.join(strArr, ',');

Başka bir çözüm (belki de en verimli)

    int[] array = {1, 2, 3};
    StringBuilder builder = new StringBuilder();

    if (array.length != 0) {
        builder.append(array[0]);
        for (int i = 1; i < array.length; i++ )
        {
            builder.append(",");
            builder.append(array[i]);
        }
    }

basit tutun ve döngü için bir standart kullanın:

for(int i = 0 ; i < array.length ; i ++ ){
    builder.append(array[i]);
    if( i != array.length - 1 ){
        builder.append(',');
    }
}

veya sadece apache commons-lang kullanın StringUtils.join ()

Açık döngüler her zaman örtük olanlardan daha iyi çalışır.

builder.append( "" + array[0] );
for( int i = 1; i != array.length; i += 1 ) {
   builder.append( ", " + array[i] );
}

Sıfır uzunlukta bir dizi ile uğraşıyorsanız, her şeyi bir if ifadesine sarmalısınız.

Klasik bir dizin döngüsüne dönüştürürseniz, evet.

Veya bittikten sonra son virgülleri silebilirsiniz. Bunun gibi:

int[] array = {1, 2, 3...};
StringBuilder

builder = new StringBuilder();

for(int i : array)
{
    builder.append(i + ",");
}

if(builder.charAt((builder.length() - 1) == ','))
    builder.deleteCharAt(builder.length() - 1);

Ben, ben sadece kullanmak StringUtils.join()gelen commons-lang .

Sınıf Ayırıcıya ihtiyacınız var .

Separator s = new Separator(", ");
for(int i : array)
{
     builder.append(s).append(i);
}

Sınıfın uygulanması Separatorçok basittir. toString()Boş bir dize döndüren ilk çağrı dışında her çağrıda döndürülen bir dize sarar .

Java.util.AbstractCollection.toString () öğesine dayanarak, sınırlayıcıdan kaçınmak için erken çıkar.

StringBuffer buffer = new StringBuffer();
Iterator iter = s.iterator();
for (;;) {
  buffer.append(iter.next());
  if (! iter.hasNext())
    break;
  buffer.append(delimiter);
}

Etkili ve zariftir, ancak diğer bazı cevaplar kadar açık değildir.

İşte bir çözüm:

int[] array = {1, 2, 3...};
StringBuilder builder = new StringBuilder();
bool firstiteration=true;

for(int i : array)
{
    if(!firstiteration)
        builder.append(",");

    builder.append("" + i);
    firstiteration=false;
}

İlk yinelemeyi arayın :)  

Araç setinde belirtildiği gibi, Java 8'de şimdi Koleksiyonerlerimiz var . Kod şöyle görünecektir:

String joined = array.stream().map(Object::toString).collect(Collectors.joining(", "));

Bence bu tam olarak aradığınızı yapıyor ve bu birçok şey için kullanabileceğiniz bir model.

Yine başka bir seçenek.

StringBuilder builder = new StringBuilder();
for(int i : array)
    builder.append(',').append(i);
String text = builder.toString();
if (text.startsWith(",")) text=text.substring(1);

Burada açıklanan çözümlerin birçoğu üstte IMHO'nun, özellikle de dış kütüphanelere dayanan çözümlerin biraz üzerindedir. Her zaman kullandığım virgülle ayrılmış bir liste elde etmek için güzel, temiz ve net bir deyim var. Koşullu (?) Operatöre dayanır:

Düzenleme : Orijinal çözüm doğru, ancak yorumlara göre optimal değil. İkinci kez denemek:

    int[] array = {1, 2, 3};
    StringBuilder builder = new StringBuilder();
    for (int i = 0 ;  i < array.length; i++)
           builder.append(i == 0 ? "" : ",").append(array[i]); 

İşte dizi ve StringBuilder bildirimi de dahil olmak üzere 4 kod satırında gidin.

İşte bu sonuçlarla çalıştırdığım (uygulamak zorunda olduğum şeyle ilgili) bir SSCCE karşılaştırması:

elapsed time with checks at every iteration: 12055(ms)
elapsed time with deletion at the end: 11977(ms)

En azından benim bilgisayarımda her işleminde çek atlama özellikle verilerin aklı başında hacimleri için fark daha hızlı değil, ama olup daha hızlı.

import java.util.ArrayList;
import java.util.List;


public class TestCommas {

  public static String GetUrlsIn(int aProjectID, List<String> aUrls, boolean aPreferChecks)
  {

    if (aPreferChecks) {

      StringBuffer sql = new StringBuffer("select * from mytable_" + aProjectID + " WHERE hash IN ");

      StringBuffer inHashes = new StringBuffer("(");
      StringBuffer inURLs = new StringBuffer("(");

      if (aUrls.size() > 0)
      {

      for (String url : aUrls)
      {

        if (inHashes.length() > 0) {
        inHashes.append(",");
        inURLs.append(",");
        }

        inHashes.append(url.hashCode());

        inURLs.append("\"").append(url.replace("\"", "\\\"")).append("\"");//.append(",");

      }

      }

      inHashes.append(")");
      inURLs.append(")");

      return sql.append(inHashes).append(" AND url IN ").append(inURLs).toString();
    }

    else {

      StringBuffer sql = new StringBuffer("select * from mytable" + aProjectID + " WHERE hash IN ");

      StringBuffer inHashes = new StringBuffer("(");
      StringBuffer inURLs = new StringBuffer("(");

      if (aUrls.size() > 0)
      {

      for (String url : aUrls)
      {
        inHashes.append(url.hashCode()).append(","); 

        inURLs.append("\"").append(url.replace("\"", "\\\"")).append("\"").append(",");
      }

      }

      inHashes.deleteCharAt(inHashes.length()-1);
      inURLs.deleteCharAt(inURLs.length()-1);

      inHashes.append(")");
      inURLs.append(")");

      return sql.append(inHashes).append(" AND url IN ").append(inURLs).toString();
    }

  }

  public static void main(String[] args) { 
        List<String> urls = new ArrayList<String>();

    for (int i = 0; i < 10000; i++) {
      urls.add("http://www.google.com/" + System.currentTimeMillis());
      urls.add("http://www.yahoo.com/" + System.currentTimeMillis());
      urls.add("http://www.bing.com/" + System.currentTimeMillis());
    }


    long startTime = System.currentTimeMillis();
    for (int i = 0; i < 300; i++) {
      GetUrlsIn(5, urls, true);
    }
    long endTime = System.currentTimeMillis();
    System.out.println("elapsed time with checks at every iteration: " + (endTime-startTime) + "(ms)");

    startTime = System.currentTimeMillis();
    for (int i = 0; i < 300; i++) {
      GetUrlsIn(5, urls, false);
    }
    endTime = System.currentTimeMillis();
    System.out.println("elapsed time with deletion at the end: " + (endTime-startTime) + "(ms)");
  }
}

Başka bir yaklaşım, dizinin uzunluğunun (varsa) ayrı bir değişkende saklanmasıdır (her seferinde uzunluğun yeniden kontrol edilmesinden daha etkilidir). Son olarak, virgülün eklenip eklenmeyeceğini belirlemek için dizininizi bu uzunlukla karşılaştırabilirsiniz.

DÜZENLEME: Başka bir husus, her yinelemede bir koşulun denetlenmesine karşı son karakteri (dize kopyasına neden olabilir) kaldırmanın performans maliyetini tartmaktır.

Bir diziyi yalnızca virgülle ayrılmış bir diziye dönüştürüyorsanız, birçok dilde tam olarak bunun için birleştirme işlevi bulunur. Bir diziyi, her öğe arasında sınırlayıcı olan bir dizeye dönüştürür.

Bu durumda, son tekrar olup olmadığını gerçekten bilmeye gerek yoktur. Bunu çözebilmemiz için birçok yol var. Bunun bir yolu:

String del = null;
for(int i : array)
{
    if (del != null)
       builder.append(del);
    else
       del = ",";
    builder.append(i);
}

Burada iki alternatif yol var:

1: Apache Commons String Utils

2: firsttrue olarak ayarlanmış bir boole çağrısı yapın . Her yinelemede firstyanlışsa virgül ekleyin; bundan sonra, firstfalse olarak ayarlayın .

Sabit bir dizi olduğundan, gelişmiş için kaçınmak daha kolay olurdu ... Nesne bir koleksiyon ise, bir yineleyici daha kolay olurdu.

int nums[] = getNumbersArray();
StringBuilder builder = new StringBuilder();

// non enhanced version
for(int i = 0; i < nums.length; i++){
   builder.append(nums[i]);
   if(i < nums.length - 1){
       builder.append(",");
   }   
}

//using iterator
Iterator<int> numIter = Arrays.asList(nums).iterator();

while(numIter.hasNext()){
   int num = numIter.next();
   builder.append(num);
   if(numIter.hasNext()){
      builder.append(",");
   }
}