Thread: Needing assistance with pointers

You don't need to use pointer there. Your program will not even compile. You should do it this way:

Code:

#include <iostream>
#include <cstdlib>
using namespace std;
int main()
{
	char input[128];
	cout<<"Enter command> ";
	cin>>input;
	system(input);
	return 0;
}

agarwaga, it's great that you know about the std namespace, but it really doesn't help to use it if you aren't using libraries that are defined under that namespace. <iostream.h> should be <iostream>, <stdio.h> should be <cstdio>, and siavoshkc, <stdlib.h> should be <cstdlib>

With pointer:

Code:

#include <iostream>
#include <cstdlib>
using namespace std;
int main()
{
         char input[128];
         char* ptr=&input[0];
         cout<<"Enter command> ";
         cin>>ptr;
         system(ptr);
         return 0;
}

Or better:

Code:

int main()
{
          char *ptr = new char[128];
          cout<<"Enter command> ";
          cin>>ptr;
          system(ptr);
          return 0;
}

Originally Posted by siavoshkc

With pointer:

Code:

#include <iostream>
#include <cstdlib>
using namespace std;
int main()
{
         char input[128];
         char* ptr=&input[0];
         cout<<"Enter command> ";
         cin>>ptr;
         system(ptr);
         return 0;
}

Or better:

Code:

int main()
{
          char *ptr = new char[128];
          cout<<"Enter command> ";
          cin>>ptr;
          system(ptr);
          return 0;
}

better still

Code:

#include <iostream>
#include <string>
#include <cstdlib>
using namespace std;
int main()
{
         string input;
         cout<<"Enter command> ";
         cin>>input;
         system(input.c_str());
         return 0;
}

Try using pointers, ChaosEngine.

Originally Posted by siavoshkc

Try using pointers, ChaosEngine.

No. Pointers are completely unneccessary for this task. Anytime you can avoid them, you should. C++ provides several features (references, auto_ptr, vector, string) to avoid using pointers, which are notoriously error-prone.

Learn to use them.

Yes, but starter of the thread wants to practice pointers I think. Look at the posts before sending new one please.

i want to use pointers though, yes their is an easier way of doing it, but i am not challenged from that

I agree. pointer should be used in its own place, minimum required.

Originally Posted by bikr692002

Hey, I'm trying to break in to my OWN computer so I can make it more secure and develop skills but I am getting confused with pointers...

Code:

#include <iostream>
#include <windows.h>
#define cmd.exe "C;\windows\system32\cmd.exe" // improper format 1
using namespace std;
int main()
{
    char *Input[128];  // problem #2
    char Command[128];
    Input=&Command; 
    cin>>Command;
    cout<<*Input;
    Command="MSG.exe * Hi";
    system("cmd.exe"+" "+Command);  // problem #3


}

Is there also any other way I can substitute system() for winexec()?

#1
#define cmd.exe "C;\windows\system32\cmd.exe"
the . is illegal, cmd is not a structure
the ; should be :
each \ should be \\

#2
char *Input[128];
why create 128 pointers? simply
char *Input;
should suffice.

#3
system("cmd.exe"+" "+Command);
character arrays cannot be added. Use strcat() to combine them then call the function.

Other errors can be fixed after these, unless everyone else has pulled you away from fixing your original project.

There are more errors than those.