[MySQL][PHP]Powtórzenia kodu Opcje

[julek12]

Witam,
chciałbym się spytać czy da się jakoś zminimalizować ten kod, bo wydaje mi się on trochę za długi

[PHP] pobierz, plaintext 
<?php
try
{
	$pdo = new PDO('mysql:host=localhost;dbname=xxxx', 'xxxx', 'xxx', array(PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8'));
	$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
	function showPoll()
	{
		global $pdo;
 
		// Losowanie rekordu
		$sql = $pdo->query('SELECT * FROM `questions` WHERE `status` = 0');
		$row = $sql->fetchAll(PDO::FETCH_COLUMN);
		$rand_keys = array_rand($row);
 
		$sql = $pdo->prepare('SELECT * FROM `questions` q JOIN `answers` a USING(qid) WHERE a.`qid` = :qid'); //SELECT
 
		if ($_SERVER['REQUEST_METHOD'] == 'POST')
		{
			$stmt = $pdo->prepare('UPDATE `questions` q JOIN `answers` a USING(qid) SET q.`votes` = q.`votes` + 1, a.`vote` = a.`vote` + 1 WHERE q.`qid` = :qid AND a.`aid` = :aid'); //UPDATE
			$stmt->bindValue(':qid', $_POST['qid'], PDO::PARAM_INT); //UPDATE
			$stmt->bindValue(':aid', $_POST['answer'], PDO::PARAM_INT); //UPDATE
			$stmt->execute(); //UPDATE
 
			$sql->bindValue(':qid', $_POST['qid'], PDO::PARAM_INT); //SELECT
			$sql->execute(); //SELECT
			$data = $sql->fetchAll();
			$question = $data[0];
 
			echo '<ul><div style="max-width: 120px;">
			<li class="first"><b>'.$question['question'];
			echo '<br />Głos został oddany!</li>';
			foreach($data as $answer)
			{
				$percent = round(($answer['vote'] / $answer['votes']) * 100);
				echo '<li>'.$answer['answer'].'<br />
				głosów: '.$answer['vote'].', '.$percent.'%<br />
				<div style="border: 1px solid rgb(158, 152, 131); background: rgb(74, 8, 5); height: 10px; width: '.$percent.'%;"></div></li>';
			}
			$sql->closeCursor();
		}
		else
		{
			$sql->bindValue(':qid', $row[$rand_keys], PDO::PARAM_INT);
			$sql->execute();
			$data = $sql->fetchAll();
			$question = $data[0];
 
			echo '<ul><div style="max-width: 120px;">
			<li class="first"><b>'.$question['question'];
 
			if (2==3)
			{
				echo '<br />Głosowałeś już!</li>';
				foreach($data as $answer)
				{
					$percent = round(($answer['vote'] / $answer['votes']) * 100);
					echo '<li>'.$answer['answer'].'<br />
					głosów: '.$answer['vote'].', '.$percent.'%<br />
					<div style="border: 1px solid rgb(158, 152, 131); background: rgb(74, 8, 5); height: 10px; width: '.$percent.'%;"></div></li>';
				}
				$sql->closeCursor();
			}
			else
			{
				echo '</li><form id="poll" method="post" action="">';
				foreach($data as $answer)
				{
					echo '<li><input type="radio" name="answer" value='.$answer['aid'].' /> '.$answer['answer'].'</li>';
				}
				$sql->closeCursor();
				echo '<li><input type="text" name="qid" value='.$row[$rand_keys].' style="display: none;" /><div align="center"><input class="button" type="submit" value="Oddaj głos!" /></div></li></form>';
			}
		}
		echo '</b></div></ul>';
	}
	echo showPoll();
}
catch(PDOException $e)
{
	echo $e->getMessage();
}
?>
[PHP] pobierz, plaintext 

Ten post edytował julek12 24.08.2009, 08:00:29

[nospor]

if ($_SERVER['REQUEST_METHOD'] == 'POST')
na:
if (!empty($_POST))

[PHP] pobierz, plaintext 
if (2==3)
			{
				echo '<br />Głosowałeś już!</li>';
				foreach($data as $answer)
				{
					$percent = round(($answer['vote'] / $answer['votes']) * 100);
					echo '<li>'.$answer['answer'].'<br />
					głosów: '.$answer['vote'].', '.$percent.'%<br />
					<div style="border: 1px solid rgb(158, 152, 131); background: rgb(74, 8, 5); height: 10px; width: '.$percent.'%;"></div></li>';
				}
				$sql->closeCursor();
			}
			else
			{
				echo '</li><form id="poll" method="post" action="">';
				foreach($data as $answer)
				{
					echo '<li><input type="radio" name="answer" value='.$answer['aid'].' /> '.$answer['answer'].'</li>';
				}
				$sql->closeCursor();
				echo '<li><input type="text" name="qid" value='.$row[$rand_keys].' style="display: none;" /><div align="center"><input class="button" type="submit" value="Oddaj głos!" /></div></li></form>';
			}
[PHP] pobierz, plaintext 

2==3 Co to ma byc? Po co dajesz warunek, który nigdy nie będzie spelniony a w tym warunku masa kodu. A potem sie dziwisz ze masz za duzo kodu

[julek12]

warunek zostanie napisany potem wiem ze daje taki warunek zeby sprawdzić else a jak dam 2==2 to wtedy warunek a reszta kodu ok czy coś jeszcze?

[nospor]

kod do wyswietlania procentow moglbys dac w funkcje. przeciez dwa razy generujesz to samo.

[julek12]

ok a jak wygląda to:

[PHP] pobierz, plaintext 
$sql = $pdo->prepare('SELECT * FROM `questions` q JOIN `answers` a USING(qid) WHERE a.`qid` = :qid'); //SELECT
 
		if ($_SERVER['REQUEST_METHOD'] == 'POST')
		{
			$stmt = $pdo->prepare('UPDATE `questions` q JOIN `answers` a USING(qid) SET q.`votes` = q.`votes` + 1, a.`vote` = a.`vote` + 1 WHERE q.`qid` = :qid AND a.`aid` = :aid'); //UPDATE
			$stmt->bindValue(':qid', $_POST['qid'], PDO::PARAM_INT); //UPDATE
			$stmt->bindValue(':aid', $_POST['answer'], PDO::PARAM_INT); //UPDATE
			$stmt->execute(); //UPDATE
 
			$sql->bindValue(':qid', $_POST['qid'], PDO::PARAM_INT); //SELECT
			$sql->execute(); //SELECT
[PHP] pobierz, plaintext 

I do tego chciałbym dać jeszcze inserta da się to jakoś krócej zapisać?

Ten post edytował julek12 24.08.2009, 08:01:33

[phpion]

Losowanie rekordu również jest nieco popaćkane. Po co pobierać wszystko (prawie) z tabeli tylko po to aby (jak zakładam) docelowo wylosować z niej 1 rekord. Możesz przecież skorzystać z:

[SQL] pobierz, plaintext 
SELECT * FROM tabela ORDER BY RAND() LIMIT 1;
[SQL] pobierz, plaintext 

Możesz również pobrawić się w optymalizację tego zapytania poprzez pobranie liczby rekordów w jednym zapytaniu, a potem wykonanie drugiego z odpowiednim offsetem i limitem.

[julek12]

Jak te procenty w funkcje dać skoro one potrzebują zapytania?

[wookieb]

Potrzebuje wyniku zapytania.

[julek12]

teraz skrypt wygląda tak:

[PHP] pobierz, plaintext 
<?php
try
{
	$pdo = new PDO('mysql:host=localhost;dbname=xxxx', 'xxxx', 'xxxx', array(PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8'));
	$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
	function showPoll()
	{
		global $pdo;
 
		// Losowanie rekordu
		$sql = $pdo->query('SELECT `qid` FROM `questions` WHERE `status` = 0 ORDER BY RAND() LIMIT 1');
		$row = $sql->fetch();
 
		$sql = $pdo->prepare('SELECT * FROM `questions` q JOIN `answers` a USING(qid) WHERE a.`qid` = :qid'); //SELECT
 
		if (!empty($_POST))
		{
			$stmt = $pdo->prepare('UPDATE `questions` q JOIN `answers` a USING(qid) SET q.`votes` = q.`votes` + 1, a.`vote` = a.`vote` + 1 WHERE q.`qid` = :qid AND a.`aid` = :aid'); //UPDATE
			$stmt->bindValue(':qid', $_POST['qid'], PDO::PARAM_INT); //UPDATE
			$stmt->bindValue(':aid', $_POST['answer'], PDO::PARAM_INT); //UPDATE
			$stmt->execute(); //UPDATE
 
			$sql->bindValue(':qid', $_POST['qid'], PDO::PARAM_INT); //SELECT
			$sql->execute(); //SELECT
			$data = $sql->fetchAll();
			$question = $data[0];
 
			echo '<ul><div style="max-width: 120px;">
			<li class="first"><b>'.$question['question'];
			echo '<br />Głos został oddany!</li>';
			foreach($data as $answer)
			{
				$percent = round(($answer['vote'] / $answer['votes']) * 100);
				echo '<li>'.$answer['answer'].'<br />
				głosów: '.$answer['vote'].', '.$percent.'%<br />
				<div style="border: 1px solid rgb(158, 152, 131); background: rgb(74, 8, 5); height: 10px; width: '.$percent.'%;"></div></li>';
			}
			$sql->closeCursor();
		}
		else
		{
			$sql->bindValue(':qid', $row['qid'], PDO::PARAM_INT);
			$sql->execute();
			$data = $sql->fetchAll();
			$question = $data[0];
 
			echo '<ul><div style="max-width: 120px;">
			<li class="first"><b>'.$question['question'];
 
			if (2==3)
			{
				echo '<br />Głosowałeś już!</li>';
				foreach($data as $answer)
				{
					$percent = round(($answer['vote'] / $answer['votes']) * 100);
					echo '<li>'.$answer['answer'].'<br />
					głosów: '.$answer['vote'].', '.$percent.'%<br />
					<div style="border: 1px solid rgb(158, 152, 131); background: rgb(74, 8, 5); height: 10px; width: '.$percent.'%;"></div></li>';
				}
				$sql->closeCursor();
			}
			else
			{
				echo '</li><form id="poll" method="post" action="">';
				foreach($data as $answer)
				{
					echo '<li><input type="radio" name="answer" value='.$answer['aid'].' /> '.$answer['answer'].'</li>';
				}
				$sql->closeCursor();
				echo '<li><input type="text" name="qid" value='.$row['qid'].' style="display: none;" /><div align="center"><input class="button" type="submit" value="Oddaj głos!" /></div></li></form>';
			}
		}
		echo '</b></div></ul>';
	}
	echo showPoll();
}
catch(PDOException $e)
{
	echo $e->getMessage();
}
?>
[PHP] pobierz, plaintext 

A baza danych tak:

[SQL] pobierz, plaintext 
--
-- Struktura tabeli dla  `answers`
--
 
CREATE TABLE IF NOT EXISTS `answers` (
  `aid` int(10) UNSIGNED NOT NULL AUTO_INCREMENT,
  `qid` int(10) NOT NULL,
  `answer` text COLLATE utf8_polish_ci NOT NULL,
  `vote` int(10) NOT NULL DEFAULT '0',
  PRIMARY KEY (`aid`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=5 ;
 
--
-- Zrzut danych tabeli `answers`
--
 
INSERT INTO `answers` (`aid`, `qid`, `answer`, `vote`) VALUES
(1, 2, 'asddasdadasdasdasdasdasdasdds', 11),
(2, 1, 'wswewew', 2),
(3, 1, 'cxcxzczxcxz', 0),
(4, 2, 'czcsadasqwsacaxc', 0);
 
-- --------------------------------------------------------
 
--
-- Struktura tabeli dla  `logs`
--
 
CREATE TABLE IF NOT EXISTS `logs` (
  `lid` int(10) UNSIGNED NOT NULL AUTO_INCREMENT,
  `ip` varchar(50) COLLATE utf8_polish_ci NOT NULL,
  `user-agent` text COLLATE utf8_polish_ci NOT NULL,
  `time` bigint(30) NOT NULL,
  PRIMARY KEY (`lid`)
) ENGINE=MyISAM DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=1 ;
 
--
-- Zrzut danych tabeli `logs`
--
 
 
-- --------------------------------------------------------
 
--
-- Struktura tabeli dla  `questions`
--
 
CREATE TABLE IF NOT EXISTS `questions` (
  `qid` int(10) UNSIGNED NOT NULL AUTO_INCREMENT,
  `question` text COLLATE utf8_polish_ci NOT NULL,
  `status` int(1) NOT NULL DEFAULT '0',
  `votes` int(10) NOT NULL DEFAULT '0',
  PRIMARY KEY (`qid`)
) ENGINE=MyISAM  DEFAULT CHARSET=utf8 COLLATE=utf8_polish_ci AUTO_INCREMENT=11 ;
 
--
-- Zrzut danych tabeli `questions`
--
 
INSERT INTO `questions` (`qid`, `question`, `status`, `votes`) VALUES
(1, 'sgfsdfgdfgsdf', 0, 16),
(2, 'dfgdgfdgdfgdf', 0, 18);
[SQL] pobierz, plaintext 

[wookieb]

Ale żeś nazmieniał. Łooo
W dodatku liczenie procentów nadal nie zrobiłeś w funkcji? why?

[nospor]

no ale miales zamienic procenty na funkcje. nie umiesz do funkcji jako parametr przekazac tablicy na ktorej generujesz te procenty?

[julek12]

no nie za bardzo :|

spróbuje dojść:P

Ten post edytował julek12 24.08.2009, 08:34:40

[wookieb]

Więc wracamy do żłobka.. http://pl.php.net/manual/pl/functions.arguments.php

[julek12]

wyszło (chyba) nie korzystałem z manuala tylko sam:

[PHP] pobierz, plaintext 
<?php
try
{
	$pdo = new PDO('mysql:host=localhost;dbname=xxxx', 'xxx', 'xxxx', array(PDO::MYSQL_ATTR_INIT_COMMAND => 'SET NAMES utf8'));
	$pdo->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
	function percent($data)
	{
		foreach($data as $answer)
		{
			$percent = round(($answer['vote'] / $answer['votes']) * 100);
			echo '<li>'.$answer['answer'].'<br />
			głosów: '.$answer['vote'].', '.$percent.'%<br />
			<div style="border: 1px solid rgb(158, 152, 131); background: rgb(74, 8, 5); height: 10px; width: '.$percent.'%;"></div></li>';
		}
	}
 
	function showPoll()
	{
		global $pdo;
 
		// Losowanie rekordu
		$sql = $pdo->query('SELECT `qid` FROM `questions` WHERE `status` = 0 ORDER BY RAND() LIMIT 1');
		$row = $sql->fetch();
 
		$sql = $pdo->prepare('SELECT * FROM `questions` q JOIN `answers` a USING(qid) WHERE a.`qid` = :qid'); //SELECT
 
		if (!empty($_POST))
		{
			$stmt = $pdo->prepare('UPDATE `questions` q JOIN `answers` a USING(qid) SET q.`votes` = q.`votes` + 1, a.`vote` = a.`vote` + 1 WHERE q.`qid` = :qid AND a.`aid` = :aid'); //UPDATE
			$stmt->bindValue(':qid', $_POST['qid'], PDO::PARAM_INT); //UPDATE
			$stmt->bindValue(':aid', $_POST['answer'], PDO::PARAM_INT); //UPDATE
			$stmt->execute(); //UPDATE
 
			$sql->bindValue(':qid', $_POST['qid'], PDO::PARAM_INT); //SELECT
			$sql->execute(); //SELECT
			$data = $sql->fetchAll();
			$question = $data[0];
 
			echo '<ul><div style="max-width: 120px;">
			<li class="first"><b>'.$question['question'].'
			<br />Głos został oddany!</li>'.percent($data);
		}
		else
		{
			$sql->bindValue(':qid', $row['qid'], PDO::PARAM_INT);
			$sql->execute();
			$data = $sql->fetchAll();
			$question = $data[0];
 
			echo '<ul><div style="max-width: 120px;">
			<li class="first"><b>'.$question['question'];
 
			if (2==3)
			{
				echo '<br />Głosowałeś już!</li>'.percent($data);
				$sql->closeCursor();
			}
			else
			{
				echo '</li><form id="poll" method="post" action="">';
				foreach($data as $answer)
				{
					echo '<li><input type="radio" name="answer" value='.$answer['aid'].' /> '.$answer['answer'].'</li>';
				}
				$sql->closeCursor();
				echo '<li><input type="text" name="qid" value='.$row['qid'].' style="display: none;" /><div align="center"><input class="button" type="submit" value="Oddaj głos!" /></div></li></form>';
			}
		}
		echo '</b></div></ul>';
	}
	echo showPoll();
}
catch(PDOException $e)
{
	echo $e->getMessage();
}
?>
[PHP] pobierz, plaintext 

Ten post edytował julek12 24.08.2009, 08:43:47

[nospor]

czemu robisz wyniki funkcji traktujesz jako string i cos z nimi robisz, skoro zadna z twoich funkcji nic nie zwraca? Do zwracania sluzy return, a ty go nie uzywasz.

[julek12]

To co zamiast echo używać return czy co?

[wookieb]

wyszło (chyba) nie korzystałem z manuala tylko sam:

Naprawdę? To lepiej byś to nie robił sam.

To co zamiast echo używać return czy co?

Używać MANUALA!!!! http://pl.php.net/manual/pl/functions.returning-values.php

[julek12]

wiem do czego służy return ale nie wiem jak go użyć

[wookieb]

wiem do czego służy return ale nie wiem jak go użyć

A ja nie wiem jak wtłoczyć ci do głowy zasadniczą myśl. CZYTAĆ MANUAL!!!! W pierwszym przykładzie w linku, który ci podalem masz jak BYK pokazane jak się używa i to w dodatku z połączenie z ECHO (gdyby było print to pewnie byś się nie domyślił?)