Thread: Need a little help with my project

First of all, I'd like to introduce myself to the C-Programming.com boards. I'm taking CS in college and not going to class for the summer made me rusty. If anyone can provide me with some insight with why I get errors when building this program, it would be helpful.

Code:

#include <cstring>
#include <iostream>
#include <cmath>
using namespace std;

void encode(char[] ,char[] ,char[] );

void main()
{
	char table[5][10];
	char statement[50];
	char keyword[10];

	cout<<"Please enter your statement,max 50 characters, all lowercase, with no spaces."<<endl;
	cin.get(statement,50);
	cout<<"Please enter your keyword. Do not repeat any letters.";
	cin>>keyword;
	
	cout<<"Our original statement is "<<statement<<endl;
	
	encode(statement,keyword,table);

	for (int y=0;y<strlen(keyword);y++)
		{				
		for (int x=0;x<ceil(strlen(statement)/strlen(keyword))+1;x++)
			{
			cout<<table[x][y];
			}
		cout<<' ';		
		}										
		
	cout<<endl<<endl;
	
}

void encode(char statement[], char keyword[], char table[])
{
	int i=0;
	int keylength = strlen(keyword);
	int messagelength = strlen(statement);

	for(int x=0;x<ceil(messagelength/keylength)+1;x++)		  
		for(int y=0;y<keylength;y++)	
		{
			table[x][y] = statement[i];
			i++;
		}
}

Thanks in advance. ^_^

All right, I changed my function header to: void encode(char statement[], char keyword[], char table[5][])
and my function call to: encode(statement,keyword,table[5]);

however, this gives me another error:
'<Unknown>' Missing Subscript
and points to my function header before my function code.

Once again I am lost, and the way things are going, I'm going to be here alot more often this semester.

I apologize for my stupidity, but...wha?

My new semester's resolution is not to take the summer off of using C++.

Anyway, you lost me there.

Code:

#include <cstring>
#include <iostream>
#include <cmath>
using namespace std;

void encode(char[] ,char[] ,char**);

int main()
{
	char table[5][10];
	char statement[50];
	char keyword[10];

	cout<<"Please enter your statement,max 50 characters, all lowercase, with no spaces."<<endl;
	cin.get(statement,50);
	cout<<"Please enter your keyword. Do not repeat any letters.";
	cin>>keyword;
	
	cout<<"Our original statement is "<<statement<<endl;
	
	encode(statement,keyword,table[5]);

	for (int y=0;y<strlen(keyword);y++)
		{				
		for (int x=0;x<ceil(strlen(statement)/strlen(keyword))+1;x++)
			{
			cout<<table[x][y];
			}
		cout<<' ';		
		}										
		
	cout<<endl<<endl;

	return 0;
}

void encode(char statement[], char keyword[], char table[5][])
{
	int i=0;
	int keylength = strlen(keyword);
	int messagelength = strlen(statement);

	for(int x=0;x<ceil(messagelength/keylength)+1;x++)		  
		for(int y=0;y<keylength;y++)	
		{
			table[x][y] = statement[i];
			i++;
		}
}

That is exactly what I have right now.

Once again I apologize for my stupidity. Once I re-familirize myself with the language, I won't have any lapses like that.

I want to pass the entire 2D array "table" to the function.

What the function does is take the message, insert it into the table in row-major order. After this, the message is printed in column major order. The actual keyword is arbitrary, because the important part is the length of the keyword, as we do not have to sort the array. (originally, we had to sort the array and print it based on the alphabetical order of the letters in the keyword, but too many people were having trouble implementing this.)

The weird symbols are because you don't initialize the contents of table and you don't fill it up all the way in encode. Your statement is 24 characters and your key is seven characters. So you have seven sets of letters and you add a letter to each set until you have added all 24 letters, but you leave the last four spots open. When you output the table you are outputting four letters per set, regardless of how many letters have been added to that set.

You could initialize table in the beginning using memset. If you set it to all 0's then you can check for a 0 in the table and stop outputting when you reach it. Or you could add a counter to stop outputting when the message length is reached. Or you could initialize to the ' ' (space) character and it will output spaces instead of gobbledy-gook.