Подсчет негативных элементов массива преобразованных из строки

[coNsept]

#Всем доброго времени суточек, уважаемые пользователи данного форума.
.So, I have some trouble with string conversation

В структуре маю следующий массив:

 struct VectorStruct {
	char *Vector[100]; 
	int n;
}; VectorStruct Index;

Здесь суммирую негативные элементы, но меня шлют далеко и на долго, мол секция записи "access violation when read":

int Sum = 0;
for (int i = 0; i < *pSize; i++)
{
    if (atoi(*Table[i].Vector) < 0)
    {
        Sum+= atoi(*Table[i].Vector);
    }
}

p.s. По идее все смотреться неплохо, в чем может быть проблема?

[Stilet]

char *Vector[100];

Это массив строк, или тут подразумевается строка из 100 символов?

[blob]

Выложи кусок кода побольше.
Надо полагать, что VectorStruct у тебя хранит одно число в формате строки, так ? Что за структура у Table ? Думаю ответ родится сразу, как только пояснишь

[coNsept]

Это массив строк, или тут подразумевается строка из 100 символов?

Как массив строк.

Весь код:

// Main.cpp : Defines the entry point for the console application.
//

#include "stdafx.h"
#include <stdio.h>
#include <conio.h>
#include <windows.h>
#include <iostream>
#include <string>
#include <string.h>

struct VectorStruct {
	char *Vector[100]; 
	int n;
}; VectorStruct Index;

VectorStruct Access;
VectorStruct Table[100];

int Size, *pSize;

void Interface()
{
	printf("--------------------------------------------------------------------------------");
	printf("\t\t\t   - Operatiile cu vector -\n");
	printf("--------------------------------------------------------------------------------");
	printf("[1].#Initializarea\n");
	printf("[2].#Eliminarea\n");
	printf("[3].#Modificarea dimensiunii\n");
	printf("[4].#Acces la elementele\n");
	printf("[5].#Calcul produsului elementelor negative\n");
	printf("[6].#Afisarea\n");
	printf("[7].#Iesire din program\n");
	printf("--------------------------------------------------------------------------------");
}

void scanVector(VectorStruct Table[])
{ 
	for (int i = 0; i < *pSize; i++)
	{
		printf("[%d]: ", i + 1); 
		scanf("%s", &Table[i].Vector);
	}
	Access.n = *pSize;
}

void printVector(VectorStruct Table[])
{
	for (int i = 0; i < *pSize; i++)
	{
		printf("[%s]", Table[i].Vector);
	}
}

int _tmain(int argc, _TCHAR* argv[])
{
	bool Step = TRUE, Select = TRUE;

	Access.n = 0;

	while (Step)
	{
		Interface();

		switch (_getch())
		{
		case '1':
			{ 
				if (Select)
				{
					printf("\nIntroduceti dimensiunea: "); scanf("%d", &Size); printf("\n"); pSize = &Size; 
					// ----
					scanVector(&Table[0]);
					// ----
					Select = FALSE;
				}

				printf("\n");
				// ----
				printVector(&Table[0]);
				// ----
				printf("\n\n"); system("PAUSE"); system("CLS");

			} break;

		case '2':
			{
				printf("\n");
				for (int i = 0; i < *pSize; i++)
				{
					memset(Table[i].Vector, 0, *pSize); printf("[%s]", Table[i].Vector); 
				} 
				Select = TRUE;
				printf("\n\n"); system("PAUSE"); system("CLS");
			} break;

		case '3':
			{
				int TempSz;

				printf("\n");
				printf("Introduceti dimensiunea: "); scanf("%d", &TempSz); printf("\n"); *pSize = TempSz;	
				// ----
				for (int i = Access.n; i < *pSize; i++)
				{
					printf("[%d]: ", i + 1); 
					scanf("%s", &Table[i].Vector);			
				}
				printf("\n");
				// ----
				printVector(&Table[0]);
				// ----
				printf("\n\n"); system("PAUSE"); system("CLS");
			} break;

		case '4':
			{
				int Element;

				printf("\nIntroduceti i - numarul elemntului\n");
				printf("\n -> "); scanf("%d", &Element);

				for (int i = 0; i < *pSize; i++)
				{
					if (i == Element)
					{ printf("\nIntroduceti valoare noua: "); scanf("%s", &Table[i].Vector); }
				}

				printf("\n");
				// ----
				printVector(&Table[0]);
				// ----

				printf("\n\n"); system("PAUSE"); system("CLS");
			} break;

		case '5':
			{
				printf("\n");

				int Sum = 0;

				for (int i = 0; i < *pSize; i++)
				{
					if (atoi(Access.Vector[i]) < 0)
					{
						Sum+= atoi(Access.Vector[i]);
					}
				}

				printf("\n Suma elementelor negative = %d", Sum);

				printf("\n\n"); system("PAUSE"); system("CLS");
			} break;

		case '6':
			{
				printf("\n"); printVector(&Table[0]); printf("\n\n"); system("PAUSE"); system("CLS");
			} break;

		case '7':
			{
				ExitProcess(0); // sau Step = FALSE;
			} break;

		default: system("CLS"); MessageBoxA(NULL, "Ati apasat butonul gresit!", "#Mesaj", MB_ICONWARNING); 
		}
	}
	printf("\n"); system("PAUSE"); return 0;
}

5-ый кейс

[pproger]

2coNsept
код очень страшный. чего тока стоит эта часть

scanf("%s", &Table[i].Vector);

в данном случае scanf ожидает вторым параметром char *, а ты ей передаешь указатель на массив из 100 элементов типа char *. сюда же, нигде не увидел, где ты выделяешь память под эти строки. жееесть, просто жесть.
пс. чо за итальянский?
ппс. лучше напиши, что должно быть в итоге

[coNsept]

Хм, а разве под статичный строковый массив нужно выделять память?
У меня просто такое понятие, если char* тогда выделяю память, если массив статичный к примеру char* [], тогда я этого не делаю. (Точнее сказать я даже и не в курсе был нужно ли это, так как и так вижу сколько возможно использовать элементов, не изучил я еще всех тонкостей).
Кстати, буду рад любой критики кода, жду исправлений и пояснений дабы в следующий раз писать код более правильным и элегантным. За ранее благодарен.

ппс. лучше напиши, что должно быть в итоге

Прочти тему топика. (А дальше я сам все исправлю и сделаю что нужно)

[pproger]

2coNsept
это не строковый массив. у тебя массив из 100 указателей на тип char. он будет строковым массивом (или двумерным символьным массивом, как угодно), если ты выделишь для каждого элемента память под строку. пока у тебя кучка указателей ни на что.
код критиковать не буду, очень уж я пылкий на критику, понимаешь. могу посоветовать только пользоваться нормальным компилятором, собирать свои программы с высоким уровнем показа варнингов и внимательно их читать.
этому коду поможет только ggdG

[coNsept]

Такой вариант по лучше, на взгляд?

// Main.cpp : Defines the entry point for the console application.
//

#include "stdafx.h"
#include <stdio.h>
#include <conio.h>
#include <windows.h>
#include <iostream>
#include <fstream>

int Size = 0, RecordSz, *pSize;

struct VectorStruct 
{ 
	// ----
	char **Vector;
	// ----

	void Initialize(char InputString[], int Counter)
	{
		Vector[Counter] = new char[strlen(InputString) + 1];
		strcpy(Vector[Counter], InputString);
	}

    void FreeMemory()
	{
		for (int i = 0; i < *pSize; i++)
		{
			delete [] Vector[i]; Vector[i] = NULL;
		}
	}
};

void Interface();
void printVector(VectorStruct p);
int VectorProd(VectorStruct p);

void main(void)
{
	// ----
	char InputString[100];
	bool Step = TRUE;
	VectorStruct Index;
	// ----

	pSize = &Size;

	while (Step)
	{
		system("CLS");
		// ----------
		Interface();
		// ----------

		switch (_getch())
		{
		case '1':
			{
				printf("\n");
				printf("Introduceti dimensiunea: "); scanf("%d", &Size); RecordSz = Size; pSize = &Size; 		
				printf("\n");

				for (int i = 0; i < *pSize; i++)
				{
					printf(" -> "); scanf("%s", &InputString);
					// ----------
					Index.Initialize(InputString, i);
					// ----------
				}

				printf("\n");

				// ----------
				printVector(Index);
				// ----------
			} break;

		case '2':
			{
				// ----------
				Index.FreeMemory();
				// ----------
				printf("\n");
				// ----------
				printVector(Index);
				// ----------
			} break;

		case '3':
			{
				int NewSize;

				printf("\n");
				printf("Introduceti dimensiunea: "); scanf("%d", &NewSize); pSize = &NewSize;
				printf("\n");

				for (int i = RecordSz; i < NewSize; i++)
				{
					printf(" -> "); scanf("%s", &InputString);
					// ----------
					Index.Initialize(InputString, i);
					// ----------
				}
				
				printf("\n");
				   
				// ----------
				printVector(Index);
				// ----------

			} break;

		case '4':
			{
				// ----
				int Select; 
				char ChangeString[100];
				// ----

				printf("\n");
				printf("(Total elemente %d) | Alegeti element: ", *pSize); scanf("%d", &Select);
				printf("\n");

				if (Select > *pSize)
				{ MessageBoxA(NULL, "Asa element nu exista", "Eroare!", MB_ICONWARNING); break; }

				for (int i = 0; i < *pSize; i++)
				{
					if (Select == i)
					{
						printf(" -> "); scanf("%s", &ChangeString);
						// ----------
						Index.Initialize(ChangeString, i);
						// ----------
					}
				}   
				// ----------
				printVector(Index);
				// ---------- 

			} break;

		case '5':
			{
				printf("\nProdusul elementelor negative = %d", VectorProd(Index)); 

			} break;

		case VK_ESCAPE:
			{
				ExitProcess(0);
			} break;

		default: MessageBoxA(NULL, "Datele introduse sunt incorecte", "Eroare!", MB_ICONWARNING);
		}
		_getch(); 
	} 

	printf("\n"); system("PAUSE");
}

void Interface()
{
	printf("================================================================================");
	printf("                                  [VECTOR]\n");
	printf("================================================================================");
	printf("#--1 : Initializare\n");
	printf("#--2 : Stergere\n");
	printf("#--3 : Modificarea dimensiunii\n");
	printf("#--4 : Acces/modificarea elementelor\n");
	printf("#--5 : Calcularea produsului elementelor negative\n");
	printf("#ESC : Iesire din program\n");
	printf("================================================================================");
}

void printVector(VectorStruct p)
{
	for (int i = 0; i < *pSize; i++) 
	{ 
		printf("[%s]", p.Vector[i]); 
	} 
}

int VectorProd(VectorStruct p)
{
	int Sum = 1;

	for (int i = 0; i < *pSize; i++)
	{
		if (atoi(p.Vector[i]) < 0)
		{
			Sum*= atoi(p.Vector[i]);
		}
	}
	return Sum;
}

[pproger]

2coNsept

Такой вариант по лучше, на взгляд?

неа

	char **Vector;
	// ----

	void Initialize(char InputString[], int Counter)
	{
		Vector[Counter] = new char[strlen(InputString) + 1];
		strcpy(Vector[Counter], InputString);
	}

ты используешь указатель на указатель, но память под этот массив указателей (на который должен указывать твой указатель на указатель) не выделяешь. а выделяешь сразу память под указатель)
т.е уже при обращении Vector[Counter] твоя прога должна упасть. может в дебаге ты и выживешь. но врядли
ты же понимаешь, что это равносильно *(Vector + Counter)? и если по этому смещению лежит не "что-то из мусора", а 0, упадет. ибо нулевой указатель нельзя разыменовывать)

"аписняю".

char *c; указатель на тип char. можно выделить для него память, как c = new char[128]; и он будет указывать на память длиной 128 байт. тут все тривиально.
далее. char **c. указатель на указатель. чтобы его использовать, необъодимо выделить память под обычные указатели:
c = new char*[10]; получили в памяти массив из 10-указателей. далее уже для этих указателей можно выделить память, где можно хранить данные

for (int i = 0; i < 10; i++)
c[i] = new char[128];

для 10 указателей выделяем память из 128 байт. и вот только теперь память можно заполнять. например
strcpy(c[2], "string");

теперь сравни со своим кодом.

оставил бы
char *Vector[100];
как в исходном варианте, твоя инициализация (дальше не смотрел) была бы правильная. но тут у тебя массив указателей динамический

короче, k&r до посинения и последующего просветления)

сюда же, структуры передаешь по значению. трагеееедияяяя. но объективно да, вариант можно считать "получше"

вот прогнал то я тираду....