Adding exception handling for better user experience

[tinodj]

This line

php-amqplib/PhpAmqpLib/Channel/AMQPChannel.php

Line 1029 in 31c6ec8

call_user_func($this->callbacks[$consumer_tag], $message);

might get replaced with

            if (is_callable($this->callbacks[$consumer_tag])) {
                call_user_func($this->callbacks[$consumer_tag], $message);
            }else{
                throw new AMQPInvalidArgumentException(sprintf('%s is not callable.',var_export($this->callbacks[$consumer_tag], true)));
            }

for better user experience.

[lukebakken]

Would you mind checking if there are other places this could apply and open a pull request with a test to address them? Thank you.

[ramunasd]

@tinodj callback check in this place would hurt performance, it's better to check it before assignment, in method basic_consume()

[tinodj]

@tinodj callback check in this place would hurt performance, it's better to check it before assignment, in method basic_consume()

I agree - you are right.