czemu program nie chce się łaskawie zkończyć
#include <stdio.h>
void filecopy(FILE*, FILE*);
int main(int argc, char* argv[])
{
FILE* fp =fopen("dane.txt","r");
char* program = argv[0];
printf("%d\n", argc);
if (argc == 1)
filecopy(fp, stdout);
return 0;
}
void filecopy(FILE* ifp, FILE* ofp)
{
int c;
while ((c = getc(stdin)) != 0) // <<<<<------ jak wpisze 0 z klawiatury
fputc(c, ofp);
}
bartek164 napisał(a):
void filecopy(FILE* ifp, FILE* ofp) { int c; while ((c = getc(stdin)) != 0) // <<<<<------ jak wpisze 0 z klawiatury fputc(c, ofp); }
A nie powinno być czasem:
C while ((c = getc(stdin)) != '0') // <<<<<------ jak wpisze 0 z klawiatury
możecie zobaczyć dlaczego to nie działa?
int main(int argc, char *argv[]){
//char* temp[] = argv; nie można tak przypisać?
int i;
char* temp[argc];
for(i=0;i<argc;++i)
temp[i]=argv[i];
while( ++*temp !=NULL)
printf("%s\n",*temp);
argv?
bartek164 napisał(a):
while( *(temp++) != NULL) printf("%s\n",*temp);
To jest zły sposób na iterowanie. To, że dojdziesz do końca tablicy, nie oznacza, że temp ani tym bardziej *temp będzie NULL, mogą tam być jakieś śmieci. Skoro wiesz, ze masz argc elementów tablicy, iteruj w pętli for aż do ostatniego indeksu. Po to właśnie to argc jest przesyłane, żeby wiedzieć ile tych argumentów jest na liście.
temp jest wskaznikiem na pierwszy element tablicy temp[]. temp++ oznacza temp=temp+1, czyli zwiększ wskażnik o 1 i przypisz go do poprzedniej wartości. To oznacza, że próbujesz przesunąć początek tablicy, zamiast przesuwać się po jej elementach.
no a tak?
#include <stdio.h>
#include <stdlib.h>
int main(int argc, char* argv[])
{
char** temp = &argv;
while (*(temp++) != NULL)
printf("%s\n", *temp);
return 0;
}
powinno być raczej
void filecopy(FILE* ifp, FILE* ofp)
{
int c;
while ((c = getc(ifp)) != EOF) // tu były DWA błędy
fputc(c, ofp);
}
Zarejestruj się i dołącz do największej społeczności programistów w Polsce.
Otrzymaj wsparcie, dziel się wiedzą i rozwijaj swoje umiejętności z najlepszymi.