Skip to content

Conversation

@paranoiq
Copy link

@paranoiq paranoiq commented Jul 7, 2013

tahle drobnost mě tam neuvěřitelně štve. má to nějaký důvod?

IDE kvůli tomu háže warningy, kdykoliv použiju $e->id jako klíč pole

v projektech to řeším překrytím anotace na vyšší úrovni, ale je to zbytečné. lepší aby to bylo správně hned od počátku

@PetrP
Copy link
Owner

PetrP commented Jul 17, 2013

Validace na typ int|string není to samé jako validace na typ id. viz kód

Takže typ nelze takhle volně zaměnit.

Řešit by to šlo tím že by se jednalo o macro:

/**
 * @property-read int|string $id {id}
 */

Které ide vyhazuje "chybu" při použití knihovny s nesprávném typem? Chápal bych kdyby ti to zvyrazňovalo v tvém kódu, ale vyhozovat chyby na "knihovny" je divné.

@paranoiq
Copy link
Author

zneužíváš standartní typovou anotaci k vlastnímu účelu a znemožňuješ tím správnou funkci statické analýzy kódu (to se nemusí týkat jen IDE)

"id" není platné jméno datového typu ani pseudo-typu, takže vyvozovat, že jde o chybu IDE je chybné, bez ohledu na to o jaké IDE jde. "id" prostě není typ

@property-read int|string $id {id} je podle mě ok

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants