0

Всем доброго времени суток. Назрела такая проблема: при передаче параметров в консоли Linux выдает ошибку "Ошибка сегментирования (сделан дамп памяти)". Вот код программы (нужно вывести на экран требуемое число Фиббоначи по индексу):

int main(int argc, char* argv[]) {
    cout << "Your value is ";
    for (int i = 0; argc; i++){
        int a = atoi(argv[i]);
        if (a <= 2) cout << "1"; 
        else{
            int x = 1;
            int y = 0;
            int ansver;
            for (int j = 1; j <= a; j++)
            {
                    ansver = x + y;
                    x = y;
                    y = ansver;
            }
            cout << ansver;
            }
        }
    return 0;
}

2 ответа 2

3

И сколько раз выполнится цикл

for (int i = 0; argc; i++)

Если argc всегда больше нуля?...

И еще - argv[0] - это просто имя самой программы (как правило), но никогда - не аргумент командной строки.

Словом, вот:

int main(int argc, char* argv[]) {
    cout << "Your value is ";
    for (int i = 1; i < argc; i++){
        int a = atoi(argv[i]);
        if (a <= 2) cout << "1  ";
        else
        {
            int x = 1;
            int y = 0;
            int answer;
            for (int j = 1; j <= a; j++)
            {
                ansver = x + y;
                x = y;
                y = answer;
            }
            cout << answer << "  ";
        }
    }
    return 0;
}
5
  • получилось, спасибо! а если мне потребуется соединить данную программку с еще одним .cpp файлом, в котором я задаю новые ключи с нужными мне функциями, как это сделать, не подскажите?
    – Kyrlik
    8 мар 2017 в 15:41
  • Не очень понял, что вы хотите спросить...
    – Harry
    8 мар 2017 в 15:42
  • например: " ls -aR ", только вместо -aR я прописываю новый ключ (допустим -z) и задаю ему нужную мне функцию. если я пытаюсь объединить мою мини-программу и файл с прописанными новыми ключами, то пишет "множественное определение main". если попытаться объединить два данных файла(т.е. в мою программу тупо вставить нужный код и забыть о проблемме), то, по-моему, это будет неверно
    – Kyrlik
    8 мар 2017 в 15:49
  • Все равно не понял. Так что выскажусь чисто теоретически - оформите этот код для чисел Фибоначчи как отдельную функцию. main должна заниматься только разбором командной строки и вызовом соответствующих функций (и, само собой, в программе она должна быть одна).
    – Harry
    8 мар 2017 в 16:42
  • я бы добавил, что разбор строки надо бы структурировать. Посмотрите на исходники линуховых утилит, там есть хороший паттерн разбора. Написав один раз, дальше нужно только добавлять указатели на функции обработки в массивчик 10 мар 2017 в 9:59
1

Так а что это за цикл такой

for (int i = 0; argc; i++)

Это бесконечный цикл. Неудивительно, что в какой-то момент доступ к argv[i] вылетает, учитывая, что argv[argc] всегда равно null-указателю. Может имелось в виду

for (int i = 0; i < argc; i++)

?

Ваш ответ

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

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