Create lab3.2 #3
Conversation
lab3.2
Outdated
| perror("myls: openning dir"); | ||
| return 1; | ||
| } | ||
| struct dirent* ep; |
There was a problem hiding this comment.
Опять не понятно, что за ep)
Всё же, не стоит жертвовать читаемостью кода ради сокращения нескольких букв в названии переменной
| perror("mymkdir: removing dir"); | ||
| return 1; | ||
| } | ||
| return 0; |
There was a problem hiding this comment.
И снова какие-то непонятные 0 и 1)
| int mycat(char* name){ | ||
| FILE* file = fopen(name, "r"); | ||
| if (file == NULL){ | ||
| perror("mycat: openning file:"); |
There was a problem hiding this comment.
А тут типо норм, файл не открылся - написали сообщение и пошли дальше?
lab3.2
Outdated
| size_t size; | ||
| char buffer[4096]; | ||
| while((size = fread(buffer, sizeof(char), 4096, file))){ | ||
| if (write(0,buffer,size )== 0){ |
There was a problem hiding this comment.
Ну, ты, вроде, везде пользовался буферизированным вводом/выводом с fread/fwrite. Почему вдруг тут решил напрямую системные вызовы дергать? Почему тот же fwrite() не использовать?
lab3.2
Outdated
| buffer[len] = '\n'; | ||
| buffer[len +1] = '\0'; | ||
| if (write(1, buffer, len + 1) == 0){ | ||
| perror("mycatvsl "); |
lab3.2
Outdated
| } | ||
|
|
||
| printf("%s info: number of hard links: %lu.", name, st.st_nlink); | ||
| char rights[10]; |
There was a problem hiding this comment.
Ты можешь при объявлении массива его сразу нулями инициализировать char rights[10] = {0};, тогда не придется потом последний символ занулять
lab3.2
Outdated
| perror("mychmod: getting info about file"); | ||
| return 1; | ||
| } | ||
| st.st_mode = (st.st_mode >> 8) << 8; |
There was a problem hiding this comment.
Очень странно, почему сдвиги на 8, когда бит используется 9..
Можно было еще написать что-то типа st.st_mode &= ~((mode_t) 0777);
There was a problem hiding this comment.
И если в своем коде используешь какие-то такие неочевидные шаги, лучше всегда оставлять комментарии, почему здесь именно так. Ибо ты сам будешь читать код через месяц и уже забудешь, о чем здесь вообще речь была, почему здесь 8/9/15, а не 379, например
lab3.2
Outdated
| } | ||
|
|
||
| for (int i = 0; i < 3; i++){ | ||
| st.st_mode = rights[3 * i] == 'r' ? (st.st_mode | (0400 >> (3 * i))) : st.st_mode; |
There was a problem hiding this comment.
Если хочется коротенько и красиво, то можно вот так сделать)
st.st_mode |= rights[3 * i] == 'r' ? (0400 >> (3 * i)) : 0;
lab3.2
Outdated
| // printf("argv[0] %s\n", argv[0]); | ||
| // } | ||
| char* exe_name = strrchr(argv[0], '/') + 1; | ||
| if (exe_name == (char* ) NULL + 1 ){ |
There was a problem hiding this comment.
Выглядит это, конечно, страшновато)
Вообще, есть уже готовая функция, которая по пути вернет тебе имя файла. Советую поискать, посмотреть)
lab3.2
Outdated
|
|
||
| if (!strcmp(exe_name, "mymkdir")){ | ||
| if (argc != 2){ | ||
| fprintf(stderr, "ivalid number of arguments(requars 1 arg: dir name)"); |
There was a problem hiding this comment.
Вообще, правильно пишется require, а не requar. Лучше будет написать required 1 arg
lab3.2
Outdated
| } | ||
|
|
||
|
|
||
| int myrm(char* name){ |
There was a problem hiding this comment.
А вот твои изменения, в которых ты половину лабы удалил))
No description provided.