Jump to content
php.lv forumi

Ieteikumi un labojumi iekš index.php


lostz

Recommended Posts

Sveiki rakstu ar tādu jautājumu, varat ieteikt ko varētu pielabot manam indexam un stilam, jo zinu ka viņā ir daudz nepilnību, paviršību, kļūdu, un kā būtu efektīvāk tasīt, zinu pāris metodes kad taisa ka index sastāv tikai no include, citi taisa ka html kopa ar php indexā, bet kādas būtu vel un kuras ir efektīvākas protams?

 

Galvenā: http://paste.php.lv/28a51f032f899efabf2615d5c0afba81?lang=php

Stils: http://paste.php.lv/d9fae6cf1c197f0163e86d98ecad5048?lang=css

Link to comment
Share on other sites

Lostz, nekad nav izcēlies ar valīdu kodu!!! :D

vaine? :D

 

nu css vis ok, bet es pat nezinu kapēc html met ārā kļūdas kur tās itkā nav, vai arī es vienkārši nesaskatu.

Link to comment
Share on other sites

title tags ārpus head taga

<br> vajag arī aizvērt - <br />

 

Nav kļūda, bet <?print$copy_rights?>, "pareizāk" būtu: <?=$copy_rights;?> (ja izmanto shorttags)

 

Varētu arī normāli tabulēt kodu, lai ir kāda pārskatāmība...

Edited by briedis
Link to comment
Share on other sites

Sa-tabulē-visu kodu, izlabo tās kļūdas un ieliec te vēlreiz :)

Reku ir itkā šādi?

 

http://paste.php.lv/40e53d0e1ebf60a00422f59ddd64e013?lang=php

 

Ps. to <br> droši var neņemt vērā ņemšu tos ārā un tā vietā pēctam gan jau kaut kas būs.

Edited by lostz
Link to comment
Share on other sites

Varēji tak vienkārši full string replace <br> ar <br />

 

Kā tu domā, kas ir pārskatāmāk?

 

  <div id="sidebar1">

  <div class="panelis">
<div class="panelish2">Autorizēšanās</div>
<div class="texts"><center>
<?
include('inc/login.php');
?>
</center>
</div>
</div>
<div class="atstarpe"></div>

 <div class="panelis">
<div class="panelish2">Aptauja</div>
<div class="texts"><center>
<br>
<br />
</center>
</div>
</div>
<div class="atstarpe"></div>

 

vai

 

<div id="sidebar1">
<div class="panelis">
	<div class="panelish2">Autorizēšanās</div>
	<div class="texts">
		<center><? include('inc/login.php'); ?></center>
	</div>
</div>
<div class="atstarpe"></div>
<div class="panelis">
	<div class="panelish2">Aptauja</div>
	<div class="texts">
		<center><br><br /></center>
	</div>
</div>
<div class="atstarpe"></div>
</div>

Link to comment
Share on other sites

Varēji tak vienkārši full string replace <br> ar <br />

 

Kā tu domā, kas ir pārskatāmāk?

 

gan jau ka 2 :]

Bet nau vai tad kļūdu vai kaut ko kas būtu labojams?

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...