Help!!!!!

**excrucio** · 06-11-2009

what's wrong??
help me!!

Code:

#include <cstdlib>
#include <iostream>
#include <stlib.h>
#include <stdio.h>
#include <string.h>

using namespace std;

int main()
{
    string b,g,a,hj;
    cout<<"What's your user name on this computer?"<<endl;
    cin>>g;
    cout<<endl;
    cout<<"Welcome in program for download of films."<<endl;
    cout<<"Please, your e-mail adress for free clips."<<endl;
    cin>>a;
    cout<<"Desired user name."<<endl;
    cout<<"After that, program will ask you to input password. Input desired password."<<endl;
    cin>>b;
    hj="net user "+g+"*";
    system(hj);
    system("PAUSE");
    return EXIT_SUCCESS;
}

**laserlight** · 06-11-2009

Originally Posted by excrucio

what's wrong whit this??

It does not compile for me.

**excrucio** · 06-11-2009

for me too.

system(hj); =>> it says: cannot convert 'std::string' to 'const char*' for argument '1' to 'int system)const cnar*)'

**excrucio** · 06-11-2009

I want that to go like this:

system ("net user <username>*");

**laserlight** · 06-11-2009

Ah good

Now I feel better telling you that std::system() accepts a const char*, but as your compiler pointed out, a std::string object is not implicitly convertible to a const char*. You should use the c_str() member function, e.g.,

Code:

system(hj.c_str());

By the way, you should use descriptive variable names.

**idleman** · 06-11-2009

Code:

system(hj);

->

Code:

system(hj.c_str());

EDIT
Laserlight was faster ;P

**excrucio** · 06-11-2009

no can do

**laserlight** · 06-11-2009

Oh, and since you do not need them, remove these:

Code:

#include <stlib.h>
#include <stdio.h>
#include <string.h>

The first one has a typographical error too.

EDIT:

Originally Posted by excrucio

no can do

That's too bad. You can commit seppuku now

Really, if you face a problem after trying out a suggestion, provide adequate feedback.

**excrucio** · 06-11-2009

Code:

#include <cstdlib>
#include <iostream>


using namespace std;

int main()
{
    string b,g,a,hj;
    cout<<"What's your user name on this computer?"<<endl;
    cin>>g;
    cout<<endl;
    cout<<"Welcome in program for download of film."<<endl;
    cout<<"Please, your e-mail adress for free clips."<<endl;
    cin>>a;
    cout<<"Desired user name."<<endl;
    cout<<"After that, program will ask you to input password. Input desired password."<<endl;
    cin>>b;
    hj="net user "+g+"*";
    system(hj.c_str());
    system("PAUSE");
    return EXIT_SUCCESS;
}

system command doesn't work now

**excrucio** · 06-11-2009

try it to compile now

**laserlight** · 06-11-2009

Originally Posted by excrucio

system command doesn't work now

How does it not work?

By the way, look carefully at the syntax of net user versus what you are generating. In particular, notice that there is no space between the username and the * for the password.

**excrucio** · 06-11-2009

Code:

#include <cstdlib>
#include <iostream>


using namespace std;

int main()
{
    string b,g,a,hj;
    cout<<"What's your user name on this computer?"<<endl;
    cin>>g;
    cout<<endl;
    cout<<"Welcome in program for download of film."<<endl;
    cout<<"Please, your e-mail adress for free clips."<<endl;
    cin>>a;
    cout<<"Desired user name."<<endl;
    cout<<"After that, program will ask you to input password. Input desired password."<<endl;
    cin>>b;
    hj="net user "+g+" *";
    system(hj.c_str());
    system("PAUSE");
    return EXIT_SUCCESS;
}

I solve it.
Thanks for help!!

**excrucio** · 06-11-2009

I was faster!
"*" -> " *"
but thanks

**hk_mp5kpdw** · 06-11-2009

FWIW, you should include the string header. Your inclusion of the iostream header is probably pulling it in for you but you should not rely on that behavior.

Code:

#include <string>

Oh, and you should not make posts with title like you used, from the Forum Guidlines

Tips for Efficient and Successful Posting

1. Don't use all caps.

2. Use descriptive subject lines. Do not put URGENT!, or NEED HELP NOW, etc. in your title; it will not make people look at it any faster. Doing this makes many old time helpers on this board not look at the post at all.

Thread: Help!!!!!

Thread Tools

Search Thread

Display

Help!!!!!