ayuda con un error

[Fayca]

alguien me puede decir por que no me sirbe esto?

Código:

#include <iostream>
int main(int argc, char** argv){
    if (argc == 4){
        if (argv[1] == "copy"){
            FILE *f1, *f2;
            f1 = fopen (argv[2], "rb");
            if (f1 == NULL)  perror("Error:"); return -1;
            f2 = fopen (argv[3], "wb");
            if (f2 == NULL) perror("Error:"); return -1;
            char buffer[1000];
            int leidos;
            leidos = fread (buffer, 1, 1000, f1);
            if (leidos == NULL) perror("Error"); return -1;
            fwrite (buffer, 1, leidos, f2);
            while (leidos!=0){
                fwrite (buffer, 1, leidos, f2);
                leidos = fread (buffer, 1, 1000, f1);
            }
            fclose(f1);
            fclose(f2);

        }
    }
    return 0;
}

no me da ningun error al compilar
trato que sirba para copiar archivos

[CalgaryCorpus]

Usa llaves para agrupar instrucciones

Código:

if (f1 == NULL)  perror("Error:"); return -1;

es lo mismo que

Código:

if (f1 == NULL)  
   perror("Error:"); 

return -1;

o sea, return -1 se ejecuta incondicionalmente

[Fayca]

creeo "no estoy seguro" que el error esta en esta linea

if (argv[1] == "copy"){

si el error es ese como le puedo hacer para esa comparacion?

[_Lucifer_]

Con la función strcmp.

Saludos

[Fayca]

Gracias¡¡
ya solucione el error
solo era el error del if y hacer la comparacion con strcmp
perdon por la pregunta tonta es que estoy aprendiendo solo

[_Lucifer_]

Entonces te recomiendo que comiences usando un buen estilo para programar, por ejemplo evita hacer:

Código:

...
if (f1 == NULL){  perror("Error:"); return -1;}
...

Es mucho mejor:

Código:

...
if (f1 == NULL)
{ 
   perror("Error:");
   return -1;
}
...

Yo regularmente uso el estilo Ansi para las llaves y la identación, pero hay quienes prefieren la K&R:

Código:

...
if (f1 == NULL) { 
   perror("Error:");
   return -1;
}
...

Incluso en algunos textos recomiendan que en vez de :

Código:

...
if (f1 == NULL)
...

hagas:

Código:

...
if (NULL == f1)
...

En caso de que te "comas" un "=" el compilador mostrará un error, pero esos ya son detalles.

Y cosas por el estilo, lo hacen más faćil de leer (sobre todo para otras personas distntas a quienes hicieron el código).

Saludos