- html - 出于某种原因,IE8 对我的 Sass 文件中继承的 html5 CSS 不友好?
- JMeter 在响应断言中使用 span 标签的问题
- html - 在 :hover and :active? 上具有不同效果的 CSS 动画
- html - 相对于居中的 html 内容固定的 CSS 重复背景?
我正在尝试执行一个 Pascal 三角形,我需要将一个数组复制到另一个数组,我目前正在使用复制功能,但遇到了著名的段错误。
这是我的代码:
#include <iostream>
#include <algorithm>
#include <cstring>
using namespace std;
void print(const int *tab, const int &nbr){
for(int i = 0; i<nbr;i++){
cout << tab[i];
}
cout << endl;
}
int main()
{
int *tab;
int *tab1;
int index = 0;
cout << "enter a number less than or equal to 20!" << endl;
int number = 40;
while(number > 20){
cin >> number;
cout << endl;
}
int a = 1;
while(index < number){
tab[0] = 1;
for(int i=1;i<=index;i++){
tab[i] = i;
}
print(tab,index);
std::copy(tab,tab+index,tab1);
index++;
}
return 0;
}
我在使用 memcpy 函数时遇到了同样的错误,任何人都可以吗
最佳答案
(在您的版本之前)
getting segmentation fault when copying arrays using std::copy
问题出现在std::copy
拥有
int *tab;
int *tab1;
...
tab[i] = tab1[i];
tab 和 tab1 未初始化,它们不指向用作数组的内存块,因此行为未定义(在您的情况下是段错误) 每次它们被取消引用时
关于关于number 的代码,你可能想要类似的东西
int tab[20];
int tab1[20]
警告在
for(int i=1;i<=index;i++){
你似乎假设数组的第一个索引是 1,而它是 0
您的代码(在您的版本之后)删除未定义行为的建议,还有一些其他更改,我评论了这些修改。
#include <iostream>
#include <algorithm>
#include <cstring>
using namespace std;
void print(const int *tab, const int &nbr){
for(int i = 0; i<nbr;i++){
cout << tab[i] << ' '; // add a space to separate numbers
}
cout << endl;
}
int main()
{
int number;
do { // your case it typically a "do while"
// print moved inside to clearly indicate the expected input
// even after a number invalid
// and also request a number > 0 else no sence after
cout << "enter a number between 1 and 20!" << endl;
if (!(cin >> number)) { // detect the error else if a non number you loop forever
cerr << "invalid input" << endl;
cin.clear(); // clear the error
// bypass invalid input
string s;
if (! (cin >> s)) {
// EOF !
return -1;
}
number = 0; // to reloop
}
} while ((number > 20) || (number <= 0));
int * tab = new int[number]; // added missing initialization
int * tab1 = new int[number]; // added missing initialization
for (int index = 0; index < number; ++index) {
tab[0] = 1;
for(int i=1; i<=index; i++) {
tab[i] = i;
}
print(tab,index);
std::copy(tab, tab+index, tab1);
}
// free resources
delete [] tab;
delete [] tab1;
return 0;
}
编译和执行:
pi@raspberrypi:/tmp $ g++ -pedantic -Wextra -Wall cp.cc
pi@raspberrypi:/tmp $ ./a.out
enter a number between 1 and 20!
aze
invalid input
enter a number between 1 and 20!
-1
enter a number between 1 and 20!
21
enter a number between 1 and 20!
20
1
1 1
1 1 2
1 1 2 3
1 1 2 3 4
1 1 2 3 4 5
1 1 2 3 4 5 6
1 1 2 3 4 5 6 7
1 1 2 3 4 5 6 7 8
1 1 2 3 4 5 6 7 8 9
1 1 2 3 4 5 6 7 8 9 10
1 1 2 3 4 5 6 7 8 9 10 11
1 1 2 3 4 5 6 7 8 9 10 11 12
1 1 2 3 4 5 6 7 8 9 10 11 12 13
1 1 2 3 4 5 6 7 8 9 10 11 12 13 14
1 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15
1 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16
1 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17
1 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18
pi@raspberrypi:/tmp $
然而在
for (int index = 0; index < number; ++index) {
tab[0] = 1;
for(int i=1; i<=index; i++) {
tab[i] = i;
}
print(tab,index);
std::copy(tab, tab+index, tab1);
}
tab白白初始化了很多次,每个entry只初始化一次就够了
std::copy(tab, tab+index, tab1);
没用,因为 tab1 从未使用过。
可以删除所有与 tab1 相关的内容,只需要:
tab[0] = 1;
for (int index = 1; index < number; ++index) {
tab[index] = index;
print(tab,index);
}
在 valgrind 下执行以检查内存访问和泄漏(已删除 tab1):
pi@raspberrypi:/tmp $ valgrind ./a.out
==16633== Memcheck, a memory error detector
==16633== Copyright (C) 2002-2017, and GNU GPL'd, by Julian Seward et al.
==16633== Using Valgrind-3.13.0 and LibVEX; rerun with -h for copyright info
==16633== Command: ./a.out
==16633==
enter a number between 1 and 20!
10
1
1 1
1 1 2
1 1 2 3
1 1 2 3 4
1 1 2 3 4 5
1 1 2 3 4 5 6
1 1 2 3 4 5 6 7
1 1 2 3 4 5 6 7 8
==16633==
==16633== HEAP SUMMARY:
==16633== in use at exit: 0 bytes in 0 blocks
==16633== total heap usage: 4 allocs, 4 frees, 22,312 bytes allocated
==16633==
==16633== All heap blocks were freed -- no leaks are possible
==16633==
==16633== For counts of detected and suppressed errors, rerun with: -v
==16633== ERROR SUMMARY: 0 errors from 0 contexts (suppressed: 6 from 3)
pi@raspberrypi:/tmp $
另请注意,您错过了打印print
中的最后一个元素for(int i = 0; i<nbr;i++){
可以
for(int i = 0; i<=nbr;i++){
关于c++ - 使用 std::copy 复制数组时出现段错误,我们在Stack Overflow上找到一个类似的问题: https://stackoverflow.com/questions/56045325/
我正在开发一个小型图书馆,我需要做的一件事是让访问者访问一些数据并返回结果。 在一些较旧的 C++ 代码中,访问者需要声明一个 typedef return_type .例如,boost::stati
我正在尝试使用std:map类型的键和值制作std::any Visual Studio 2017 std::map m("lastname", "Ivanov"); std::cout (m["la
我已经在 C++ 的 map 中声明了一个集合为 std::map> .如何循环访问或打印设定值? 最佳答案 如果你知道如何迭代 std::map或 std::set单独地,您应该可以毫无问题地组合迭
如何循环? 我已经试过了: //----- code std::vector >::iterator it; for ( it = users.begin(); it != users.end();
我有两个用例。 A.我想同步访问两个线程的队列。 B.我想同步两个线程对队列的访问并使用条件变量,因为其中一个线程将等待另一个线程将内容存储到队列中。 对于用例 A,我看到了使用 std::lock_
我正在查看这两种类型特征的文档,但不确定有什么区别。我不是语言律师,但据我所知,它们都适用于“memcpy-able”类型。 它们可以互换使用吗? 最佳答案 不,这些术语不能互换使用。这两个术语都表示
我有以下测试代码,其中有一个参数 fS,它是 ofstream 的容器: #include #include #include #include int
这是这个问题的延续 c++ function ptr in unorderer_map, compile time error 我试图使用 std::function 而不是函数指针,并且只有当函数是
std::unordered_map str_bool_map = { {"a", true}, {"b", false}, {"c", true} }; 我们可以在此映射上使
我有以下对象 std::vector> vectorList; 然后我添加到这个使用 std::vector vec_tmp; vec_tmp.push_back(strDRG); vec_tmp.p
为什么 std::initializer_list不支持std::get<> , std::tuple_size和 std::tuple_element ?在constexpr中用得很多现在的表达式,
我有一个像这样定义的变量 auto drum = std::make_tuple ( std::make_tuple ( 0.3f , Ex
假设我有一个私有(private)std::map在我的类(class)里std::map 。我怎样才能将其转换为std::map返回给用户?我想要下面的原型(prototype) const std
假设我有一个私有(private)std::map在我的类(class)里std::map 。我怎样才能将其转换为std::map返回给用户?我想要下面的原型(prototype) const std
问题 我正在尝试将 lambda 闭包传递给 std::thread,它使用任意封闭参数调用任意封闭函数。 template std::thread timed_thread(Function&& f
我想创建一个模板类,可以容纳容器和容器的任意组合。例如,std::vector或 std::map ,例如。 我尝试了很多组合,但我必须承认模板的复杂性让我不知所措。我编译的关闭是这样的: templ
我有一个 std::vector>我将其分配给相同类型的第二个 vector 。 我收到这个编译器错误: /opt/gcc-8.2.0/include/c++/8.2.0/bits/stl_algob
有时候,我们有一个工厂可以生成一个 std::unique_ptr vector ,后来我们想在类/线程/你命名的之间共享这些指针。因此,最好改用 std::shared_ptr 。当然有一种方法可以
这个问题在这里已经有了答案: Sorting a vector of custom objects (14 个答案) 关闭 6 年前。 我创建了一个 vector vector ,我想根据我定义的参
我有三个类(class)成员: public: std::vector > getObjects(); std::vector > getObjects() const; privat
我是一名优秀的程序员,十分优秀!