Оцените

[RussDragon]

#include <stdio.h>
#define IN 1
#define OUT 0



int main() {

    int c, i, state, nb, nl;
    int count[10];
    nb = 0;
    
    for (i = 0; i<10; ++i)
    count[i] = 0;
    
    
    while((c = getchar()) && (c != EOF))
    {
            
            if (c == ' ' || c == '\n' || c == '\t')
            state = OUT;
        
                else if (state == OUT)
                {
                    state = IN;
                    ++nb;
                }
        
        if (c != ' ' && c != '\n' && c != '\t'){
            ++nl;
            count[nb] = nl;
        }
        
        if (state == OUT)
            nl = 0;
    }
    
    for(i = 1; i < 10; ++i)
    {
        printf("%d: %d\n", i, count[i]);
    }
    
    
        
        
    
    
}

Понимаю, здесь многое можно урезать, но скажите, насколько это плохой код для начинающего?

[Stilet]

Это хороший код даже для проффи, если он дает правильный результат.

[RussDragon]

Его смысл в принципе в одном, принимать входной поток, смотреть является ли символ частью слова, если да то считать их количество, до тех пор пока не встретится символ по типу перехода на новую строку, табуляции или пробела, далее записывать данные в массив. Вывод от текста "test test" такой — 0 4 4 0 0 0 0 0 0 0 ( Да, как записывать в первую ячейку массива и придумал, но подумал что не критично )

UPD:
Поправил вывод, теперь он начинает вывод с 1 и записывает в таком формате:
1: 4
2: 4
3: 0
И т.д.

UPD2:
Нашел несколько ошибок при вводе русских слов. Например при написании "Привет" он записывает не 6, а 12 символов. Может я конечно чего-то пока не знаю, но надо бы поправить.

[Stilet]

он записывает не 6, а 12 символов.

Потому что считает русское слово юникодом, а в нем один символ описывается двумя байтами.

[RussDragon]

Ну, как я и говорил, чего-то я не знал)
Ну тогда пожалуй, это полностью рабочая версия программы

[three_cats]

Молодец! Так держать!

[RussDragon]

int Calc(int, int, int);

int main()
{
    Calc(12, 13, '+');
}


int Calc(int i, int c, int g) {
    int b;
    if (g == '+') {
        b = i +c;
    }
    else if (g == '*')
    {
        b = i * c;
}
    else if (g == '/') {
        b = i / c;
    }
    else if(g == '-') {
        b = i - c;
    }
    return printf("%d\n", b);
}

Тот же вопрос что и в первом, как все плохо?
Да, я специально не использовал switch так как не дошел до него в учебнике.
И да, код написал за 10 минут от нечего делать)

[s1r0]

Слишком маленький код, чтобы по нему можно было что-то сказать. И да, тут лучше и нагляднее будет использовать switch.

P.S. Посмотри, что произойдет, если записать в main такой вызов: Calc(12, 0, '/');

[RussDragon]

У меня вызывается LLBD, надо поправить, но думать ради такого бесполезного кода не хочется)

[three_cats]

Такой код не каждому дано написать...