Jump to content
php.lv forumi

mysql_query($users) or ...


laucinieks

Recommended Posts

Kāpēc vispār sesijā jāglabā tā kļūda?

 

$username = $_POST['username'];


if(Users::validUsername($username)){
 Users::loginUser($username);
 header("location:gatavs.php");
 exit();
}else{
 echo "Slikts lietotājvārds";
}

Edited by briedis
Link to comment
Share on other sites

Hmm, kas tas tāds pa pierakstu ? :) Pirmo reizi tādu redzu. Varbūt vari pastāstīt, vairāk par to? (Domāju par Users::valid...)

Šeit ir mans kods - http://paste.php.lv/...eb6ae3?lang=php

28. rindā var redzēt visu, it kā jau labāk, bet kā var nodefinēt tajā die lai viņš aiziet vienu lapu atpakaļ ar header?

L.

Edited by laucinieks
Link to comment
Share on other sites

Tā ir klase Users ar statisku metodi valid.. http://php.net/manual/en/language.oop5.php kas teiksim atgrieztu vai lietotāja vārds ir derīgs.

 

Sāc mācīties ne-rakstīt spagetti kodu (kur viss ir kopā - html, php kods un darbības ar datubāzi), nodali vismaz darbības ar datubāzi atsevišķas metodēs.

Link to comment
Share on other sites

Tu runā par šo?

public function edit_user()
{
 if (isset($_GET['action']) == "edit")
 {
$id = (int)$_GET['id'];
$select_user = "SELECT * FROM users WHERE id =".$id;
$mysql_query_user = mysql_query($select_user) or die(mysql_error());
$user = mysql_fetch_array($mysql_query_user);
?>                
<tr style="background: #FBD3B1;">
<form method="post" action="">
 	<td><input type="text" name="email" value="<?php echo $user['email']; ?>" /></td>
 	<td><input type="text" name="phone" value="<?php echo $user['phone']; ?>" /></td>
 	<td><input type="text" name="website" value="<?php echo $user['website']; ?>" /></td>
 	<td><input type="submit" name="submit_users" value="" class="submit_button" /></td>
</form>
</tr>
<?php
 }
}

Ja jā, kā tad viņu var sadalīt divās daļās. Viņš man ir class.users.php failā un users.php failā izsauc kā $users->edit_user();

 

Hmm... Ja tas ir OOP, kas tad ir tas, ko es tagad rakstu (ar class failiem un pēctam padod citā failā ar $mainigais = new class; $mainigais->blabla(); )

Edited by laucinieks
Link to comment
Share on other sites

kaut vai:

-pārbaudīt, vai action == edit nevajag metodē user_edit(), lieks couplings un lieki ierobežo klases izmantošanu citā kontekstā, piem unittestos.

-to html ko metode uztaisa vislabāk vispār tur nelikt, bet nu vismaz atgriezt kā stringu, nevis uzreiz izpildīšanas brīdī bērt ārā outputā.

Edited by v3rb0
Link to comment
Share on other sites

Tas, ka tiek izmantotas klases, gluži kodu nepadara objektu-orientētu. Vai arī Tu domā, ka :: un -> ir viss, ar ko atšķiras objektu orientētā programmēšana no procedurālās? :)

gribeju zinat kapec gan doma, ka vinjam kods NAV OOP ?

Pec ka tu secinaji, redzot TIKAI vienu f-ju ?

Tas ka tiek iekseji izmantots sql pieprasijums, neveidojot tam atsevikju Objektu ?

A varbut nekur citur to SQL vairs nevajadzes...

Es gan neizdaritu secinajumus neredzot parejo koda dalju ...

 

Un ja tas nav OOP un nav arii proceduralaa ?

 

P.S. Tas ka SQL un HTML outputu, labak adalit tas tiesa...

Link to comment
Share on other sites

Join the conversation

You can post now and register later. If you have an account, sign in now to post with your account.

Guest
Reply to this topic...

×   Pasted as rich text.   Paste as plain text instead

  Only 75 emoji are allowed.

×   Your link has been automatically embedded.   Display as a link instead

×   Your previous content has been restored.   Clear editor

×   You cannot paste images directly. Upload or insert images from URL.

Loading...
×
×
  • Create New...