Linked Link経由でループする

Question

私はバッファクラス（BufferCharacter）からLinkedListを取得しようとしていて、ReaderクラスのLinkedListのすべての要素をループしています。しかし、私がをBufferCharacterクラスのget()メソッドで返すようにしようとすると、それをReaderクラスでループすることはできません。私はBufferクラスのリストをループして、そこから各要素を返そうとしましたが、どちらも機能しません。

ご協力いただきありがとうございます。

public class CharacterBuffer { 

    private char ch; 
    private LinkedList buffer = new LinkedList(); 
    private boolean filled; 

    public void put(char ch) { 
     buffer.addLast(ch); 
    } 

    public void filled() { 
     filled = true; 
    } 

    public Object get() throws InterruptedException { 
     while (buffer.isEmpty()) { 
      // wait(); 
      return "Waiting"; 
     } 
     return buffer; 
    } 

    public synchronized void putSync(char ch) { 
     buffer.addLast(ch); 
    } 

    public synchronized Object getSync() throws InterruptedException { 
     while (buffer.isEmpty()) { 
      // wait(); 
      return "---------"; 
     } 

     for(int i = 0; i<buffer.size(); i++){ 
      System.out.println(buffer.get(i)); 
     } 

     return buffer; 

    } 

    public int size(){ 
     return buffer.size(); 
    } 

} 

public class Reader extends Thread { 

    private GUIMutex gui; 
    private CharacterBuffer buffer; 
    private boolean isSynced; 

    public Reader(GUIMutex gui, CharacterBuffer buffer, boolean isSynced) { 
     this.gui = gui; 
     this.buffer = buffer; 
     this.isSynced = isSynced; 
    } 

    public void run() { 
     String data = "test"; 
     while (true) { 

      try { 
       // data = buffer.get(); 
       if (isSynced) { 
        gui.setReaderText(buffer.getSync() + "\n"); 
       } else { 
        for(int i = 0; i<buffer.get().size(); i++){ 
         gui.setReaderText(i); 
        } 
        gui.setReaderText(buffer.get() + "\n"); 
       } 
       Thread.sleep(700); 
      } catch (InterruptedException e) { 
      } 

     } 
    } 
} 

Answer 1

あなたはあなたが何を話しているのか理解していないと思います。そこでここでいくつかの光を当てようとします。

最後に、あなたは複数の要素を含む何らかの "コレクション"クラスについて話しています。あなたのケースではLinkedList。そのようなクラスを使用するには、提供するAPIを明確に理解する必要があります。

バッファを使用して、個別のchar値を格納することを考えました。これは、putSync()を使用して追加します。

しかし···正確にはgetSync()さんとなっていますか？

あなたの場合、単にバッファを返すだけです。おそらくが間違っているです。理解する

synchronized boolean hasNext() 

と

synchronized char getNext() 

最初のメソッドを呼び出すことができ、あなたのクラスのユーザー：

は代わりに、次のようなメソッドを持ってしたい他の文字があります。もしそうであれば、2番目のメソッドはそれらの値を返します。

これは、コードを改善するための最初の簡単な方法です。より妥当な方法は、getIterator()メソッドを実装して、Iteratorインターフェイスを実装するオブジェクトを返すことです。

その他の注意事項：「内蔵」の場合LinkedList;このクラスがジェネリックをサポートしていることをご理解ください！

したがって、あなたは次のようにそれを使用する必要があります。厳密に型指定されたコレクションを使用してからのすべての利点を得るために

private final List<Character> buffer = new LinkedList<>(); 

！

編集：お客様のご意見に、私はを使用していると思います。リンクされたリストはここでは間違ったアプローチです。

代わりにリストを使用しての、あなたがQueueを使用したい、のような：クラスはあなたの一方の当事者がキューテールで要素を追加することができる機能を提供します

private final Queue<Character> buffer = new ConcurrentLinkedQueue<>(); 

。もう1つのパーティーは、キューヘッドから要素を削除します。

追加ボーナス：そのクラスは既にあなたのために同期作業を行っているので、気にする必要はありません！

Answer 2

使用StringBuilderの代わりに

StringBuilder sb = new StringBuilder(128); 
    // add chars using sb.append(char) 
    for (int i = 0, n = sb.length(); i < n; i++) 
    { 
    char c = sb.charAt(i); 
    } 

    or 

    String s = sb.toString();