Ti sconsigliamo di usare sempre null poiché può portare a un ulteriore NPE, che è difficile da eseguire il debug (e costano molto se si verificano nel codice di produzione).

Soluzione 1 (metodo di sovraccarico)

Se non viene fornito deviceName , puoi invece fornire uno di default. Il più grande svantaggio di questo approccio è il pericolo in genericDeviceMap.put(deviceName, device) perché può ignorare in modo silenzioso la voce la cui chiave è il nome predefinito (quindi, perdere la traccia del precedente Device ).

public void attachDevice(Device device)
{
    attachDevice(device, "DefaultName");       
}

public void attachDevice(Device device, String deviceName)
{
    ..
    device.setName(deviceName);
    genericDeviceMap.put(deviceName, device);
    ..
}

Soluzione 2 (metodo di estrazione)

Forse con la tua architettura attuale non ha senso aggiungere una voce a genericDeviceMap quando attachDevice viene chiamato senza un nome. Se è così, un buon approccio è solo estrarre il comportamento comune tra il due attachDevice in metodi privati. Personalmente non mi piace questo approccio per 2 motivi:

• Il comportamento tra i due attachDevice non è lo stesso, uno ha un effetto collaterale ( device.setName(deviceName) ) e l'altro no
• L'effetto collaterale di per sé che spesso porta a bug sottili perché modifichi un oggetto proveniente da un ambito esterno

Codice:

public void attachDevice(Device device)
{
    preAttachDevice();
    postAttachDevice();      
}

public void attachDevice(Device device, String deviceName)
{
    preAttachDevice();
    device.setName(deviceName);
    genericDeviceMap.put(deviceName, device);
    postAttachDevice();
}

private void preAttachDevice()
{
    ...
}

private void postAttachDevice()
{
    ...
}

Soluzione 3 (metodo di rimozione)

Il mio preferito, ma il più difficile. Chiediti se hai davvero bisogno di questi due metodi? Ha davvero senso poter chiamare attachDevice con un nome o no? Non dovresti essere in grado di dire che attachDevice deve essere chiamato con un nome?

In questo caso il codice è semplificato con un solo metodo

public void attachDevice(Device device, String deviceName)
{
    ..
    device.setName(deviceName);
    genericDeviceMap.put(deviceName, device);
    ..
}

O d'altra parte, hai davvero bisogno di mantenere un Map di nomi di dispositivi e dispositivi e di impostare il nome del dispositivo? In caso contrario, puoi sbarazzarti del secondo metodo e mantenere solo il primo.

public void attachDevice(Device device)
{
    ...
    ...     
}

Direi che il tuo capo squadra ti sta chiedendo di creare codice errato. Il dover passare un valore arbitrario come parametro name quando non si desidera specificare un nome è disordinato e confuso per tutti gli sviluppatori che desiderano utilizzare il metodo.

Invece, mantieni i tuoi due metodi (anche se rinominerei il tuo ultimo con qualcosa come attachNamedDevice per rendere chiaro che fa qualcosa di diverso da attachDevice , quindi sposta il codice comune in metodi privati:

public void attachDevice(Device device)
{
    preDeviceNameSetup(device);
    postDeviceNameSetup(device);        
}

public void attachNamedDevice(Device device, String deviceName)
{
    preDeviceNameSetup(device);
    device.setName(deviceName);
    genericDeviceMap.put(deviceName, device);
    postDeviceNameSetup(device);        
}

private void preDeviceNameSetup(Device device)
{
    ...
}

private void postDeviceNameSetup(Device device)
{
    ...
}

In tal modo, mantieni l'API pulita, ma evita la duplicazione del codice nell'implementazione.

Farei quanto segue:

1. Rimuovi la seconda funzione con String come secondo parametro.
2. Chiama device.setName(deviceName) prima di chiamare attachDevice(device) . Non è responsabilità del metodo attachDevice impostare il nome del dispositivo.
3. Modifica il metodo attachDevice per aggiungere il dispositivo a genericDeviceMap se ha un nome, come mostrato nello snippet di codice seguente.

Se hai anche bisogno di aggiungere il dispositivo a genericDeviceMap quando non ha un nome, probabilmente stai meglio impostando un valore predefinito nel costruttore della classe Device o qualcosa del genere.

public void attachDevice(Device device)
{
    // ...
    if (device.getName())
    {
        genericDeviceMap.put(deviceName, device);
    }
    // ...
}

Ciò mantiene le tue preoccupazioni separate e ti aiuta a stare lontano dalle funzioni con effetti collaterali.