การทำซ้ำครั้งล่าสุดของ Enhanced for loop ใน java

มีวิธีการตรวจสอบว่าการวนซ้ำเป็นครั้งสุดท้ายหรือไม่ รหัสของฉันมีลักษณะดังนี้:

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

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

ตอนนี้สิ่งที่ฉันไม่ต้องการที่จะผนวกเครื่องหมายจุลภาคในการทำซ้ำครั้งล่าสุด ขณะนี้มีวิธีการตรวจสอบว่าเป็นซ้ำล่าสุดหรือฉันติดกับห่วงหรือใช้ตัวนับภายนอกเพื่อติดตาม

อีกทางเลือกหนึ่งคือการใส่เครื่องหมายจุลภาคต่อท้ายก่อนที่คุณจะต่อท้าย i ไม่ใช่แค่การวนซ้ำครั้งแรก (โปรดอย่าใช้"" + iโดยวิธี - คุณไม่ต้องการการต่อข้อมูลที่นี่จริงๆและ StringBuilder มีการผนวกที่ดีอย่างสมบูรณ์ (int) เกินพิกัด)

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

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

สิ่งที่ดีเกี่ยวกับเรื่องนี้คือมันจะทำงานกับสิ่งใดก็ได้Iterable- คุณไม่สามารถจัดทำดัชนีสิ่งต่าง ๆ ได้เสมอไป ("เพิ่มเครื่องหมายจุลภาคแล้วลบออกในตอนท้าย" เป็นคำแนะนำที่ดีเมื่อคุณใช้ StringBuilder - แต่มันไม่ได้ผลกับสิ่งต่าง ๆ เช่นการเขียนไปยังกระแสข้อมูลมันอาจเป็นวิธีที่ดีที่สุดสำหรับปัญหานี้ )

อีกวิธีในการทำสิ่งนี้:

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

อัปเดต: สำหรับ Java 8 ตอนนี้คุณมีนักสะสม

มันอาจจะง่ายกว่าที่จะต่อท้าย จากนั้นเมื่อคุณทำลูปเสร็จแล้วให้ลบตัวละครตัวสุดท้ายออก มีเงื่อนไขน้อยลงเช่นกัน

คุณสามารถใช้StringBuilderเป็นdeleteCharAt(int index)ดัชนีความเป็นอยู่length() - 1

บางทีคุณกำลังใช้เครื่องมือที่ผิดสำหรับงาน

นี่เป็นคู่มือมากกว่าสิ่งที่คุณกำลังทำอยู่ แต่มันก็ดูดีกว่าถ้าไม่ใช่ "โรงเรียนเก่า"

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

นี่เป็นคำถามที่เกิดขึ้นซ้ำ ๆกับ StackOverflow สิ่งที่คุณต้องการคือStringUtilsและเรียกวิธีการเข้าร่วม

StringUtils.join(strArr, ',');

วิธีแก้ไขปัญหาอื่น (อาจมีประสิทธิภาพมากที่สุด)

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

ทำให้มันง่ายและใช้มาตรฐานสำหรับการวนซ้ำ:

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

หรือเพียงแค่ใช้ apache commons-lang StringUtils.join ()

การวนซ้ำอย่างชัดเจนมักจะทำงานได้ดีกว่าการวนซ้ำ

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

คุณควรห่อสิ่งทั้งหมดไว้ในคำสั่ง if ในกรณีที่คุณจัดการกับอาเรย์ที่มีความยาวเป็นศูนย์

หากคุณแปลงเป็นลูปดัชนีแบบคลาสสิคใช่

หรือคุณสามารถลบเครื่องหมายจุลภาคสุดท้ายหลังจากเสร็จสิ้น ชอบมาก

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

ฉันฉันใช้StringUtils.join()จากคอมมอนส์ - langเท่านั้น

คุณจำเป็นต้องมีชั้นแยก

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

การใช้งานของชั้นเรียนSeparatorเป็นไปข้างหน้า มันล้อมรอบสตริงที่ส่งคืนในทุกการโทรtoString()ยกเว้นการโทรครั้งแรกซึ่งส่งกลับสตริงว่าง

ขึ้นอยู่กับ java.util.AbstractCollection.toString () มันออกก่อนเวลาเพื่อหลีกเลี่ยงตัวคั่น

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

มันมีประสิทธิภาพและสวยงาม แต่ไม่ชัดเจนเท่าคำตอบอื่น ๆ

นี่คือทางออก:

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

มองหาการทำซ้ำครั้งแรก :)  

ในฐานะที่เป็นเครื่องมือที่กล่าวถึงในชวา 8 ตอนนี้เรามีนักสะสม นี่คือสิ่งที่รหัสจะมีลักษณะ:

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

ฉันคิดว่านั่นเป็นสิ่งที่คุณกำลังมองหาและเป็นรูปแบบที่คุณสามารถใช้กับสิ่งอื่น ๆ อีกมากมาย

อีกตัวเลือกหนึ่ง

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

วิธีแก้ปัญหาหลายอย่างที่อธิบายไว้ที่นี่เป็นเพียงเล็กน้อยเหนือ IMHO โดยเฉพาะอย่างยิ่งที่พึ่งพาไลบรารีภายนอก มีสำนวนที่ดีสะอาดและชัดเจนสำหรับการบรรลุรายการที่คั่นด้วยเครื่องหมายจุลภาคที่ฉันใช้อยู่เสมอ มันขึ้นอยู่กับผู้ประกอบการเงื่อนไข (?):

แก้ไข : โซลูชันดั้งเดิมถูกต้อง แต่ไม่เหมาะสมตามความคิดเห็น ลองครั้งที่สอง:

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

ที่นั่นคุณไปในรหัส 4 บรรทัดรวมถึงการประกาศของอาร์เรย์และ StringBuilder

นี่เป็นเกณฑ์มาตรฐาน SSCCE ที่ฉันใช้ (เกี่ยวข้องกับสิ่งที่ฉันต้องนำไปใช้) กับผลลัพธ์เหล่านี้:

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

ตัวอย่างของฉันอย่างน้อยกระโดดข้ามการตรวจสอบในทุกย้ำไม่เห็นได้ชัดเร็วขึ้นโดยเฉพาะอย่างยิ่งสำหรับปริมาณของข้อมูลที่มีเหตุผล แต่ก็เป็นได้เร็วขึ้น

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

อีกวิธีคือการจัดเก็บความยาวของอาร์เรย์ (ถ้ามี) เก็บไว้ในตัวแปรแยกต่างหาก (มีประสิทธิภาพมากกว่าการตรวจสอบความยาวอีกครั้งในแต่ละครั้ง) จากนั้นคุณสามารถเปรียบเทียบดัชนีของคุณกับความยาวนั้นเพื่อกำหนดว่าจะเพิ่มจุลภาคสุดท้ายหรือไม่

แก้ไข: สิ่งที่ต้องพิจารณาอีกประการหนึ่งคือการชั่งน้ำหนักค่าใช้จ่ายด้านประสิทธิภาพของการลบอักขระตัวสุดท้าย (ซึ่งอาจทำให้สำเนาสตริง) ไม่ให้ทำการตรวจสอบเงื่อนไขในการทำซ้ำแต่ละครั้ง

หากคุณเพียงแค่เปลี่ยนอาเรย์เป็นอาเรย์ที่คั่นด้วยจุลภาคหลายภาษามีฟังก์ชั่นการเข้าร่วมสำหรับสิ่งนี้ มันเปลี่ยนอาร์เรย์เป็นสตริงที่มีตัวคั่นระหว่างแต่ละองค์ประกอบ

ในกรณีนี้คุณไม่จำเป็นต้องรู้เลยว่านี่เป็นการทำซ้ำครั้งสุดท้ายหรือไม่ มีหลายวิธีที่เราสามารถแก้ปัญหานี้ได้ วิธีหนึ่งจะเป็น:

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

สองเส้นทางสำรองที่นี่:

1: Apache Utons String Utils

2: เก็บบูลีนที่เรียกว่าfirstตั้งค่าเป็นจริง ในแต่ละการวนซ้ำหากfirstเป็นเท็จให้เพิ่มจุลภาคของคุณ หลังจากนั้นตั้งค่าfirstเป็นเท็จ

เนื่องจากเป็นอาเรย์คงที่มันจะง่ายกว่าที่จะหลีกเลี่ยงการปรับปรุงสำหรับ ... ถ้า Object เป็นคอลเลกชันตัววนซ้ำจะง่ายขึ้น

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