Đã Hủy

(code) review

A project in commissioned in India is delayed by several weeks:

History:

We set-up the requirements (incl. mock-ups) for the new website (see attachments).

We awarded the website to a company in India.

We decided to go for zend

The staff was almost overstrained. The project was late for more the 3 month…

In the meantime the have delivered something – you see it here:

Frontend:

[url removed, login to view]

User: champs

champunity

Backend follows separately if you have free capacity in the next days/ weeks:

The task:

- We need someone who is able help us in the first step: bug fixing with current system. We would like to come up the version which could be public-beta – i.e. it must function – it does not have to be perfect.

I thinks it could be useful if you also check the code in a first place an than provide an estimation on

a) If the code is readable/ usable from your perspective at all

b) What you think would be required to for you to have test system to fix the obvious bugs. In second step to additional fix bugs from the users perspective (we use jira for this – I will provide access to you later).

I hope this information is sufficient as of now. Please come back to us if you have any questions.

Timing and budget mentioned below are indicative. Please feel free to comment.

Regards,

Sven

PS: a friend already carried out some code review. Below you find is commens (in German – maybe you can read it.

1) fehlende Dokumentation

1.1) Struktur: wo finde ich was? -> Bugfixes

1.2) Systemanforderungen: was brauche ich? -> Entwicklungsumgebung / Betrieb

1.3) Quelltext: was tut eine eine Klasse/Funktion? -> erwartetes Verhalten

2) fehlende Tests

2.1) mindestens Unittests für Businesslogik, z.B. Rankingalgorithmus

2.2) idealerweise auch Integrationstests, die z.B. den Prozessfluss prüfen

2.3) mittelfristig auch zum Abfangen von SQL-Injections

3) Veraltete Bibliotheken, z.B. ZendFramework [url removed, login to view]

3.1) Wieso ZF1 und nicht ZF2, das auch schon länger stabil ist?

3.2) [url removed, login to view] listet einige Security Fixes für

ZF1

4) Hardgecodete Zeitzone Calcutta?!?

5) Absicherung des Servers

5.1) [url removed, login to view] -> Passwort

5.2) dabei kann ich Euch auch gerne unterstützen

6) wozu quasi leere Dateien ?!?, z.B.

application/modules/default/views/scripts/index/[url removed, login to view]

7) Syntaxfehler im JavaScript und HTML

8) "nicht gut riechender" Code, z.B. select4JoinResult() und Co.

8.1) sieht extrem unstrukturiert, teilweise falsch und v.a. fehleranfällig aus

(und das einzige an Doku sind dann auskommentierte Codestellen...)

9) keine Überprüfung von erwarteten Parametern führen zu kaputten Seiten, z.B.

[url removed, login to view]

9.1) damit verwandt: keine Plausibilitätsprüfung der URL-Routen, wie z.B.

[url removed, login to view]

9.2) wo dann auffällt, dass Stacktrace + Parameter angezeigt werden -> für eine

Entwicklungsumgebung okay, aber sollte produktiv nicht angezeigt werden

10) Bzgl. der Berücksichtigung der Nicht-Funktionalen Anforderungen habe ich

keine Hinweise gefunden (insbesondere failover, scalability und caching)

10.1) eine Trennung von Logik und Darstellung gibt es, scheint mir aber

teilweise recht unstrukturiert

Kỹ năng: MySQL, PHP, Zend

Xem thêm: review javascript, review bug, html free code, get html code url, free backend html, bug review, jira system requirements, company find staff, zum, wo, sql l, sind, profile id, need new security code, jira, gut, first review, code india, code review, javascript function timing, beta test website, test beta website, phtml, indicative, code review javascript

Về Bên Thuê:
( 18 nhận xét ) Hamburg, Germany

Mã Dự Án: #4545001

1 freelancer đang chào giá trung bình $150 cho công việc này

a25team

Hired by the Employer

$150 USD trong 7 ngày
(71 Đánh Giá)
7.4