Perché così, strutturalmente/sintatticamente, non ha assolutamente senso. Lì stai definendo un metodo (shouldOverrideUrlLoading) ... NON lo stai invocando. Quindi la stringa letterale in rosso
public boolean shouldOverrideUrlLoading(WebView view, "http://www.milanclubalife.it")
in quel punto è assolutamente errata.
Non me ne intendo di Android ma ridefinire quel shouldOverrideUrlLoading presumo abbia un senso del tipo: "guarda webview, ti passo un WebViewClient per cui se riceve notifica per gestire un url, fallo aprire nella web view (e non altrove, in altro modo)".
Correggetemi se sbaglio.
In sostanza, lì non devi mettere qualcosa di specifico, perché applica un concetto generico.