2

upd.: Переименовал вопрос. Из-под терминала код работает.

Предпринял попытку переселиться на Linux (xubuntu 12.10 x64) c Windows. Первым делом были установлены среды разработки QTcreator и MonoDevelop и была написана простенькая программа для теста. Суть программы заключается в выдаче пользователю n случайных чисел от 1 до m пока не будет введён 0. Код:

#include<stdio.h>
#include<stdlib.h>
#include<time.h>

int dN(int N);   
void MdN(int M, int N);

int main(int argc, char ** argv)
{
    int m = 1, n = 0;
    int c;
    srand(time(NULL));
    while (m != 0)
    {
        printf("\nВведите два числа (количество и количество граней дайсов) или 0 для выхода\n");
        m = 0;
        c = scanf("%d", &m);
        if (c != 1)
        {
             puts("Ошибка ввода");
             return 0;
        }
        if (m != 0)
        {
            n = 0;
            while (n == 0)
            {
                printf("\b\bd");
                c = scanf("%d", &n);
                if (c != 1)
                {
                     puts("Ошибка ввода");
                     return 0;
                }
            }
            MdN(m, n);
        }
    }
    printf("\n");
    return 0;
}

int dN(int N)
{
    return rand()%N + 1;
}

void MdN(int M, int N)
{
    int s = 0, tmp;
    while (M>0)
    {
        s += tmp = dN(N);
        printf("+%d", tmp);
        M--;
    }
    printf("=%d;\r \n", s);
}

В обеих средах код работал неверно. В QtCreator программа бесконечно запрашивала ввод (в т.ч. и символьного типа). В MonoDevelop программа не запрашивала (и не ожидала) ввод, но выводила

\nВведите два числа (количество и количество граней дайсов) или 0 для выхода\n

Ошибка ввода

(выполнялось c==EOF).

Устанавливал среды через "Центр приложений Ubuntu", настроек по дефолту не менял.

//тут был ещё более простой код с той же проблемой

11
  • Попробуйте включить ctype.h
    – renegator
    26 мар 2013 в 9:08
  • Не оказало влияния на работу программы.
    – bessgeor
    26 мар 2013 в 17:22
  • @bessgeor, ну Вы меня прямо заинтересовали (последним маленьким кодом). А что Вы от него ждете? Если печать введенного числа, то добавьте %d в соответствующий printf(). Также советую завершать вывод символом '\\n'.
    – avp
    26 мар 2013 в 20:52
  • Забыл %d по неопытности и т.к. спешил. Исправил это, ситуация с некорректной работой программы сохранилась. Сейчас попробую применить флаги компилятора, как посоветовал KoVadim, может так смогу увидеть причину проблемы.
    – bessgeor
    26 мар 2013 в 21:46
  • 1
    @bessgeorg, ну а теперь замените в if (c == 1) { puts("Input error"); .... c == 1 на c != 1, глядишь и лучше станет.
    – avp
    27 мар 2013 в 10:30

4 ответа 4

5

Посмотрел код, стало страшно. Вот мой вариант (написан в лоб, по факту можно было бы вынести в отдельные функции)

#include<stdio.h>
#include<stdlib.h>
#include<time.h>
#include <ctype.h>

int dN(int N)
{
    srand(time(NULL));
    return rand()%N + 1;
}

void MdN(int M, int N)
{
   printf("==== MdN(%d,%d)===\n", M, N); // это для отладки
    int s = 0, tmp;
    if (N == 0) // защита, Ваш код падал при нуле
      return;
    while (M>0)
    {
        s += tmp = dN(N);
        printf("+%d", tmp);
        M--;
    }
    printf("=%d;\r \n", s);
}

int main()
{
    int m = 0;
    int n = 0;
    while (1)
    {
        printf("Enter two numbers (MdN) or zero to exit\n");
        int r = scanf("%d", &m); // ввод первого
        if (r != 1) {
          puts("first number is not numer\n");
          break;
        }
        if (m == 0) {
            // пользователь ничего не ввел либо ввел один ноль
            puts("exit\n");
            break;
        }

        // удалось прочитать только одно число. Попробуем прочитать ещё одно
        r = scanf("%d", &n);
        if (r != 1) {
            puts("ups, second number is not number\n");
            break;
        }

        MdN(m,n);
    }
    printf("end\n");
    return 0;
}

scanf возвращает результат и корректно будет проверять, что он возвратил. По факту - возвращается количество переменных, которые удалось вычитать. 0 - если ничего не прочитали. Те переменные, которые удалось прочитать, будут иметь значения. Если случится что то странное, scanf возвратит -1 (по честному - константу EOF).

Можно делать ввод в виде sscanf("%d %d", &m, &n). Но если пользователь введет только один ноль, то код будет ждать, пока будет введет ещё один символ. Поэтому я разбил ввод на две части.

Как ни странно, в обеих средах код работал неверно.

логично, так как с 99% вероятностью компилятор один и тот же - gcc.

Для Вашего случая больша подошла бы getchar() с проверкой введенного значения

очень вредный совет. С int еще проверку сделаете, а с float?

Почему же оно работает в Visual studio? есть подозрение, что там sscanf("%d %d", &m, &n) при вводе одно числа и перевода строки возвращает 1 и заполненное m.

upd

Учитывая последние изменения в примере в вопросе. Если Вы только начинаете изучать С и делаете много ошибок (зачастую элементарных), то нужно компилятору намекнуть, что нужно быть строже. Да, он будет придираться по пустякам, но на первое время самое оно.

Я рекомендую такой наборчик

-std=c99 -pedantic -Wall -Wshadow -Wpointer-arith -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes

Как использовать:

  • если это в консоли, то просто `gcc имяфайла.с -std=c99 -pedantic -Wall -Wshadow -Wpointer-arith -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes
  • если это monodevelop. В дереве "решений" слева выделяете свой проект, кликаете правой, "свойства", в окне "свойства проекта" находите "Генерация кода", выбираете вкладку с таким же именем и внизу видите окно "Extra Compiler Options", куда и вставляете вышеприведенный список.
  • В QtCreator нужно открыть pro файл и добавить туда строку вида QMAKE_CFLAGS = -std=c99 -pedantic -Wall -Wshadow -Wpointer-arith -Wcast-qual -Wstrict-prototypes -Wmissing-prototypes

Что значат эти флаги?

  • std=c99 следовать стандарту с99. Сейчас есть и другие, но этот более промышленный (но это мое мнение)
  • pedantic быть попридирчивей
  • Wall включает большой набор проверок. К примеру придерется к коду вида if (a = 0) (некоторые любители любят это лечить переворачиванием условия, а некоторые используют для сознательного запутывания). Также эта опция включает -Wformat, который заставляет компилятор пересчитывать параметры у функций вида printf/scanf и ругаться, если их не хватает или типы не совпадают.
  • Wshadow придираться, когда в функции используется перекрывание переменных. К примеру, внутри функции объявляется переменная, имя которой совпадает с параметром. В некоторых случаях может приводить к веселым ошибкам.
  • Wpointer-arith в новых версиях это уже включается в -pedantic. Придирается, когда с химичат с арифметикой указателей.
  • Wcast-qual придирается, когда делается приведение типов, которое потенциально опасное. это часто вылазит у новичков, когда они осваивают работу со строками в стиле с.
  • Wstrict-prototypes придерется к int main(), так как желательно писать int main(int argc, char ** argv).
  • Wmissing-prototypes потребует определять прототипы используемых функций. То есть, добавлять строки вида int dN(int N);.

Конечно, некоторые найдут эти вещи абсурдными и мешающими (особенно для "продвинутых программистов"). Но я в своих проектах время от времени включаю подобные наборы и смотрю - не вылезло ли глупостей - опечатку то сделать легко.

На первый порах рекомендую все эти замечания компилятора устранять.

14
  • в данной теме задачи считывания float не стоит, поэтому getchar() считаю вполне уместным.
    – margosh
    26 мар 2013 в 12:38
  • а сделать проверку на переполнение, а сделать ввод отрицательных чисел? Ввод через getchar в данном случае - это велосипед.
    – KoVadim
    26 мар 2013 в 12:46
  • 1
    В данном случае это не должно давать описываемые автором вопроса проблемы.
    – KoVadim
    27 мар 2013 в 8:22
  • 1
    Ключи компилятора, которые я привел, попытаются выявить часть ошибок, что бы ТС не переспрашивал о элементарных ошибках. ТС пишет, что ему не нравиться и это точно не связано с srand.
    – KoVadim
    27 мар 2013 в 9:15
  • 1
    @bessgeor, ну что же у Вас в самом деле не работает? Вот скопипастил Ваш код (ничего в нем не трогал), собрал и запустил: avp@avp-ubu1:~/hashcode$ gcc t22.c avp@avp-ubu1:~/hashcode$ ./a.out Enter two numbers (MdN) or zero to exit 3 3Enter non-zero N4 2+2+2=6; Enter two numbers (MdN) or zero to exit 5 5Enter non-zero N6 2+2+2+2+2=10; Enter two numbers (MdN) or zero to exit 0 0avp@avp-ubu1:~/hashcode$ Как видите ввод-вывод работает. Это Linux (Ubuntu 12.04.2, gnome-terminal). С \\n Вы, конечно, не дружите, но в принципе работает.
    – avp
    27 мар 2013 в 20:08
5

Вероятнее всего, Ваша проблема кроется в функции scanf(), т.к. данная функция завершает считывание числа, когда встречает первый нечисловой символ (link), соответственно, у Вас в основном цикле:

  1. переменная m остается равной своему предыдущему значению,
  2. после введенный символ пробует считаться еще раз, не считывается снова, и в n остается либо предыдущее значение, либо мусор,
  3. выполняются действия для неадекватных m и n, но введенный символ никуда из потока ввода не делся, и происходит зацикливание.

Есть у Вас и еще ошибка - Вы не проверяете значение, сохраняемое в n, а в функции dN() делите на него, хотя никто не мешает ввести вторым запрашиваемым числом - 0.

5
  • Переделал (изменил код в вопросе) ввод под использование getchar(). Проблема сохранилась в том же виде, за исключением того, что в MonoDevelop зацикливания больше не происходит и вывод происходит лишь один раз (т.к. в теле цикла перед вводом m приравнивается 0, а ввода не происходит).
    – bessgeor
    26 мар 2013 в 8:56
  • компилила с getchar() в gcc, все там нормально вводится... попробуйте еще проще примерчик запустить просто с вводом чилел и их последующей печатью. KoVadim прав, все-таки стоит пользоваться тем же scanf(), но с проверкой результата, дабы избежать зацикливания.
    – margosh
    26 мар 2013 в 12:42
  • gcc генерирует исполняемый файл, запуск которого ни к чему не ведёт
    – bessgeor
    26 мар 2013 в 17:23
  • @margosh, Вы что, поиздеваться над новичком (это о совете заменить scanf на getchar) решили? -- Оно, конечно, можно (в конце-то концов свой scanf получится), но все-таки для ввода чисел значительно проще изучить, как использовать scanf из библиотеки.
    – avp
    27 мар 2013 в 8:15
  • @avp, нет, я решила,что автор просто проверяет ввод,и раз с использованием scanf() было сразу не разобраться(в изначальной версии он все зацикливал), то проще пользоваться getchar(), но если корень проблемы в считывании именно чисел, то она неудобна. Поправлю ответ, чтобы это больше никого не сумщало.
    – margosh
    27 мар 2013 в 9:52
1

Почему же оно работает в Visual studio

У мелкомягких все как не у людей

А вообще-то я советую компилить gcc из командной строки

1
  • 1
    @akalend: вы считаете, что то, что программа не работает — это нормально, так "у людей" и должно быть?
    – VladD
    26 мар 2013 в 18:00
0

Доброго времени суток! По поводу Ваше простой задачи(ввести число, вывести число), вы не корректно используете функцию printf - вы забыли передать форматную строку, в стандарте сказано, что при этом

Программа будет иметь неопределенное поведение

Что вы и имеете на разных платформах (зависает, не выводит и т.п.) Вот исправленный вариант:

#include <stdio.h>

int main()
{
    int a, b;
    printf("test output\n");
    b = scanf("%d", &a);
    if (b == EOF)
        puts("input error");
    // не забываем %d!
    printf("input test: %d", a);
    return 0;
}

Теперь по поводу самого вопроса, я не очень понял, что именно не так работает в Вашем коде? Просто у меня исправленный код Вашей программы работает вроде нормально, может я что-то не уловил?

2
  • В малой задаче исправление %d не помогло. Проблема заключается в следующем: во всех версиях кода, способных компилироваться а) gcc генерирует исполняемый файл, запуск которого ни к чему не ведёт б) QT Creator открывает окно терминала и читает в нём всё подряд ничего не выводя в) MonoDevelop не запрашивая ввода выводит всё что может (на что не стоит однозначно false условий, например, 1!=1). Например, выполнение вашего кода, приведённого выше приводит к такому вот выводу: test output input error input test: 0
    – bessgeor
    26 мар 2013 в 21:56
  • Посмотрите, может это поможет: stackoverflow.com/questions/3255035/qt-creator-run-in-terminal 29 мар 2013 в 15:20

Ваш ответ

By clicking “Отправить ответ”, you agree to our terms of service and acknowledge that you have read and understand our privacy policy and code of conduct.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.