Изучаю ООП и возникают ошибки "unexpected '(', expecting ',' or ';'"

@kuznets · Регистрация: 22.11.2013

Студворк — интернет-сервис помощи студентам

Здравствуйте, я изучаю ООП и практикуя возникают вопросы не подвласные моему разуму, пока ) Надеюсь тут найдется добрые люди, кто направить на праведный путь, как с решением ошибок так и с избавлением от говнокода.
Сразу оговорюсь, я предпочтительно понимаю русский язык и порой мне сложно понять, что на их русском мне говорят программисты, так что если не сложно постарайтесь не злоупотреблять )

Итак.
class_Comentary.php

PHP
1
2
3
4
5
6
7
8
9
10
11
class Commentary
{
    $db=MyMysqli::getDB(); //Ошибка ругается сюда! (7 строка) Класс подключения к БД рабочий, и метод db_query тоже.
    
    public function getCommentary($query)
    {
        if($result=$this->db->db_query($query)){
            return $result;
        }
    }
}

viev.tpl

PHP
1
2
3
<?php $query='SELECT * FROM `commentary`'; ?>
<?php $getCom=new Commentary; ?>
<?php $result=$getCom->getCommentary($query); ?>

Получаю ошибку: Parse error: syntax error, unexpected '(', expecting ',' or ';' in \www\libs\class_Commentary.php on line 7

KOPOJI · 22.11.2013, 23:55

так нельзя делать. Собственно, в ошибке вам и пишет: ошибка синтаксиса, не ожидается скобка, ожидается запятая или точка с запятой.
Почитайте тему FAQ: Объектно-ориентированное программирование на PHP

@kuznets · 23.11.2013, 00:35 **[ТС]**

Я честно говоря не понял в чем ошибка синтаксиса.
Класс MyMysqli статический обращаюсь я к нему правильно

PHP
1
2
$db=MyMysqli::getDB();
echo ClassName::StaticFunc(); // Печатаем данные, возвращаемые методом класса

Может дело в области видимости? Но и здесь я не нашел четкого ответа http://www.php.net/manual/ru/l... otayim.php

DrobyshevAlex · 23.11.2013, 01:11

Наверно самые основы стоит почитать, прежде чем браться за ООП

http://www.php.net/manual/ru/l... erties.php

Переменные, которые являются членами класса, называются "свойства". Также их называют, используя другие термины, такие как "аттрибуты" или "поля", но в рамках этой документации, мы будем называть их свойствами. Они определяются с помощью ключевых слов public, protected, или private, следуя правилам правильного описания переменных. Это описание может содержать инициализацию, но инициализация дожна применяться для константных значений - то есть, переменные должны быть вычислены во время компиляции и не должны зависеть от информации программы во время выполнения для их вычисления.

@kuznets · 23.11.2013, 11:23 **[ТС]**

Сообщение от DrobyshevAlex

Наверно самые основы стоит почитать, прежде чем браться за ООП

Честно говоря читал, но

PHP
1
2
3
4
5
class Commentary
{
    public $db=MyMysqli::getDB();
    public function getCommentary($query)
    ........

Так была та же ошибка. Но я понял, что свойства нужно обязательно определять.
Сейчас сделал так:

PHP
1
2
3
4
5
6
7
8
9
class Commentary
{
    public function getCommentary($query)
    {
        if($result=MyMysqli::getDB()->db_query($query)){
            return $result;
        }
    }
}

И все работает, но мне хочется понять, первую схему public $db=MyMysqli::getDB(); и что в ней не так.

KOPOJI · 23.11.2013, 11:28

kuznets, еще раз внимательнее прочитайте сообщение выше. Там должно быть константное, уже вычисленное значение.

@nonamez123 · 25.11.2013, 14:03

по красто я думаю должно быть что-то похожее на

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
class Commentary
{
    
    private $db = FALSE;
    
    function __construct() 
    {
        $this->db = MyMysqli::getDB();
    }
    
    public function getCommentary($query)
    {
        if ($this->db && $result = $this->db->db_query($query))
            return $result;
    }
}

Не по теме:

P.S.
Забегая далеко вперед тут очень хорошо бы было использовать http://ru.wikipedia.org/wiki/%... 0.B0_PHP_5

KOPOJI · 25.11.2013, 15:35

Сообщение от nonamez123

Забегая далеко вперед тут очень хорошо бы было использовать

чем хорошо тут использовать синглтон?

@nonamez123 · 25.11.2013, 17:58

Сообщение от KOPOJI

чем хорошо тут использовать синглтон?

что бы 10 подключений к бд не было.

KOPOJI · 25.11.2013, 19:02

и что делать, если захочется подключения к разным серверам?

@kuznets · 25.11.2013, 19:13 **[ТС]**

В классе MyMysqli я уже использую синглтон, так что 10 подключений не будет. В данном случае будет использоваться одна база.

@OnYourLips · 25.11.2013, 22:03

Сообщение от kuznets

В классе MyMysqli я уже использую синглтон

Пойдем дальше: теперь попробуй убрать синглтон и передавать инстанс этого класса в те места, где он будет использоваться.

@kuznets · 26.11.2013, 00:14 **[ТС]**

А смысл его убирать? Я пока не вижу его минусов, ну вроде все говорят, что не стоит его пихать где попало, но здесь он вроде в тему.

П.С. Я не понимаю что значит "инстанс этого класса".

DrobyshevAlex · 26.11.2013, 00:48

Сообщение от kuznets

А смысл его убирать?

Если Вы считаете что написав

PHP
1
class A {}

Вы можете сказать что знаете, понимаете и используете ООП для своих проектов - то это далеко не так...
Суть ООП не просто написание классов, а написания их так, что бы они были переносимы, что бы проекты были модульными и легко расширяемыми, а так же, что бы затронув один класс, было как можно меньше (а в идеале что бы вообще не было) изменений в других классах.
Не нужно писать фразы типа

Сообщение от kuznets

В данном случае будет

Старайтесь писать так, что бы этот класс потом можно было легко использовать для других проектов, в которых возможно понадобится несколько коннектов к разным бд.

Почитайте книги по ООП не именно пхп, а там где описывают различные подходы, паттерны проектирования, а потом посмотрите как пишут код в фреймворках, зная паттерны будетп роще понять что там делается и для чего.

Сообщение от kuznets

Я пока не вижу его минусов

Предположим, что на хостинге где размещается сайт нет mysqli, или выбрана вообще другая субд, не mysql, вот напишите класс так, что бы было легко перейти на другую субд, не меняя класса комментариев.

Передав в конструктор класса объект для доступа к бд, при изменении субд, можно просо передать экземпляр другого класса, например MongoDB.
а для обеспечения совместимости, нужно использовать интерфейсы, которые гарантируют, что класс содержит необходимые методы.

@kuznets · 26.11.2013, 00:58 **[ТС]**

Вы можете сказать что знаете, понимаете и используете ООП для своих проектов - то это далеко не так...

Я пока учусь, какое там понимание. Но если мне говорят, убирай, то мне интересно почему. Потому, что я хочу понимать, а не просто убирать.

Старайтесь писать так, что бы этот класс потом можно было легко использовать для других проектов, в которых возможно понадобится несколько коннектов к разным бд.

Ехал с работу сегодня и думал об этом и лучше, что пришло в голову для разных БД свой класс, вызываешь то, что тебе нужно. Возможно спустя (что скорее всего) n времени я буду думать по другому. Разбор сейчас фреймворков почти бесполезен, я там смотрю в книгу и вижу, сами знаете что. Хотя так, или иначе приходится смотреть yii.
Читать то я много, что читаю, но времени в сутках всего 24ч ) а еще надо прочитанное попробовать.

DrobyshevAlex · 26.11.2013, 01:02

Ну так по этому я и посоветовал начать всё таки с паттернов, перед разбиранием кода фв

Когда будите в фв видетьп отом знакомый код, будите вспоминать что это за паттерн, и для чего его используют)

@OnYourLips · 26.11.2013, 09:18

Сообщение от kuznets

А смысл его убирать? Я пока не вижу его минусов, ну вроде все говорят, что не стоит его пихать где попало, но здесь он вроде в тему.

Это антипаттерн.
Я не знаю ситуаций, где без него нельзя было бы обойтись.
При этом минусы очевидны:

Сообщение от http://www.rsdn.ru/forum/design/2615563.1

1. Синглтон нарушает SRP (Single Responsibility Principle) — класс синглтона, помимо того чтобы выполнять свои непосредственные обязанности, занимается еще и контролированием количества своих экземпляров.
2. Глобальное состояние. Про вред глобальных переменных вроде бы уже все знают, но тут та же самая проблема. Когда мы получаем доступ к экземпляру класса, мы не знаем текущее состояние этого класса, и кто и когда его менял, и это состояние может быть вовсе не таким, как ожидается. Иными словами, корректность работы с синглтоном зависит от порядка обращений к нему, что вызывает неявную зависимость подсистем друг от друга и, как следствие, серьезно усложняет разработку.
3. Зависимость обычного класса от синглтона не видна в публичном контракте класса. Так как обычно экземпляр синглтона не передается в параметрах метода, а получается напрямую, через GetInstance(), то для выявления зависимости класса от синглтона надо залезть в тело каждого метода — просто просмотреть публичный контракт объекта недостаточно.
4. Наличие синглтона понижает тестируемость приложения в целом и классов, которые используют синглтон, в частности. Во-первых, вместо синглтона нельзя подпихнуть Mock-объект, а во-вторых, если синглтон имеет интерфейс для изменения своего состояния, то тесты начинают зависеть друг от друга.
Говоря же проще — синглтон повышает связность, и все вышеперечисленное, в том или ином виде, есть следствие повышения связности.

DrobyshevAlex · 26.11.2013, 11:15

имхо, пункты надуманы и накручены какими то частными случаями и недопониманием для чего используют синглтон.

Сообщение от OnYourLips

класс синглтона, помимо того чтобы выполнять свои непосредственные обязанности, занимается еще и контролированием количества своих экземпляров

Даже если перевести слово сингтон, то станет видно, что он и должен контролировать количество своих экземпляров, а точнее оно должно быть равно 1.
Собственно даже открыв вики, и прочитав первую фразу, можно увидеть что человек, писавший первый пункт, не до конца понимает что такое синглтон.

Вообщем первые три пункта очень спорные...

Я не сторонник синглтонов, но и пункты, описывающие минусы, приведённые на том сайте, основаны на недопонимании что такое синглтон и где его можно использовать...
У ТС один из примеров, неправильного использования синглтонов.
И действительно, синглтон сильно связывает приложение, но точно так его связывает любая функция из набора расширений пхп, типа SimpleXML, mb_strlen... нужно в идеале вообще всё обернуть своими классами

@OnYourLips · 26.11.2013, 21:40

DrobyshevAlex, приведи мне ситуацию, где использование синглтона оправдано.
Вышеприведенный пример - класс доступа к базе - яркий пример, где синглтон использовать ни в коем случае нельзя.

DrobyshevAlex · 26.11.2013, 22:57

Сообщение от OnYourLips

Вышеприведенный пример - класс доступа к базе - яркий пример, где синглтон использовать ни в коем случае нельзя.

Я сказал что это не так?

Сообщение от DrobyshevAlex

У ТС один из примеров, неправильного использования синглтонов.

Сообщение от OnYourLips

приведи мне ситуацию, где использование синглтона оправдано.

Сообщение от DrobyshevAlex

не сторонник синглтонов, но и пункты, описывающие минусы, приведённые на том сайте, основаны на недопонимании что такое синглтон и где его можно использовать...

Мне кажется Вы меня не так поняли.
Я же говорю, что я сам не сторонник синглтонов.

Но Вы почитайте те пункты.
Синглтон что должен делать? Следить что бы экземпляр был 1.
Что в первом пункте написано?

Сообщение от OnYourLips

помимо того чтобы выполнять свои непосредственные обязанности, занимается еще и контролированием количества своих экземпляров.

То есть тут как раз основная задача синглтона, выставлена как будто это его баг

Так раз задачей синглтона является слежение за количеством объектов, то зачем приводить пример что он даёт связывание, на основании кода ТС? Связывание даёт любая функция, мало функций в пхп устаревших? тот де split. В идеале, все функции надо обернуть, что бы даже функций ядра языка на котором пишется проект не было

Синглтоны можно вообще не использовать, я не спорю.
Но я писал выше о том, что пункты там описаны не правильно.
Да минус - это связывание различных классов приложения в случае как у ТС. Но не кто не заставляет использовать синглтон так... это не задача дать глобальный доступ. Задача что бы объект был 1

Сообщение от OnYourLips

Зависимость обычного класса от синглтона не видна в публичном контракте класса.

Откуда опять взялась зависимость? Опять же из за неправильной трактовки понятия синглтона. Дело же не в паттерне а в умении его готовить

Сообщение от OnYourLips

Наличие синглтона понижает тестируемость приложения в целом и классов, которые используют синглтон, в частности. Во-первых, вместо синглтона нельзя подпихнуть Mock-объект

Легко...

Сообщение от OnYourLips

если синглтон имеет интерфейс для изменения своего состояния, то тесты начинают зависеть друг от друга.

непонятное использование синглтона.
если у шевралет камаро есть колёса и мощный мотор, это не значит что на нём поля пахать нужно...

Сообщение от OnYourLips

Про вред глобальных переменных вроде бы уже все знают, но тут та же самая проблема.

Тут вообще о чём разговор? При чём тут глобальные переменные?
Автор использует сингтон так?

PHP
1
for (Counter::GetInstance()->i = 0;  Counter::GetInstance()->i < 10; Counter::GetInstance()->i ++)

Если да, то вопросов больше не имею...

Код без синглтона

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
class Config {...};
 
class App
{
public:
    App(Config* config): m_config(config) {}
 
private:
   Config* m_config;
};
 
Config* config = new Config;
App login(Config->GetAuthConfig());
App game(Config->GetWorldConfig());

код с синглтоном

C++
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
class Config {
    static Config* GetInstance();
};
#define Config::GetInstance()
 
class App
{
public:
    App(Config* config): m_config(config) {}
 
private:
   Config* m_config;
};
App login(Config->GetAuthConfig());
App game(Config->GetWorldConfig());

Теперь предположим что вам нужно иметь всегда один общий сокет-менеджер, подписку на события...
Вам нужно быть уверенным что это тот же объект. С синглтоном вы уверены, что везде у вас один объект.

Вопрос: чем код с синглтоном более связан чем без синглтона?

Новые блоги и статьи Все статьи Все блоги /
Отчёт о затраченных материалах за определенный период с макетом печатной формы Maks 21.04.2026 Отчёт из решения ниже размещён в конфигурации КА2. Задача: разработка отчёта по затраченным материалам за определённый период, с возможностью вывода печатной формы отчёта с шапкой и подвалом. В. . .	Отчёт о спецтехнике находящейся в ремонте Maks 20.04.2026 Отчёт из решения ниже размещен в конфигурации КА2. Задача: отобразить спецтехнику, которая на данный момент находится в ремонте. Есть нетиповой документ "Заявка на ремонт спецтехники" который. . .	Памятка для бота и "визитка" для читателей "Semantic Universe Layer (Слой семантической вселенной)" Hrethgir 19.04.2026 Сгенерировано для краткого описания по случаю сборки и компиляции скелета серверного приложения. И пусть после этого скажут, что статьи сгенерированные AI - туфта и не интересно. И это не реклама -. . .	Запрет удаления строк ТЧ документа при определённом условии Maks 19.04.2026 Алгоритм из решения ниже реализован на примере нетипового документа "Аккумуляторы", разработанного в конфигурации КА2. У данного документа есть ТЧ, в которой в зависимости от прав доступа. . .
Модель заражения группы наркоманов alhaos 17.04.2026 Условия задачи сформулированы тут Суть: - Группа наркоманов из 10 человек. - Только один инфицирован ВИЧ. - Колются одной иглой. - Колются раз в день. - Колются последовательно через. . .	Мысли в слух. Про "навсегда". kumehtar 16.04.2026 Подумалось тут, что наверное очень глупо использовать во всяких своих установках понятие "навсегда". Это очень сильное понятие, и я только начинаю понимать край его смысла, не смотря на то что давно. . .	My Business CRM MaGz GoLd 16.04.2026 Всем привет, недавно возникла потребность создать CRM, для личных нужд. Собственно программа предоставляет из себя базу данных клиентов, в которой можно фиксировать звонки, стадии сделки, а также. . .	Знаешь почему 90% людей редко бывают счастливыми? kumehtar 14.04.2026 Потому что они ждут. Ждут выходных, ждут отпуска, ждут удачного момента. . . а удачный момент так и не приходит.

KOPOJI Почетный модератор 16844 / 6724 / 880 Регистрация: 12.06.2012 Сообщений: 19,967
	22.11.2013, 23:55
	так нельзя делать. Собственно, в ошибке вам и пишет: ошибка синтаксиса, не ожидается скобка, ожидается запятая или точка с запятой. Почитайте тему FAQ: Объектно-ориентированное программирование на PHP 0

KOPOJI Почетный модератор 16844 / 6724 / 880 Регистрация: 12.06.2012 Сообщений: 19,967
	23.11.2013, 11:28
	kuznets, еще раз внимательнее прочитайте сообщение выше. Там должно быть константное, уже вычисленное значение. 0

KOPOJI Почетный модератор 16844 / 6724 / 880 Регистрация: 12.06.2012 Сообщений: 19,967
	25.11.2013, 19:02
	и что делать, если захочется подключения к разным серверам? 0

@kuznets 0 / 0 / 0 Регистрация: 22.11.2013 Сообщений: 79
	25.11.2013, 19:13 [ТС]
	В классе MyMysqli я уже использую синглтон, так что 10 подключений не будет. В данном случае будет использоваться одна база. 0

@kuznets 0 / 0 / 0 Регистрация: 22.11.2013 Сообщений: 79
	26.11.2013, 00:14 [ТС]
	А смысл его убирать? Я пока не вижу его минусов, ну вроде все говорят, что не стоит его пихать где попало, но здесь он вроде в тему. П.С. Я не понимаю что значит "инстанс этого класса". 0

DrobyshevAlex 1178 / 1128 / 94 Регистрация: 31.05.2012 Сообщений: 3,060
	26.11.2013, 01:02
	Ну так по этому я и посоветовал начать всё таки с паттернов, перед разбиранием кода фв Когда будите в фв видетьп отом знакомый код, будите вспоминать что это за паттерн, и для чего его используют) 0

@OnYourLips 508 / 358 / 13 Регистрация: 12.03.2012 Сообщений: 1,896
	26.11.2013, 21:40
	DrobyshevAlex, приведи мне ситуацию, где использование синглтона оправдано. Вышеприведенный пример - класс доступа к базе - яркий пример, где синглтон использовать ни в коем случае нельзя. 0