in function 'main'

$ cat shell.c
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
void sh(char *);
int main()
{
void sh(char *cmd)
int num;
num = system(cmd);
}

$ gcc shell.c
shell.c: In function `sh':
shell.c:9: parse error before "num"
shell.c:8: parm types given both in parmlist and separately
$
What do I do to correct this error??

I don't know... why are you writing this program? What is sh? What is cmd for?

Quote:

---

Originally posted by zmonc
$ cat shell.c
int main(void)
{
void sh (char *);
void sh (char *cmd);
int num;
num = system(cmd);
}

---

1) Why do you have 'sh' defined twice?
2) You have no variable named 'cmd'.
3) You don't define the 'sh' function.

There are your problems and your fixes.

Quzah.

sorry...I am newbie
I have modifed the code and compiled and got the new error message:

$ cat shell.c
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
void sh(char *);
int main()
{
void sh(char *cmd)
int num;
num = system(cmd);
}

$ gcc shell.c
shell.c: In function `sh':
shell.c:9: parse error before "num"
shell.c:8: parm types given both in parmlist and separately
$


what's wrong in this now?

Use code tags, also few problems:

>>void sh(char *);
>>void sh(char *cmd)

First you defined, now you didn't write the function, do something like this, first declare the function, then write it, or write it outside 'main'. Your main function doesn't return anything, before you close the main function (before the '}') return a value using 'return' like 'return 0;'.

thank you verymuch guys

First of all when you code you have to have a goal, what do you wont to do with this program. Ask yourself that and then come back, if we dont know your intentions then we cant know your problem...

But if i were you i would start with searching a tutorial on the net...(there are plenty of them -- + you could visit the faq board of this site once in a while).

Then to get to your code
Why do you include header files such as stdlib and string.h i dont see any file or string handling in your code...
Second of all Check your compiler errors-also compiler warnings because they may indicate that something nifty is going on in your code.
[quote]
shell.c:9: parse error before "num"
[/code]
Parse error..... means add a ";" sign (without the quotes) behind that no-make-sense-line-of-code -void sh(char *cmd) )

Apparently you have a long way to go start by making the infamous Hello program
+ init main returns 0; (in ur case)

But before you start asking yourself WTF are they talking about i suggest you search some tutorials....

P.S: Sry if i was rude at any point but i understand your position completely, when i started coding it took me 4 months to find out how this thing actually works. That was because i didnt searched the net for tutorials and because i hadnt followed any programming clas, nor did i had a book about programming. But ill never forget when i made my first application that actually did something usefull and worked without bugs in it....


BTW: Welcome to the board!!!
;)
And i just looked on the net and this seems pretty good... just went trough ex1 and only thing is they write main instead of int main check it out .
:o 2 late now going to bed :o