Ввод/вывод. Покритикуйте код

@vcorp · Регистрация: 01.03.2015

Студворк — интернет-сервис помощи студентам

Как можно это написать еще лучше (производительнее, эстетичнее, логичнее, экономнее по памяти и строкам)?

C
1
2
3
4
5
6
7
8
9
10
11
12
13
    char *DATA_TEXT = (char*)malloc(12);
    FILE *FILE_TEXT = fopen("textfile.txt", "r");
    int c;
    unsigned char i = 0;
    do{
        c = fgetc(FILE_TEXT);
        if (c != EOF)
            *(DATA_TEXT+i++) = c;
        else
            break;
    } while(1);
    for (unsigned char j = 0; j < i;)
        putchar(*(DATA_TEXT+j++));

Заранее благодарю за здравую критику.

@easybudda · 14.03.2015, 02:01

Не по теме:

Сообщение от vcorp

Как можно это написать еще лучше

Похоже, лучше некуда...

@korvin_ · 15.03.2015, 10:28

Не по теме:

Сообщение от easybudda

Похоже, лучше некуда...

Мой сарказмометр сломался. Это ведь сарказм, правда? =))

Сообщение от vcorp

Заранее благодарю за здравую критику.

1) Захрадкоженное имя файла — дурной тон.
2) Где проверка, что файл открылся?
3) Где закрытие файла?
4) Почему 12? Если в файле больше 12-ти байт, добро пожаловать за границу массива.
5) Инкрементировать счётчики в выражениях — дурной тон.
6) Зачем тут вообще массив, если ты сразу вывод делаешь?
7) Использование unsigned char для счётчиков не имеет смысла в большинстве случаев. В этом тоже.

C
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
#include <stdio.h>
#include <stdlib.h>
 
int
cat(char *filename)
{
    FILE *fd;
    int c;
 
    fd = fopen(filename, "r");
    if(fd == NULL)
        return 1;
    while((c = fgetc(fd)) != EOF)
        putchar(c);
 
    fclose(fd);
    return 0;
}
 
int
main(int argc, char *argv[])
{
    int i;
 
    for(i = 1; i < argc; i++)
        if(cat(argv[i]) != 0)
            fprintf(stderr, "can't open file: %s\n", argv[i]);
 
    return 0;
}

Bash
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
----[ d:\Sources\C ]----
% tcc -o cat.exe cat.c
 
----[ d:\Sources\C ]----
% type hello.txt
Hello
----[ d:\Sources\C ]----
% type world.txt
World!
 
----[ d:\Sources\C ]----
% cat hello.txt world.txt
Hello World!
 
----[ d:\Sources\C ]----
%

@vcorp · 15.03.2015, 12:40 **[ТС]**

>korvin_,
Спасибо. Объясни еще, пожалуйста, почему unsigned char в качестве счетчика не имеет смысла.

@korvin_ · 15.03.2015, 15:03

Сообщение от vcorp

Спасибо. Объясни еще, пожалуйста, почему unsigned char в качестве счетчика не имеет смысла.

http://stackoverflow.com/quest... -variables

Новые блоги и статьи Все статьи Все блоги /
Автозаполнение реквизита при выборе элемента справочника Maks 27.03.2026 Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. При выборе "Спецтехники" (Тип Справочник. Спецтехника), заполняется. . .	Сумматор с применением элементов трёх состояний. Hrethgir 26.03.2026 Тут. https:/ / fips. ru/ EGD/ ab3c85c8-836d-4866-871b-c2f0c5d77fbc Первый документ красиво выглядит, но без схемы. Это конечно не даёт никаких плюсов автору, но тем не менее. . . всё может быть. . .	Автозаполнение реквизитов при создании документа Maks 26.03.2026 Программный код из решения ниже размещается в модуле объекта документа, в процедуре "ПриСозданииНаСервере". Алгоритм проверки заполнения реализован для исключения перезаписи значения реквизита,. . .	Команды формы и диалоговое окно Maks 26.03.2026 1. Команда формы "ЗаполнитьЗапчасти". Программный код из решения ниже на примере нетипового документа "ЗаявкаНаРемонтСпецтехники" разработанного в конфигурации КА2. В качестве источника данных. . .
Кому нужен AOT? DevAlt 26.03.2026 Решил сделать простой ланчер Написал заготовку: dotnet new console --aot -o UrlHandler var items = args. Split(":"); var tag = items; var id = items; var executable = args;. . .	Отправка уведомления на почту при изменении наименования справочника Maks 24.03.2026 Программная отправка письма электронной почты на примере изменения наименования типового справочника "Склады" в конфигурации БП3. Перед реализацией необходимо выполнить настройку системной учетной. . .	модель ЗдравоСохранения 5. Меньше увольнений- больше дохода! anaschu 24.03.2026 Теперь система здравосохранения уменьшает количество увольнений. 9TO2GP2bpX4 a42b81fb172ffc12ca589c7898261ccb/ https:/ / rutube. ru/ video/ a42b81fb172ffc12ca589c7898261ccb/ Слева синяя линия -. . .	Midnight Chicago Blues kumehtar 24.03.2026 Такой Midnight Chicago Blues, знаешь?. . Когда вечерние улицы становятся ночными, а ты не можешь уснуть. Ты идёшь в любимый старый бар, и бармен наливает тебе виски. Ты смотришь на пролетающие. . .

@vcorp 0 / 0 / 1 Регистрация: 01.03.2015 Сообщений: 7
	15.03.2015, 12:40 [ТС]
	>korvin_, Спасибо. Объясни еще, пожалуйста, почему unsigned char в качестве счетчика не имеет смысла. 0