Ошибка в примере K&R

[8Observer8]

Доброго времени суток! Помогите, пожалуйста, отладить пример из книги Кернигана и Ритчи. Никак не могу сообразить, как заставить его работать.

Вот пример из книги, без изменений:

// find_orig.c

#include <stdio.h>
#include <string.h>
#define MAXLINE 1000

int getline(char* s, int lim);

/* find: вывод строк, содержащих образец из 1-го аргумента */
int main (int argc, char* argv[]) {
    char line[MAXLINE];
    long lineno = 0;
    int c, except = 0, number = 0, found = 0;

    while (--argc > 0 && (*++argv)[0] == '-')
        while (c == *++argv[0])
            switch (c) {
            case 'x':
                except = 1;
                break;
            case 'n':
                number = 1;
                break;
            default:
                printf("find: illegal option %c\n", c);
                argc = 0;
                found = -1;
                break;
            }
    if (argc != 1)
        printf("Usage: find -x -n pattern\n");
    else
        while (getline(line, MAXLINE) > 0) {
            lineno++;
            if ((strstr(line, MAXLINE) > 0)) {
                if (number)
                    printf("%ld: ", lineno);
                printf("%s", line);
                found++;
            }
        }
    return found;
}

/* getline: считывает строку в s */
int getline(char* s, int lim) {
    int c, i;

    for (i = 0; i<lim-1 && (c = getchar()) != EOF && c != '\n'; i++) {
        s[i] = c;
    }

    if (c == '\n') {
        s[i++] = c;
    }
    s[i] = '\0';

    return i;
}

Ошибки, которые я исправил:
1) Заменил строку:
while (c == *++argv[0]);
на:
while (c = *++argv[0]);
2) Заметил, что ключ 'x' не обрабатывается, но это ерунда, я исправил.
3) Не смог найти ошибку. Предполагаю, что она кроется в отмеченной красным строке. Ведь нельзя же скажем объявить массив int a[10] и произвести операцию: ++a. А здесь это проделали. Как записать правильно? Заранее спасибо всем, кто откликнется на просьбу: помочь разобраться.

[Blade]

Сообщение от 8Observer8

Ведь нельзя же скажем объявить массив int a[10] и произвести операцию: ++a.

Почему? Читаем внимательно арифметику указателей.
Начните с того, что именно у вас не работает?

[JuniorProger]

У меня компилятор ругается на strstr(line, MAXLINE). Говорит, что не может найти функцию с подходящими параметрами. Оно и понятно. Посмотри объявление функции. А почему в вариантах для switch (c) символы типа case 'x' ? Переменная с объявлена как int. И еще, разве хоть раз выполнится условие (c == *++argv[0])? Переменная с объявлена как int.

[8Observer8]

> to Blade
Я нашёл это место в книге. Написано, что эта операция не разрешена! Я даже попробовал сейчас:

#include <stdio.h>
int main () {
    int a[10] = {1,2,3};
    ++a;
    return 0;
}

Компилятор отписался:
error C2105: для "++" требуется левостороннее значениe

> to JuniorProger

У меня компилятор ругается на strstr(line, MAXLINE)

Да, извиняюсь, я забыл написать, что эту ошибку я тоже исправил. Блин такой косячный пример, до этого ни одного косяка, почти.

И еще, разве хоть раз выполнится условие (c == *++argv[0])

Вы пропустили. Я написал в первом сообщении:

Ошибки, которые я исправил:
1) Заменил строку:
while (c == *++argv[0]);
на:
while (c = *++argv[0]);

А почему в вариантах для switch (c) символы типа case 'x' ? Переменная с объявлена как int

Ну здесь точно нет косяка.

Вот исправленный код (кроме except, пока он не нужен), компилится нормально, но всё равно ошибка возникает на этапе выполнения.

// find_orig.c

#include <stdio.h>
#include <string.h>
#define MAXLINE 1000

int getline(char* s, int lim);

/* find: вывод строк, содержащих образец из 1-го аргумента */
int main (int argc, char* argv[]) {
    char line[MAXLINE];
    long lineno = 0;
    int c, except = 0, number = 0, found = 0;

    while (--argc > 0 && (*++argv)[0] == '-')
        while (c = *++argv[0])
            switch (c) {
            case 'x':
                except = 1;
                break;
            case 'n':
                number = 1;
                break;
            default:
                printf("find: illegal option %c\n", c);
                argc = 0;
                found = -1;
                break;
            }
    if (argc != 1)
        printf("Usage: find -x -n pattern\n");
    else
        while (getline(line, MAXLINE) > 0) {
            lineno++;
            if ((strstr(line, argv[argc]) > 0)) {
                if (number)
                    printf("%ld: ", lineno);
                printf("%s", line);
                found++;
            }
        }
    return found;
}

/* getline: считывает строку в s */
int getline(char* s, int lim) {
    int c, i;

    for (i = 0; i<lim-1 && (c = getchar()) != EOF && c != '\n'; i++) {
        s[i] = c;
    }

    if (c == '\n') {
        s[i++] = c;
    }
    s[i] = '\0';

    return i;
}

[JuniorProger]

Ну а как насчет того, что int c ?

[8Observer8]

Я успел раньше отредактировать Так точно можно, я всегда так делаю и авторы рекомендуют.

Думаю, косяк в этой записи: (*++argv)[0]. Не нравится она мне.

[JuniorProger]

Чет я тогда не понимаю. Как он в int найдет 'x' или 'n'? Он всегда будет выходить на default (наверное).

Сообщение от 8Observer8

Думаю, косяк в этой записи: (*++argv)[0]. Не нравится она мне.

Так он ошибку выдает? Попробуй так *(++argv[0])

[8Observer8]

Чет я тогда не понимаю. Как он в int найдет 'x' или 'n'? Он всегда будет выходить на default (наверное).

Вот пример:

#include <stdio.h>

int main () {
    int c;

    c = 'x';
    switch (c) {
       case 'x':
           printf("%c\n", c);
           break;
       default:
           printf("None\n");
           break;
    }
}

Так он ошибку выдает? Попробуй так *(++argv[0])

Попробовал, не работает. Я уже запутался с этими записями и их расшифровками

[JuniorProger]

А что вообще делает программа? Я в код не вникал. Разъясни поподробнее.

Сообщение от 8Observer8

Я успел раньше отредактировать Так точно можно, я всегда так делаю и авторы рекомендуют.

Действительно можно, но я не видел чтобы кто-то так рекомендовал делать. По-моему это только ухудшает читаемость программы и создает путанницу. Лучше так не делать.

[8Observer8]

А что вообще делает программа? Я в код не вникал. Разъясни поподробнее.

В командной строке вводишь имя .exe файла, ключи (-x или -n) и образец для поиска подстроки в строкаx. Для вывода тех где есть совпадения
Ключи:
-x все строки, кроме образца
-n ставить номер перед выводимой строкой

Ну проще на примерах объяснить:

Пример 1:
find abc

Запрос(введите строки):
abc1344
dfdffdfs
ererrer
dsfabc344

Результат:
abc1344
dsfabc344

Пример 2:
find -x -n abc

Запрос(введите строки):
abc1344
dfdffdfs
ererrer
dsfabc344

Результат:
2: dfdffdfs
3: ererrer

Предыдущий можно ещё и так запускать
Пример 2(перестановка и объединение ключей):
find -nx abc