Dev-C++ Программа с массивами

@Аника · Регистрация: 22.12.2012

Author24 — интернет-сервис помощи студентам

По неработающему образцу (данному учителем) написала программу. Как следствие: прога тоже нерабочая вышла.
Образец (Пример ввода 5 целочисленных значений):

C++

#include <conio.h>
#include <iostream.h>
void main()
{
clrscr();
const int n=4;
int a[n]={0,};
for (int i=0;i<n;i++);
{
cout<<"vvedite"<<i<<"element: ";
cin>>a[i];
}
for (int i=0;i<n;i++)cout<<a[i]<<" ";
cin.get();
}

Задание для написания проги: задан массив с количеством элементов N. Сформировать 2 массива: в первый включить элементы исходного массива с четными номерами, а во второй - с нечетными.
Массив вводит пользователь.
Программа: (найдите 10 ошибок

)

C++

#include <cstdlib>
#include <iostream>
#include <conio>
 
using namespace std;
void main();           //компилятор ругается
int main(int argc, char *argv[])
{
    int a,b,c;
    const int n=100; //компилятор ругается
    clrscr();       //компилятор ругается
    int a[n]={0,};  //компилятор ругается
    for (int i=0;i<n;i+=2);
    b[i/2]=a[i];
    for (int i=1;i<n;i+=2);
    c[i/2]=a[i];   
{
    cout<<"vvedite"<<i<<"element: ";
    cin>>a[i];
}
    for (int i=0;i<n;n/2)cout<<b[i]<<" "<<c[i]<<endl;
    cin.get();
}

@Folko · 06.11.2013, 00:29

Аника, ну правильно. n - константа, означающая размер массива. Поэтому вначале объявите ее const int n = 10; а затем объявляйте массивы. Вот как то так)

ValeryS · 06.11.2013, 00:32

[quote="Аника;5292392"]Думаю, что точно не так:
там же где описывала массив а
того же размера
хотя нет размер можно в половину сократить плюс один элемент для нечетного размера

C++

1
2
3

int a[n];  //обнулять не надо все равно в него данные будут вводится
int b[n/2+1];
int c[n/2+1];

Добавлено через 2 минуты
вот как выглядит начало твоей программы

C++

int main(int argc, char *argv[])
{
 const int n=100; //не надоест сто чисел вручную вводить?
int a[n];  //обнулять не надо все равно в него данные будут вводится
int b[n/2+1];
int c[n/2+1];

теперь перепиши все с учетом замечаний и будем дальше ошибки искать

@Аника · 06.11.2013, 00:36 **[ТС]**

Вышло так:

C++

#include <cstdlib>
#include <iostream>
 
using namespace std;
int main(int argc, char *argv[])
{
    const int n=100;
    int a[n];
    int b[n/2+1];
    int c[n/2+1];
    cout<<"vvedite"<<i<<"element: ";
    cin>>a[i];
    for (int i=0;i<n;i+=2)
    b[i/2]=a[i];
    for (int i=1;i<n;i+=2)
    c[i/2]=a[i];   
    for (int i=0;n/2)cout<<b[i]<<" "<<c[i]<<endl;
    cin.get();
}

Выдает на 11 строке:
`i' undeclared (first use this function)
Следующий вопрос: где и как описать i?

@Folko · 06.11.2013, 00:36

Аника, у вас потому что там нет цикла
Перед cout<<"vvedite"<<i<<"element: "; добавьте for (int i = 0; i < n; i++)
А, и не забудьте фигурными скобками захватить cout и следующий за ним cin
Типа так

C++

for (int i = 0; i < n; i++)
{
cout<<"vvedite"<<i<<"element: ";
cin>>a[i];
}

@Аника · 06.11.2013, 00:39 **[ТС]**

Folko, т.е. должно быть так?:

C++

{
    cout<<"vvedite"<<i<<"element: ";
    cin>>a[i];
}

ValeryS · 06.11.2013, 00:40

Сообщение от Аника

cout<<"vvedite"<<i<<"element: ";
* * cin>>a[i];

Сообщение от Аника

for (int i=0;i<n;i++);
{
cout<<"vvedite"<<i<<"element: ";
cin>>a[i];
}

есть разница?

@Folko · 06.11.2013, 00:41

Аника, вот так

C++

for (int i = 0; i < n; i++)
{
cout<<"vvedite"<<i<<"element: ";
cin>>a[i];
}

вам нужно пройтись по всем индексам (т.е ячейкам). Для этого и цикл. А скобки что бы на каждый шаг выводить и вводить данные

@Аника · 06.11.2013, 00:41 **[ТС]**

name lookup of `i' changed for new ISO `for' scoping
выдает на строку
cin>>a[i];

ValeryS · 06.11.2013, 00:41

Сообщение от Аника

const int n=100;

уменьшь число хотя бы до 10
пожалей пользователя сто чисел вводить

@Folko · 06.11.2013, 00:43

Аника, а ну ка покажите теперь код.

Не по теме:

И да, не издевайтесь так над пользователями, они же устанут после сотни enter'ов

@Аника · 06.11.2013, 00:46 **[ТС]**

C++

#include <cstdlib>
#include <iostream>
 
using namespace std;
int main(int argc, char *argv[])
{
    const int n=10;
    int a[n];
    int b[n/2+1];
    int c[n/2+1];
    for (int i = 0; i < n; i++)
{
    cout<<"vvedite"<<i<<"element: ";
    cin>>a[i];
}
    for (int i=0;i<n;i+=2)
    b[i/2]=a[i];
    for (int i=1;i<n;i+=2)
    c[i/2]=a[i];   
    for (int i=0;n/2);
    cout<<b[i]<<" "<<c[i]<<endl;
    cin.get();
}

Осталась проблема с выводом.
Строка 20: expected `;' before ')' token

@Folko · 06.11.2013, 00:47

Аника, for (int i=0;n/2); после цикла не ставьте ;, он ведь что то делает
И что за цикл такой, неизменяющий i?

C++

1 2	for (int i=0;i<n/2;i++) cout<<b[i]<<" "<<c[i]<<endl;

Видимо так должно быть... я не вникал в смысл кода

ValeryS · 06.11.2013, 00:49

Сообщение от Аника

for (int i=0;n/2);

опять точка с запятой после for
нет третьего элемента заголовка цикла
сравни

Сообщение от Аника

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

торопишься наверное

@Аника · 06.11.2013, 00:53 **[ТС]**

C++

#include <cstdlib>
#include <iostream>
 
using namespace std;
int main(int argc, char *argv[])
{
    const int n=10;
    int a[n];
    int b[n/2+1];
    int c[n/2+1];
    for (int i=0;i<n;i++)
{
    cout<<"vvedite"<<i<<"element: ";
    cin>>a[i];
}
    for (int i=0;i<n;i+=2)
    b[i/2]=a[i];
    for (int i=1;i<n;i+=2)
    c[i/2]=a[i];   
    for (int i=0;i<n;n/2)
    cout<<b[i]<<c[i]<<endl;
    cin.get();
}

Программа зациклилась.

@Folko · 06.11.2013, 00:54

Аника, for (int i=0;i<n;n/2)
i должна меняться. А тут у вас ничего не меняется. Я же вам написал правильный вариант

@Аника · 06.11.2013, 00:57 **[ТС]**

Теперь вообще выходит безо всяких действий.
Ввела 10 чисел, и до свидания.

ValeryS · 06.11.2013, 00:58

Сообщение от Аника

for (int i=0;i<n;n/2)
* * cout<<b[i]<<c[i]<<endl;

даже если исправишь цикл получишь на экране тот же массив
b[i]<<c[i] идут друг за другом
я бы вот так сделал

C++

cout<<"array B"<<endl;
for(int i=0;i<n/2;i++)
   cout<<b[i]<<" ";
cout<<endl<<"array C"<<endl;
for(int i=0;i<n/2;i++)
   cout<<c[i]<<" ";
  cin.get();

@Folko · 06.11.2013, 00:58

Аника, ну вот же...

C++

1 2	for (int i=0;i<n/2;i++) cout<<b[i]<<" "<<c[i]<<endl;

вам как раз надо вывести массивы с размерами половина n

ValeryS · 06.11.2013, 01:01

Folko,
на экране исходный массив получишь
почему смотри выше

@Folko · 06.11.2013, 01:03

ValeryS, я только исправил ее код. Если делать как лучше, там само собой лучше раздельно вывести массивы, что бы не было путаницы

@Folko 267 / 255 / 27 Регистрация: 27.09.2013 Сообщений: 876 Записей в блоге: 1
	06.11.2013, 00:29	21
	Аника, ну правильно. n - константа, означающая размер массива. Поэтому вначале объявите ее const int n = 10; а затем объявляйте массивы. Вот как то так) 0

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,523
	06.11.2013, 00:40	26
	Сообщение от Аника cout<<"vvedite"<<i<<"element: "; * * cin>>a[i]; Сообщение от Аника for (int i=0;i<n;i++); { cout<<"vvedite"<<i<<"element: "; cin>>a[i]; } есть разница? 0

@Аника 1 / 1 / 0 Регистрация: 22.12.2012 Сообщений: 160
	06.11.2013, 00:41 [ТС]	28
	name lookup of `i' changed for new ISO `for' scoping выдает на строку cin>>a[i]; 0

@Folko 267 / 255 / 27 Регистрация: 27.09.2013 Сообщений: 876 Записей в блоге: 1
	06.11.2013, 00:43	30
	Аника, а ну ка покажите теперь код. Не по теме: И да, не издевайтесь так над пользователями, они же устанут после сотни enter'ов 0

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,523
	06.11.2013, 00:49	33
	Сообщение от Аника for (int i=0;n/2); опять точка с запятой после for нет третьего элемента заголовка цикла сравни Сообщение от Аника for (int i = 0; i < n; i++) торопишься наверное 1

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,523
	06.11.2013, 00:41	29
	Сообщение от Аника const int n=100; уменьшь число хотя бы до 10 пожалей пользователя сто чисел вводить 0

@Folko 267 / 255 / 27 Регистрация: 27.09.2013 Сообщений: 876 Записей в блоге: 1
	06.11.2013, 00:54	35
	Аника, for (int i=0;i<n;n/2) i должна меняться. А тут у вас ничего не меняется. Я же вам написал правильный вариант 0

@Аника 1 / 1 / 0 Регистрация: 22.12.2012 Сообщений: 160
	06.11.2013, 00:57 [ТС]	36
	Теперь вообще выходит безо всяких действий. Ввела 10 чисел, и до свидания. 0

ValeryS Модератор 8908 / 6677 / 918 Регистрация: 14.02.2011 Сообщений: 23,523
	06.11.2013, 01:01	39
	Folko, на экране исходный массив получишь почему смотри выше 0

@Folko 267 / 255 / 27 Регистрация: 27.09.2013 Сообщений: 876 Записей в блоге: 1
	06.11.2013, 01:03	40
	ValeryS, я только исправил ее код. Если делать как лучше, там само собой лучше раздельно вывести массивы, что бы не было путаницы 0