0
#include <iostream>

using std::cin;
using std::cout;

int main(int argc, char *argv[]){
    if(argc <= 1){
        cout << "Not enough args.\n";
        return 1;
    }
    int i;
    int j = 0;
    for(i = 2; i <= argc; i++){
        while(argv[i][j]){
            cout << argv[i][j];
            j++;
        }
        cout << "\n";
    }
    return 0;
}

Что здесь не так?

1
  • 1
    счёт в массиве начинается с "0" 21 ноя 2013 в 5:01

1 ответ 1

-1

Вот здесь i должно быть 0, т.к. ка правильно заметил ProkletyiPirat первый элемент массива 0, то i=2 будет выводить параметры, начиная с 3-го

for(i = 2; i <= argc; i++){
    while(argv[i][j]){
        cout << argv[i][j];
        j++;
    }
    cout << "\n";
}

правильно

for(i = 0; i <= argc; i++){
    while(argv[i][j]){
        cout << argv[i][j];
        j++;
    }
    cout << "\n";
}
2
  • 5
    @CyberCoder, так тоже неправильно (поскольку argv[argc] == NULL (!!!), то получите Segmentation Fault), да и j не мешает обнулять перед while(). Правильно: for(i = 0; i < argc; i++){ j = 0; while (argv[i][j]){ cout << argv[i][j]; j++; } cout << "\n"; } а еще проще: for (i = 0; argv[i]; i++) cout << argv[i] << '\n'; Если имя программы (argv[0]) печатать не надо, то соответственно for (i = 1; argv[i]; i++) ...
    – avp
    21 ноя 2013 в 7:26
  • согласен, не особо всматривался в код, увидел неправильный начальный индекс и скопировал код, изменив начальное значение переменной.
    – CyberCoder
    22 ноя 2013 в 11:29

Ваш ответ

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

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