¿Esta función de cadena inversa en C está mal escrita? / ¿Cómo mejorar este código? [cerrado]

Here is a part of code which describes a function to reverse characters of a string (Based on exercise 1-19 of Brian W. Kernighnan Programming in C)...
( I have googled various text reverse function, but all of them us pointers or using strrev(), but I don't have an idea what a pointer is.... nor do I want to use strrev(), hence I made a reverse string function as the author wanted it to be........ ) La función:

void reverse(char s[])
{
    int i , n ;
    char j ;
    i = 0 ;
    while(s[i] != '0') //And not EOF
    {
        ++n;
    }
    for(i = 0; i < n; ++i)
    {
        j = s[i] ;
        s[i] = s[n - 1] ;
        s[n - 1] = j ;
    }
}

P.S : It would be great if you did check and help me with the whole code here, since it would be offtopic if I did post it here, The code's main return 0; however it still doesn't work....

[EDIT] Ok I am seriously sorry for troubling you for a typo... I can't delete this question since it has answers with upvotes however I'm sorry....

The correct function would be :

void reverse(char s[])
{
    int i, l;
    char temp;
    for (l = 0; s[l] != '\0'; ++l);
    l--;
    for (i = 0; i < l; ++i) {
        temp = s[i];
        s[i] = s[l-1];
        s[l-1] = temp;
        --l;
    }
}

Full Code is here :

Código

Code Working is here :

ACTUALIZACIÓN:

I created a correct and working solution for the word 'hello':

#include <stdio.h>

int main(void)
{
    char s[] = "hello";
    char temp;



    // do the swapping here..

    temp = s[0];
    s[0] = s[4] ;
    s[4] = temp ;

    temp = s[1] ;
    s[1] = s[3] ;
    s[3] = temp ; 

   temp = s[2] ;
   s[2] = s[2] ;
   s[2] = temp ;
   printf("%c, %s ", temp, s);   

}