Форум программистов, компьютерный форум, киберфорум
PHP: ООП
Войти
Регистрация
Восстановить пароль
Блоги Сообщество Поиск Заказать работу  
 
Рейтинг 4.67/18: Рейтинг темы: голосов - 18, средняя оценка - 4.67
0 / 0 / 0
Регистрация: 22.09.2017
Сообщений: 10

Тестовое задание

22.09.2017, 18:30. Показов 3653. Ответов 10
Метки oop (Все метки)

Студворк — интернет-сервис помощи студентам
Дали тестовое задание, так как я его провалил, интересно сообразит ли кто то что вообще должен делать этот код и в чем тут ошибки? Потому что выглядит очень странно

найти и исправить 4 ошибки в коде
PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
class Sales
{
    public function testPayment(array $data, $amount)
    {
        $payment = new PaypalPayment;
        try {
            $payment->validate($data, function($testAmount) {
                return $this->test($amount, $testAmount);
            });
        } catch (\PaypalException $e) {
            new \RuntimeException('something weird', 23, $e);
        }
        
        return $payment;
    }
}
 
class PaypalPayment
{
    private $testAmount = 200;
    
    public function validate(array $data, callable $comparator)
    {
        if (!$comparator($this->testAmount)) {
            throw new \PaypalException("Incorrect amount");
        }
    }
    
    private function test($amount, $testAmount) {
        return $amount == $testAmount;
    }
}
 
(new Sales)->testPayment([], 200);
0
cpp_developer
Эксперт
20123 / 5690 / 1417
Регистрация: 09.04.2010
Сообщений: 22,546
Блог
22.09.2017, 18:30
Ответы с готовыми решениями:

Корректно ли тестовое задание
Честно говоря не знаю, куда лучше сунуть данную тему, но так как она относится к PHP и ООП, запилил сюда. В общем я получил тестовое...

Тестовое задание в использованием словаря
Здравствуйте! Не могу разобраться, пожалуйста, помогите. Выполняю тест для C++. Мне необходимо выполнить его на PHP. В...

тестовое задание
Здравствуйте! тестовое задание : Задача создать GUI приложение для конвертации файлов данных разных форматов. Данными в...

10
Эксперт PHP
3899 / 3237 / 1353
Регистрация: 01.08.2012
Сообщений: 10,904
22.09.2017, 19:26
Цитата Сообщение от troopermanowar Посмотреть сообщение
что вообще должен делать этот код
Имхо, названия классов и функций говорят сами за себя.

Цитата Сообщение от troopermanowar Посмотреть сообщение
в чем тут ошибки?
Я так понимаю, в анонимной функции. Переменную $amount не видно, $this ссылается на Sales вместо PaypalPayment, а метод test должен быть публичным.

Думаю, должно быть так, хотя не уверен, может можно не передавать $payment и каким-то образом заставить $this ссылаться на нужный объект.
PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
class Sales
{
    public function testPayment(array $data, $amount)
    {
        $payment = new PaypalPayment;
        try {
            $payment->validate($data, function($testAmount) use ($amount, $payment) {
                return $payment->test($amount, $testAmount);
            });
        } catch (\PaypalException $e) {
            new \RuntimeException('something weird', 23, $e);
        }
        
        return $payment;
    }
}
 
class PaypalPayment
{
    private $testAmount = 200;
    
    public function validate(array $data, callable $comparator)
    {
        if (!$comparator($this->testAmount)) {
            throw new \PaypalException("Incorrect amount");
        }
    }
    
    public function test($amount, $testAmount) {
        return $amount == $testAmount;
    }
}
 
(new Sales)->testPayment([], 200);
0
1943 / 1768 / 825
Регистрация: 23.01.2014
Сообщений: 6,230
22.09.2017, 19:30
Цитата Сообщение от troopermanowar Посмотреть сообщение
найти и исправить 4 ошибки в коде
Сначала хотел все описать, потом понял что их тут гораздо больше 4 и стало лень, в общем вот:
PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
<?php
 
class PaypalException extends RuntimeException {}
 
class Sales
{
    public function testPayment(array $data, $amount)
    {
        $payment = new PaypalPayment;
        try {
            $payment->validate($data, function($testAmount) use ($payment, $amount) {
                return $payment->test($amount, $testAmount);
            });
        } catch (\PaypalException $e) {
            throw new \RuntimeException('something weird', 23, $e);
        }
        
        return $payment;
    }
}
 
class PaypalPayment
{
    private $testAmount = 200;
    
    public function validate(array $data, callable $comparator)
    {
        if (!$comparator($this->testAmount)) {
            throw new \PaypalException("Incorrect amount");
        }
    }
    
    public function test($amount, $testAmount) {
        return $amount == $testAmount;
    }
}
 
try {
    (new Sales)->testPayment([], 200);
} catch (RuntimeException $e) {
    echo $e->getMessage();
}

Не по теме:

Jodah, замени 200 на другое число и увидишь еще парочку... или тройку...


В общем что тут от Вас хотели я не знаю, выглядит все очень костыльно, но по крайней мере так работает.
2
0 / 0 / 0
Регистрация: 22.09.2017
Сообщений: 10
22.09.2017, 20:09  [ТС]
Jodah, название test не очень то хорошо говорит за себя. Ну я кстати исправил вчера точно также, добавил use, поменял private на public, говорят нет, все не так должно быть.

Добавлено через 35 минут
Ага, а вот и правильный ответ:

PHP
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
class Sales
{
    public function testPayment(array $data, $amount)
    {
        $payment = new PaypalPayment;
        try {
            $closure  = function ($testAmount) use ($amount) {
                return $this->test($amount, $testAmount);
            };
            $closure = $closure->bindTo($payment, 'PaypalPayment');
            $payment->validate($data, $closure);
        } catch (\PaypalException $e) {
            throw new \RuntimeException('something weird', 23, $e);
        }
        return $payment;
    }
}
 
 
 
class PaypalPayment
{
    private $testAmount = 200;
 
    public function validate(array $data, callable $comparator)
    {
        if (!$comparator($this->testAmount)) {
            throw new \PaypalException("Incorrect amount");
        }
    }
 
    private function test($amount, $testAmount)
    {
        return $amount == $testAmount;
    }
}
0
Эксперт PHP
4925 / 3920 / 1620
Регистрация: 24.04.2014
Сообщений: 11,441
22.09.2017, 20:54
Цитата Сообщение от troopermanowar Посмотреть сообщение
вот и правильный ответ:
И в чем принципиально использование bindTo отличается от передачи через use?
0
0 / 0 / 0
Регистрация: 22.09.2017
Сообщений: 10
22.09.2017, 21:21  [ТС]
И в чем принципиально использование bindTo отличается от передачи через use?
Ну вот и я не понял. С их точки зрения этот вариант правильный а мой нет

Добавлено через 1 минуту
Типа чтоб метод test остался private
0
Эксперт PHP
4925 / 3920 / 1620
Регистрация: 24.04.2014
Сообщений: 11,441
22.09.2017, 21:30
Цитата Сообщение от troopermanowar Посмотреть сообщение
Типа чтоб метод test остался private
Они жгут. Какой смысл тогда выносить валидацию в коллбек и прятать его? Пользователя класса по идее не должно волновать все что скрыто уровнем доступа, и как такой конструкцией пользоваться в реальности не понятно.
0
Эксперт PHP
 Аватар для Fedor Vlasenko
936 / 693 / 236
Регистрация: 01.02.2015
Сообщений: 1,848
22.09.2017, 23:15
Надо было завернуть, ответив Ваш говнокод сами в нем и разбирайтесь
Это код со всех сторон является таким
0
1943 / 1768 / 825
Регистрация: 23.01.2014
Сообщений: 6,230
23.09.2017, 08:36
Цитата Сообщение от troopermanowar Посмотреть сообщение
Ага, а вот и правильный ответ:
Ни разу неправильный. Читайте внимательней.
Цитата Сообщение от pav1uxa Посмотреть сообщение
замени 200 на другое число и увидишь еще парочку... или тройку...
0
0 / 0 / 0
Регистрация: 22.09.2017
Сообщений: 10
23.09.2017, 10:26  [ТС]
Ни разу неправильный.
Имеется в виду то что они считают правильным
0
1943 / 1768 / 825
Регистрация: 23.01.2014
Сообщений: 6,230
23.09.2017, 12:28
Цитата Сообщение от troopermanowar Посмотреть сообщение
Имеется в виду то что они считают правильным
Ну так укажите им ошибки. Может возьмут тогда
0
Надоела реклама? Зарегистрируйтесь и она исчезнет полностью.
raxper
Эксперт
30234 / 6612 / 1498
Регистрация: 28.12.2010
Сообщений: 21,154
Блог
23.09.2017, 12:28
Помогаю со студенческими работами здесь

Тестовое задание
Ребят, подскажите пожалуйста, кто с опытом, есть компания, хорошие условия работы и т.п. Я сразу понимаю что не волоку, но хочу подтянуть...

Тестовое задание
Здравствуйте, форумчане! Я решил заняться программированием и в частности веб-разработкой. Ситуация такова, что я знаю всего по чуть: html,...

Тестовое задание
Добрый день! Относительно недавно начал изучать SQL, подкинули тестовое задание (во вложении). Ума не приложу как его сделать....

Тестовое задание
private class Connect (){ public Device dev1 = new Device(); public Device dev2 = new Device(); puclic void Connect (){ for...

Тестовое задание
Возникла необходимость сделать вот такое задание. Create a configurable two-level cache (for caching Objects). Level 1 is memory,...


Искать еще темы с ответами

Или воспользуйтесь поиском по форуму:
11
Ответ Создать тему
Новые блоги и статьи
PhpStorm 2025.3: WSL Terminal всегда стартует в ~
and_y87 14.12.2025
PhpStorm 2025. 3: WSL Terminal всегда стартует в ~ (home), игнорируя директорию проекта Симптом: После обновления до PhpStorm 2025. 3 встроенный терминал WSL открывается в домашней директории. . .
Access
VikBal 11.12.2025
Помогите пожалуйста !! Как объединить 2 одинаковые БД Access с разными данными.
Новый ноутбук
volvo 07.12.2025
Всем привет. По скидке в "черную пятницу" взял себе новый ноутбук Lenovo ThinkBook 16 G7 на Амазоне: Ryzen 5 7533HS 64 Gb DDR5 1Tb NVMe 16" Full HD Display Win11 Pro
Музыка, написанная Искусственным Интеллектом
volvo 04.12.2025
Всем привет. Некоторое время назад меня заинтересовало, что уже умеет ИИ в плане написания музыки для песен, и, собственно, исполнения этих самых песен. Стихов у нас много, уже вышли 4 книги, еще 3. . .
От async/await к виртуальным потокам в Python
IndentationError 23.11.2025
Армин Ронахер поставил под сомнение async/ await. Создатель Flask заявляет: цветные функции - провал, виртуальные потоки - решение. Не threading-динозавры, а новое поколение лёгких потоков. Откат?. . .
Поиск "дружественных имён" СОМ портов
Argus19 22.11.2025
Поиск "дружественных имён" СОМ портов На странице: https:/ / norseev. ru/ 2018/ 01/ 04/ comportlist_windows/ нашёл схожую тему. Там приведён код на С++, который показывает только имена СОМ портов, типа,. . .
Сколько Государство потратило денег на меня, обеспечивая инсулином.
Programma_Boinc 20.11.2025
Сколько Государство потратило денег на меня, обеспечивая инсулином. Вот решила сделать интересный приблизительный подсчет, сколько государство потратило на меня денег на покупку инсулинов. . . .
Ломающие изменения в C#.NStar Alpha
Etyuhibosecyu 20.11.2025
Уже можно не только тестировать, но и пользоваться C#. NStar - писать оконные приложения, содержащие надписи, кнопки, текстовые поля и даже изображения, например, моя игра "Три в ряд" написана на этом. . .
Мысли в слух
kumehtar 18.11.2025
Кстати, совсем недавно имел разговор на тему медитаций с людьми. И обнаружил, что они вообще не понимают что такое медитация и зачем она нужна. Самые базовые вещи. Для них это - когда просто люди. . .
Создание Single Page Application на фреймах
krapotkin 16.11.2025
Статья исключительно для начинающих. Подходы оригинальностью не блещут. В век Веб все очень привыкли к дизайну Single-Page-Application . Быстренько разберем подход "на фреймах". Мы делаем одну. . .
КиберФорум - форум программистов, компьютерный форум, программирование
Powered by vBulletin
Copyright ©2000 - 2025, CyberForum.ru