Última iteración del bucle for mejorado en java

Question

¿Hay alguna manera de determinar si el ciclo se está iterando por última vez? Mi código es como la siguiente:

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

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

Ahora la cosa es que no quiero añadir la coma en la última iteración. Ahora hay una manera de determinar si es la última iteración o estoy atrapado con el ciclo for o usando un contador externo para realizar un seguimiento.

Answer 1

Puede ser más fácil agregar siempre. Y luego, cuando hayas terminado con tu ciclo, simplemente elimina el personaje final. Tone menos condicionales de esa manera también.

Puede utilizar StringBuilder 's deleteCharAt(int index) con índice es length() - 1

Answer 2

Si convierte a un bucle índice clásico, sí.

O simplemente podría eliminar la última coma después de que haya terminado. De este modo:

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); 

Me

, sólo tiene que utilizar StringUtils.join() de commons-lang.

Answer 3

que sea sencillo y utilizar un estándar para el bucle:

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

o simplemente usan Apache commons-lang StringUtils.join() bucles

Answer 4

explícitos siempre funcionan mejor que las implícitas.

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

Usted debe envolver todo el asunto en una sentencia if en caso de que está tratando con una matriz de longitud cero.

Answer 5

Otra alternativa es agregar la coma antes de agregar i, pero no en la primera iteración de . (Por favor, no use "" + i, por cierto - que realmente no quiere concatenación aquí, y StringBuilder tiene un append en perfecto estado() Sobrecarga int.)

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

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

Lo bueno de esto es que lo hará trabaje con cualquier Iterable - no siempre puede indexar cosas. (La frase "agregue la coma y luego elimínela al final" es una buena sugerencia cuando realmente usa StringBuilder, pero no funciona para cosas como escribir en streams. Sin embargo, posiblemente sea el mejor enfoque para este problema exacto.)

Answer 6

Otro enfoque es tener la longitud de la matriz (si está disponible) almacenada en una variable separada (más eficiente que volver a verificar la longitud cada vez). Luego puede comparar su índice con esa longitud para determinar si agrega o no la coma final.

EDITAR: Otra consideración es ponderar el costo de rendimiento de la eliminación de un carácter final (que puede causar una copia de cadena) en contra de tener un condicional se compruebe en cada iteración.

Answer 7

Aquí es una solución:

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; 
} 

Busca la primera iteración :)  

Answer 8

dos caminos alternativos aquí:

1: Apache Commons String Utils

2: Mantener un booleano llamado first, establecido en verdadero. En cada iteración, si first es falso, agregue su coma; después de eso, configure first en falso.

Answer 9

Quizás esté utilizando la herramienta incorrecta para el trabajo.

Esto es más manual que lo que está haciendo pero está en una forma más elegante si no un poco "vieja escuela"

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

Answer 10

Otra solución (tal vez el más eficiente)

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]); 
     } 
    } 

Answer 11

otra manera de hacer esto:

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

actualización: para Java 8, que ahora tienen Collectors

Answer 12

Si solo está convirtiendo una matriz en una matriz delimitada por comas, muchos idiomas tienen una función de combinación para exactamente eso. Convierte una matriz en una cadena con un delimitador entre cada elemento.

Answer 13

Necesita Class Separator.

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

La implementación de la clase Separator es sencilla. Envuelve una cadena que se devuelve en cada llamada de toString(), excepto la primera llamada, que devuelve una cadena vacía.

Answer 14

Basado en java.util.AbstractCollection.toString(), sale antes para evitar el delimitador.

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

Es eficiente y elegante, pero no tan evidente como algunas de las otras respuestas.

Answer 15

Otra opción más.

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

Answer 16

Esto es casi una repetición de this StackOverflow question. Lo que quiere es StringUtils, y para llamar al join método.

StringUtils.join(strArr, ','); 

Answer 17

Muchas de las soluciones que aquí se describen son un poco exageradas, en mi humilde opinión, especialmente aquellas que dependen de bibliotecas externas. Hay una agradable y clara expresión idiomática para lograr una lista separada por comas que siempre he usado. Se basa en el operador condicional (?):

Editar: Solución original correcta, pero no óptima según los comentarios.Tratando una segunda vez:

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

Hay que ir, en 4 líneas de código, incluyendo la declaración de la matriz y el StringBuilder.

Answer 18

En este caso, realmente no hay necesidad de saber si es la última repetición. Hay muchas maneras en que podemos resolver esto. Una forma sería:

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

Answer 19

Aquí es un punto de referencia SSCCE Corrí (relacionado con lo que tenía que poner en práctica) con estos resultados:

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

En mi ejemplo, al menos, saltarse la comprobación en cada iteración no es notablemente más rápido, especialmente para los volúmenes de datos, pero es más rápido.

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)"); 
    } 
} 

Answer 20

Como se ha mencionado conjunto de herramientas, en Java 8 ahora tenemos Collectors. Esto es lo que el código se vería así:

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

Creo que hace exactamente lo que está buscando, y es un patrón que se podría utilizar para muchas otras cosas.

Answer 21

Dado que es una matriz fija, sería más fácil simplemente evitar la mejorada para ... Si el Objeto es una colección, un iterador sería más fácil.

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(","); 
    } 
}