Правильно ли написал программы?

[Lynsha]

Правильно ли написал программы?
Задача №1
Написать программу, которая удаляет из бинарного файла все нечетные числа. Программа должна содержать следующие функции: 1. функция, которая заполняет, созданный в функции main, бинарный файл целыми числами. 2. функция, которая выводит содержимое бинарного файла. 3. функция, которая удаляет из бинарного файла все четные числа. (Дополнительных массивов и файлов не использовать).

#include<stdio.h>
#include<io.h>
#include<conio.h>
// указатель на файл
FILE*fp;
int count;
int fill_file();
int show_file();
int udalenie_ne4etnuh();

void main (void)
fp=fopen("file","wb+");
if(fp!=NULL)
{
fill_file()
printf("Soderjimoe faila:\n");
show_file();
udalenie_ne4etnuh();
printf("Soderjimoe faila posle udaleniya ne4etnuh 4isel:\n");
show_file();
fclose(fp);
}
getch();
// функция заполнения файла целыми числами 
//окончание ввода-при первой же ошибке ввода
int fill_file()
int num;
printf("Vvedite celye 4isla:\n");
while(1)
{
if(!scanf("%d",&num))
return 0;
fwrite (&num, sizeof(int),1,fp);
}
}
int show_file()
{
int num;
fseek(fp,0,SEEK_SET);
for(;;)
{
fread(&num, sizeof(int),1,fp);
if(feof(fp)) break;
printf("%d",num);

printf("\n");
return 0;
}
int udalenie_ne4etnyh()
{
int num;
long int p;
fpos_t pos1, pos2;
fseek (fp,0,SEEK_SET);
fgetpos (fp, &pos1);
fgetpos (fp, &pos2);
for(;;)
{
fread(&num, sizeof(int),1,fp);
if(feof(fp))
break;
fgetpos(fp,&pos2);
if(num%2!=0)
continue;
else
{
fgetpos (fp, &pos1);
fwrite (&num, sizeof(int),1,fp);
p=ftell(fp);
fgetpos (fp, &pos1);
fgetpos (fp, &pos2);
}
}
fseek(fp,0,SEEK_SET);
chsize(fileno(fp),p);
}

-----------------------------------------------------------
Задача №2
Ввести символьную строку. Посчитать количество слов в предложении. Если предложение состоит не из одного слова, то переписать второе слово в предложении справа налево. Вывести преобразованную строку. (При выводе строк использовать функцию puts и дополнительных строк не использовать)

#include <stdio.h>
#include <string.h>

void InputString(char *str);
void FormatString(char *str);

void main()
{
const int size = 255; // Размер строки
char str[size]; 	// Строка
InputString(str);// Ввод строки
// Решение 
FormatString(str);
}
void InputString(char *str)
{
printf("Input string: ");
gets(str);
}
void FormatString(char *str)
{
int numWords = 0;
int startWord = 0;
int endWord = 0;
int i;
for (i = 0; i < strlen(str) - 1; i++)
{
if ((str[i] == ' ') && (str[i+1] != ' '))
{
// Окончание второго слова
if (numWords == 1)
{
if (str[i] == ' ')
{
endWord = i-1;
}
}
numWords = numWords + 1;
}
// Начало второго слова
if (numWords == 1 && startWord == 0)
{
startWord = i+1;
}
}
if (str[0] != ' ')
{
numWords = numWords + 1;
if (numWords == 2)
{
endWord = strlen(str) - 1;
}
}
// Переворачивание строки
int halfWordLen = startWord + (int)((endWord-startWord)/2); //
int k, j;
for (j = startWord, k = 0; j <= halfWordLen; j++, k++)
{
char buf;
buf = str[j];
str[j] = str[endWord-k];
str[endWord-k] = buf;
}
printf("There are %d words in the string\n" , numWords );
puts(str);
}

[Lynsha]

Вижу программистов очень много!!!

[p51x]

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

[Lynsha]

Сообщение от p51x

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

У тебя никто не просит решить LOL, а по поводу оформления пусть тебя не ..ет!!!

[NiCola999]

У тебя никто не просит решить LOL, а по поводу оформления пусть тебя не ..ет!!!

это ты зря

по теме: запускать не пробовал?

если ты хочешь, чтобы форумчане смотрели твой код учись его нормально оформлять
вместо

while(1)
{
if(!scanf("%d",&num))
return 0;
fwrite (&num, sizeof(int),1,fp);
}

надо, например, вот так:

while(1) {
         if(!scanf("%d",&num))  return 0;
         fwrite (&num, sizeof(int),1,fp);
}

а то хрен поймешь, какая скобка для какого ифа

[MeTeOpA]

Сообщение от Lynsha

У тебя никто не просит решить LOL, а по поводу оформления пусть тебя не ..ет!!!

Смысл этой темы? Не ты делал задание?

[Obey-Kun]

Без нормального оформления твой код читать лень. Есть astyle, есть gnu indent. Да что уж там, все IDE'шки умеют код форматировать.

[russian-stalker]

Сообщение от Obey-Kun

Без нормального оформления твой код читать лень. Есть astyle, есть gnu indent. Да что уж там, все IDE'шки умеют код форматировать.

...есть Tab, есть безымянный палец левой руки...

[ACE Valery]

Lynsha, чтобы узнать, правильно ли - запустите.
Лень запускать - нечего оскорблять участников форума.
Короче, закрыто.