Try { smth } catch { – MySQL – Fórum – Programujte.com

Dobý deň,  prosím Vás kde mám chybu dostávam hlásenie:Parse error: syntax error, unexpected 'catch' (T_CATCH)

function opendatabase(){
$db =mysqli_connect(MYSQLHOST,MYSQLUSER,MYSQLPASS) or die(mysql_error());
try {
    if (!$db){
        $exceptionstring = "Error connecting to database: <br />";
        $exceptionstring .= mysql_errno() . ": " . mysql_error();
        throw new exception ($exceptionstring);
    } else {   
        mysql_select_db("test",$db) or die(mysql_error());
        return $db;
    } catch (Exception $e) {
        echo $e->getmessage();
        die();
    }
}

#1 noro
Try a catch mají být na stejné úrovni, ty máš catch do try zanořený.

#2 ďakujem, mám dalšie hlasenie

Warning: mysql_select_db() expects parameter 2 to be resource, object given in

#1 noro
Doporučuji tento silně zastaralý a dále nepodporovaný kód zahodit. Zapomeň na funkci die(). Nauč se PDO, které je bezpečnější a pohodlnější:

<?php
function opendatabase($host, $user, $password) {
    $options = array(
        PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
        PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8",
    );
    $dsn = "mysql:host=$host;dbname=test";
    try {
        $db =  new PDO($dsn, $user, $password, $options);
    } catch (Exception $e) {
        throw new Exception("Nepodařilo se připojení k databázi: "
            . $e->getmessage();
    }
    return $db;
}

#4 Kit
Ďakujem popozerám to. Problém je v tom že každý požuvá inakší zápis.

#5 noro
Normálně používám jiný (kratší) zápis, ale v daném případě jsem se snažil přizpůsobit tvému zápisu, aby to fungovalo zhruba stejně. Ve zjednodušené podobě to vypadá takto: 

<?php
function opendatabase($host, $user, $password) {
    $options = array(
        PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
        PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8",
    );
    $dsn = "mysql:host=$host;dbname=test";
    $db =  new PDO($dsn, $user, $password, $options);
    return $db;
}

Jak vidíš, blok try { } catch { } jsem zcela vyhodil jako zbytečný, protože PDO již má vše potřebné. Navíc v této podobě to již nastavuje korektní práci s češtinou v UTF-8 a vyhazování výjimek při všech chybách práce s databází. Všechny detekce návratových kódů z databáze je možné vyhodit a soustředit se na tvorbu aplikace.

Nemám totiž rád "else", nepoužívám ho. Pokud je to možné, vyhýbám se i "if", "while" a "switch". A když už použiji "switch", tak zásadně bez "break".

A až budeš mít chuť to dělat, tak, jak se v OOP jazyce dělat má, zapomeň na standalone metody a databázi si zabal do objektu implementující vlastní databázový interface, kterému již hotový PDO (nebo jiný driver) předáš.

#7 ondrej39
Sám jsem si to udělal tak, že si PDO balím do třídy MyPDO (resp. SQLitePDO a PgPDO), která pak slouží jako továrna na objekty třídy PDOStatement. S těmi už pak pracuji bez obalu.

#8 Kit
A aký program požuváte? Ja používam text editor. S programami som mal problém, veľa veci čo som chcel sa my písane v programoch sa mi nezobrazovalo. A text editore nebol problém. Neviem prečo

#9 noro
A prečo sa píš niekedy mysql a niekedy mysqli?

#9 noro
Osobně používám:

• PHP / HTML - PhpStorm od JetBrains
• node.js / js a ts frontend - Webstorm od JetBrains
• Java - IntelliJ IDEA of JetBrains
• C / C++ - Visual Studio 2013 od Microsoftu nebo CLion od JetBrains
• C# - Visual Studio 2013 od Microsoftu

Volba prostředí je ale velmi subjektivní. Já jsem bez IDE bezradný, pokud si to ale dobře pamatuji, třeba konkrétně Kit nepoužívá IDE vůbec.

#10 noro
Velmi jednoduše řečeno je MySQLi nové rozhraní sloužící pro komunikaci s MySQL databázi z PHP. Nabízí různé opravy a přidává nějaké vlastnosti (to vše se dá dohledat na internetu). Důležitý je ovšem fakt, že MySQLi je bezpečnější díky prepared příkazům a také to, že původní mysql_* driver je považován za zastaralý a ačkoliv stále funguje, již několik let se hovoří o tom, že budoucí verze PHP ho podporovat vůbec nebudou.

Pokud si máš ale zvolit čistý php driver pro manipulaci databáze, pak je jednoznačně nejlepší volbou stále PDO.

<?php
$host = "localhost";
$user = "user";
$password = "";
$db = "test";

function opendatabase() {
     $options = array(
        PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
        PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8",
    );
    $dsn = "mysq:host=localhost;dbname=test";
    $db =  new PDO($dsn, 'user', '', $options);
    return $db;
$db->close();
}
?>

Skúšal som vrchný kód dostal som hlásenie:

-Fatal error: Uncaught exception 'PDOException' with message 'could not find driver' in

-PDOException: could not find driver in

Stále tam je nijaký problém potrebujem niečo nainštalovať?

#9 noro
Používám textový editor Vim, ale většina ostatních programátorů používá nějaké IDE. Obojí má své pro i proti.

#10 noro
PHP má k databázi MySQL tři ovladače: Zastaralý a zavrhovaný procedurální MySQL, nověji hybridní ovladač MySQLi, se kterým se dá pracovat podobojí a nejnověji plně objektové PDO, které sjednocuje ovládání různých databází typu SQL. Druhý a třetí v sobě obsahuje ochranu před útoky SQL Injection. Třetí se mi jeví jako nejpohodlnější, viz můj příklad.

#12 noro
Tady vidíš výhodu PDO: Když tomu něco chybí, vyhodí ti to jako výjimku a problém správně pojmenuje. Máš tam v proměnné $dsn překlep - místo slova "mysq" patří "mysql". To slovo označuje druh databáze, "mysq" pochopitelně neexistuje.

K čemu tam máš $db->close(); na konci? To nedělej. Je to zbytečné a v uvedeném případě i nefunkční.

Aj tak som to mal mám tam ďalšie hlásenie

Deprecated: mysql_real_escape_string(): The mysql extension is deprecated and will be removed in the future: use mysqli or PDO instead

A keď tam dám mysqli

-Fatal error: Uncaught exception 'PDOException' with message 'could not find driver'

-PDOException: could not find driver

#15 noro
Funkce mysql_real_escape_string() je v PDO nahrazena metodou $db->quote(), která zároveň doplní i potřebné apostrofy kolem výstupního stringu. Je tak chytrá, že to dělá podle typu databáze, ke které je připojena. PostgreSQL tedy escapuje jinak než MySQL.

Sám ji však nepoužívám, protože jsem si oblíbil prepared statements, které se bez toho zcela obejdou.

#16 noro
Psal jsem, že tam patří "mysql".

#16 noro
Do konstruktoru PDO vkládáš ty databáze. Typy databází jsou třeba SqlLite, MS Access, MySQL, PosgtreSQL, Oracle nebo MsSQL. Zde máš seznam databází, které PDO podporuje.

mysql_*, mysqli_* a PDO v PHP ale nejsou typy databází, to jsou drivery, které používáš pro interakci s ní. Do konstruktoru PDO patří typy databází, nikoliv drivery.

Pohlo sa to ďakujem .

Select má vyzerať nijak takto?

$querystr = $conn->prepare("SELECT userid FROM user WHERE name =LOWER(" .  $db->quote( $_GET['sstring']) . ")");

#20 noro 

$lowerCaseString = strtolower($_GET['sstring']);

$statement = $conn->prepare("SELECT userid  FROM user WHERE name = ?");
$successfullyExecuted = $statement->execute([
    $lowerCaseString,
]);

if ($successfullyExecuted === true) {
    while ($row = $statement->fetch()) {
        // work with the $row variable
    }
}

Nechtěj po databází, aby za tebe prováděla agregační funkce typu LOWER, pokud nemusí.

#21 ondrej39

Ďakujem Si super!!!!  funguje to. A na konci to netreba zatvoriť?

ukončil som to takto:

$lowerCaseString = strtolower($_GET['sstring']);

$querystr = $db->prepare("SELECT user_id  FROM users WHERE user_name = ?");
$successfullyExecuted = $querystr->execute([
    $lowerCaseString,
]);

if ($successfullyExecuted === true) {
    while ($row = $querystr->fetch()) {
    //Then return with an error.
?><span style="color: #FF0000;">Name not found...</span><?php
        // work with the $row variable
    }
}
else {
//At this point we would go to the processing script.
?><span style="color: #FF0000;">Form would now submit...</span><?php
}

Toto Dostávam na konci:

Name not found...Form would now submit... Name not found...Name not found...Form would now submit...

#22 noro
K čemu potřebuješ proměnnou $successfullyExecuted ? Prostě to vyhoď.

Databázi otevřenou přes PDO není potřeba zavírat, protože se sama zavře v destruktoru, jakmile $db opustí scope, tedy "zmizí z dohledu".

#23 Kit
$successfullyExecuted som robil podľa Ondreja. Dal som to preč ale teraz sa mi zobrazuje len čo je za else:

$db = opendatabase();
$lowerCaseString = strtolower($_GET['sstring']);

$stmt =( $db->prepare("SELECT user_id  FROM users WHERE user_name =$lowerCaseString"));

if ($stmt  === 0){

while ($row =$stmt->fetch())
    //Then return with an error.
?><span style="color: #FF0000;">Name not found...</span><?php
        // work with the $row variable
    
}
else {
//At this point we would go to the processing script.
?><span style="color: #FF0000;">Form would now submit...</span><?php
}
?>

#24 noro

Myslel jsem, že odstraníš i patřičný if a else a ponecháš jen ten cyklus while.

<?php
// ...

while ($row = $statement->fetch(PDO::FETCH_ASSOC)) {
    echo $row['user_id'] . "<br>";
}

#25 Kit

Takt som to našiel aj ja na internete, ale ja som práve if a else chcel použiť ale nikde som to nenašiel len takto:

$db = opendatabase();
//Set up the dynamic query string.
$querystr = "SELECT userid FROM user WHERE name =
LOWER('" . mysql_real_escape_string ( $_GET['sstring']) . "')";
if ($userquery = mysql_query ($querystr)){
if (mysql_num_rows ($userquery) == 0){
//Then return with an error.
?><span style="color: #FF0000;">Name not found...</span><?php
} else {
//At this point we would go to the processing script.
?><span style="color: #FF0000;">Form would now submit...</span><?php
}
} else {
echo mysql_error();
}

Ale pre PDO som nenašiel. Neviem to dokončiť

#26 noro
Zkus naznačit účel toho požadovaného ifu. Potřebuješ počet získaných záznamů?

Pokud chceš jen zjistit, jestli jsi našel alespoň jeden záznam, tak v tom mém posledním příkladu místo while jen použiješ if.

Uvědom si, že dotaz SELECT je úspěšný i v případě, když nic nenajde. Prázdný výstup není chybou.

Má zistiť či je db. Požadované meno alebo neni. Či že jeden záznam.

#28 noro
Takže už víš, jak na to. Viz výše.

Skúšal som to aj takto. Nepíše to chybu ale ani žiadne hlásenie aj keď je tam správne meno čo je DB

$lowerCaseString = strtolower($_GET['sstring']);
$stmt =( $db->prepare("SELECT user_id  FROM users WHERE user_name =$lowerCaseString"));
while ($row =$stmt->fetch(PDO::FETCH_ASSOC)) {
    echo $row['user_name'] . $lowerCaseString."<br>";
}

Neni tam nijaká chyba?

#30 noro
Však ti tam chybí řádky, které jsme si už vyjasnili a to while jsi za if nezaměnil. A proměnná $lowerCaseString nemá v tom SQL dotazu co pohledávat - místo ní patří otazník.

Projdi si celou diskuzi znovu a udělej to pořádně.

ďakujem ide to

#32 noro
Ještě bys neměl zapomínat na bloky try { } catch () { }, které jsem sice ze svých ukázek vypustil, ale ještě to neznamená, že je nepoužívám. Jen je mám vně takových celků. Záleží na vývojáři, jakou potřebuje jejich granularitu - může stačit i jediný blok try .. catch na celou aplikaci.

Obvykle však mám dvě až tři vrstvy ošetření výjimek, aby se aplikace i v případě chyby chovala "tak nějak rozumně". V žádném případě není nutné ošetření výjimky cpát do bloku (resp. třídy, metody), ve kterém k té výjimce může dojít.

#33 Kit
Prosím Vás a ako môžem zobraziť celú DB ? Takto sa mi to podarilo poskladať len sa to zobrazuje po jednom.

$query = $db->prepare("SELECT content FROM block  ORDER BY RAND() LIMIT 1");
$query->execute(array(':block'));
 
   $i=0;
   while ($row = $query ->fetch(PDO::FETCH_NUM, PDO::FETCH_ORI_NEXT)){
 
    echo $row[$i] . "\n";$i++;
 }

#34 noro
Bohužel tomuto zápisu vůbec nerozumím a ani netuším, co to má dělat.

Víš vůbec, co znamená ta klauzule "LIMIT 1"? Pokud ne, tak proč to tam dáváš?

ďakujem za odpoveď len som to našiel ne jednej stránke.  PDO funkciami nemám skúsenosti ale čo som potreboval našiel som na jednej stránke :http://code.tutsplus.com/…y--net-25338 .Nič lepšie som nenašiel.

#36 noro
Ta stránka vypadá velmi dobře. Má jen jednu vadu: Je tam toho moc a je možné, že právě z toho jsi zmaten.

Chybí mi tam použití čtvrtého parametru konstruktoru třídy PDO, ale to chybí snad ve všech návodech (kromě mých). Citelně tam chybí zmínka o způsobu používání znakové sady UTF-8, bez kterého je databáze v našem prostředí nepoužitelná.

Takže zde máš mnou vypilované připojení k MySQL pro české prostředí: 

<?php
$options = array(
    PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
    PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8",
);
$dsn = "mysql:host=$host;dbname=$dbname";
$db =  new PDO($dsn, $user, $password, $options);

Můžeš to celé zavřít do try .. catch dle potřeby. Na rozdíl od nich mi to vyhodí výjimku i při chybě otevření databáze, což považuji za užitečné.

Zbytek klidně dělej podle toho návodu.