Doy por interiorizado el gitignore

Trabajo con ramas: veo un comportamiento anómalo

| * | | 4b01555 - Merge pull request #1 from vctorChamizo/feature/iteration1 (3 days ago) <Víctor Chamizo>
| |\ \ \
| | | | |
| | * | | bc7bed3 - Iteration 1 done (3 days ago)
| |/ / /
| | | |
| | * | 623ee04 - Iteration 2 done (3 days ago)
| |/ /
| | |
| | * b15fb1f - Iteration 3 done (3 days ago)
| |/
| |
| * 609b5e2 - Added gitignore file (3 days ago)
|/
|

• e2f1cfe - Update README.md (11 days ago)

Es como si integrases primero la iteración 3 antes que la 2 y esta antes que la 1... En cualquier caso, mucho mejor trabajo con las branches, lo vas pillando muy bien tio, enhorabuena.

Paso a revisar el código

Línea 31: sacada de rabo, enhorabuena, pero recuerda... KISS, que es más o menos lo que haces en la 41

Linea 95: fenomenal el trabajo con funciones de responsabilidad única! Ese es el buen camino. Sólo una cosa.

Linea 108 countSubstring(lorem_chain, "et" -> Te das cuenta de que no estás teniendo en cuenta "Et"? quizás sería mejor así? countSubstring(lorem_chain, "et" || "Et") No se si compilaría, prueba y me cuentas

Línea 116, el segundo parámetro "" te sobra, pero demuestra que te has leido la docu!

Good job!

• Miraré lo de las ramas, porque no me había dado cuenta, per tienes razón, aparantemente el gráfico muestra que no se han mezclado como deberían.

• La linea 108 que me comentas, lo he probado; pero tiene un comportamiento curiso ya que poniendo el código que me sugieres, unicamente me detecta el primero de los "parametros" (et), sin embargo, al suitituir el || lógico por un && lógico, tiene el resultado opuesto, unicamente detecta el "Et". Investigaré el por qué.

Gracias.