Aykut Yaman aykutyaman

inline strings used for app states instead of reusable constants/enums (”Idle”, “Loading”, "Act::Fetch" , etc).

I will discuss it with three examples what I mean.

We can't have silly typos and autocomplete of IDEs will predict them correctly too. So for example if we type "Load" the IDE will autocomplete as "Loading". And when we type something that is not part of the union the compiler will alert us in that spot.
If a naming change occurs we are also forced to handle the change. So, it means we don't have the possibility to skip renaming where they are used.

	var trampoline = fn => (n) => {
	// make sure we call trampoline function only once

	var result = fn(n);

	while (typeof result === 'function') {
	result = result();
	}

	return result;

	const compose = (...args) => n => args.reverse().reduce((acc, f) => f(acc), n);

	var mapReducer = fn => combineFn => (acc, v) => combineFn(acc, fn(v));
	var filterReducer = fn => combineFn => (acc, v) => fn(v) ? combineFn(acc, v) : acc;

	var add1 = n => n + 1;
	var isOdd = n => n % 2 === 1;
	var sum = (acc, n) => Number(acc) + n;

	var list = [1, 3, 4, 6, 9, 12, 13, 16, 21];