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

[8Observer8]

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

Согласен. Буду предварительным просмотром теперь пользоваться

[JuniorProger]

Условие (c = *++argv[0]) выполняется всегда, т.к. это не проверка, а присваивание.

[JuniorProger]

думаю надо так

...........	
    while ((--argc > 0) && ((*argv[0]) == '-')){
        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).......

[8Observer8]

> to JuniorProger

Условие (c = *++argv[0]) выполняется всегда, т.к. это не проверка, а присваивание.

Не согласен, вот пример:

#include <stdio.h>

int main () {
    int c;

    char* argv[] = {"abcdef", "567"};

    while (c = *++argv[0]) {
        printf("%c", c);
    }
    printf("\n");

    return 0;
}

Цикл while заканчивает работу, когда с == '\0'

[Blade]

ТС, о чем вы вообще говорите? Пример из книги, без всяких исправлений, прекрасно компилируется.
http://codepad.org/FCILMdmk
Вы можете объяснить, что вас конкретно не устраивает.

[8Observer8]

> to Blade

ТС, о чем вы вообще говорите? Пример из книги, без всяких исправлений, прекрасно компилируется.
http://codepad.org/FCILMdmk
Вы можете объяснить, что вас конкретно не устраивает.

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

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

думаю надо так

...........	
    while ((--argc > 0) && ((*argv[0]) == '-')){
        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).......

К сожалению, это неверно. Здесь должно быть два цикла. Внешний цикл передвигается по ключам (через символы '-'), а внутренний по символам после символа '-' Это для того, чтобы мы могли вводить, как find -x -n образец, так и find -xn образец.

Я ещё попробовал заменить строку:
int main (int argc, char* argv[]) {
на строку:
int main (int argc, char** argv) {
Всё равно не работает.

[8Observer8]

Я нашёл эту книгу на английском. Пример работает. Пока не понял почему работает и где была ошибка.

    // find_eng.c

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

   int getline(char *line, int max);

   /* find: print lines that match pattern from 1st arg */
   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) != NULL) != except) {
                   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;
}

[8Observer8]

Всем, спасибо! Тему можно закрывать. Понял в чём ошибка.

Надо заменить строку:
if ((strstr(line, argv[argc]) > 0)) {
На строку:
if ((strstr(line, *argv) != NULL) != except) {

[8Observer8]

Поторопился я с закрытием темы. Остались "непонятки". Поясните кто-нибудь, пожалуйста, почему конструкция (*++argv)[0] (из предыдущего кода) работает, а по-моему не должна. Вот пример, где она не работает:

#include <stdio.h>
#include <string.h>

int main() {
    char* argv[] = {"abc", "123"};

    (*++argv)[0];

    return 0;
}

[JuniorProger]

Что ты хочешь получить этой конструкцией (*++argv)[0] ?

Если надо букву b, то надо сделать так *(++argv[0]) или *++argv[0] (первый вариант понятнее). Если число 3, то *(++(++argv[1])) или *++(++argv[1]).
Т.е. сначала выбираешь элемент массива argv: argv[0] = "abc" (указатель на букве а) , argv[1] = "123" (указатель на цифре 1).
Затем сдвигаешь указатель операцией ++: ++argv[0] = "bc" (указатель перешел на b), ++(++argv[0]) = "c"(указатель перешел на с), (++argv[1]) = "23".
Если нужно получить конкретное значение, например 2, то добавляешь * : *(++argv[1]) = "2".
Думаю понятно объяснил.