[SOLVED] 2 easy'ish questions.

[LemonInflux]

I am making an file uploader, and I have got everything done except the register form. You see, this hoster, unlike other codes, offers users individual galleries and directories so they can organize their own set of images. However, there are 2 problems I need sorting out. Before we begin, here is the code that processes the registration:

 

<?php

$username = $_POST['username']; 
$password = $_POST['password']; 

$username = strtolower($username);
$password = strtolower($password);

$all_users = file("user.db.txt");
    foreach($all_users as $user_line)
    {
                $user_arr = explode(" | ", $user_line);
        if($user_arr[0] == $username){ echo 'This username has already been taken! <a href="register.php">Try again</a>' }
    }

$File = "user.db.txt";
$Handle = fopen($File, 'a');
$Data = $username;
fwrite($Handle, $Data);
$Data = ' | ';
fwrite($Handle, $Data);
$Data = md5($password);
fwrite($Handle, $Data);
$Data = ' | ';
fwrite($Handle, $Data);
$Data = "4\n";
fwrite($Handle, $Data);
print "You have registered successfully! click <a href=index.htm>here</a> to return to the login screen";
fclose($Handle);

mkdir("$username", 0700);
?>

 

So what this basically does is convert their username and password to lowercase, write it to user.db.txt, and create them a directory. Now, the first of my two problems:

 

How do you write to a .php file, not a .txt? I've tried the code above and just replacing .txt with .php, but it doesn't seem to work.

 

The second is, the code isn't functioning because of this code:

 

$all_users = file("user.db.txt");
    foreach($all_users as $user_line)
    {
                $user_arr = explode(" | ", $user_line);
        if($user_arr[0] == $username){ echo 'This username has already been taken! <a href="register.php">Try again</a>' }
    }

 

this was my attempt at a code that would stop users from signing up with someone else's username, thus making it possible for them to access the other user's gallery. So my question is simply, what's wrong with it?

 

NOTE: it is meant to explode ' | ' not '|'.

 

Opinions? Tom.

[LemonInflux]

bump

[MadTechie]

1. its the same as a text file, just need to make it executale

2. if thats codes at fault then wheres the fault /problem?

[LemonInflux]

1) CHMOD? It's 777

2) It's in that code, I just don't know where. But it worked fine (apart from the registering already made users issue) before I added it in, and now it doesn't.

[ToonMariner]

make sure you use umask when using chmod

[LemonInflux]

it's a windows server, it doesn't use CHMOD o_O Besides, host automatically 777's everything online, linux or windows.

[ToonMariner]

chmod id a php function that works on all platforms - even windows has the permissions system!!!!

[LemonInflux]

I'll try it. Any ideas for question 2?

[LemonInflux]

It appears I owe you an apology. Strange, everything else auto 777'd o_O. Thanks a bunch

 

So yeah, question 2.

 

<?php

$username = $_POST['username']; 
$password = $_POST['password']; 

$username = strtolower($username);
$password = strtolower($password);

$all_users = file("users.db.php");
    foreach($all_users as $user_line)
    {
                $user_arr = explode(" | ", $user_line);
        if($user_arr[0] == $username){ echo 'This username has already been taken! <a href="register.php">Try again</a>' }
    }

$File = "users.db.php";
$Handle = fopen($File, 'a');
$Data = $username;
fwrite($Handle, $Data);
$Data = ' | ';
fwrite($Handle, $Data);
$Data = md5($password);
fwrite($Handle, $Data);
$Data = ' | ';
fwrite($Handle, $Data);
$Data = "4\n";
fwrite($Handle, $Data);
print "You have registered successfully! click <a href=index.htm>here</a> to return to the login screen";
fclose($Handle);

mkdir("$username", 0700);
?>

 

I think the problem is that $user_line is still undefined. But I can't think how I'd define it o_O how do you say 'array[0] on every line' in PHP?

[LemonInflux]

Bump..

[LemonInflux]

And again...

[LemonInflux]

bump

[btherl]

A better topic name might help to bring attention to your questions.

 

As for your question 2, try printout out your variables to check their values.  Use var_dump() rather than print if possible, as it is more detailed.

 

In your code there is no need to define $user_line, as it is defined by the foreach loop.

[LemonInflux]

ah, ok. thanks. Trying now.

[LemonInflux]

it didn't work :\ you did mean to just have 'foreach($all_users)', right?

[darkfreaks]

look it up on php.net

[LemonInflux]

I did -.- They're not much help though if you want a block of text sorting. I've found them only helpful whilst writing, not checking. But maybe it's just me..

[LemonInflux]

bump

[LemonInflux]

bump again. Someone must know the answer

[LemonInflux]

*sigh* anyone?

[MadTechie]

try reading the logic....!

 

Sighs..

 

<?php

$username = $_POST['username']; 
$password = $_POST['password']; 

$username = strtolower($username);
$password = strtolower($password);

$all_users = file("users.db.php");
$freeuser = true;
foreach($all_users as $user_line)
{
$user_arr = explode(" | ", $user_line);
if($user_arr[0] == $username)
{
	echo 'This username has already been taken! <a href="register.php">Try again</a>';
	$freeuser = false;
	break;
}
}
if($freeuser)
{
$File = "users.db.php";
$Handle = fopen($File, 'a');
$Data = $username;
fwrite($Handle, $Data);
$Data = ' | ';
fwrite($Handle, $Data);
$Data = md5($password);
fwrite($Handle, $Data);
$Data = ' | ';
fwrite($Handle, $Data);
$Data = "4\n";
fwrite($Handle, $Data);
print "You have registered successfully! click <a href=index.htm>here</a> to return to the login screen";
fclose($Handle);

mkdir("$username", 0700);
}
?>

[LemonInflux]

It worked! Thanks ever so much for your help, there was no way I was going to get that

[leela]

Hai ..i really dont know how to post a question..

 

i am in need of this ...Plz help me

 

1) How to count the no of downloads of a particular file/doc/image in PHP.

 

Thanks in advance.....Looking for your help 

[MadTechie]

You need to click new topic

 

from here

 

you would need to create either a FlatFile storage or MySQL to store the downloads

 

MySQL would be better..

 

please start the new thread..