Добрый день.

Я новичок в C++, работал с Delphi, на C не писал.

Читаю Либерти - Освой самостоятельно С++ за 21 день.
Решил попробовать для тренировки написать простой класс для работы с dbf.
Буду очень благодарен за замечания к моему коду: ошибки, неверные подходы,
скрытые угрозы, реализация чере ж..., таких косяков думаю здесь много.

Вообще, C++ понравился очень, возможности впечатляют, но иногда некоторые простые вещи
не понятно как делать, например работа с char.

Делалось все в Dev-C++ 4.9.9.2. Исходники присоединены. 
Заранее спасибо за любые комментарии.


Присоединённый файл ( Кол-во скачиваний: 20 )
 dbf.rar 4,33 Kb

а может просто небольшой кусок кода или функцию выкинешь, что тебя смущает?
тока не огромное полотно, а кусок.

мне, к примеру, лень качать, запускать Dev-C++ 4.9.9.2....

Поскольку мало что понимаю что хорошо-плохо хочется комментарий по всему коду, там немного, строк 200
компилиться и в TC++

Не разраривается   у меня линь   

Это сообщение отредактировал(а) GrayCardinal - 6.11.2007, 16:44

Вот блин
Положил zip

Присоединённый файл ( Кол-во скачиваний: 5 )
 dbf.zip 4,47 Kb

tatan, 
Так, на вскидку :
1. if не надо закрывать точкой с запятой.
2. Определение класса должно быть в хэйдере.

Лучше сделать "inline" (в каждую строчку, в начало). И тоже последние ";" не нужны.
3. for тоже не надо ";" в конце.
4. Рекомендую { ставить в той же строчке, что и оператор.
5. Все переменные  должны быть в начале функции.
Вроде всё 

unrar есть 


С-стиль, весьма спорно.

Изварт чистейшей воды.
Проще так (на коленке):


Добавлено через 48 секунд
JackYF, 
Ничего спорного. Спросили как надо, я ответил  

Добавлено через 1 минуту и 15 секунд
JackYF, 
Унрар есть, я в курсе.

to GrayCardinal
 Все принято, большое спасибо
Ну кроме про "{", но это кажется к области религиозных войн )

с sprintf - о как, не придумал бы сам-то, согласен у меня изврат, я долго думал как записать строку формата

Далее. Указатели, перед тем как их удалять лучше проверять на NULL. И соответственно, после удаления присваивать NULL - для гарантии что не удалишь два раза один и тот же указатель.

Добавлено через 1 минуту и 3 секунды
tatan, 
Завсегда пожалуйста   

Указатели кстати проверял
Но в предыдущей теме ктото сказал что проверка указателей на валидность почти ошибка в проектировании, все и убрал оставив на потом

tatan, 
Не читал тот флуд, но на нуль всегда проверяю. Так жить проще   

Добавил в список рекомендаций - после удаления указателям присваивать NULL. "Так жить проще" (с) 

А кто сказал, что так надо? 



А я рекомендую так не делать .



Ещё проще .


Добавлено через 48 секунд

Нужно:

archimed7592, 
А передёргивать не надо