Ayuda para dejar mas simple este codigo

[aguml]

Lo que me falta por modificar creo que tiene que quedar algo tal que asi:

Código PHP:

do {
    valor1 = cadena[contador];
    valor2 = valor2 ^ valor1 + contador;
    ++contador;
    valor3 = (cadena[(valor2 % valor4) + 1] + valor2 + (valor1 + valor2) * 2 + retval) * valor2;
    retval = valor3;
} while (contador < sizefilelic); 


Pero creo que hago algo mal.