Escenario de entrevista "Últimos 100 bytes"


78

Recibí esta pregunta en una entrevista el otro día y me gustaría saber algunas de las mejores respuestas posibles (no respondí muy bien jaja):

Escenario: hay una página web que supervisa los bytes enviados a través de una red. Cada vez que se envía un byte, se llama a la función recordByte () pasando ese byte, esto podría suceder cientos de miles de veces al día. Hay un botón en esta página que, cuando se presiona, muestra los últimos 100 bytes pasados ​​a recordByte () en la pantalla (lo hace llamando al método de impresión a continuación).

El siguiente código es lo que me dieron y me pidieron que completara:

public class networkTraffic {
    public void recordByte(Byte b){
    }
    public String print() {
    }
}

¿Cuál es la mejor forma de almacenar los 100 bytes? ¿Una lista? Curioso cuál es la mejor manera de hacer esto.


70
El búfer circular usando una matriz es una forma. Inicialícelo con ceros, luego realice un seguimiento de la cabeza y la longitud. Luego, puede usar la cabeza y la longitud para recorrer el búfer para imprimirlo. El uso eficiente de la memoria y la CPU, además de adaptarse a las necesidades históricas.
Tim Lloyd


2
¿Es necesario mantener todos los bytes o solo los últimos 100?
jpredham

2
Usaría una pila, solo presione los bytes que se envían y luego muestre los últimos 100 resultados.
Alvin Baena

5
@alvinbaena ¿qué sucede cuando pasan días o semanas de recordByte () sin que nadie llame a print ()?
Russell Borogove

Respuestas:


150

Algo como esto ( búfer circular ):

byte[] buffer = new byte[100];
int index = 0;

public void recordByte(Byte b) {
   index = (index + 1) % 100;
   buffer[index] = b; 
}

public void print() {
   for(int i = index; i < index + 100; i++) {
       System.out.print(buffer[i % 100]);
   }
}

Los beneficios de usar un búfer circular:

  1. Puedes reservar el espacio estáticamente. En una aplicación de red en tiempo real (VoIP, streaming, ..) esto se hace a menudo porque no es necesario almacenar todos los datos de una transmisión, sino solo una ventana que contiene los nuevos bytes que se van a procesar.
  2. Es rápido: se puede implementar con una matriz con un costo de lectura y escritura de O (1).

6
Te encontrarás con un comportamiento inesperado cuando alcances el valor máximo de int, pero por lo demás parece correcto.
CassOnMars

2
cerca pero sujeto a errores cuando el índice crece demasiado. En su lugar, use esto en recordByte: index = (index + 1)% 100; matriz [índice] = b;
DwB

3
Es mejor usar una condición al final para envolver en lugar de un módulo, probablemente sea un poco más claro y un poco más rápido (y si está haciendo esto para cada byte enviado, el rendimiento será importante).
jmoreno

4
Creo que debe realizar un seguimiento de la longitud, de lo contrario, si se llama a print antes de que se registren 100 bytes, entonces imprime 100 bytes de todos modos, algunos de los cuales son valores de matriz unitarios (ceros)
Mike Q

3
Puede utilizar 128 en lugar de 100 bytes. Eso desperdiciará 28 bytes, pero la operación de módulo será más rápida. Para una función tan corta, la mejora de la velocidad será significativa.
Mackie Messer

34

No sé Java, pero debe haber un concepto de cola mediante el cual pondría en cola bytes hasta que el número de elementos en la cola llegara a 100, momento en el que sacaría de la cola un byte y luego pondría en cola otro.

public void recordByte(Byte b)
{ 
  if (queue.ItemCount >= 100)
  {
    queue.dequeue();    
  }
  queue.enqueue(b);
}

Puede imprimir mirando los elementos:

public String print() 
{ 
  foreach (Byte b in queue)
  {
    print("X", b);  // some hexadecimal print function
  }
}  

3
+1 para una cola. LinkedList implementa la interfaz Queue y debe permitir que las operaciones add () (poner en cola) y eliminar () (sacar de cola) se ejecuten en el tiempo O (1).
sceaj

Stack fue mi primer pensamiento, pero la pregunta no dice cómo quiere que se presenten los datos (en orden de aparición frente al último byte primero). .. después de todos los últimos 100 bytes de cualquier cosa no va a ser exactamente útil más que para métricas / informes.
Matthew Cox

@MatthewCox Sí, me refiero a que, como pregunta de la entrevista, no fue realmente útil excepto como una prueba de resolución de problemas, pero tenía curiosidad sobre la mejor manera de hacerlo.
Yottagray

@MatthewCox Técnicamente, una pila no permite el acceso a los datos más antiguos (las primeras entradas), solo LA última entrada, de ahí la cola.
Sceaj

@sceaj Yo diría que ese punto. Todavía tiene iteradores ... no está limitado solo al primer elemento de la pila. Con su punto, lo mismo se aplica a la inversa. Solo tendría acceso a los bytes más antiguos y no a los más nuevos, ya que una cola solo se elimina del frente.
Matthew Cox

26

Búfer circular usando matriz:

  1. Matriz de 100 bytes
  2. Mantenga un registro de dónde está el índice de cabeza i
  3. Para recordByte()poner el byte actual en A [i] e i = i + 1% 100;
  4. Para print(), return subarreglo (i + 1, 100) concatenar con subarreglo (0, i)

Cola usando la lista vinculada (o la cola de Java):

  1. Para recordByte()agregar un nuevo byte al final
  2. Si la nueva longitud es superior a 100, elimine el primer elemento
  3. Para print()simplemente imprimir la lista

9

Aquí está mi código. Puede parecer un poco oscuro, pero estoy bastante seguro de que esta es la forma más rápida de hacerlo (al menos estaría en C ++, no estoy tan seguro de Java):

public class networkTraffic {
    public networkTraffic() {
      _ary = new byte[100];
      _idx = _ary.length;
    }

    public void recordByte(Byte b){
      _ary[--_idx] = b;
      if (_idx == 0) {
        _idx = _ary.length;
      }   
    }

    private int _idx;
    private byte[] _ary;
}

Algunos puntos a tener en cuenta:

  • No se asignan / desasignan datos al llamar a recordByte ().
  • No usé%, porque es más lento que una comparación directa y usando el if (la predicción de rama también podría ayudar aquí)
  • --_idxes más rápido que _idx--porque no interviene ninguna variable temporal.
  • Cuento hacia atrás hasta 0, porque entonces no tengo que entrar _ary.lengthcada vez en la llamada, sino solo cada 100 veces cuando se alcanza la primera entrada. Quizás esto no sea necesario, el compilador podría encargarse de ello.
  • si hubo menos de 100 llamadas a recordByte (), el resto son ceros.

1
Voto a favor porque tienes razón, pero en Java estaría menos preocupado por la variable temporal y por evitar verificar la longitud. Ambas son cosas que esperaría que cualquier JIT decente optimizara.
Daniel Pryden

Yo votaría a favor si agrega el print()método requerido también.
icza

4

Lo más fácil es meterlo en una matriz. El tamaño máximo que puede acomodar la matriz es de 100 bytes. Siga agregando bytes a medida que se transmiten desde la web. Después de que los primeros 100 bytes estén en la matriz, cuando llegue el byte 101, elimine el byte de la cabecera (es decir, el 0). Sigue haciendo esto. Esto es básicamente una cola. Concepto FIFO. Una vez realizada la descarga, le quedan los últimos 100 bytes.

No solo después de la descarga, sino en cualquier momento dado, esta matriz tendrá los últimos 100 bytes.

@Yottagray ¿No llegas a donde está el problema? Parece haber una serie de enfoques genéricos (matriz, matriz circular, etc.) y una serie de enfoques específicos del lenguaje (byteArray, etc.). ¿Me estoy perdiendo de algo?


¿Qué sucede si se llama a print () después de que se hayan registrado más de 100 bytes?
jpredham

no registra más de 100 bytes. detente cuando <= 100.
Srikar Appalaraju

1
Eso obtendría los primeros 100 bytes, no los últimos 100.
interjay

Supongo que realmente no entiendo cómo manejar el mantenimiento de solo los últimos 100 bytes y hacerlo de una manera realmente eficiente. Su respuesta no tiene ningún sentido, ¿qué pasa si su matriz llega a 100 y se llama de nuevo a recordByte ()? Su solución solo contiene los primeros 100 bytes
Yottagray

1

Solución multiproceso con E / S sin bloqueo:

private static final int N = 100;
private volatile byte[] buffer1 = new byte[N];
private volatile byte[] buffer2 = new byte[N];
private volatile int index = -1;
private volatile int tag;

synchronized public void recordByte(byte b) {
  index++;
  if (index == N * 2) {
    //both buffers are full
    buffer1 = buffer2;
    buffer2 = new byte[N];
    index = N;
  }
  if (index < N) {
    buffer1[index] = b;
  } else { 
    buffer2[index - N] = b;
  }
}

public void print() {
  byte[] localBuffer1, localBuffer2;
  int localIndex, localTag;
  synchronized (this) {
   localBuffer1 = buffer1;
   localBuffer2 = buffer2;
   localIndex = index;
   localTag = tag++;
  }
  int buffer1Start = localIndex - N >= 0 ? localIndex - N + 1 : 0;
  int buffer1End = localIndex < N ? localIndex : N - 1;      
  printSlice(localBuffer1, buffer1Start, buffer1End, localTag);
  if (localIndex >= N) {
    printSlice(localBuffer2, 0, localIndex - N, localTag);
  }
}

private void printSlice(byte[] buffer, int start, int end, int tag) {
  for(int i = start; i <= end; i++) {
    System.out.println(tag + ": "+ buffer[i]);
  }
}

0

Sólo por el gusto de hacerlo. ¿Qué tal usar un ArrayList<Byte>? Di por qué no

public class networkTraffic {
    static ArrayList<Byte> networkMonitor;          // ArrayList<Byte> reference
    static { networkMonitor = new ArrayList<Byte>(100); }   // Static Initialization Block
    public void recordByte(Byte b){
        networkMonitor.add(b);
        while(networkMonitor.size() > 100){
            networkMonitor.remove(0);
        }
    }
    public void print() {
        for (int i = 0; i < networkMonitor.size(); i++) {
            System.out.println(networkMonitor.get(i));
        }
        // if(networkMonitor.size() < 100){
        //  for(int i = networkMonitor.size(); i < 100; i++){
        //      System.out.println("Emtpy byte");
        //  }
        // }
    }
}
Al usar nuestro sitio, usted reconoce que ha leído y comprende nuestra Política de Cookies y Política de Privacidad.
Licensed under cc by-sa 3.0 with attribution required.