ET12DE1Computacion / Fulbo12

Proyecto interno para simulador de Futbol
0 stars 10 forks source link

Quimica - MismaNacionalidad #3

Closed luchoxx87 closed 2 years ago

luchoxx87 commented 2 years ago

Se pretende conocer si un Futbolista tiene la misma Nacionalidad que otro Futbolista. Recomiendo que se implemente primero el método bool MismaNacionalidad(Persona) en Persona:

// Clase Persona
bool MismaNacionalidad(Persona otra)

y se implemente de igual forma en Futbolista:

// Clase Futbolista
bool MismaNacionalidad(Futbolista otro)

Ejemplos de salidas

// Devuelve true porque ambos son argentinos
diegoMaradona.MismaNacionalidad(lioMessi);

// Devuelve false porque cr7 es portugués
lioMessi.MismaNacionalidad(cr7);
luchoxx87 commented 2 years ago

@Liono02 o @MMorandiniGit ¿quién se ve haciendolo primero en el transcurso de este finde?

Se pretende conocer si un Futbolista tiene la misma Nacionalidad que otro Futbolista. Recomiendo que se implemente primero el método bool MismaNacionalidad(Persona) en Persona:

// Clase Persona
bool MismaNacionalidad(Persona persona)

y se implemente de igual forma en Futbolista:

// Clase Futbolista
bool MismaNacionalidad(Futbolista futbolista)
Liono02 commented 2 years ago

Profe,ahi envie el commit del método de MismaNacionalidad,dígame si le llega por favor. Desde ya,muchas gracias!!

El vie, 22 de oct. de 2021 a la(s) 14:35, Luis Duran ( @.***) escribió:

@Liono02 https://github.com/Liono02 o @MMorandiniGit https://github.com/MMorandiniGit ¿quién se ve haciendolo primero en el transcurso de este finde?

Se pretende conocer si un Futbolista tiene la misma Nacionalidad que otro Futbolista. Recomiendo que se implemente primero el método bool MismaNacionalidad(Persona) en Persona:

// Clase Persona bool MismaNacionalidad(Persona persona)

y se implemente de igual forma en Futbolista:

// Clase Futbolista bool MismaNacionalidad(Futbolista futbolista)

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ET12DE1Computacion/Fulbo12/issues/3#issuecomment-949834361, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUQTOYULLZUEO3P7HCMCXOTUIGOH5ANCNFSM5GKI6YUA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

luchoxx87 commented 2 years ago

No veo ningún commit @Liono02 o algún repo tuyo forkeado (como hiciste cuando agregaste filas en el proyecto de sql).

Recorda que podes colaborar con este proyecto haciéndote un fork y después con pull request contra tu fork. link de utilidad

Liono02 commented 2 years ago

Listo,creo que ahora si está publicado. perdón 😅

El sáb, 23 de oct. de 2021 a la(s) 22:27, Luis Duran ( @.***) escribió:

No veo ningún commit @Liono02 https://github.com/Liono02 o algún repo tuyo forkeado (como hiciste cuando agregaste filas en el proyecto de sql).

Recorda que podes colaborar con este proyecto haciéndote un fork y después con pull request contra tu fork. link de utilidad

— You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub https://github.com/ET12DE1Computacion/Fulbo12/issues/3#issuecomment-950243050, or unsubscribe https://github.com/notifications/unsubscribe-auth/AUQTOYWA6NLRAHHF472R433UINOIZANCNFSM5GKI6YUA . Triage notifications on the go with GitHub Mobile for iOS https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675 or Android https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub.

luchoxx87 commented 2 years ago

Ahi lo vi, no pasa nada @Liono02 , gracias a vos por tu colaboración. Chusmea porfas lo que te puse en #4

luchoxx87 commented 2 years ago

Buenas @Liono02, vi los cambios que sugeriste; te di el feedback en #5 ; te recomiendo porfas que veas lo que puse ahí y hagas las correcciones haciendo commit y push sobre el branch que propusiste para el PR.