Ввод массива в виде элементов

[artem7902]

Здравствуйте, прошу помочь с вводом массива через консоль в виде символов с помощью gets(). Никак не могу понять почему не работает. Заранее спасибо за помощь.

#include <QCoreApplication>
#include <iostream>
#include <cmath>
#include <cstdlib>
#include <cctype>
#include <cstdio>
using namespace std;
void vvodprot();
void vvodosnowa(char str[],int arr[],int &size);
int main(int argc, char *argv[])
{
    QCoreApplication a(argc, argv);
    vvodprot();
    return a.exec();
}
void vvodprot()
{
int size=0;
int arr[size];
char str[80];
cout << "Введiть елементи масиву через пробiл\n";
gets(str);
vvodosnowa(str,arr,size);
}
void vvodosnowa(char str[],int arr[],int &size)
{
int i=0; size=0;
forever
{
while(str[i]!='\0' && str[i]!=' ')i++;
if (str[i]=='\0')return;
arr[size]=0;
while(str[i]!='\0' && str[i]!=' '){
 if (str[i]!=isdigit(str[i])){
     cout << str[i] << " Не цифра! \n";
     return;
 }}
arr[size] = arr[size]*10+(str[i]-48);i++;
cout << arr [size] << "\n";
size++;}}

[8Observer8]

void vvodprot() 
{ 
int size=0; 
int arr[size];

Здесь ошибка. Массив нулевой длинны

[artem7902]

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

void vvodprot() 
{ 
int size=0; 
int arr[size];

Здесь ошибка. Массив нулевой длинны

Спасибо исправил, но все равно после ввода через пробел чисел программа ничего не показывает.

[8Observer8]

Неохота в вашей программе разбираться. Это значит, что надо будет её до конца продумать

Расскажу как бы я поступил. Я бы вместо статического массива использовал бы vector, а вместо С-строк - С++-строки

Уберите эту строку:

#include <QCoreApplication>

И соответсвенно эти:

QCoreApplication a(argc, argv);

return a.exec();

Можно и без них обойтись.

[8Observer8]

Вставьте этот код и убедитесь, что он работает:

#include <iostream>

int main()
{
    std::cout << "Hello, world!" << std::endl;
    return 0;
}

Вместо C-строк всегда используйте C++-строки:

#include <iostream>
#include <string>

int main()
{
    std::string str = "Hello, world!";
    std::cout << str << std::endl;
    return 0;
}

Вместо статичекого массива "int arr[size];" используйте динамический массив std::vector

#include <iostream>
#include <vector>

int main()
{
    // Fill array
    std::vector<int> arr;
    arr.push_back( 3434 );
    arr.push_back( 645 );
    arr.push_back( 6767 );
    arr.push_back( 21564 );
    arr.push_back( 4455 );

    // Show array
    for ( std::size_t i = 0 ; i < arr.size(); ++i ) {
        std::cout << arr[i] << std::endl;
    }

    return 0;
}

В vector'е можно хранить строки (да и объекты других типы):

#include <iostream>
#include <string>
#include <vector>

int main()
{
    // Fill array
    std::vector<std::string> arr;
    arr.push_back( "I" );
    arr.push_back( "love" );
    arr.push_back( "computers" );
    arr.push_back( "and" );
    arr.push_back( "programming!" );

    // Show array
    for ( std::size_t i = 0 ; i < arr.size(); ++i ) {
        std::cout << arr[i] << " ";
    }
    std::cout << std::endl;

    return 0;
}

[8Observer8]

Вот так можно ввести массив целых чисел:

Enter a number (q=quit):
> 3434
> 4532
> 454
> 56
> 7
> 65656
> 7
> 8787
> q

Array: 3434 4532 454 56 7 65656 7 8787

Press <RETURN> to close this window...

main.cpp

#include <iostream>
#include <string>
#include <vector>

std::vector<int> getArrya();
void showArray( const std::vector<int> &arr );

int main()
{
    // Fill array
    std::vector<int> arr;
    arr = getArrya();

    // Do something with array
    // ...

    // Show array
    std::cout << "\nArray: ";
    showArray( arr );
    std::cout << std::endl;

    return 0;
}

std::vector<int> getArrya()
{
    std::vector<int> arr;

    std::cout << "Enter a number (q=quit):\n";
    std::string str;
    while( true ) {
        std::cout << "> ";
        if ( !std::getline( std::cin, str ) || str == "q" ) {
            break;
        }

        int value = std::stoi( str );
        arr.push_back( value );
    }

    return arr;
}

void showArray( const std::vector<int> &arr )
{
    for ( std::size_t i = 0 ; i < arr.size(); ++i ) {
        std::cout << arr[i] << " ";
    }
    std::cout << std::endl;
}

[WorldMaster]

Жесть ... разбираться в коде значит лень ... а писать столько текста не лень .. Где логика то??

Вот правильные функции:

void vvodprot()
{
const int size=10;
int arr[size];
char str[80];
//cout << "Введiть елементи масиву через пробiл\n";
cout << "Ennter array elemnts\n";
gets(str);

int arr_len = sizeof ( arr ) / sizeof ( *arr );
vvodosnowa(str,arr,arr_len);
}


void vvodosnowa(char str[],int arr[],int &size)
{
int i=0; size=0;
while(1)
{
    while(str[i]!='\0' && str[i]!=' ')i++;
    if (str[i]=='\0')return;
    arr[size]=0;
    while(str[i]!='\0' && str[i]!=' '){
     if (str[i]!=isdigit(str[i])){
         cout << str[i] << " Not number! \n";
         return;
     }}
    arr[size] = arr[size]*10+(str[i]-48);i++;
    cout << arr [size] << "\n";
    size++;
}
}

Правда вывод почему то смущает. Наверное во второй функции где то ошибка.

[artem7902]

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