fix for NSInternalInconsistencyException - caused by nil bridge in TransportHandler

[joshbalfour]

Love the lib! Building a multiplayer game and it works well 😃

When using the library I got this error (#30):

NSInternalInconsistencyException. This is probably because you've explicitly synthesized the bridge in NetworkManager, even though it's inherited from RCTEventEmitter.

To fix this I refactored out the RCTEventEmitter into a separate object which is instantiated on app start, which TransportHandler can then use when it needs.

I followed this guide https://gist.github.com/brennanMKE/1ebba84a0fd7c2e8b481e4f8a5349b99 I'm not a swift programmer by any stretch, so there may be a better way of doing this but this works 😊

[michaeldanielbell]

@joshbalfour Nice job, out of interest, once devices are connected in your multiplayer game. How are you communicating between devices? Using the BluetoothCP.sendMessage? Or an alternative way?

[joshbalfour]

@michaeldanielbell Thanks!

I am, a simplified version of how i'm doing this is:

const {
    getConnectedPeers,
    addReceivedMessageListener,
    sendMessage,
} = require('react-native-bluetooth-cross-platform')

import newGameStateHandler from './newGameState'
import peerIdentifiedHandler from './peerIdentifiedHandler'

export const broadcast = ({ type, data }) => {
    getConnectedPeers(peers => {
        peers.forEach(peer => {
            console.log('sending', { type, data })
            sendMessage(JSON.stringify({ type, data }), peer.id)
        })
    })
}

export const emit = ({ type, data, id }) => {
    console.log(`emitting to ${id}`, { type, data })
    sendMessage(JSON.stringify({ type, data }), peer.id)
}

addReceivedMessageListener(({ message, ...peer }) => {
    let msg
    try {
        msg = JSON.parse(message)
    } catch (e) {
        console.error('non json parsable message recieved')
    }
    if (msg) {
        const { type, data } = msg
        if (!type) {
            console.error('message without type recieved')
        }

        switch (type) {
            case 'new-game-state':
                newGameStateHandler({ gameState: data, peer })
                break;
            case 'identify':
                peerIdentifiedHandler({ peer, identity: data })
                break;
            default:
                break;
        }
    }
})

[michaeldanielbell]

👍

[alexkendall]

Hi @joshbalfour. Thanks for the changes! Sorry I haven't been able to get onto this sooner. Planning on testing these and merging them in.

[vimalathiyagu]

@alexkendall : it would be great if you could merge these changes into the lib if all well. Since we are using this library and the issue is causing crashes consistently...