Skip to content

Conversation

@limingxinleo
Copy link
Member

@limingxinleo limingxinleo commented May 17, 2020

fix #1

@limingxinleo limingxinleo requested review from Reasno and huangzhhui May 17, 2020 03:19
README-CN.md Outdated

require_once __DIR__ . '/vendor/autoload.php';

class DemoConsumer extends Amqp\Message\ConsumerMessage
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里这么写感觉有点乱,改成匿名类的写法会不会好一些呢?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

嗯,我今天再改改

README-CN.md Outdated

$app->addProcess(function () use ($container) {
$message = new DemoConsumer();
$consumer = $container->get(Amqp\Consumer::class);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这里直接用$this就行

@limingxinleo limingxinleo merged commit 4c419d9 into master Jun 5, 2020
@limingxinleo limingxinleo deleted the v1.0-limx-amqp-doc branch June 5, 2020 10:26
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.

基于nano 实现amqp 消息投递和消费

4 participants