Optimización de algoritmo de fuerza bruta

Iniciado por kutcher, 2 Septiembre 2014, 22:30 PM

0 Miembros y 3 Visitantes están viendo este tema.

kutcher

Buenas, tengo el siguiente algoritmo:

Código (cpp) [Seleccionar]
void bruteforce(int inc, int fin, char *rotation)
{
    int i = 0, j, len = strlen(rotation);
    long int r = 0, end, c;
    char test[200];
    for(j = inc; j < fin+1; j++)
    {
        r = 0;
        end = (long int)pow(len, j);
        while(r != end)
        {
            c = r;
            while(i < j)
            {
                test[i] = rotation[c % len];
                c = (long int)c / len;
                i++;
            }
            test[i] = '\0';
            puts(test);
            i = 0;
            r++;
        }
    }
}


Mi intención es evitar el uso de pow porque esta consume demasiado recursos, lo que busco es la forma de representarlo con solo operaciones aritméticas .... alguna idea

Saludos kutcher   

MCKSys Argentina

MCKSys Argentina

"Si piensas que algo está bien sólo porque todo el mundo lo cree, no estás pensando."


kutcher

Cita de: MCKSys Argentina en  2 Septiembre 2014, 22:41 PM
Revisa este post.

La verdad no entendí mucho, por otro lado hice una función pow que solo trabaja con enteros pensé que de esta manera podría ser mas rápida pero no:

Código (cpp) [Seleccionar]
int pow2(int x, int n)
{
    int p;
    for ( p = 1 ; n > 0 ; --n )
        p *= x;
    return p;
}


Saludos kutcher

eferion

Cita de: kutcher en  3 Septiembre 2014, 02:06 AM
La verdad no entendí mucho, por otro lado hice una función pow que solo trabaja con enteros pensé que de esta manera podría ser mas rápida pero no:

Código (cpp) [Seleccionar]
int pow2(int x, int n)
{
    int p;
    for ( p = 1 ; n > 0 ; --n )
        p *= x;
    return p;
}


Saludos kutcher

La librería estándar ha sido programada y optimizada por gente bastante ducha tanto en matemáticas como en programación... es complicado sacar una versión que sea más eficiente (no es imposible pero...)

En cualquier caso yo creo que puedes optimizar la función "arrastrando" el valor de "end", de esta forma sólo llamas a pow una vez durante la ejecución de la función.


void bruteforce(int inc, int fin, char *rotation)
{
    int i = 0, j, len = strlen(rotation);
    long int r = 0, c;
    char test[200];
    long int end = (long int)pow( len, inc );
    for(j = inc; j < fin+1; j++)
    {
        r = 0;
        end *= len;
        while(r != end)
        {
            c = r;
            while(i < j)
            {
                test[i] = rotation[c % len];
                c = (long int)c / len;
                i++;
            }
            test[i] = '\0';
            puts(test);
            i = 0;
            r++;
        }
    }
}


kutcher

Cita de: eferion en  3 Septiembre 2014, 08:16 AM
En cualquier caso yo creo que puedes optimizar la función "arrastrando" el valor de "end", de esta forma sólo llamas a pow una vez durante la ejecución de la función.

Hola eferion probé lo que indicaste pero no funciona y ademas provoca que entre en un ciclo infinito .. ahora tengo otra pow pero mas optimizada que creo es mas rápida según noto es este:

Código (cpp) [Seleccionar]
int ipow(int base, int exp)
{
    int result = 1;
    while(exp)
    {
        if(exp & 1)
           result *= base;
        exp >>= 1;
        base *= base;
    }
    return result;
}


Saludos kutcher

rir3760

Primero las recomendaciones:

1) Las conversiones explicitas al tipo long int no son necesarias ya que se realizan de forma automática, por ello hay que eliminarlas.

2) Los dos bucles while se pueden sustituir por bucles for para compactar el código fuente.

3) Puedes utilizar una sola vez la función pow justo antes del bucle externo, la actualización del valor de la variable "end" se debe realizar justo al final del mentado bucle.

Con los cambios:
void bruteforce(int inc, int fin, char *rotation)
{
   int i, j, len = strlen(rotation);
   long int r, end, c;
   char test[200];
   
   end = pow(len, inc);
   for(j = inc; j < fin + 1; j++){
      for (r = 0; r != end; r++){
         c = r;
         for (i = 0; i < j; i++){
            test[i] = rotation[c % len];
            c = c / len;
         }
         printf("%.*s\n", i, test);
      }
     
      end *= len;
   }
}

Como no publicas el programa completo te toca a ti verificar que funcione correctamente (ya nos avisas si hubo algún resbalón).

----

Un ultimo comentario: si se trata de una solución de fuerza bruta la prioridad debe ser la claridad del código fuente, si esta fuera la eficiencia lo mejor es simplemente ... cambiar de algoritmo.

Un saludo
C retains the basic philosophy that programmers know what they are doing; it only requires that they state their intentions explicitly.
--
Kernighan & Ritchie, The C programming language

kutcher

Cita de: rir3760 en  3 Septiembre 2014, 19:29 PM
Como no publicas el programa completo te toca a ti verificar que funcione correctamente (ya nos avisas si hubo algún resbalón).

Hola rir3760 ahora si va bien en cuanto al código completo es el siguiente:

Código (cpp) [Seleccionar]
#include<stdio.h>
#include<string.h>
#include<math.h>
#include<stdlib.h>
#include<time.h>

void bruteforce(int inc,int fin,char *rotation);

int main(void)
{
    int inc, fin;
    unsigned long t_start = 0, t_end = 0;
    char rotation[200] = "ABCDEFGHIJKLMNOPQRSTUVWXYZ"
                         "abcdefghijklmnopqrstuvwxyz"
                         "1234567890<>,?;.:/!з*╡∙%$г"
                         "ди+=})]р@ч^\\_`ш|-[({'#\"щ~&\0";

    printf("\n Comenzar de: ");
    scanf("%d", &inc);
    printf("\n Terminar en: ");
    scanf("%d", &fin);

    t_start = clock();
    bruteforce(inc, fin, rotation);
    t_end = clock();

    printf("Terminado en : %ld segundos.\n", (t_end - t_start)/1000);

    return 0;
}

void bruteforce(int inc, int fin, char *rotation)
{
    int i, j, len = strlen(rotation);
    long int r, end, c;
    char test[200];

    end = pow(len, inc);
    for(j = inc; j < fin + 1; j++)
    {
        for(r = 0; r != end; r++)
        {
            c = r;
            for(i = 0; i < j; i++)
            {
                test[i] = rotation[c % len];
                c = c / len;
            }
            puts(test);
        }
        end *= len;
    }
}


Solo le cambie lo del printf ya que considero que es mas lento que puts

Saludos kutcher

rir3760

Cita de: kutcher en  3 Septiembre 2014, 19:56 PMSolo le cambie lo del printf ya que considero que es mas lento que puts
En ese caso debes agregar manualmente el carácter '\0' justo antes de la llamada a función (esa es la ventaja de printf):
test[i] = '\0';
puts(test);


Un saludo
C retains the basic philosophy that programmers know what they are doing; it only requires that they state their intentions explicitly.
--
Kernighan & Ritchie, The C programming language