Revisione del codice ragionevole per il progetto su specifica?

Do I have the wrong perspective here?

In parte, sì. Può essere molto difficile per una persona non tecnica discernere se qualcosa che hanno è una robaccia ben costruita o mal tenuta. Ad esempio, facevo parte di una squadra che è stata invitata a consultare una grande base di codice per un prodotto simile a Photoshop che era stato venduto dai suoi proprietari alla fine degli anni '90.

Tuttavia, posso capire la tua esitazione. Hai fatto del lavoro e stai producendo valore per loro e non vuoi che controllino il tuo codice, rubino le idee principali e producano il loro prodotto senza darti nulla. Questa è una preoccupazione legittima.

Quindi cosa fare? Sarebbe possibile trovare qualcuno di cui entrambi ti fidi per la recensione? Qualcuno di cui potresti essere sicuro non guarderebbe solo il tuo codice e farà una riscrittura banale, ma a chi potrebbero fidarsi per dire se è un codice ragionevolmente gestibile? Se è così, potrebbe essere la soluzione migliore. In caso contrario, potresti essere in un vicolo cieco.

Consideralo in questo modo, se gli hai permesso di esaminarlo e hanno rubato le tue idee e riscritto con il loro codice, cosa ti costerebbe? Hai già fatto il lavoro gratuitamente. Puoi rivenderlo ad un altro cliente? Se lo esaminano e non lo trovano nei loro standard, ma non lo riscrivono da soli, ti ritroverai nella stessa posizione.

Nei commenti chiedi:

I guess following that logic, he should be fine with me finding an MBA to grill him on his business acumen.

Sì, assolutamente! Dovresti sapere con chi stai facendo affari. Controlla il Better Business Bureau per vedere se hanno presentato reclami contro di loro (se ritieni che la BBB sia utile). Sicuramente chiedi a un imprenditore affermato che conosci di controllare il loro piano aziendale se sei preoccupato della loro capacità di pagarti in futuro. Si chiama due diligence ed è una pratica commerciale ampiamente accettata. Se non lo fai, ci possono essere casi in cui non hai nessuno da incolpare ma te stesso per non ottenere ciò che ti aspetti dalla relazione. (Sono stato lì! Non è divertente.) Buona fortuna!

Quanto è grande il codebase in totale? Se il fondatore vuole solo un'idea generale della tua abilità di programmazione, forse sarebbe disposto a far controllare una parte di esso su una terza parte. È quindi possibile selezionare il codice inviato in modo che includa alcuni "bit intelligenti" ma non tutti. Ciò ti consentirebbe di mantenere il tuo IP, pur facendo comunque uno show di buona fede.

Se un nuovo sviluppatore inizia a contribuire a un progetto esistente, una revisione del codice è perfettamente ragionevole, ma capisco perfettamente le tue preoccupazioni: mi sentirei esattamente allo stesso modo.

Le revisioni del codice possono essere processi spinosi se non vengono condotti in modo corretto. Sarei propenso a lasciare che la recensione accadesse e lasciare che ne facessero quello che vogliono, spiegando che si tratta di un work in progress. Non ho mai sentito di due sviluppatori completamente d'accordo su un approccio per un pezzo di software di qualsiasi dimensione - quindi prendi qualsiasi feedback in buona grazia. Per inciso, credo che ci siano strumenti che ti permetteranno di fare una revisione online senza che alcun codice passi realmente alle mani.

Sentiti libero di chiedere le credenziali di questo ingegnere - ti darà un vantaggio su cosa aspettarti. Se non hanno mai codificato nella lingua di destinazione, potrebbe essere ... come posso mettere questo ... interessante!

Se potessi offrire una perla di consiglio, è d'accordo su quanto ci si aspetta da entrambe le parti fin dall'inizio (vedi la nota cautelativa qui sotto!) sebbene sia ovviamente facile dirlo adesso. Se senti di essere sfruttato, vai via. Non hanno pagato per la tua esperienza e la buona volontà dovrebbe solo andare così lontano.

Alcuni miei amici hanno creato uno start up e il proprietario e l'analista di business hanno chiesto che il codice fosse revisionato prima di andare in diretta dopo 2 anni di duro innesto su tutti i lati. Lo sviluppatore ha rifiutato e ha dovuto ricominciare da capo.