Вопрос по символьным массивам. Как улучшить код.

[Aerial]

Задание вот:

Создать массив из 20 символьных значений.  Сформировать из его значений три других массива: 
	состоящих из цифр, из букв и из символов, не являющихся ни буквами, ни цифрами.

Вот что сделал я:

#include "stdafx.h"
#include <iostream>
#include <conio.h>
#include <locale.h>
using namespace std;
int _tmain(int argc, _TCHAR* argv[])
{setlocale(LC_ALL,"Russian");
	/*Создать массив из 20 символьных значений.  Сформировать из его значений три других массива: 
	состоящих из цифр, из букв и из символов, не являющихся ни буквами, ни цифрами.*/
	const int N=20;int k=0,z=0,m=0;int i=0;
	char abvgd[N];
	char cifri_[N],bykvi_[N],none_[N];
	cout<<"Введите строку, значений символьного массива(20 элементов)"<<endl;
	cin.getline(abvgd,N);
	for(int i=0;i<N;i++)
	{
		if((abvgd[i]>='0')&&(abvgd[i]<='9'))
		{cifri_[k]=abvgd[i];k++;}
		else if(((abvgd[i]>='A')&&(abvgd[i]<='Z'))||((abvgd[i]>='a')&&(abvgd[i]<='z')))
		{bykvi_[z]=abvgd[i];z++;}
		else {none_[m]=abvgd[i];m++;}		 
	}
	char* cifri=new char[k+1];
	char* bykvi=new char[z+1];
	char* none=new char[m+1];
	for(i=0;i<k;i++)
	{
		cifri[i]=cifri_[i];
	}cifri[k]=NULL;
	for(i=0;i<z;i++)
	{
		bykvi[i]=bykvi_[i];
	}bykvi[z]=NULL;
	for(i=0;i<m;i++)
	{
		none[i]=none_[i];
	}none[m]=NULL;
	
	cout<<"Мы получили 3 массива, массив из цифер: "<<endl<<cifri<<endl<<"Массив из букв и символов: "<<endl<<bykvi<<endl;
	cout<<"Массив из остальных знаков: "<<none;
	delete[]cifri;delete[]bykvi;delete[]none;
	_getch();
	return 0;
}

Меня интересует как можно усовершенствовать код в целом. И\или как более эффективно\лучше\точнее задать сортировку элементов массива:

for(int i=0;i<N;i++)
	{
		if((abvgd[i]>='0')&&(abvgd[i]<='9'))
		{cifri_[k]=abvgd[i];k++;}
		else if(((abvgd[i]>='A')&&(abvgd[i]<='Z'))||((abvgd[i]>='a')&&(abvgd[i]<='z')))
		{bykvi_[z]=abvgd[i];z++;}
		else {none_[m]=abvgd[i];m++;}		 
	}

p.s. если что Visual Studio 2008

[NiCola999]

ну главное работает ?=)), код ужасен, а так вроде ничего
алгоритм такой:
A[20]
1)посчитать кол-во цифр,букв и символов в массиве A
2)создать 3 массива
3)записать туда в них что надо

у тебя как раз так и сделано

замечания: лишние скобки в ифах

вот это лишнее:

for(i=0;i<k;i++)
	{
		cifri[i]=cifri_[i];
	}cifri[k]=NULL;
	for(i=0;i<z;i++)
	{
		bykvi[i]=bykvi_[i];
	}bykvi[z]=NULL;
	for(i=0;i<m;i++)
	{
		none[i]=none_[i];
	}none[m]=NULL;

у тебя массивы

 char cifri_[N],bykvi_[N],none_[N];

после окончания работы сами удалятся
да и вообще эти массивы не нужны вовсе

а очистку динамических массивов ты делаешь с помощью delete

еще не понятно зачем ты создаешь массивы размером на 1 больше, чем надо:

char* cifri=new char[k+1];

вот как бы написал я:

//массив уже заполненный N=20
int m=0, n=0, k=0; // кол-во букв цифр и символов
for(int i=0; i<N; i++) {
     if ( A[i] >= '0' && A[i] <= '9' ) m++;
     else if( (A[i] >= 'A' && A[i] <= 'Z' ) || (A[i] >= 'z' && A[i] <= 'z' ) n++;
     else k++;
}
char *digits = new char[m];
char *letters = new char[n];
char *symbols = new char[k];
m=0; n=0; k=0;
for(int i=0; i<N; i++) {
     if ( A[i] >= '0' && A[i] <= '9' )
            digits[m++] = A[i];
     else if( (A[i] >= 'A' && A[i] <= 'Z' ) || (A[i] >= 'z' && A[i] <= 'z' )
            letters[n++] = A[i];
     else
           symbols[k++] = A[i];
}
// вывод масивов или еще че-нить
...
delete[] digits;  delete[] letters; delete[] symbols;

не проверял, писал тут

[Aerial]

Сообщение от NiCola999

ну главное работает ?=)), код ужасен, а так вроде ничего

Главное что код ужасен;( То, что работает хорошо, но хотелось бы лучше написать код.
До алгоритма в принципе догадаться способен, а исполнение подводит.

вот это лишнее:

for(i=0;i<k;i++)
	{
		cifri[i]=cifri_[i];
	}cifri[k]=NULL;
	for(i=0;i<z;i++)
	{
		bykvi[i]=bykvi_[i];
	}bykvi[z]=NULL;
	for(i=0;i<m;i++)
	{
		none[i]=none_[i];
	}none[m]=NULL;

Этот кусок сделал, чтобы не было "лишних" пустых-чтоли, элементов.

еще не понятно зачем ты создаешь массивы размером на 1 больше, чем надо:

char* cifri=new char[k+1];

Это от незнания наверно А вообще для нулевого элемента:

cifri[k]=NULL;
bykvi[z]=NULL;
none[m]=NULL;

Очень интересует как можно улучшить:

for(int i=0;i<N;i++)
	{
		if((abvgd[i]>='0')&&(abvgd[i]<='9'))
		{cifri_[k]=abvgd[i];k++;}
		else if(((abvgd[i]>='A')&&(abvgd[i]<='Z'))||((abvgd[i]>='a')&&(abvgd[i]<='z')))
		{bykvi_[z]=abvgd[i];z++;}
		else {none_[m]=abvgd[i];m++;}
	}

Сообщение от NiCola999

вот как бы написал я:

//массив уже заполненный N=20
int m=0, n=0, k=0; // кол-во букв цифр и символов
for(int i=0; i<N; i++) {
     if ( A[i] >= '0' && A[i] <= '9' ) m++;
     else if( (A[i] >= 'A' && A[i] <= 'Z' ) || (A[i] >= 'z' && A[i] <= 'z' ) n++;
     else k++;
}
char *digits = new char[m];
char *letters = new char[n];
char *symbols = new char[k];
m=0; n=0; k=0;
for(int i=0; i<N; i++) {
     if ( A[i] >= '0' && A[i] <= '9' )
            digits[m++] = A[i];
     else if( (A[i] >= 'A' && A[i] <= 'Z' ) || (A[i] >= 'z' && A[i] <= 'z' )
            letters[n++] = A[i];
     else
           symbols[k++] = A[i];
}
// вывод масивов или еще че-нить
...
delete[] digits;  delete[] letters; delete[] symbols;

не проверял, писал тут

Спасибо, приму к сведению. Интересует вопрос: стоит\нужно добавлять нулевой элемент в конец массива?
И, если не трудно приведите пример(алгоритм\команды) как можно предотвратить перезапись последующих элементов символьного массива в коде, если ввожу элементы посимвольно?

.......
char A[N];
.....
for(int i=0;i<N;i++)
{cin<<A[i];}
....

Что-то вроде того.

Сообщение от NiCola999

для какого нулевого элемента? у тебя k - кол-во цифр которое надо записать в массив, индексы в нем будут от 0 до k-1
можно както проверку букв упростить , но чет бошка щас не варит

Что-то из серии "Далее вы узнаете, что в C++ символьные строки хранятся в массиве типа char, который заканчивается символом NULL (или ASCII 0)."
Была мысль с использованием ASCII(возможно ли?стоит ли?), но как, не представляю.

[NiCola999]

пардон... уже не соображаю. Забыл что у вас char, тогда там правильно k+1 итд. Нулевой символ добавлять не нужно, компилятор сам должен это сделать поидее...

можно както проверку букв упростить , но чет бошка щас не варит

[Aerial]

NiCola999, спасибо.
Хотелось бы узнать ответы на след. вопросы, есди не сильно затруднит.

Очень интересует как можно улучшить:

for(int i=0;i<N;i++)
	{
		if((abvgd[i]>='0')&&(abvgd[i]<='9'))
		{cifri_[k]=abvgd[i];k++;}
		else if(((abvgd[i]>='A')&&(abvgd[i]<='Z'))||((abvgd[i]>='a')&&(abvgd[i]<='z')))
		{bykvi_[z]=abvgd[i];z++;}
		else {none_[m]=abvgd[i];m++;}
	}

И, если не трудно приведите пример(алгоритм\команды) как можно предотвратить перезапись последующих элементов символьного массива в коде, если ввожу элементы посимвольно?

.......
char A[N];
.....
for(int i=0;i<N;i++)
{cin<<A[i];}
....

Что-то вроде того.

Была мысль упростить\улучшить проверку букв\символов\цифер с использованием ASCII(возможно ли?стоит ли?), но как, не представляю.