evgueni-petrov-aka-espetrov / 18203 Goto Github PK

Line 21 in c29e99e

}else{ printf_s("%s", "Bad input"); }

Пожалуйста, оформляйте все управляющие инструкции так:

if (...) {
    ...
} else {
    ...
}

Shishkin: use strlen

18203/Ivanov/1.0/Main.cpp

Line 2 in daae167

int amountOfNum (char b[]) {

Ivanov: FILE* toOpen = fopen(...); if (toOpen == NULL) { ... }

Line 79 in 2e71a63

if (!(toOpen = fopen(filename, mode))) {

Shishkin: return errornum != 0;

18203/Nalepova/0/Исходный код.cpp

Line 21 in 332697d

if (errornum != 0) return 1;

Алдонин: (1) это не "быстрая сортировка"; (2) падает на первом тесте

18203/Aldonin/3.1/main.c

Line 10 in 20dd0da

void quickSort(int *stack,int size)

Налепова: падает 3й тест

Line 25 in e1e34e6

int main() {

c:\Users\espetrov\WorkingPlace\Преп\18203\Nalepova\0>cl /RTC1 /W4 "Исходный код.cpp"
Microsoft (R) C/C++ Optimizing Compiler Version 19.00.24215.1 for x64
Copyright (C) Microsoft Corporation. All rights reserved.

Исходный код.cpp
C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\INCLUDE\xlocale(341): warning C4530: C++ exception handler used, but unwind semantics are not enabled. Specify /EHsc
Исходный код.cpp(51): warning C4267: '=': conversion from 'size_t' to 'int', possible loss of data
Исходный код.cpp(56): warning C4244: 'argument': conversion from 'int' to 'short', possible loss of data
Исходный код.cpp(56): warning C4244: '=': conversion from 'double' to 'int', possible loss of data
Исходный код.cpp(64): warning C4244: 'argument': conversion from 'int' to 'short', possible loss of data
C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\INCLUDE\exception(359): warning C4577: 'noexcept' used with no exception handling mode specified; termination on exception is not guaranteed. Specify /EHsc
Microsoft (R) Incremental Linker Version 14.00.24215.1
Copyright (C) Microsoft Corporation. All rights reserved.

"/out:Исходный код.exe"
"Исходный код.obj"

c:\Users\espetrov\WorkingPlace\Преп\18203\Nalepova\0>......\labs\0-num-sys\testlab0.exe "Исходный код.exe"

Testing Lab 0 Number Systems...
TEST 1/30: PASSED
TEST 2/30: PASSED
TEST 3/30: FAILED

:-(

Exe ╚ёїюфэ√щ ъюф.exe failed for input file in.txt in the current directory.
Please fix and try again.

c:\Users\espetrov\WorkingPlace\Преп\18203\Nalepova\0>type in.txt
2 2
2.01

c:\Users\espetrov\WorkingPlace\Преп\18203\Nalepova\0>type out.txt
2.01
c:\Users\espetrov\WorkingPlace\Преп\18203\Nalepova\0>

Кондратьев: coding style: должно быть не более одной ; в строке, исключая for(;;)

18203/Aldonin/1.1/lab1_1.c

Line 13 in c29e99e

 char s[57]; int a,b; double res; //s - строка ввода-вывода, a,b - системы счисления для ввода и вывода соотв. 

Aldonin: не проходит в режиме контроля выхода за границу массива во время исполнения

Line 35 in 2e71a63

int main()

c:\Users\espetrov\WorkingPlace\Преп\18203\Aldonin\1.1>cl /W4 /D_CRT_SECURE_NO_WARNINGS /RTC1 *.c
Microsoft (R) C/C++ Optimizing Compiler Version 19.00.24215.1 for x64
Copyright (C) Microsoft Corporation. All rights reserved.

lab1_1.c
lab1_1.c(43): warning C4267: 'initializing': conversion from 'size_t' to 'int', possible loss of data
lab1_1.c(75): warning C4244: 'initializing': conversion from 'int' to 'unsigned char', possible loss of data
Microsoft (R) Incremental Linker Version 14.00.24215.1
Copyright (C) Microsoft Corporation. All rights reserved.

/out:lab1_1.exe
lab1_1.obj

c:\Users\espetrov\WorkingPlace\Преп\18203\Aldonin\1.1>......\labs\1-1-rabin-karp\testLab11.exe lab1_1.exe

Testing Lab 1-1 Rabin-Karp...
TEST 1/19: PASSED
TEST 2/19: PASSED
TEST 3/19:
Exe "lab1_1.exe" terminated with exception 0x80000003

:-(

Exe lab1_1.exe failed for input file in.txt in the current directory.
Please fix and try again.

Shishkin: swaparr -> reverse_array, move int x inside loop

Line 83 in 2c50ce4

void swaparr (int n[], int s) {

Кондратьев: приоритет && выше, чем || -- нужно скобки вокруг ||

Line 112 in c29e99e

 while (i >= pointIndex && (*(res + i) < '0' || *(res + i) > '9' && *(res + i) < 'a' || *(res + i) > 'f')) { 

Ivanov: формально это С++

18203/Savrova/0/num_sys.c

Line 110 in be0a33a

bool checkInput(int a, int b, char* numberA) {

Savrova: не тот отступ тут и в main

Line 58 in be0a33a

int dot(0);

Ищенко: мы пишем на Си

18203/Ischenko/1.0/moor.cpp

Line 2 in c29e99e

#include<iostream>

Shishkin: fix identation, use meaningful names, simplify

18203/Ivanov/1.0/Main.cpp

Line 9 in daae167

int convcheck (char s[], float n[], int b, int c, int am) {

Алдонин: (1) не используется; (2) конфликтует со стандартной atoi

18203/Aldonin/4/calc.c

Line 6 in 20dd0da

int atoi(char c)

Ivanov: или void, или добавить return

Line 86 in 2e71a63

char* fgetsWithoutNewline(char* buffer, int maxCount, FILE* stream) {

Кондратьев: используйте именованную константу

18203/Shishkin/3.0/heap.c

Line 13 in c29e99e

 char s[57]; int a,b; double res; //s - строка ввода-вывода, a,b - системы счисления для ввода и вывода соотв. 

#define MAX_NUMBER_OF_DIGITS 57, а лучше -- enum {MAX_NUMBER_OF_DIGITS=57};

Shishkin: описание var надо унести в цикл

Line 11 in 0940c8a

int var;

Shishkin: лучше int j = num-2; и т.д. всюду

18203/Kondratiev/4/Calc.cpp

Line 36 in 332697d

i = num-2;

Кондратьев: исправьте coding style

Line 16 in c29e99e

}else

Shishkin: what happens if input is "90"?, divide into several small functions

18203/Kondratiev/3.0/HeapSort.cpp

Line 41 in daae167

long long int conv1 (float n[], double m1[], int b, int am) {

Кондратьев: перегрузка функций -- это С++; мы пишем на Си

Line 11 in c29e99e

int makeMaxFirst(int* a, int index1, int index2, int index3);

Ищенко: мы пишем на Си

18203/Ischenko/0/lab0.cpp

Line 1 in c29e99e

#include <iostream>

Shishkin: из цикла можно выйти с помощью break; и тогда не нужна переменная whileexit

18203/Shishkin/3.0/heap.c

Line 18 in 0940c8a

if (var == 0) pos = whileexit;

Ivanov: malloc_s --> safeMalloc

Line 5 in be0a33a

void* malloc_s(size_t size);

Ivanov: достаточно одного из *1.0 и double

18203/Kondratiev/3.1/QuickSort.cpp

Line 77 in be0a33a

result += (double)fromChar*1.0/(double)factor;

Shishkin: не проходят тесты, если собрать debug + /RTC1; неинициализированные переменные

nums2.3.1.c(50) : warning C4701: potentially uninitialized local variable 'result' used
nums2.3.1.c(127) : warning C4701: potentially uninitialized local variable 'pointplace' used
nums2.3.1.c(91) : warning C4701: potentially uninitialized local variable 'number2' used

Кондратьев: те же замечания, что для пирамидальной сортировки и перевода из системы счисления в систему счисления

Line 9 in c29e99e

scanf_s("%d /n", &length);

Shishkin: fout остался незакрытым

Line 53 in 332697d

printf("File could not be opened.");

Кондратьев: тут указатели не нужны

Line 26 in c29e99e

bool goodInput(char* s,int* system1, int* system2) {

Через указатель передаем изменяемые формальные параметры
Через указатель на const передаем большие неизменяемые формальные параметры (строки, массивы)
Небольшие неизменяемые формальные параметры (int, double и т.п.) передаем по значению

Ivanov: утечка памяти тут и в других местах, т.к. free не вызван ни разу

18203/Kondratiev/3.0/HeapSort.cpp

Line 92 in be0a33a

return strcat(fromDecIntToB(dec / b, b), itoa(dec%b));

Кондратьев: scanf("%d")

Line 15 in c29e99e

scanf_s("%d /n", &length);

перевод строки записывается \n

строка с форматом в scanf -- это мини-программа чтения из потока stdin, составленная из следующих команд:

пробел --> пропустить в stdin все пробелы, табуляции, переводы строк
спецификатор формата, начинающийся с % --> если в stdin запись значения указанного после % типа, то считать эту запись, преобразовать в значение, присвоить значение соотв. переменной; иначе прекратить чтение из stdin
любой другой символ --> если в stdin этот символ, то пропустить его; иначе прекратить чтение из stdin

Поэтому scanf("%d /n", &length); считывает length, пропускает все пробельные символы, и либо удаляет из stdin символы / и n (если есть), либо просто прекращает чтение из stdin.
В лабе про сортировку все это можно заменить на один scanf("%d").

Nemtsev: зависает для массива 4, -1, 2 (первый тест)

18203/Nemtsev/3.1/main.c

Line 42 in be0a33a

QuickSort(A, numbersCount);

Кондратьев: bool -- это С++; мы пишем на Си

18203/Aldonin/1.1/lab1_1.c

Line 26 in c29e99e

bool goodInput(char* s,int* system1, int* system2) {

Алдонин: access violation (segfault) на первом тесте

18203/Aldonin/4/main.c

Line 5 in 20dd0da

int main()

Aldonin: тут надо -1

Line 58 in 2e71a63

fprintf(output,"%d ",j+i);

Алдонин: не проходит первый тест

18203/Aldonin/1.0/lab1.c

Line 15 in 20dd0da

int main()

c:\Users\espetrov\WorkingPlace\Преп\18203\Aldonin\1.0>......\labs\1-0-boyer-moore\testLab10.exe lab1.exe

Testing Lab 1-0 Boyer-Moore...
TEST 1/19: wrong output -- FAILED

:-(

Exe lab1.exe failed for input file in.txt in the current directory.
Please fix and try again.

c:\Users\espetrov\WorkingPlace\Преп\18203\Aldonin\1.0>type in.txt
example
this is simple example
c:\Users\espetrov\WorkingPlace\Преп\18203\Aldonin\1.0>type out.txt
7 14 15 14 13 12 11 21 23 22 21 20 19 18 17

Кондратьева: выход за границу массива на первом тесте

18203/Kondratieva/3.1/qsort.c

Line 46 in c8c9a8d

int main() {

Кондратьев: используйте именованную константу

18203/Nalepova/3.0/quick_sort.cpp

Line 111 in c29e99e

 i = 56; //удаление ненужных символов из правой части строки 

Nalepova: нет парного free

Line 36 in be0a33a

a = (int*)malloc(n * sizeof(int));

Shishkin: сделайте, пожалуйста, проверку без вложенных циклов

18203/Kondratieva/3.0/heapSort.c

Line 10 in 332697d

 int check (char symbols[], int num) { //ïðîâåðêà ïðî÷òåííûõ öèôð(return correct=0 or incorrect=1) 

Кондратьева: coding style: malloc и free должны быть в одной области видимости

Line 18 in c8c9a8d

heap = (int*)malloc(sizeof(int)*heapSize);

Алдонин: не проходит первый тест

18203/Aldonin/0/main.c

Line 4 in 20dd0da

int main()

Алдонин: <> используются для стандартных .h (stdio.h, stdlib.h и т.п.); для остальных .h используются ""

18203/Aldonin/4/calc.c

Line 2 in 20dd0da

#include <calc.h>

Shishkin: лишние переменные