Feda /dev/ - No Javascript allowed

Entrevista para junior, dime todo lo que esta mal en este pseudo-codigo:

def update_items(request, new_value):
  let old_items = find_items(request)
  for old_item in old_items:
    if old_item == request.condition:
      update_item(old_item, new_value)
  return 200

Solucion: todo, el peor codigo que puede escribir un programador.

La verdad es que ese código se entiende bastante bien.
Mejor hacerlo The Pythonic Way™

from functools import partial

def update_items(request, new_value):
  items = [item for item in find_items(request) if item == request.condition]
  _update_item = partial(update_item, new_value=new_value)
  map(_update_item, items)
  return 200

#60405 igual de mal

el if dentro del for jode el branch prediction
en lugar de hacerlo en el servicio, seguramente podrias filtrar en DB
luego el update es de item por item, en lugar de batching, las operaciones deben ser siempre batching por defecto

por ejemplo:

def update_items(request, new_value):
  let old_items = find_items_with_condition(request, request.condition)
  let updated_items = new Items[old_items.size]
  for i in old_items.size:
      update_item[i] = update_item()
  let updated_items_call = update_items(updated_items)
  return updated_items_call == success

• tienes solo 2 calls a DB, find_items y update_items vs 2items + 2items
• no tienes logica rara y todo es lineal en tu servicio asique va rapido brrr
• podrias hacer streaming, paralelizar los iteradores
• el uso de network y bandwith es mejor y mas eficiente
• si tienes una GC es mas eficiente

si este metodo te tardaba 100ms, ahora te tardara 20ms, si lo optimizas te tardara <10ms.

es un ejemplo.

def process_item(item: Item)

vs

def process_items(items: []Item)

nse, me parecen preguntas para junior, xq ninguno del hilo las pasa. he puesto este ejemplo en el hilo ya 4 o 5 veces jaja

#60408 yo creo que ha mezclao 3 lenguajes

#60408 #60409 he puesto arriba pseudo-codigo.

#60399desu:

dime todo lo que esta mal en este pseudo-codigo

let asignar una variable a una expresión, let name = expression.

es el naming habitual.

#60410 bueno en eso consiste la entrevista, en hacer asumciones y modificar las cosas para que sean lo mejor posible dentro de las constraints.

#60411

• Tengo que asumir que la condición se puede filtrar en BD?
• Tengo que asumir que tan siquiera hay una BD?
• Tengo que asumir que update_item no tiene ningún sistema para ir almacenando las actualizaciones y hacerlas al final?
• Tengo que asumir que solo debo devolver un 200 y no los objectos ya guardados?
• En tu solución tengo que asumir que update_item solo cambia los valores pero no los guarda, y update_items, no cambia valores pero si guarda

Porque puestos a asumir lo primero que pregunto es porque tengo la request entera dentro de un servicio

#60415 Por asumir puedes hasta asumir que el PC no esté enchufao a la corriente

#60418 arriba te he respondido, resuelvemelo. o tienes mas dudas? preguntame y te las respondo todas, quiero ver tu respuesta.