prosze o przeanalizowanie kodu :) Opcje

[rozny]

Witam wszystkich (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Bardzo prosze o przeanalizowanie tego kodu pod wzgledem poprawnosci skladni itp.

Kod dziala tak jak to sobie zalozylem wiec bledow zadnych typu parsy, czy inne errory nie ma, jednak zalezy mi na optymalizacji tego kodu, a niestety doswiadczenie mam niewielkie i podejrzewam ze jest to raczej przerosniety smok (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[PHP] pobierz, plaintext 
<?php
 
		session_start();
 
		if (!session_is_registered('logged_in')) {
		echo '<p class=top>Admin Panel</p><form action=' .$PHP_SELF. ' method=post class=login>
		<input type=text name=login> LOGIN<br />
		<input type=password name=passwd> HASŁO<br />
		<input type=submit value=zaloguj>
		</form>';
		}
 
		if (isset($_POST['login'])&&isset($_POST['passwd'])) {
 
		$connect = mysql_connect('localhost', '*', '*')
 
		or die('Brak polaczenia z serwerem mySQL.<br />Błąd: '.mysql_error());
 
		echo ('Polaczenie z serwerem jest aktywne.<br/>');
 
		$db = mysql_select_db('users', $connect)
 
		or die('Brak polaczenia z baza danych<br />Blad: '.mysql_error());
 
		echo ('Polaczenie z baza danych jest aktywe.<br />');
 
 
		$sql = "SELECT * FROM admini WHERE login='".$_POST['login']."' LIMIT 0,30";
 
 
		$query=mysql_query($sql) or die ("Bład: ".mysql_error());
 
		$fetch = mysql_fetch_array($query);
 
		if ($fetch) {
			if ($_POST['passwd'] == $fetch['password']) {
 
				session_register('logged_in');
				header('Location: index.php');
 
			}
			else {
 
				echo 'bledne haslo';
 
			}
		}
 
		else {
 
			echo 'bledny login';
 
		}
 
 
		mysql_close();
 
		}
 
		if (isset($_GET['id'])) {
			if ($_GET['id']==logout) {
 
			session_destroy();
			session_write_close();
			header('Location: index.php');
			}
		}
 
		if (session_is_registered('logged_in')) {
		require_once('cfg/menu.cfg');
		}
		?>
[PHP] pobierz, plaintext 

pozdrawiam (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

[rozny]

Spoko Cysiaczek, juz dalem rade (IMG:http://forum.php.pl/style_emoticons/default/smile.gif) Pomogl mi dokladnie ten fragment, ktory zacytowales, zeby nie bylo, ze nie czytam tego co mi tu ludzie podsylaja (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg) Dzieki Wielkie raz jeszcze (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Jak co to bede jeszcze pytal o rozne sprawy w tym topicu (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)


To jeszcze takie pytanie. Czym zainteresowac sie jezeli chce wyswietlic okreslona ilosc znakow w jednej linii. Chodzi mi o to, ze chce wyswietlac w 1 linii np max 60 znakow, reszte przeniesc do 2 i tam tez wyswietlic max 60 itd... nie zalatwie tego tabelka ani divem o okreslonej szerokosci. Czy da sie to jakos zrobic w php, czy raczej musial bym pogrzebac w css? Juz wiem, ze bedzie jeszcze temat wysietlenia tylko czesci newsa, a do calego textu link <wiecej>, ale widzialem gdzies tutka jak to zrobic wiec raczej was tym nie bede meczyl (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

PS. Przegrzebalem forum wszukajac podobnego tematu, ale nie znalazlem nic o wyswietlaniu okreslonej ilosci znakow w 1 linii.

Pozdrawiam (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

EDIT: Poradzilem sobie w CSS,ale nie jest to zbyt eleganckie rozwiazanie (nie ogranicza ilosci znakow), jak ktos ma ochote to moze przedstawic rozwiazanie w php (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

EDIT:

[SQL] pobierz, plaintext 
$sql = 'INSERT INTO `news` (`id`, `data`, `tytul`, `tresc`, `autor`, `obrazek`) VALUES ('NULL', '"$data"', '"$tytul"', '"$text"', '"$autor"' , 'NULL');';
[SQL] pobierz, plaintext 

Witam to znowu ja (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

Tym razem pytanie co w tym zapytaniu jest niepoprawne... wywala mi:

Parse error: parse error, unexpected T_STRING in D:\Program Files\programy\VertrigoServ\www\serwis\administracja\cfg\add_form.php on line 25

Jak robilem:

[SQL] pobierz, plaintext 
$sql = 'INSERT INTO `news` (`id`, `data`, `tytul`, `tresc`, `autor`, `obrazek`) VALUES (NULL,' .$data. ', '.$tytul.', '.$text.', '.$autor.', NULL);';
[SQL] pobierz, plaintext 

Wywalalo mi:

You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near ':12:th, cinek, qazokm33, cinek, NULL)' at line 1

No i jak ja mam poprawnie to zapisac? Jak nie zastosuje kropek, wywala parse error, jak stosuje ' ' albo " " albo i ' " " ' to tez wyskakuje parse error ;(

pomozcie prosze (IMG:http://forum.php.pl/style_emoticons/default/smile.gif)

EDIT: dobra nie wazne juz (IMG:http://forum.php.pl/style_emoticons/default/winksmiley.jpg)

Ten post edytował rozny 14.12.2006, 12:39:17