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

@troopermanowar · Регистрация: 22.09.2017

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

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

найти и исправить 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);

@Jodah · 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);

@pav1uxa · 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 на другое число и увидишь еще парочку... или тройку...

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

@troopermanowar · 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;
    }
}

@Jewbacabra · 22.09.2017, 20:54

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

вот и правильный ответ:

И в чем принципиально использование bindTo отличается от передачи через use?

@troopermanowar · 22.09.2017, 21:21 **[ТС]**

И в чем принципиально использование bindTo отличается от передачи через use?

Ну вот и я не понял. С их точки зрения этот вариант правильный а мой нет

Добавлено через 1 минуту
Типа чтоб метод test остался private

@Jewbacabra · 22.09.2017, 21:30

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

Типа чтоб метод test остался private

Они жгут. Какой смысл тогда выносить валидацию в коллбек и прятать его? Пользователя класса по идее не должно волновать все что скрыто уровнем доступа, и как такой конструкцией пользоваться в реальности не понятно.

@Fedor Vlasenko · 22.09.2017, 23:15

Надо было завернуть, ответив Ваш говнокод сами в нем и разбирайтесь
Это код со всех сторон является таким

@pav1uxa · 23.09.2017, 08:36

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

Ага, а вот и правильный ответ:

Ни разу неправильный. Читайте внимательней.

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

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

@troopermanowar · 23.09.2017, 10:26 **[ТС]**

Ни разу неправильный.

Имеется в виду то что они считают правильным

@pav1uxa · 23.09.2017, 12:28

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

Имеется в виду то что они считают правильным

Ну так укажите им ошибки. Может возьмут тогда

Новые блоги и статьи Все статьи Все блоги /
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 . Быстренько разберем подход "на фреймах". Мы делаем одну. . .

@Fedor Vlasenko 936 / 693 / 236 Регистрация: 01.02.2015 Сообщений: 1,848
	22.09.2017, 23:15
	Надо было завернуть, ответив Ваш говнокод сами в нем и разбирайтесь Это код со всех сторон является таким 0