Anonymní profil Quido – Programujte.com

Zdravím, jak by šlo importovat funkci/ popřípadě celý .py soubor do jiného souboru?

Např. měl bych hru kde by hráč přišel k důležitému rozhodnutí. Pokud by se rozhodnul pro první možnost, spustil by se kód z prvního souboru, na jehož konci by se program opět vrátil do hlavního programu a pokud pro druhou, spustil by se kód z druhého souboru?

Děkuju předem.

#3 Aricak
Zkus se zeptat na fóru joomly.

#15 peter
V pohodě. Dlouho jsem laboroval s tím, aby model nebyl příliš velký. Proto jsem ho rozdělil do domén. A protože ty domény potřebovaly konstruktor, udělal jsem ho co nejjednodušší - nacpal jsem do objektu vstupní data. V této konstrukci už nejsem omezen počtem tabulek v DB ani počtem domén v modelu, ale přitom velikost jednotlivých souborů zpravidla nepřekročí mých oblíbených 65 řádek. Zároveň si neblokuji možnost rozšíření na další databázové stroje, např. PostgreSQL, SQLite, DB4, filesystém apod.

V mém případě volající nemusí znát strukturu databáze. Stačí vědět, jakou strukturu mají mít data, která mohou být ve stromové struktuře a mohou být určena pro naplnění více tabulek.

Pokud by se mělo logovat, stačí tuto funkci aktivovat v modelu. Obvykle to však není nutné, protože mé databáze se logují samy nezávisle na aplikaci. Zalogují se tedy i změny udělané přes PMA.

PDO jsem dlouho nemohl přijít na chuť. Až do doby, kdy jsem pochopil, že umí vytvářet virtuální procesory pro práci s daty. Od té doby na PDO nedám dopustit.

Nevím, co je SMTH - snad STMT - statement. Nejsem příznivcem takových zkratek a nepoužívám je.

#13 peter
Promiň, ale máš tam hromady skrytých závislostí. Z mého pohledu je to nepoužitelné. Vždyť se ta třída ani nedá testovat. Navíc ta třída ani ty SQL dotazy nijak nezpracovává a nutíš volajícího, aby znal strukturu tabulek.

#96 Swarley
Tak to z hlavy nevím. Jak někdo začne míchat dohromady SQL, PHP a HTML, tak se v tom ztrácím...

Večer se na to mrknu.

#99 Swarley
Proč tam máš ten count? Je to drahý a podle mne docela zbytečný dotaz.

#94 Swarley
Asi ano. HTML už do PHP nedávám, na to mám šablony.

#1 Facedown
Použij návrhový vzor Observer. Databázi a úložiště budeš mít jako abonenty. Data pak pošleš vydavateli, který je rozešle všem abonentům v seznamu.

Observer má v PHP i své rozhraní SplObserver, které můžeš implementovat, ale nemusíš.

#1 Radka Holasová
Chybí uvedení lokality. Například do Prahy bych rozhodně dojíždět nechtěl (a do Turnova také ne).

#8 Pavel
A my Ti děkujeme za poskytnutí řešení problému.

#45 peter
A před zpracováním POST nesmí být ani ten DOCTYPE.

#18 peter
Nemám ta práva, přesun do nových vláken bych udělal už dávno.

#36 Swarley
Že nezpracováváš obsah formuláře a proto v $_POST nic není. Když se podíváš na příspěvek od Kremble, tak to je typická ukázka, jak to vypadat nemá a jak to nemůže fungovat.

#33 Swarley
Proč tam máš to echo? Stejně se nezobrazí.

#29 Swarley
Evidentně máš práznou proměnnou $_POST. Běžná to chyba. Jestli dobře počítám, tak by měla být na 7. řádku a nikoli na 37. Někde ti tam lítá zbytečných 30 řádek, nejspíš s formulářem.

#6 Gary
Ten form dej pryč (sem nepatří).

#25 Swarley
Zahoď Db.php a místo něj zkus tohle: 

<?php
$dsn = "mysql:host=127.0.0.1;dbname=zoznam_filmov;charset=utf8";
$db = new PDO($dsn, "root", "");
$db->setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
$query = "INSERT INTO filmy (nazov, rok, krajina, dlzka) VALUES (?, ?, ?, ?)";
$insert = $db->prepare($query);
$insert->execute(array($_POST['nazov'], $_POST['rok'], $_POST['krajina'], $_POST['dlzka']));

#1 Krokeťák
Najdi si někde kočičí jména, rozděl je na slabiky a pak ty slabiky mixuj.

#22 Kremlbe
Nemáš to dobře. Založ si vlastní vlákno.

#4 peter
Naučil jsem se URL nepřepisovat, ale pouze vše přesměrovat na index.php, který obdrží původní URL. Dobře se tak dělá třeba RestAPI. Ten můj router v PHP má jen 7 řádek a úplně to stačí.

#1 MyName
Doporučuji: Bruce Eckel - Thinking in Java.

#7 Flowy
Nevidím důvod, proč by cancel ve vyskakovacím okně měl vyvolávat výjimku. Vždyť to není chyba!

Argumentaci o délce stacktrace považuji za nesmyslnou. Výjimky nemají sloužit pro řízení programu, ale pokud dojde k chybě, vyhazuji výjimku bez ohledu na délku stacktrace.

#1 thomas
Když voláš nějakou funkčnost v programu, tak předpokládáš, že to dopadne dobře a obdržíš nějaký výsledek nějakého typu. Občas se něco nepodaří a nedostaneš žádný výsledek. Nějak však musíš zjistit, proč jsi ten výsledek nedostal. Zpráva o chybě má ale jinou strukturu a proto se musí přenášet jiným způsobem než výsledek. Nejlépe výjimkami.

#9 remmidemmi
Takovou krabičku bez uína mám. Dělal jsem ji ve 12 letech, kdy žádná uína nebyla a neměl jsem ani ten měřák - musel jsem si ho vyrobit.

#3 JirkaEr
"mnohonásobně vnořené if" by vypadaly asi hodně blbě. Naštěstí se nedělají.

#1 Jojo
Domnívám se, že je to poněkud zbytečná vlastnost, protože zvýraznění syntaxe funguje mnohem lépe. Když nebudeš dělat prázdné řádky uvnitř metod a budeš správně odsazovat, tak zvýrazňování bloků ani nebudeš potřebovat.

#7 Peter125
V tom případě nedělej SELECT a INSERT jako dva dotazy, ale jako jeden INSERT se SELECTem uvnitř. Bude to rychlejší a spolehlivější.

V původním příspěvku máš $_POST[cas], což nejsou data z databáze, ale od uživatele, která je nutné ošetřit.

#1 Facedown
Konstruktor v OOP slouží k vytvoření konzistentního objektu. Pokud má víc než 3 parametry, asi není něco v pořádku a bylo by dobré třídu rozdělit. Do konstruktoru pak místo skalárů předáš objekty nebo kolekce.

Rozhraní by mělo být pokud možno tenké, tedy s minimem metod. Raději více tenkých rozhraní než jedno tlusté. Pokud možno bez zbytečných getterů a setterů, protože ty počet metod často neúměrně zvyšují. Počet parametrů metod by měl být v rozumných mezích (0..3)

#4 q2iz
Začni se učit rovnou OOP, na které je PHP velmi dobře vybaveno. Ty základy, které se učíš, se ti budou také hodit. Vždyť objekt je také jen proměnná, která toho však může umět mnohem víc než jen držet data.

#1 David
Přínosnější IMHO bude, když budeme o PHP diskutovat přímo tady.

#1 Peter125
http://php.net/manual/en/datetime.createfromformat.php 

#4 ondra
V MySQL je to skoro stejné, pouze s odlišnou syntaxí.

#6 VladislavK
O programátořině jsem nic nepsal.

Peníze nejsou důležité, když jsou.

#4 VladislavK
Ozvalo se 8 lidí, jejichž hlavní motivací není zajímavý druh práce, ale peníze.

#8 peter
PDO sežere méně času než ovladač mysql.

Je zbytečné přecházet na céčko, PHP je dost rychlé už samo o sobě.

#5 peter
http://programujte.com/clanek/2009103100-php-a-mysql-mysqli-1-dil/

#8 Mudr. Jiří Kopeček
Pořadí je správné.

Data jsou možná chybně uložena. Defaultním kódováním je totiž Latin1. V tom případě je nutné tabulky zrušit, založit znovu s kódováním UTF-8 a nahrát tam data znovu. V každém kroku se dá udělat chyba.

#6 peter
Funkce PDO už není nutné obalovat, zejména pokud se vyhneš použití metody query().

#4 peter
Místo mysql_query se už dávno používá mysqli_query nebo PDO::query.

V další verzi PHP už funkce mysql_query nebude.

#1 Jan Klimeš
Rozhodně se nauč OOP, bez nich používáš Javu jen tak napůl. Hledej jména Bruce Eckel, Robert C. Martin a Martin Fowler. Napsali hromadu knih o OOP a spousta je toho volně ke stažení v PDF. Jejich přednášky najdeš i na Youtube.

#1 svoby1
Používá se na to Taylorův rozvoj:

sin(x) = x - x**3/3! + x**5/5! - x**7/7! ...

Běžně se to však pro mikroprocesory dělá převodní tabulkou.

#14 Matěj Andrle
Vypovídá to o využití protected módu procesoru a mnohem lepší adresaci paměti, protože nejsi omezen na 640 KiB.

#1 Matěj Andrle
Mrkni na tohle: http://dee.cz/machina/czech.html

Je to pokus o 32bitový DOS inspirovaný např. hrou Doom.

#8 JoDiK
procedure GetDir(D: Byte; var S: String);

Nic nevrací - je to  příkaz, aby název aktuálního adresáře byl uložen do stringu S. Nikde není výsledek, nedá se tedy použít ve výrazu.

Totéž GetTime() - v manuálu mají chybu. Místo "Vrací aktuální čas, ..." patří "Uloží aktuální čas do proměnných v parametrech". Opět nic nevrací.

#4 Gooo
Procedura nevrací žádný výsledek. Je to příkaz, co se má udělat.

#2 TomasRe
Kdyby to bylo čitelné, tak by ti možná někdo i poradil.

#21 peter
Po Matesovi nemůžeš chtít, aby byl normální :-)

#1 jávíte 

<form method="POST">
<button name="tlacitko">text</button>
</form>

<?php
if (isset($_POST['tlacitko'])) {
    // zapis do DB
}

#16 Matěj Andrle
BTW: Kdybys tady tak nekřičel a nenadával, možná by ti někdo i poradil.

#11 KIIV
A co L-Alt 35?

#23 peter
No, a protoze mne nebavi to kopirovat, tak jsem to ulozil do $n.

Proč zrovna do proměnné $n? Proč ne do proměnné $jmeno?

#16 hbo83
Metody GET a POST jsou rozdílné. Jak název napovídá, GET je určen pro získávání dat ze serveru, POST pro ukládání dat na server. Neplést.

#13 kikot
Zapomněl jsi napsat svůj názor na Arduino.

Mně se Arduino zamlouvá čím dál víc a asi si nějaké pořídím na experimenty. Už jsem si vymyslel rameno robota se třemi stupni volnosti a Arduino bude skvělé pro jeho řízení.

#1 jávíte
Nejjednodušší bude ten AdSense. Obvyklá míra prokliků je 1-2 %, nesnaž se s tím jakkoli manipulovat. Při takové návštěvnosti ti to hodí už docela zajímavý boční příjem.

#3 marioff
Ptal ses na fóru Nette? Tohle s PHP nemá moc společného.

#7 peter
V PHP se s polem téměř nepracuje, na takové výpočty ani nebylo stavěno.

#1 peter
Viděl bych 2 cesty: Přepsat do něčeho vhodnějšího (např. do Fortranu) nebo k tomu použít odpovídající nástroj, např. Matlab nebo Octave.

#5 ondrej39
Ty asserty se vypínají pouhou změnou parametru při spouštění aplikace. Ve zdrojáku zůstávají.

Je možné, že autor prostě nepočítal s tím, že by tam ten NULL mohl nastat, ale jen si to chtěl posichrovat assertem. To je v naprostém pořádku.

Na řízení běhu programu se však assert použít nesmí. Proto se také AssertException nezachytává.

#3 ondrej39
Ano, asserty jsou pouze na detekci vlastních chyb programátora. U odladěných aplikací se většinou vypínají kvůli úspoře výkonu, proto s jejich funkčností za běhu nesmíš počítat.

NullObject nevyhazuje žádný assert. Tváří se, jako kdyby fungoval, ale nic nedělá. Nemusíš pak při spouštění metod testovat, zda objekt existuje nebo ne a rovnou tu metodu spustíš. Místo  

if (uzel != NULL && uzel->rodic_ != NULL && uzel->rodic_->rodic_ != NULL)
	return uzel->rodic_->rodic_;
else
	return NULL;

píšeš jen  

return uzel->rodic_->rodic_;

#1 ondrej39
Assert v běžném provozu neměl být závislý na uživatelských datech, může však detekovat selhání programu.

Místo "return NULL;" je často výhodnější NullObject.

#1 Mishsuli
Klasika: Před voláním header('Location: ...'); nesmí být žádný textový výstup.

Nauč se formátovat kód. Tohle se nedá číst.

Máš tam SQL injection. Sprav si to.

#1 moula

• ne
• ne
• záznamy se mohou promíchat

Ten lock je potřebný, pokud zapisuješ delší záznam než asi 8 KB.

#20 JMM
Místo "?article=..." přece máš "/blog/..."

K čemu je tam dobrý ten switch?

#8 JMM
Zkus se podívat, jak se dělá REST API, které by se na to skvěle hodilo. Vidím tady navigaci

• /blog/id
• /krajina/id
• /portret/id
• /recenze/id

 To se dá udělat touto továrničkou

<?php
$parsedURL = parse_url($_SERVER['REQUEST_URI']);
list($akce, $id) = explode("/", $parsedURL["path"]);
$class = ucfirst($akce) . "_GET";
echo new $class((int) $id);

ke které přidáš třídy Blog_GET, Krajina_GET, Portret_GET a Recenze_GET.

#12 SVKSuli
Když to dám na HD a zvětším na fullscreen, tak je to písmo stále příliš prťavé. Jak s takovými blechami někdo může pracovat?

#3 Tomáš
Každé políčko může přece obsahovat souřadnice místa, kde se má vykreslit. Můžeš tak snadno generovat herní plány pro 3, 4, 5,... hráčů a dokonce ten herní plán může být i šišatý dle potřeby. Algoritmy však zůstanou stejné.

#1 Tomáš
Tak, že pole deklaruješ jako pole objektů. Jinak "Člověče nezlob se" není dvourozměrné pole, ale kruhový seznam.

#7 p3can
ORM je přece jen berlička pro programátory, kteří neumí SQL.

#7 p3can
Ano, "standardní postup" ve tvé vizi je udělat všechny atributy public přes nejrůznější gettery a settery a pak se v tom rochnit.

A vlajky nám sem do toho nepleť.

#4 sleepy
Nevýhodou tvého přístupu je, že tím zbytečně narušíš zapouzdření objektu.

#3 ondra
Však si tu serializační a deserializační metodu dej přímo do té třídy, jejíž instance budeš chtít ukládat. Je to standardní přístup ve stylu OOP. Měly by se jmenovat ve všech třídách stejně, např. serialize().

#6 Xena
Nechápu, proč ti Kit neporadí, ...

Nebudu mu radit, jak se má střílet do nohy a pořádně to udělat nechce.

SQL injection je tam stále - kdokoli se může stát adminem databáze.

#3 Jarda
Jenže ten starý ovladač v novém PHP už nebude. Proč to nechceš udělat rovnou pořádně, abys to pak nemusel předělávat?

Pro osobní účely to přece nemusíš nijak zaheslovávat.

#1 Jarda
Máš tam SQL injekci.

Starým ovladačem MySQL se už nezabývám. Zahoď ho a nauč se PDO.

#23 ZAJDAN
Nedivím se, že tě na Rootovi nemají rádi. Hodíš sem správný kus kódu, sám ho máš blbě a pak se divíš, že ti to nejede a že ti nikdo neporadí.

Jestli se chceš naučit programovat, zamysli se nad tím, z čeho se skládá tabulka.

A také se konečně nauč pojmenovávat proměnné.

#20 ZAJDAN
V databázi to není sloupec, ale řádek.

#18 ZAJDAN
Proč jsi tu proměnnou nazval $column, když v ní nemáš sloupec, ale řádek?

Když správně pojmenuješ proměnné, polovina problémů se vyřeší sama.

#16 ZAJDAN
Asi to nic nenašlo.

#14 ZAJDAN
A co máš ve výsledku?

Zkus se trochu zamyslet nad názvy proměnných, které jsou nyní jak z učebnice. Místo $prepared by asi mělo být $select a místo $result bych raději viděl například $customers. A místo $resultJson? Tak to nevím. Nabízí se $customersJson, ale dvou- a víceslovní názvy proměnných moc rád nemám. Záleží na konkrétním použití.

#11 ZAJDAN
No vidíš, že to jde. Jde to i lépe, ale s tímto si nějakou dobu určitě vystačíš.

#10 ZAJDAN

Vidím, že jsi úplně vedle. Seznam si přece můžeš projít cyklem foreach.

S hvězdičkou v SELECTu zacházej opatrně. Ne vždy je to přesně to, co od databáze požaduješ.

#7 ZAJDAN
Seznam, slovník a množina jsou logické struktury (kolekce), které jsou uloženy na fyzické struktuře - zpravidla pole nebo strom.

#7 ZAJDAN
Slovníku se dřív říkávalo "asociativní pole". V OOP se používá pojem "Dictionary", tedy slovník.

Abys z těch dat neměl mišmaš, používej fetchAll() takto: 

$result = $prepared->fetchAll(\PDO::FETCH_ASSOC);

Ještě je další možnost - seznam anonymních objeků: 

$result = $prepared->fetchAll(\PDO::FETCH_OBJ);

#5 ZAJDAN
To ti neprojde, protože fetchAll() dodává seznam slovníků a ty s tím zacházíš jako kdyby to byl slovník - a říkáš tomu pole.

#3 ZAJDAN
Opravdu ty skripty správně vyhazují výjimky např. u chybného SQL dotazu? Vyzkoušej si to.

V manuálových stránkách ses asi trochu přehlédl. O tom předchozím zápisu tam není ani ň.

#1 ZAJDAN
Chybička se vloudila 

PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,

Možná ti to pak vrátí výsledek nebo alespoň výjimku. 

A ještě něco:

$prepared = $pdo->prepare(" SELECT * FROM customer WHERE uniq = :uniq ");
$prepared->bindParam(':uniq', $uniq, PDO::PARAM_STR);
$prepared->execute();   
$result = $prepared->fetchAll();

#1 Michal
Z mého pohledu jsou ty kódy zcela začátečnické. Nemají s OOP ani nic společného - je to jako kdyby sis přečetl první kapitolu z učebnice Javy a rozhodl se to vyzkoušet. Přístup je to správný (přečíst si kapitolu a ihned vyzkoušet), ale na hodnocení kvality programátora je skutečně příliš brzy.

V prvním případě máš chybně odsazování, ve druhém je chybně to chybové hlášení i následující podmínka, která tam vůbec nemá být. Názvy tříd se píší s velkým počátečním písmenem.

Takže pokračuj.

#1 Tomas
Zjednodušeně:

• funkce zjišťuje stav a vrací jeho hodnotu (v OOP getter)
• procedura slouží k modifikaci stavu (v OOP setter)

Funkce i procedury mohou mít parametry, mělo by se preferovat předávání hodnotou.

#3 JoDiK
Standardní Pascal má druhou mocninu? Stejně raději používám ve všech jazycích prosté násobení.

#10 ingiraxo
Akorát že ta tvoje funkce nezvládne číslice "٠١٢٣٤٥٦٧٨٩"

#9 vitamin
Procedurální záležitosti raději přenechávám systémovým knihovnám. Lexery dělám jen výjimečně, na ty se continue občas hodí. Ve vyšších vrstvách k tomu obvykle není důvod.

#7 vitamin
goto/break/continue neumí předat hodnotu, proto je return jako alternativa zpravidla výhodnější.

#5 vitamin
Break v cyklu i ve switchi jsou také zbytečné a obcházet je přes goto tuplem.

#2 hlucheucho
go... co? To prý kdysi bylo ve Fortranu.

#1 Jan Linek
A co o tom píší ostatní uživatelé na fóru php-f? Díval ses tam vůbec?

#2 RomanZ
On si to tady jen plete s chatem.

#6 JoDiK
Z editoru (IDE) přece spouštím kompilaci a testy toho programu. Až je program napsaný, tak už nepotřebuji, aby se mi po jeho skončení zastavovala výstupní konzole a musel klepat na enter.

#4 JoDiK
To je docela hloupý nápad dávat do aplikace readkey nebo readln. Tu prodlevu po skončení programu by měl zajistit editor.

#5 Frederik
Kromě toho je dobrým zvykem psát za interpunkcí a klíčovými slovy mezeru. Zkus použít program indent nebo astyle a hned ti ukáže další chybu, která sice nezpůsobuje nekonečný cyklus, ale způsobí nefunkčnost.

#3 Frederik
Trojitý cyklus je prostě moc a když máš tak hloupé názvy proměnných, tak se ani nedivím, že chybu nevidíš. Přejmenuj si je a chybu rychle najdeš.